Hi, On 09-19 16:01, Stefan Agner wrote: > Hi Ivan, > > Thanks for the quick response! > > On 2024-09-19 11:52, Ivan T. Ivanov wrote: > > Hi Stefan, > > > >> On 19 Sep 2024, at 12:36, Stefan Agner <ste...@agner.ch> wrote: > >> > >> Hi Ivan, > >> > >> I am looking into enabling NVMe boot support using U-Boot on Raspberry > >> Pi 5. Which brings me to some questions wrt PCIe support (see below). > >> > >> On 2024-01-10 13:29, Ivan T. Ivanov wrote: > >>> > > > > > >>> * I am dropping PCIe patch for now. I made some progress on porting > >>> changes > >>> from vendor Linux tree to U-Boot. Unfortunatly testing it is little bit > >>> tricky. They are many devices behind PCIe, but more or less all of them > >>> requires missing either "reset-controller" or "clock-controller" or > >>> "pin-controller" drivers. I was able to probe "cdns,macb" device, but > >>> access to ethernet PHY over MDIO bus is stucking. Then I ported > >>> "raspberrypi,rp1-adc" driver from vendor Linux tree, but it requires > >>> missing clock. And on top of that machine that I used for developing this > >>> crashed and I lost my PCIe changes :-|. Anyway. > >> > >> Have you tried using a M.2 HAT? This likely won't require much in terms > >> of enabling the device. > >> > >> You write that you made some progress, is that compared to v3? Do you > >> mind sharing the latest version of your patches? > > > > > > No further progress on PCIe for U-Boot from my side was made, sorry. > > > > So v3 the patch you've shared in v3 is your latest state? I was > wondering since you wrote in this v4 patch set "I made some progress on > porting changes > from vendor Linux tree to U-Boot."...
And then " .. And on top of that machine that I used for developing this crashed and I lost my PCIe changes ... ". Reading the whole paragraph now I agree it is not completely clear what happened. > > > > Stan is working on adding support for 2712 PCIe in Linux [1]. Perhaps > > once this is complete we can work on adding U-Boot support for the same, > > unless someone don’t beat us, of course. > > Hm, it seems the link did not make it. I guess you meant to link this > patch set [1] > > -- > Stefan > > [1] > https://lore.kernel.org/lkml/20240910151845.17308-1-svarba...@suse.de/T/ Yes. Regards, Ivan