Re: [PATCH v1] dt-bindings: usb: Fix EIC7700 USB reset's issue

[Conor Dooley]

On Wed, Apr 08, 2026 at 09:48:43AM +0200, Krzysztof Kozlowski wrote:
> On Tue, Apr 07, 2026 at 02:17:02PM +0800, caohang@xxxxxxxxxxxxxxxxxx wrote:
> > From: Hang Cao <caohang@xxxxxxxxxxxxxxxxxx>
> > 
> > The EIC7700 USB controller requires a USB PHY RESET operation.PHY RESET
> 
> Missing space after full stop.
> 
> > operation was missed in the verification version, as it was performed in
> > ESWIN's U-Boot.
> > 
> > If a non-ESWIN provided loader is used, this issue will occur, resulting
> > in USB not work.This patch does not introduce any backward incompatibility
> > since the dts is not upstream yet.
> 
> So U-Boot will be affected, no?

Is it even really affected? I don't think there's any bootloader for this
other than what ESWIN is shipping downstream, outside of people's development
trees. And any software that expected two resets will work just as badly as
it always did when a third one is added.

> And even if DTS is not upstreamed, what about all out of tree DTS?
> This is an already released ABI, so at least explain that driver does
> not care about resets here and grabs them all.
> 
> > 
> > Fixes: c640a4239db5 ("dt-bindings: usb: Add ESWIN EIC7700 USB controller")
> 
> 
> Best regards,
> Krzysztof
>

Attachment: signature.asc
Description: PGP signature