Re: [PATCH v4 0/6] rpi5: initial support

Ivan T. Ivanov
2024-09-19T14:19:29Z

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 <stefan@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-svarbanov@suse.de/T/

Yes.

Regards,
Ivan