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

[Vignesh Raman]

Hi Konrad,

On 11/09/23 17:20, Konrad Dybcio wrote:
> On 8.09.2023 17: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 <dmitry.baryshkov@xxxxxxxxxx>
> > Suggested-by: Maxime Ripard <mripard@xxxxxxxxxx>
> > Signed-off-by: Helen Koike <helen.koike@xxxxxxxxxxxxx>
> > Signed-off-by: David Heidelberg <david.heidelberg@xxxxxxxxxxxxx>
> > Signed-off-by: Vignesh Raman <vignesh.raman@xxxxxxxxxxxxx>
> > ---
> For such a small change, maybe you can just use sed in your
> scripts to alter this?

This was the initial idea 
https://patchwork.kernel.org/project/linux-mediatek/patch/20230825122435.316272-3-vignesh.raman@xxxxxxxxxxxxx/

Since there was a review comment that CI should not make changes to the 
source tree before building, we went with DT overlay.

Regards,
Vignesh