mbox series

[0/6] PCI: pci-bridge-emul: Various fixes

Message ID 20211124155944.1290-1-pali@kernel.org
Headers show
Series PCI: pci-bridge-emul: Various fixes | expand

Message

Pali Rohár Nov. 24, 2021, 3:59 p.m. UTC
This patch series contains various fixes for pci-bridge-emul code.
This code is used only by pci-aardvark.c and pci-mvebu.c drivers.

Pali Rohár (6):
  PCI: pci-bridge-emul: Make expansion ROM Base Address register
    read-only
  PCI: pci-bridge-emul: Properly mark reserved PCIe bits in PCI config
    space
  PCI: pci-bridge-emul: Add definitions for missing capabilities
    registers
  PCI: pci-bridge-emul: Fix definitions of reserved bits
  PCI: pci-bridge-emul: Correctly set PCIe capabilities
  PCI: pci-bridge-emul: Set PCI_STATUS_CAP_LIST for PCIe device

 drivers/pci/controller/pci-aardvark.c |   4 +-
 drivers/pci/controller/pci-mvebu.c    |   8 ++
 drivers/pci/pci-bridge-emul.c         | 113 ++++++++++++++++++++++----
 3 files changed, 108 insertions(+), 17 deletions(-)

Comments

Marek Behún Nov. 30, 2021, 1:26 p.m. UTC | #1
On Wed, 24 Nov 2021 16:59:38 +0100
Pali Rohár <pali@kernel.org> wrote:

> This patch series contains various fixes for pci-bridge-emul code.
> This code is used only by pci-aardvark.c and pci-mvebu.c drivers.
> 
> Pali Rohár (6):
>   PCI: pci-bridge-emul: Make expansion ROM Base Address register
>     read-only
>   PCI: pci-bridge-emul: Properly mark reserved PCIe bits in PCI config
>     space
>   PCI: pci-bridge-emul: Add definitions for missing capabilities
>     registers
>   PCI: pci-bridge-emul: Fix definitions of reserved bits
>   PCI: pci-bridge-emul: Correctly set PCIe capabilities
>   PCI: pci-bridge-emul: Set PCI_STATUS_CAP_LIST for PCIe device
> 
>  drivers/pci/controller/pci-aardvark.c |   4 +-
>  drivers/pci/controller/pci-mvebu.c    |   8 ++
>  drivers/pci/pci-bridge-emul.c         | 113 ++++++++++++++++++++++----
>  3 files changed, 108 insertions(+), 17 deletions(-)
> 

I will send v2 of this series with the third patch dropped, since it
will be applied via aardvark batch 3 series.

Marek
Pali Rohár Dec. 2, 2021, 8:20 p.m. UTC | #2
On Tuesday 30 November 2021 14:26:52 Marek Behún wrote:
> On Wed, 24 Nov 2021 16:59:38 +0100
> Pali Rohár <pali@kernel.org> wrote:
> 
> > This patch series contains various fixes for pci-bridge-emul code.
> > This code is used only by pci-aardvark.c and pci-mvebu.c drivers.
> > 
> > Pali Rohár (6):
> >   PCI: pci-bridge-emul: Make expansion ROM Base Address register
> >     read-only
> >   PCI: pci-bridge-emul: Properly mark reserved PCIe bits in PCI config
> >     space
> >   PCI: pci-bridge-emul: Add definitions for missing capabilities
> >     registers
> >   PCI: pci-bridge-emul: Fix definitions of reserved bits
> >   PCI: pci-bridge-emul: Correctly set PCIe capabilities
> >   PCI: pci-bridge-emul: Set PCI_STATUS_CAP_LIST for PCIe device
> > 
> >  drivers/pci/controller/pci-aardvark.c |   4 +-
> >  drivers/pci/controller/pci-mvebu.c    |   8 ++
> >  drivers/pci/pci-bridge-emul.c         | 113 ++++++++++++++++++++++----
> >  3 files changed, 108 insertions(+), 17 deletions(-)
> > 
> 
> I will send v2 of this series with the third patch dropped, since it
> will be applied via aardvark batch 3 series.
> 
> Marek

I marked third patch at patchwork as "Superseded" (as now it was applied
into pci/aardvark branch) and so there is no need to resend same patches
again.
Lorenzo Pieralisi Dec. 6, 2021, 12:19 p.m. UTC | #3
On Wed, 24 Nov 2021 16:59:38 +0100, Pali Rohár wrote:
> This patch series contains various fixes for pci-bridge-emul code.
> This code is used only by pci-aardvark.c and pci-mvebu.c drivers.
> 
> Pali Rohár (6):
>   PCI: pci-bridge-emul: Make expansion ROM Base Address register
>     read-only
>   PCI: pci-bridge-emul: Properly mark reserved PCIe bits in PCI config
>     space
>   PCI: pci-bridge-emul: Add definitions for missing capabilities
>     registers
>   PCI: pci-bridge-emul: Fix definitions of reserved bits
>   PCI: pci-bridge-emul: Correctly set PCIe capabilities
>   PCI: pci-bridge-emul: Set PCI_STATUS_CAP_LIST for PCIe device
> 
> [...]

Applied to pci/bridge-emul, thanks!

[1/6] PCI: pci-bridge-emul: Make expansion ROM Base Address register read-only
      https://git.kernel.org/lpieralisi/pci/c/1c1a3b4d3e
[2/6] PCI: pci-bridge-emul: Properly mark reserved PCIe bits in PCI config space
      https://git.kernel.org/lpieralisi/pci/c/7b067ac63a
[3/6] PCI: pci-bridge-emul: Add definitions for missing capabilities registers
      https://git.kernel.org/lpieralisi/pci/c/faa3e547f4
[4/6] PCI: pci-bridge-emul: Fix definitions of reserved bits
      https://git.kernel.org/lpieralisi/pci/c/a3ab28a2e3
[5/6] PCI: pci-bridge-emul: Correctly set PCIe capabilities
      https://git.kernel.org/lpieralisi/pci/c/0f7ba81700
[6/6] PCI: pci-bridge-emul: Set PCI_STATUS_CAP_LIST for PCIe device
      https://git.kernel.org/lpieralisi/pci/c/32051099e8

Thanks,
Lorenzo
Pali Rohár Dec. 6, 2021, 12:24 p.m. UTC | #4
On Monday 06 December 2021 12:19:57 Lorenzo Pieralisi wrote:
> On Wed, 24 Nov 2021 16:59:38 +0100, Pali Rohár wrote:
> > This patch series contains various fixes for pci-bridge-emul code.
> > This code is used only by pci-aardvark.c and pci-mvebu.c drivers.
> > 
> > Pali Rohár (6):
> >   PCI: pci-bridge-emul: Make expansion ROM Base Address register
> >     read-only
> >   PCI: pci-bridge-emul: Properly mark reserved PCIe bits in PCI config
> >     space
> >   PCI: pci-bridge-emul: Add definitions for missing capabilities
> >     registers
> >   PCI: pci-bridge-emul: Fix definitions of reserved bits
> >   PCI: pci-bridge-emul: Correctly set PCIe capabilities
> >   PCI: pci-bridge-emul: Set PCI_STATUS_CAP_LIST for PCIe device
> > 
> > [...]
> 
> Applied to pci/bridge-emul, thanks!
> 
> [1/6] PCI: pci-bridge-emul: Make expansion ROM Base Address register read-only
>       https://git.kernel.org/lpieralisi/pci/c/1c1a3b4d3e
> [2/6] PCI: pci-bridge-emul: Properly mark reserved PCIe bits in PCI config space
>       https://git.kernel.org/lpieralisi/pci/c/7b067ac63a
> [3/6] PCI: pci-bridge-emul: Add definitions for missing capabilities registers
>       https://git.kernel.org/lpieralisi/pci/c/faa3e547f4

Hello Lorenzo! This patch "PCI: pci-bridge-emul: Add definitions for
missing capabilities registers" is now in your two different branches:
pci/bridge-emul and pci/aardvark. Not sure if you want to have this same
patch on two places... So please check.

> [4/6] PCI: pci-bridge-emul: Fix definitions of reserved bits
>       https://git.kernel.org/lpieralisi/pci/c/a3ab28a2e3
> [5/6] PCI: pci-bridge-emul: Correctly set PCIe capabilities
>       https://git.kernel.org/lpieralisi/pci/c/0f7ba81700
> [6/6] PCI: pci-bridge-emul: Set PCI_STATUS_CAP_LIST for PCIe device
>       https://git.kernel.org/lpieralisi/pci/c/32051099e8
> 
> Thanks,
> Lorenzo
Lorenzo Pieralisi Dec. 6, 2021, 2:26 p.m. UTC | #5
On Mon, Dec 06, 2021 at 01:24:51PM +0100, Pali Rohár wrote:
> On Monday 06 December 2021 12:19:57 Lorenzo Pieralisi wrote:
> > On Wed, 24 Nov 2021 16:59:38 +0100, Pali Rohár wrote:
> > > This patch series contains various fixes for pci-bridge-emul code.
> > > This code is used only by pci-aardvark.c and pci-mvebu.c drivers.
> > > 
> > > Pali Rohár (6):
> > >   PCI: pci-bridge-emul: Make expansion ROM Base Address register
> > >     read-only
> > >   PCI: pci-bridge-emul: Properly mark reserved PCIe bits in PCI config
> > >     space
> > >   PCI: pci-bridge-emul: Add definitions for missing capabilities
> > >     registers
> > >   PCI: pci-bridge-emul: Fix definitions of reserved bits
> > >   PCI: pci-bridge-emul: Correctly set PCIe capabilities
> > >   PCI: pci-bridge-emul: Set PCI_STATUS_CAP_LIST for PCIe device
> > > 
> > > [...]
> > 
> > Applied to pci/bridge-emul, thanks!
> > 
> > [1/6] PCI: pci-bridge-emul: Make expansion ROM Base Address register read-only
> >       https://git.kernel.org/lpieralisi/pci/c/1c1a3b4d3e
> > [2/6] PCI: pci-bridge-emul: Properly mark reserved PCIe bits in PCI config space
> >       https://git.kernel.org/lpieralisi/pci/c/7b067ac63a
> > [3/6] PCI: pci-bridge-emul: Add definitions for missing capabilities registers
> >       https://git.kernel.org/lpieralisi/pci/c/faa3e547f4
> 
> Hello Lorenzo! This patch "PCI: pci-bridge-emul: Add definitions for
> missing capabilities registers" is now in your two different branches:
> pci/bridge-emul and pci/aardvark. Not sure if you want to have this same
> patch on two places... So please check.

Dropped, thanks.

Lorenzo

> > [4/6] PCI: pci-bridge-emul: Fix definitions of reserved bits
> >       https://git.kernel.org/lpieralisi/pci/c/a3ab28a2e3
> > [5/6] PCI: pci-bridge-emul: Correctly set PCIe capabilities
> >       https://git.kernel.org/lpieralisi/pci/c/0f7ba81700
> > [6/6] PCI: pci-bridge-emul: Set PCI_STATUS_CAP_LIST for PCIe device
> >       https://git.kernel.org/lpieralisi/pci/c/32051099e8
> > 
> > Thanks,
> > Lorenzo