Piteball
/
android_kernel_samsung_msm8976
mirror of https://github.com/team-infusion-developers/android_kernel_samsung_msm8976.git
1
1
Fork 0
Code Issues Projects Releases Wiki Activity

crypto: ice: fix ICE HW setup sequence 

ICE driver should setup ICE hardware before any caller can configure
keys in ICE hardware. Currently, ICE sets up clocks, bus vote and power
regulator respectively. This order is a little wrong as setting up
clocks could fail if power regulator is not enabled.

Change-Id: I980abd9c55ce22dac1272d57c4d1af7e8960d4da
Signed-off-by: Dinesh K Garg <dineshg@codeaurora.org>
Signed-off-by: AnilKumar Chimata <anilc@codeaurora.org>
This commit is contained in:
Dinesh K Garg 2015-09-30 12:07:10 -07:00 committed by AnilKumar Chimata
parent e44612924e
commit bf4af210b0
1 changed files with 73 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

97
drivers/crypto/msm/ice.c
View File

@ -1435,58 +1435,107 @@ out:
return ice_dev;
}
int qcom_ice_setup_ice_hw(const char *storage_type, int enable)
static int enable_ice_setup(struct ice_device *ice_dev)
{
int ret = -1, vote;
struct ice_device *ice_dev = NULL;
ice_dev = get_ice_device_from_storage_type(storage_type);
if (!ice_dev)
goto out;
/* Setup Regulator */
if (ice_dev->is_regulator_available) {
if (qcom_ice_get_vreg(ice_dev)) {
pr_err("%s: Could not get regulator\n", __func__);
goto out;
}
ret = regulator_enable(ice_dev->reg);
if (ret) {
pr_err("%s:%p: Could not enable regulator\n",
__func__, ice_dev);
goto out;
}
}
/* Setup Clocks */
if (qcom_ice_enable_clocks(ice_dev, enable)) {
pr_err("%s:%p:%s Could not enable clocks\n", __func__, ice_dev,
ice_dev->ice_instance_type);
goto out;
if (qcom_ice_enable_clocks(ice_dev, true)) {
pr_err("%s:%p:%s Could not enable clocks\n", __func__,
ice_dev, ice_dev->ice_instance_type);
goto out_reg;
}
/* Setup Bus Vote */
vote = qcom_ice_get_bus_vote(ice_dev, enable ? "MAX" : "MIN");
vote = qcom_ice_get_bus_vote(ice_dev, "MAX");
if (vote < 0)
goto out_clocks;
ret = qcom_ice_set_bus_vote(ice_dev, vote);
if (ret) {
pr_err("%s:%p: failed %d\n", __func__, ice_dev, ret);
goto out_clocks;
}
/* Setup Regulator */
return ret;
out_clocks:
qcom_ice_enable_clocks(ice_dev, false);
out_reg:
regulator_disable(ice_dev->reg);
out:
return ret;
}
static int disable_ice_setup(struct ice_device *ice_dev)
{
int ret = -1, vote;
/* Setup Bus Vote */
vote = qcom_ice_get_bus_vote(ice_dev, "MIN");
if (vote < 0) {
pr_err("%s:%p: Unable to get bus vote\n", __func__, ice_dev);
goto out_disable_clocks;
}
ret = qcom_ice_set_bus_vote(ice_dev, vote);
if (ret)
pr_err("%s:%p: failed %d\n", __func__, ice_dev, ret);
out_disable_clocks:
/* Setup Clocks */
if (qcom_ice_enable_clocks(ice_dev, false))
pr_err("%s:%p:%s Could not disable clocks\n", __func__,
ice_dev, ice_dev->ice_instance_type);
/* Setup Regulator */
if (ice_dev->is_regulator_available) {
if (qcom_ice_get_vreg(ice_dev)) {
pr_err("%s: Could not get regulator\n", __func__);
goto out_bw_vote;
goto out;
}
ret = (enable) ? regulator_enable(ice_dev->reg)
: regulator_disable(ice_dev->reg);
ret = regulator_disable(ice_dev->reg);
if (ret) {
pr_err("%s:%p: Could not enable/disable regulator\n",
pr_err("%s:%p: Could not disable regulator\n",
__func__, ice_dev);
goto out_bw_vote;
goto out;
}
}
return ret;
out_bw_vote:
if (vote)
qcom_ice_set_bus_vote(ice_dev, !vote);
out_clocks:
if (enable)
qcom_ice_enable_clocks(ice_dev, !enable);
out:
return ret;
}
int qcom_ice_setup_ice_hw(const char *storage_type, int enable)
{
int ret = -1;
struct ice_device *ice_dev = NULL;
ice_dev = get_ice_device_from_storage_type(storage_type);
if (!ice_dev)
return ret;
if (enable)
return enable_ice_setup(ice_dev);
else
return disable_ice_setup(ice_dev);
}
struct qcom_ice_variant_ops *qcom_ice_get_variant_ops(struct device_node *node)
{
if (node) {