RE: [PATCH RESEND v3 0/5] scsi: ufs: Fix regulator operations and remove "<name>-fixed-regulator" device tree property

[Stanley Chu <stanley.chu-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>]

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