Hi Avri, Alim, Pedrom, Gentle ping for this series. On Mon, 2019-03-18 at 10:14 +0800, Stanley Chu wrote: > Hi, > > Sorry please help review V3 instead of previous versions. > In this version I fix and add more details in commit messages, and add one patch to fix "undefined voltage range" issue as well. > > This patch series fixes UFS regulator operations, including voltage and current (re-)configuration flow during initialization or different power modes. > > Besides, remove "<name>-fixed-regulator" device tree property because it is not necessary anymore after fixes. > > V3: > - Fix and add more details in commit messages. > - Add one patch "scsi: ufs: Avoid configuring undefined voltage range on a regulator". > > V2: > - Add two patches to prepare to and remove "<name>-fixed-regulator" device tree property. > - Add more details on patch "scsi: ufs: remove unused min_uA field in struct ufs_vreg" (Marc Gonzalez). > > Stanley Chu (5): > scsi: ufs: Remove unused min_uA field in struct ufs_vreg > scsi: ufs: Avoid configuring undefined voltage range on a regulator > scsi: ufs: Fix regulator load and icc-level configuration > scsi: ufs: Change "<name>-max-microamp" to non-mandatory property > scsi: ufs: Remove "<name>-fixed-regulator" device tree property > > drivers/scsi/ufs/ufs.h | 1 - > drivers/scsi/ufs/ufshcd-pltfrm.c | 14 +++----------- > drivers/scsi/ufs/ufshcd.c | 28 ++++++++++++++++++++-------- > 3 files changed, 23 insertions(+), 20 deletions(-) > Thanks, Stanley