[PATCH 05/11] arm: dts: imx6qdl-gw*: add dr_mode prop to dt to avoid error

Tim Harvey tharvey at gateworks.com
Tue Apr 6 01:46:28 CEST 2021


On Mon, Apr 5, 2021 at 2:27 PM Peter Robinson <pbrobinson at gmail.com> wrote:
>
> On Mon, Mar 1, 2021 at 10:34 PM Tim Harvey <tharvey at gateworks.com> wrote:
> >
> > The fsl-usb dt bindings in Linux default dr_mode to 'host' for
> > backward compatibility however U-Boot prints an error if
> > this property does not exist. Declare it in the Gateworks
> > Ventana device-trees to avoid the error.
>
> Is there any reason not to fix the driver not to error if this
> property is missing rather than working around it like this?
>

Peter,

The usb_get_dr_mode function that produces the error is used by a
number of drivers that are not all going to have the same default. I
think it's best to augment the device-tree anyway (which I will likely
do in Linux as well).

Tim


More information about the U-Boot mailing list