On Wed, 20 Mar 2024 at 18:04, Caleb Connolly <[email protected]> wrote: > > > > On 20/03/2024 12:33, Caleb Connolly wrote: > > > > > > On 19/03/2024 13:55, Sumit Garg wrote: > >> On Tue, 19 Mar 2024 at 17:52, Caleb Connolly <[email protected]> > >> wrote: > >>> > >>> The USB VBUS supply for the type-A port is enabled via a GPIO regulator. > >>> This is incorrectly modelled in Linux where only the PCIe dependency is > >>> expressed. > >> > >> Can we send a fix for the Linux kernel DTS to correctly model it? We > >> can then later get rid of this modification once that is accepted. > > > > I spoke to Bjorn about this and apparently the correct way to model this > > will be to have a usb-connector node with a vbus-supply property. There > > is some work underway in Linux to support this kind of thing already. > > > > In the mean time he suggested to just make the regulator always-on, so I > > sent a patch to do that [1]. I actually hit this issue when booting from > > USB as the PCIe drivers aren't available in the initramfs, so USB never > > turns on, and the always-on hack fixes that. > > > > In the mean time, we'll still need this vbus-supply reference as U-Boot > > doesn't automatically probe regulators. >
Try adding regulators_enable_boot_on() to the common board code. With that there shouldn't be any need for this vbus-supply reference. > oh, forgot the link! > > [1]: > https://lore.kernel.org/linux-arm-msm/[email protected]/ -Sumit
