diff options
| author | Stanley Chu <stanley.chu@mediatek.com> | 2019-03-28 12:16:25 +0300 |
|---|---|---|
| committer | Martin K. Petersen <martin.petersen@oracle.com> | 2019-04-04 06:11:35 +0300 |
| commit | 0487fff76632ec023d394a05b82e87a971db8c03 (patch) | |
| tree | 8043ef1a97cba5125bda1f85f4efcd5640eec486 /drivers/cpuidle/cpuidle-pseries.c | |
| parent | 3b141e8cfd54ba3e5c610717295b2a02aab26a05 (diff) | |
| download | linux-0487fff76632ec023d394a05b82e87a971db8c03.tar.xz | |
scsi: ufs: Fix regulator load and icc-level configuration
Currently if a regulator has "<name>-fixed-regulator" property in device
tree, it will skip current limit initialization. This lead to a zero
"max_uA" value in struct ufs_vreg.
However, "regulator_set_load" operation shall be required on regulators
which have valid current limits, otherwise a zero "max_uA" set by
"regulator_set_load" may cause unexpected behavior when this regulator is
enabled or set as high power mode.
Similarly, in device's icc_level configuration flow, the target icc_level
shall be updated if regulator also has valid current limit, otherwise a
wrong icc_level will be calculated by zero "max_uA" and thus causes
unexpected results after it is written to device.
Signed-off-by: Stanley Chu <stanley.chu@mediatek.com>
Reviewed-by: Avri Altman <avri.altman@wdc.com>
Acked-by: Alim Akhtar <alim.akhtar@samsung.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'drivers/cpuidle/cpuidle-pseries.c')
0 files changed, 0 insertions, 0 deletions