Re: [PATCH 1/2] PCI: rockchip: Enable IO base and limit registers【请注意，邮件由linux-rockchip-bounces+shawn.lin=rock-chips.com@lists.infradead.org代发】

[Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>]

On Fri, May 22, 2020 at 05:59:14PM +0530, Anand Moon wrote:
> Hi Shawn
> 
> On Fri, 22 May 2020 at 08:30, Shawn Lin <shawn.lin@rock-chips.com> wrote:
> >
> >
> > 在 2020/5/21 18:51, Anand Moon 写道:
> > > Hi Shawn,
> > >
> > > On Thu, 21 May 2020 at 06:35, Shawn Lin <shawn.lin@rock-chips.com> wrote:
> > >>
> > >> According to RK3399 user manual, bit 9 in PCIE_RC_BAR_CONF should
> > >> be set, otherwise accessing to IO base and limit registers would
> > >> fail.
> > >>
> > >> [    0.411318] pci_bus 0000:00: root bus resource [bus 00-1f]
> > >> [    0.411822] pci_bus 0000:00: root bus resource [mem 0xfa000000-0xfbdfffff]
> > >> [    0.412440] pci_bus 0000:00: root bus resource [io  0x0000-0xfffff] (bus address [0xfbe00000-0xfbefffff])
> > >> [    0.413665] pci 0000:00:00.0: bridge configuration invalid ([bus 00-00]), reconfiguring
> > >> [    0.414698] pci 0000:01:00.0: reg 0x10: initial BAR value 0x00000000 invalid
> > >> [    0.415412] pci 0000:01:00.0: reg 0x18: initial BAR value 0x00000000 invalid
> > >> [    0.418456] pci 0000:00:00.0: BAR 8: assigned [mem 0xfa000000-0xfa0fffff]
> > >> [    0.419065] pci 0000:01:00.0: BAR 1: assigned [mem 0xfa000000-0xfa007fff pref]
> > >> [    0.419728] pci 0000:01:00.0: BAR 6: assigned [mem 0xfa008000-0xfa00ffff pref]
> > >> [    0.420377] pci 0000:01:00.0: BAR 0: no space for [io  size 0x0100]
> > >> [    0.420935] pci 0000:01:00.0: BAR 0: failed to assign [io  size 0x0100]
> > >> [    0.421526] pci 0000:01:00.0: BAR 2: no space for [io  size 0x0004]
> > >> [    0.422084] pci 0000:01:00.0: BAR 2: failed to assign [io  size 0x0004]
> > >> [    0.422687] pci 0000:00:00.0: PCI bridge to [bus 01]
> > >> [    0.423135] pci 0000:00:00.0:   bridge window [mem 0xfa000000-0xfa0fffff]
> > >> [    0.423794] pcieport 0000:00:00.0: enabling device (0000 -> 0002)
> > >> [    0.424566] pcieport 0000:00:00.0: Signaling PME through PCIe PME interrupt
> > >> [    0.425182] pci 0000:01:00.0: Signaling PME through PCIe PME interrupt
> > >>
> > >> 01:00.0 Class 0700: Device 1c00:3853 (rev 10) (prog-if 05)
> > >>          Subsystem: Device 1c00:3853
> > >>          Control: I/O- Mem- BusMaster- SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- DisINTx-
> > >>          Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=fast >TAbort- <TAbort- <MAbort- >SERR- <PERR- INTx-
> > >>          Interrupt: pin A routed to IRQ 230
> > >>          Region 0: I/O ports at <unassigned> [disabled]
> > >>          Region 1: Memory at fa000000 (32-bit, prefetchable) [disabled] [size=32K]
> > >>          Region 2: I/O ports at <unassigned> [disabled]
> > >>          [virtual] Expansion ROM at fa008000 [disabled] [size=32K]
> > >>
> > >> Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>
> > >> ---
> > >
> > > I have old development board Odroid N1 (RK3399),  It has onboard PCIe
> > > 2 dual sata bridge.
> > > I have tested this patch, but I am still getting following log on
> > > Odroid N1 board.
> > > Is their any more configuration needed for sata ports ?
> >
> > Thanks for testing. I made a mistake that it should be bit 19, so
> > can you try using BIT(19)?
> >
> 
> Nop enable this bit dose not solve the issue see at my end.
> 
> But as per RK3399 TMR  17.6.7.1.45 Root Complex BAR Configuration Register
> their are many bits that are not tuned correctly.
> I tried to set some bit to BAR Configuration register. but it dose not
> work at my end.
> I feel some more core configuration is missing.
> If I have some update I will share it with you.

What's the status of this discussion and therefore this series ?

Thanks,
Lorenzo