Hi Marek and Tom,
Thanks a lot for the feedback and explanations.
On Mon, Sep 21, 2026 at 11:40:15AM +0200, Marek Vasut wrote:
> On 9/21/26 11:10 AM, Stefan Eichenberger wrote:
>
> Hello everyone,
>
> > However, if we have to handle this corner case, wouldn't it be better
> > that probe fails entirely when dwc3_generic_probe is not called by a
> > dwc3 wrapper driver? Or asking differently, is it supposed to work that
> > the "snps,dwc3" node can be used without a wrapper driver?
>
> arch/arm/boot/dts/nxp/ls/ls1021a.dtsi
>
> 792 usb3: usb@3100000 {
> 793 compatible = "snps,dwc3";
>
> It seems snps,dwc3 can be used without a wrapper ?
>
> > If that's not supposed to work, I would propose something like this at
> > the top of dwc3_generic_probe:
> > if (!strstr(dev->parent->driver->name, "dwc3"))
> > return -EINVAL;
>
> The parent driver name check sounds good !
Perfect, then I will add this check to v2.
Regards,
Stefan
