RE: [PATCH v5 4/5] scsi: ufs-exynos: add UFS host support for Exynos SoCs

From: Alim Akhtar
Date: Fri Apr 17 2020 - 13:51:44 EST


Hi Christoph,
Thanks for your feedback.

> -----Original Message-----
> From: 'Christoph Hellwig' <hch@xxxxxxxxxxxxx>
> Sent: 13 April 2020 19:09
> To: Alim Akhtar <alim.akhtar@xxxxxxxxxxx>
> Cc: 'Christoph Hellwig' <hch@xxxxxxxxxxxxx>; robh@xxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; linux-scsi@xxxxxxxxxxxxxxx; linux-samsung-
> soc@xxxxxxxxxxxxxxx; martin.petersen@xxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; krzk@xxxxxxxxxx; kwmad.kim@xxxxxxxxxxx;
> avri.altman@xxxxxxx; cang@xxxxxxxxxxxxxx; 'Seungwon Jeon'
> <essuuj@xxxxxxxxx>; stanley.chu@xxxxxxxxxxxx; linux-arm-
> kernel@xxxxxxxxxxxxxxxxxxx
> Subject: Re: [PATCH v5 4/5] scsi: ufs-exynos: add UFS host support for
Exynos
> SoCs
>
> On Sun, Apr 12, 2020 at 09:54:53PM +0530, Alim Akhtar wrote:
> > > So this doesn't actually require the various removed or not added
> > > quirks
> > after
> > > all?
> > This driver is actual consumer of those quirks, so those are still
needed.
> > On Martin's 5.7/scsi-queue need to revert " 492001990f64 scsi: ufshcd:
> > remove unused quirks"
>
> No. You need to include one patch per quirk in your series to add them
back.
Sure, will send the updated version soon.

> Please also follow all proper kernel style guidelines, as the old code
didn't
> always follow the proper style.