On Thu, Sep 14, 2023 at 11:00:58AM -0500, Bjorn Helgaas wrote: > On Fri, Aug 25, 2023 at 06:32:07PM +0900, Yoshihiro Shimoda wrote: > > Update dw_pcie_link_set_max_link_width() to set PCI_EXP_LNKCAP_MLW. > > In accordance with the DW PCIe RC/EP HW manuals [1,2,3,...] aside with > > the PORT_LINK_CTRL_OFF.LINK_CAPABLE and GEN2_CTRL_OFF.NUM_OF_LANES[8:0] > > field there is another one which needs to be updated. It's > > LINK_CAPABILITIES_REG.PCIE_CAP_MAX_LINK_WIDTH. If it isn't done at > > the very least the maximum link-width capability CSR won't expose > > the actual maximum capability. > > > > [1] DesignWare Cores PCI Express Controller Databook - DWC PCIe Root Port, > > Version 4.60a, March 2015, p.1032 > > [2] DesignWare Cores PCI Express Controller Databook - DWC PCIe Root Port, > > Version 4.70a, March 2016, p.1065 > > [3] DesignWare Cores PCI Express Controller Databook - DWC PCIe Root Port, > > Version 4.90a, March 2016, p.1057 > > ... > > [X] DesignWare Cores PCI Express Controller Databook - DWC PCIe Endpoint, > > Version 5.40a, March 2019, p.1396 > > [X+1] DesignWare Cores PCI Express Controller Databook - DWC PCIe Root Port, > > Version 5.40a, March 2019, p.1266 > > Is there value in keeping all four of these Root Port citations? I > assume that if you have the most recent one (X+1), it completely > obsoletes the older ones, so you should never have to look at the > older ones? In general the procedure may differ from one device version to another. Though it doesn't concern DW PCIe IP-cores. So by citing all these manuals I implied that all DW PCIe controllers expect the same link-width initialization procedure. Keeping that in mind I guess the text could be indeed simplified by keeping only two citations (note [X] and [X+1] refer to the Root Port and End-point HW databooks of the same IP-core version) and noting in the text that the procedure is common for the older DW PCIe controllers too. In anyway I wouldn't say that new IP-core databooks obsolete the old one since the driver supports all old and new controllers. So before introducing a generic procedure we need to make sure that it will work for all the known to be supported devices. From that perspective citing all the available databooks gets to make sense. -Serge(y) > > Bjorn
