Hi Sylwester,
I am trying to test v4 of your patch series for USB support on rpi4: https://patchwork.ozlabs.org/project/uboot/list/?series=179078
I am building u-boot 64-bit. Testing on a rpi4 B 1.2 board. But, USB is not working for me and wanted to see if you had any ideas. Are there other dependencies other than just applying your patch series?
I've pasted the u-boot console output below.
From what I can tell this board does not need the VL805 firmware
loading patch. But, I did try those patches and it did not make any difference.
Thanks, Stuart
---------------------------------------------------------------------
U-Boot 2020.07-rc1-00009-geca0ff12a6 (Jun 10 2020 - 16:02:20 -0500)
DRAM: 3.9 GiB RPI 4 Model B (0xc03112) MMC: emmc2@7e340000: 0, mmcnr@7e300000: 1 Loading Environment from FAT... *** Warning - bad CRC, using default environment
In: serial Out: vidconsole Err: vidconsole Net: eth0: genet@7d580000 starting USB... No working controllers found Hit any key to stop autoboot: 0
On Wed, Jun 10, 2020 at 4:11 PM Stuart Yoder b08248@gmail.com wrote:
Hi Sylwester,
I am trying to test v4 of your patch series for USB support on rpi4: https://patchwork.ozlabs.org/project/uboot/list/?series=179078
I am building u-boot 64-bit. Testing on a rpi4 B 1.2 board. But, USB is not working for me and wanted to see if you had any ideas. Are there other dependencies other than just applying your patch series?
I've pasted the u-boot console output below.
From what I can tell this board does not need the VL805 firmware loading patch. But, I did try those patches and it did not make any difference.
Thanks, Stuart
U-Boot 2020.07-rc1-00009-geca0ff12a6 (Jun 10 2020 - 16:02:20 -0500)
DRAM: 3.9 GiB RPI 4 Model B (0xc03112) MMC: emmc2@7e340000: 0, mmcnr@7e300000: 1 Loading Environment from FAT... *** Warning - bad CRC, using default environment
In: serial Out: vidconsole Err: vidconsole Net: eth0: genet@7d580000 starting USB... No working controllers found Hit any key to stop autoboot: 0
So, in inserting some printfs I can see that the pcie_brcmstb PCI driver is not getting probed at all. So, thinking it must have something to do with the device tree U-boot is consuming and not finding that device. I'm not that familiar with the u-boot driver model yet so any hints or tips appreciated.
Thanks, Stuart
On Wed, Jun 10, 2020 at 10:11 PM Stuart Yoder b08248@gmail.com wrote:
Hi Sylwester,
I am trying to test v4 of your patch series for USB support on rpi4: https://patchwork.ozlabs.org/project/uboot/list/?series=179078
I am building u-boot 64-bit. Testing on a rpi4 B 1.2 board. But, USB is not working for me and wanted to see if you had any ideas. Are there other dependencies other than just applying your patch series?
I've pasted the u-boot console output below.
From what I can tell this board does not need the VL805 firmware
1.2 rev boards do need the firmware loading patch series, not sure about any other patches.
loading patch. But, I did try those patches and it did not make any difference.
Thanks, Stuart
U-Boot 2020.07-rc1-00009-geca0ff12a6 (Jun 10 2020 - 16:02:20 -0500)
DRAM: 3.9 GiB RPI 4 Model B (0xc03112) MMC: emmc2@7e340000: 0, mmcnr@7e300000: 1 Loading Environment from FAT... *** Warning - bad CRC, using default environment
In: serial Out: vidconsole Err: vidconsole Net: eth0: genet@7d580000 starting USB... No working controllers found Hit any key to stop autoboot: 0
On Wed, Jun 10, 2020 at 5:37 PM Peter Robinson pbrobinson@gmail.com wrote:
On Wed, Jun 10, 2020 at 10:11 PM Stuart Yoder b08248@gmail.com wrote:
Hi Sylwester,
I am trying to test v4 of your patch series for USB support on rpi4: https://patchwork.ozlabs.org/project/uboot/list/?series=179078
I am building u-boot 64-bit. Testing on a rpi4 B 1.2 board. But, USB is not working for me and wanted to see if you had any ideas. Are there other dependencies other than just applying your patch series?
I've pasted the u-boot console output below.
From what I can tell this board does not need the VL805 firmware
1.2 rev boards do need the firmware loading patch series, not sure about any other patches.
Are you sure? A colleague of mine said that it was in 1.4 that the flash chip for the VL805 was removed. I can physically see that the flash chip is on my board.
In any case, I did figure out one thing that was going on. I was using the rpi4 DTB as it came shipped with my board. I found that on the PCIe node there was a typo in the compatible string. I fixed that: bus-range = <0x0 0x1>; - compatible = "brcm,bcm7211-pcie", "brcm,bcm7445-pcie", "brcm,pci-plat-dev"; + compatible = "brcm,bcm2711-pcie", "brcm,bcm7445-pcie", "brcm,pci-plat-dev"; max-link-speed = <0x2>;
...and USB now starts up and works.
Thanks, Stuart
I am trying to test v4 of your patch series for USB support on rpi4: https://patchwork.ozlabs.org/project/uboot/list/?series=179078
I am building u-boot 64-bit. Testing on a rpi4 B 1.2 board. But, USB is not working for me and wanted to see if you had any ideas. Are there other dependencies other than just applying your patch series?
I've pasted the u-boot console output below.
From what I can tell this board does not need the VL805 firmware
1.2 rev boards do need the firmware loading patch series, not sure about any other patches.
Are you sure? A colleague of mine said that it was in 1.4 that the flash chip for the VL805 was removed. I can physically see that the flash chip is on my board.
Yes, I am sure, the 1.2 is for the 2/4Gb model with the changes that need the firmware loading, the 1.4 according to their rev docs is limited to the 8Gb model ATM.
https://www.raspberrypi.org/documentation/hardware/raspberrypi/revision-code...
In any case, I did figure out one thing that was going on. I was using the rpi4 DTB as it came shipped with my board. I found that on the PCIe node there was a typo in the compatible string. I fixed that: bus-range = <0x0 0x1>;
compatible = "brcm,bcm7211-pcie","brcm,bcm7445-pcie", "brcm,pci-plat-dev";
compatible = "brcm,bcm2711-pcie","brcm,bcm7445-pcie", "brcm,pci-plat-dev"; max-link-speed = <0x2>;
...and USB now starts up and works.
Thanks, Stuart
Hi,
(Cc: Nicolas and Jim)
On 11.06.2020 11:18, Peter Robinson wrote:
I am trying to test v4 of your patch series for USB support on rpi4:
I am building u-boot 64-bit. Testing on a rpi4 B 1.2 board. But, USB is not working for me and wanted to see if you had any ideas. Are there other dependencies other than just applying your patch series?
I've pasted the u-boot console output below.
From what I can tell this board does not need the VL805 firmware
1.2 rev boards do need the firmware loading patch series, not sure about any other patches.
Are you sure? A colleague of mine said that it was in 1.4 that the flash chip for the VL805 was removed. I can physically see that the flash chip is on my board.
Yes, I am sure, the 1.2 is for the 2/4Gb model with the changes that need the firmware loading, the 1.4 according to their rev docs is limited to the 8Gb model ATM.
https://www.raspberrypi.org/documentation/hardware/raspberrypi/revision-code...
I'm also using rev. 1.2 board and it has the flash chip mounted and doesn't need the VL805 firmware loading patch series.
In any case, I did figure out one thing that was going on. I was using the rpi4 DTB as it came shipped with my board. I found that on the PCIe node there was a typo in the compatible string. I fixed that: bus-range = <0x0 0x1>;
compatible = "brcm,bcm7211-pcie","brcm,bcm7445-pcie", "brcm,pci-plat-dev";
compatible = "brcm,bcm2711-pcie","brcm,bcm7445-pcie", "brcm,pci-plat-dev"; max-link-speed = <0x2>;
...and USB now starts up and works.
I'm glad you figured it out. Sorry, I just read your e-mails now, after 4 days holiday.
In my patch set I used compatible as in mainline Linux kernel https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/driv...
AFAICS there exist BCM7211 and BCM2711 SoCs and instead of modifying devicetree we could also add another compatible string in the driver, as shown below.
There will likely be follow up patches adding proper support for other Broadcom STB SoCs to the pcie_brcmstb driver, taking care of any possible differences in the PCIe controller.
------8<----- diff --git a/drivers/pci/pcie_brcmstb.c b/drivers/pci/pcie_brcmstb.c index dade79e..ceae56a 100644 --- a/drivers/pci/pcie_brcmstb.c +++ b/drivers/pci/pcie_brcmstb.c @@ -608,6 +608,7 @@ static const struct dm_pci_ops brcm_pcie_ops = { };
static const struct udevice_id brcm_pcie_ids[] = { + { .compatible = "brcm,bcm7211-pcie" }, { .compatible = "brcm,bcm2711-pcie" }, { } }; ------>8-----
On Mon, 2020-06-15 at 11:28 +0200, Sylwester Nawrocki wrote:
Hi,
(Cc: Nicolas and Jim)
Jim, FYI, this relates to u-boot's pcie-brcmstb implementation.
On 11.06.2020 11:18, Peter Robinson wrote:
I am trying to test v4 of your patch series for USB support on rpi4: I am building u-boot 64-bit. Testing on a rpi4 B 1.2 board. But, USB is not working for me and wanted to see if you had any ideas. Are there other dependencies other than just applying your patch series?
I've pasted the u-boot console output below.
From what I can tell this board does not need the VL805 firmware
1.2 rev boards do need the firmware loading patch series, not sure about any other patches.
Are you sure? A colleague of mine said that it was in 1.4 that the flash chip for the VL805 was removed. I can physically see that the flash chip is on my board.
Yes, I am sure, the 1.2 is for the 2/4Gb model with the changes that need the firmware loading, the 1.4 according to their rev docs is limited to the 8Gb model ATM.
https://www.raspberrypi.org/documentation/hardware/raspberrypi/revision-code...
I'm also using rev. 1.2 board and it has the flash chip mounted and doesn't need the VL805 firmware loading patch series.
The first comercially available board I've seen without the vl805 eeprom is the new 8GB variant. There should be an unpopulated chip just below the audio Jack.
In any case, I did figure out one thing that was going on. I was using the rpi4 DTB as it came shipped with my board. I found that on the
I strongly suggest you either use the Linux upstream device-tree or if not possible, the latest device-tree maintained by the rpi guys. For example the one available on their rpi-5.7.y branch.
Lots has changed since they released the original device-tree and you're likely to run into issues like this one all over the place, if combining it with upstream code.
PCIe node there was a typo in the compatible string. I fixed that: bus-range = <0x0 0x1>;
compatible = "brcm,bcm7211-pcie","brcm,bcm7445-pcie", "brcm,pci-plat-dev";
compatible = "brcm,bcm2711-pcie","brcm,bcm7445-pcie", "brcm,pci-plat-dev"; max-link-speed = <0x2>;
...and USB now starts up and works.
I'm glad you figured it out. Sorry, I just read your e-mails now, after 4 days holiday.
In my patch set I used compatible as in mainline Linux kernel
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/driv...
AFAICS there exist BCM7211 and BCM2711 SoCs and instead of modifying devicetree we could also add another compatible string in the driver, as shown below.
There will likely be follow up patches adding proper support for other Broadcom STB SoCs to the pcie_brcmstb driver, taking care of any possible differences in the PCIe controller.
------8<----- diff --git a/drivers/pci/pcie_brcmstb.c b/drivers/pci/pcie_brcmstb.c index dade79e..ceae56a 100644 --- a/drivers/pci/pcie_brcmstb.c +++ b/drivers/pci/pcie_brcmstb.c @@ -608,6 +608,7 @@ static const struct dm_pci_ops brcm_pcie_ops = { };
static const struct udevice_id brcm_pcie_ids[] = {
- { .compatible = "brcm,bcm7211-pcie" }, { .compatible = "brcm,bcm2711-pcie" }, { }
}; ------>8-----
FYI There is a series adding bcm7211 support on Linux: https://lkml.org/lkml/2020/5/26/1154
Regards, Nicolas
On Mon, Jun 15, 2020 at 4:48 AM Nicolas Saenz Julienne nsaenzjulienne@suse.de wrote:
On Mon, 2020-06-15 at 11:28 +0200, Sylwester Nawrocki wrote:
Hi,
(Cc: Nicolas and Jim)
Jim, FYI, this relates to u-boot's pcie-brcmstb implementation.
On 11.06.2020 11:18, Peter Robinson wrote:
I am trying to test v4 of your patch series for USB support on rpi4: I am building u-boot 64-bit. Testing on a rpi4 B 1.2 board. But, USB is not working for me and wanted to see if you had any ideas. Are there other dependencies other than just applying your patch series?
I've pasted the u-boot console output below.
From what I can tell this board does not need the VL805 firmware
1.2 rev boards do need the firmware loading patch series, not sure about any other patches.
Are you sure? A colleague of mine said that it was in 1.4 that the flash chip for the VL805 was removed. I can physically see that the flash chip is on my board.
Yes, I am sure, the 1.2 is for the 2/4Gb model with the changes that need the firmware loading, the 1.4 according to their rev docs is limited to the 8Gb model ATM.
https://www.raspberrypi.org/documentation/hardware/raspberrypi/revision-code...
I'm also using rev. 1.2 board and it has the flash chip mounted and doesn't need the VL805 firmware loading patch series.
The first comercially available board I've seen without the vl805 eeprom is the new 8GB variant. There should be an unpopulated chip just below the audio Jack.
In any case, I did figure out one thing that was going on. I was using the rpi4 DTB as it came shipped with my board. I found that on the
I strongly suggest you either use the Linux upstream device-tree or if not possible, the latest device-tree maintained by the rpi guys. For example the one available on their rpi-5.7.y branch.
Lots has changed since they released the original device-tree and you're likely to run into issues like this one all over the place, if combining it with upstream code.
Thanks! Stuart
participants (4)
-
Nicolas Saenz Julienne -
Peter Robinson -
Stuart Yoder -
Sylwester Nawrocki