Re: [PATCH v9 3/9] PCI: Store all PCIe Supported Link Speeds
From: Ilpo Järvinen
Date: Tue Nov 12 2024 - 04:20:57 EST
On Mon, 11 Nov 2024, Bjorn Helgaas wrote:
> On Mon, Nov 11, 2024 at 02:23:35PM +0100, Lukas Wunner wrote:
> > On Fri, Oct 18, 2024 at 05:47:49PM +0300, Ilpo Järvinen wrote:
> > > diff --git a/include/linux/pci.h b/include/linux/pci.h
> > > index be5ed534c39c..a02b77fe7865 100644
> > > --- a/include/linux/pci.h
> > > +++ b/include/linux/pci.h
> > > @@ -303,6 +303,7 @@ enum pci_bus_speed {
> > > PCI_SPEED_UNKNOWN = 0xff,
> > > };
> > >
> > > +u8 pcie_get_supported_speeds(struct pci_dev *dev);
> > > enum pci_bus_speed pcie_get_speed_cap(struct pci_dev *dev);
> > > enum pcie_link_width pcie_get_width_cap(struct pci_dev *dev);
> >
> > I realize this is now already queued as commit 73ee11953294 on pci/bwctrl,
> > nevertheless one belated comment:
> >
> > Since there are no callers of pcie_get_supported_speeds() outside the
> > PCI core, the above declaration should probably rather live in
> > drivers/pci/pci.h.
>
> I moved them, thanks!
Thanks for taking care of that.
> I noticed duplicate declarations for pcie_get_speed_cap() and
> pcie_get_width_cap(), so I'll add a patch to drop them from
> drivers/pci/pci.h.
I also noticed the same. FWIW,
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxxxxxx>
for that patch of yours.
--
i.