On 10/07/2018 02:16 PM, Bin Meng wrote: > Hi Marek, > > On Sun, Oct 7, 2018 at 8:12 PM Marek Vasut <marek.va...@gmail.com> wrote: >> >> On 10/01/2018 01:44 PM, Marek Vasut wrote: >>> On 09/25/2018 05:26 PM, Bin Meng wrote: >>>> Hi Marek, >>>> >>>> On Sat, Sep 22, 2018 at 7:02 AM Marek Vasut <marek.va...@gmail.com> wrote: >>>>> >>>>> Add PCI entry without compatible string and with a DT node only with >>>>> reg = <...> property into the DT. This is needed for the tests to >>>>> verify whether such a setup creates an U-Boot PCI device with the >>>>> DT node associated with it in udevice.node. >>>>> >>>>> Signed-off-by: Marek Vasut <marek.vasut+rene...@gmail.com> >>>>> Cc: Simon Glass <s...@chromium.org> >>>>> Cc: Tom Rini <tr...@konsulko.com> >>>>> --- >>>>> arch/sandbox/dts/test.dts | 9 +++++++-- >>>>> 1 file changed, 7 insertions(+), 2 deletions(-) >>>>> >>>>> diff --git a/arch/sandbox/dts/test.dts b/arch/sandbox/dts/test.dts >>>>> index b8524e3b7d..c13a270c2e 100644 >>>>> --- a/arch/sandbox/dts/test.dts >>>>> +++ b/arch/sandbox/dts/test.dts >>>>> @@ -354,9 +354,14 @@ >>>>> #address-cells = <3>; >>>>> #size-cells = <2>; >>>>> ranges = <0x02000000 0 0x30000000 0x30000000 0 0x2000 >>>>> - 0x01000000 0 0x40000000 0x40000000 0 >>>>> 0x2000>; >>>>> + 0x01000000 0 0x40000000 0x40000000 0 0x2000 >>>>> + 0x00008000 0 0x00000000 0x00008000 0 0x2000>; >>>> >>>> Adding this line makes no sense. You can't translate a PCI bus >>>> configuration space address (0x8000) to something in its parent bus's >>>> (MMIO) address space. See my related comments in patch 1 and 3. >>>> >>> >>> So what should be in that line ? >> >> Ping ? > > Looks you missed my response ? > https://lists.denx.de/pipermail/u-boot/2018-October/342820.html
I don't have that mail, sorry. Can you explain your reply ? Why "There is no need to add that line in the ranges property." ? -- Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot