On Mon, May 08, 2023 at 08:30:15PM +0530, Mayuresh Chitale wrote:
On Thu, May 4, 2023 at 2:04 AM Tom Rini trini@konsulko.com wrote:
On Sat, Feb 18, 2023 at 05:55:25PM +0530, Mayuresh Chitale wrote:
Use dev_read_() api instead of the fdtdec API to fetch the host controller's reg property value. This is similar to the other host controller drivers such as Sifive, Rockchip etc. Without this change, enabling CONFIG_OF_LIVE breaks the PCIe enumeration on Qemu Risc-V virt machine. The issue is described in the link below:
Link: https://source.denx.de/u-boot/custodians/u-boot-dm/-/issues/1 Signed-off-by: Mayuresh Chitale mchitale@ventanamicro.com Reviewed-by: Simon Glass sjg@chromium.org
drivers/pci/pcie_ecam_generic.c | 21 ++++++++------------- 1 file changed, 8 insertions(+), 13 deletions(-)
This breaks some platforms such as qemu_arm at a minimum.
Ok. How do I reproduce the issue?
Right, sorry. So the failure log is: https://source.denx.de/u-boot/u-boot/-/jobs/622397 And https://u-boot.readthedocs.io/en/latest/develop/py_testing.html talks about running the pytest suite outside of CI and https://u-boot.readthedocs.io/en/latest/develop/ci_testing.html about how to trigger CI yourself. One thing that's not documented well enough is that you'll likely want https://source.denx.de/u-boot/u-boot-test-hooks as well as you can just use as-is the "travis-ci" directories (legacy name, used by both gitlab and azure as-is) as the configuration hooks for your system as well and then the network tests will run easily, locally.