Re: [PATCH v4 11/25] scsi: ufs: mediatek: Rework probe function

Nicolas Frattaroli
2026-01-08T09:15:35Z

On Tuesday, 6 January 2026 14:23:58 Central European Standard Time Peter Wang (王信友) wrote:
> On Thu, 2025-12-18 at 13:55 +0100, Nicolas Frattaroli wrote:
> >
> > Remove the ti,syscon-reset cruft.
> >
>
> Hi Nicolas,
>
> Why do we need to remove the reset node? If an error occurs and the
> host
> does not perform a reset, it could lead to error recovery failure.

Because it's not described by the binding, and appears to be a
downstream hack to work around not having the reset controller
properly described and referred to with a `resets` property.

Even if you were to use `ti,syscon-reset` to describe a reset
controller, the UFS controller driver should not be searching
for this compatible. It should access the reset through the
reset API. The common reset code can then take care of probe
ordering without every driver reinventing it.

>
> Thanks.
> Peter
>