Re: [PATCH 1/1] usb: dwc3: let role switch handle initial state

[Thinh Nguyen <Thinh.Nguyen@synopsys.com>]

On Wed, Oct 25, 2023, Alexander Stein wrote:
> Hi,
> 
> Am Samstag, 21. Oktober 2023, 00:43:39 CEST schrieb Thinh Nguyen:
> > Hi,
> > 
> > On Tue, Oct 17, 2023, Alexander Stein wrote:
> > 
> > > From: Markus Niebel <Markus.Niebel@ew.tq-group.com>
> > > 
> > > When we have a role switch device attached, we should not configure our
> > > initial role. Leave this up to the role switch device, that should
> > > detect and signal the initial role.
> > > 
> > > This fixes situations where a Type-A plug is connected already when the
> > > driver is loaded but the default role is set to none or device. In this
> > > case only an disconnect / reconnect gets the correct mode.
> > 
> > 
> > If the default role is none, why isn't there a notification to update
> > the role on initialization from the connector? The current role should
> > not be none.
> 
> dwc->role_switch_default_mode can only be none if the DT is misconfigured, 
> e.g. role-switch-default-mode = "";

No, I was referring to the current role, which is detected after
initialization. The current role should not be none. If the connector
has not notified the controller of the current role, the dwc3 driver
would set the controller to either host or device mode. The connector
can notify the dwc3 driver if it detected the current role on its
initialization.

> 
> Calls to usb_role_switch_set_role() from usb role switch drivers will not 
> affect dwc->role_switch_default_mode.
> I'm wondering if checking for a misconfigured DT is sensible. But this will be 
> detected by 'make dtbs_check'.
> 

I see that you just pushed out the fix patch for the actual issue, which
looks more reasonable.

Thanks,
Thinh