Hi Avri, On Wed, 2019-03-27 at 10:57 +0000, Avri Altman wrote: > > Hi, > > > > Resend this patch series for review. > > > > This version (v3) fixed and added more details in commit messages, and added > > 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 UFS initialization and power mode switching. > > > > In the end, remove "<name>-fixed-regulator" device tree property because it is > > not necessary anymore after these 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 regulator with undefined voltage range > > 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 > > > This series looks good to me, but better get acked-by from couple of platform guys as well. Thanks so much for review. I'll fix commit message and invite more reviewers in next version. > > Thanks, > Avri > Thanks, Stanley
