On Mon, May 08, 2023 at 08:30:15PM +0530, Mayuresh Chitale wrote: > On Thu, May 4, 2023 at 2:04 AM Tom Rini <tr...@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 <mchit...@ventanamicro.com> > > > Reviewed-by: Simon Glass <s...@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. -- Tom
signature.asc
Description: PGP signature
