RE: [PATCH v4 6/7] Revert "scsi: ufs: disable vccq if it's not needed by UFS device"

From: Avri Altman
Date: Sat Feb 09 2019 - 04:07:53 EST


Hi,
>
> This reverts commit 60f0187031c05e04cbadffb62f557d0ff3564490.
>
> Calling ufshcd_set_vccq_rail_unused() breaks UFS init on two boards.
> I would say that vccq is *not* not needed.
>
> Reviewed-by: Jeffrey Hugo <jhugo@xxxxxxxxxxxxxx>
> Signed-off-by: Marc Gonzalez <marc.w.gonzalez@xxxxxxx>
Those tags got switched off.

I still think that If you are reverting the quirk implementation,
you should remove the quirk listing as well.

Also, as the v3 discussion held on several threads,
and new people might be joining in,
maybe you could reply to this patch with a couple of sentences summing-up
the various theories that this bring-up raised.

Thanks,
Avri