On Mon, 25 May 2009 14:45:22 +0900
Kenji Kaneshige <kaneshige.kenji@xxxxxxxxxxxxxx> wrote:
> Yu Zhao wrote:
> > PCI-to-PCI Bridge 1.2 specifies that the Secondary Bus Reset bit can
> > force the assertion of RST# on the secondary interface, which can be
> > used to reset all devices including subordinates under this bus.
> > This can be used to reset a function if this function is the only
> > device under this bus.
> >
> > Signed-off-by: Yu Zhao <yu.zhao@xxxxxxxxx>
> > ---
> > drivers/pci/pci.c | 31 +++++++++++++++++++++++++++++++
> > 1 files changed, 31 insertions(+), 0 deletions(-)
> >
> > diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
> > index 19a5479..3711e74 100644
> > --- a/drivers/pci/pci.c
> > +++ b/drivers/pci/pci.c
> > @@ -2160,6 +2160,33 @@ static int pci_pm_reset(struct pci_dev *dev,
> > int probe) return 0;
> > }
> >
> > +static int pci_secondary_bus_reset(struct pci_dev *dev, int probe)
> > +{
>
> I guess many people would imagine that pci_secondary_bus_reset()
> receives the pointer to struct pci_dev corresponding to the bridge
> and resets bridge's secondary bus, though it does quite different
> things. So I think the function name should be changed.
Yu, wanna send me an update with the change? You'll need to refresh it
against my linux-next branch to avoid conflicts.
Thanks,
--
Jesse Barnes, Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
