2023-09-12 18:07:18

by Helen Koike

[permalink] [raw]
Subject: Re: [PATCH v3 2/9] arm64: dts: qcom: apq8016-sbc: Add overlay for usb host mode



On 08/09/2023 12:22, Vignesh Raman wrote:
> Due to the presence of the fastboot micro cable in the CI farm,
> it causes the hardware to remain in gadget mode instead of host mode.
> So it doesn't find the network, which results in failure to mount root
> fs via NFS.
>
> Add an overlay dtso file that sets the dr_mode to host, allowing
> the USB controllers to work in host mode. This dtso file will be used
> in drm-ci, mesa-ci.
>
> Overlay DT file uses the sugar syntax [suggested by Dmitry Baryshkov and Maxime Ripard]
>
> Suggested-by: Dmitry Baryshkov <[email protected]>
> Suggested-by: Maxime Ripard <[email protected]>
> Signed-off-by: Helen Koike <[email protected]>
> Signed-off-by: David Heidelberg <[email protected]>
> Signed-off-by: Vignesh Raman <[email protected]>

Acked-by: Helen Koike <[email protected]>

> ---
>
> v3:
> - New patch in the series to add device tree overlay in arch/arm64/boot/dts/qcom
>
> ---
> arch/arm64/boot/dts/qcom/apq8016-sbc-usb-host.dtso | 8 ++++++++
> 1 file changed, 8 insertions(+)
> create mode 100644 arch/arm64/boot/dts/qcom/apq8016-sbc-usb-host.dtso
>
> diff --git a/arch/arm64/boot/dts/qcom/apq8016-sbc-usb-host.dtso b/arch/arm64/boot/dts/qcom/apq8016-sbc-usb-host.dtso
> new file mode 100644
> index 000000000000..a82c26b7eae8
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/apq8016-sbc-usb-host.dtso
> @@ -0,0 +1,8 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> +
> +/dts-v1/;
> +/plugin/;
> +
> +&usb {
> + dr_mode = "host";
> +};