[PATCH v1 0/6] Enable fastboot support for IPQ9574 based boards

[Varadarajan Narayanan]

On Thu, Jul 17, 2025 at 03:01:04PM +0200, Casey Connolly wrote:
>
>
> On 16/07/2025 06:57, Varadarajan Narayanan wrote:
> > On Tue, Jul 15, 2025 at 02:30:09PM +0200, Casey Connolly wrote:
> >> Hi Varadarajan,
> >>
> >> On 7/15/25 12:15, Varadarajan Narayanan wrote:
> >>> This series enables MMC based fastboot flashing for IPQ9574 boards.
> >>> For this the following are added/enabled
> >>> 	* USB2 & USB3 phys support
> >>> 	* USB Gadget support, Fastboot-MMC related configs
> >>> 	* Flashing related documentation
> >>
> >> This is awesome! Glad to see you have the superspeed phy working. I'm a bit
> >> confused though since the of_fixup code in mach-snapdragon configures all
> >> the usb controllers for high speed only and removes the reference to the
> >> superspeed phy, is there some missing commit to adjust this?
> >
> > CONFIG_EVENT is not defined in ipq defconfig. Hence of_live_build() doesn't
> > notify the EVT_OF_LIVE_BUILT event and qcom_of_fixup_nodes -> fixup_usb_nodes
> > is not called.
>
> ohh I see, hmm
>
> >
> > 	=> bdinfo
> > 	. . .
> > 	fdt_blob    = 0x000000006fe3b5e0
> > 	. . .
> >
> > 	=> fdt addr 0x6fe3b5e0
> > 	Working FDT set to 6fe3b5e0
> >
> > 	=> fdt print /soc at 0/usb at 8af8800
>
> Just fyi, the fdt command won't show changes to the live tree, it does
> direct FDT parsing whereas the live-tree is an internal representation
> in U-Boot (which is what makes it faster to access and modify).

ok

[ . . . ]

> > Initially, I manually overrode the 'phys' and 'phy-names' to just have the usb2
> > phy and it didn't seem to work. Only after bringing up the USB3 phy did it work.
> > Not sure what the relation is between the two.
>
> Hmm, you don't even get high speed USB? probably an issue with clocks then.

If 'qcom,select-utmi-as-pipe-clk;' is added to the controller DT node it
works. Should I post a new version

	(1) without SS phy + qcom,select-utmi-as-pipe-clk;

		(or)

	(2) with SS phy + qcom,select-utmi-as-pipe-clk; (i.e. same as
	    this series, with just utmi as pipe clock added)

		(or)

	(3) leave the series as is

Kindly advice.

Thanks
Varada