mbox series

[00/28] Remove PCI_IRQ_LEGACY

Message ID 20240325070944.3600338-1-dlemoal@kernel.org
Headers show
Series Remove PCI_IRQ_LEGACY | expand

Message

Damien Le Moal March 25, 2024, 7:09 a.m. UTC
This patch series removes the use of the depracated PCI_IRQ_LEGACY macro
and replace it with PCI_IRQ_INTX. No functional change.

Damien Le Moal (28):
  PCI: msi: Use PCI_IRQ_INTX
  PCI: portdrv: Use PCI_IRQ_INTX
  PCI: documentation: Use PCI_IRQ_INTX
  sound: intel: Use PCI_IRQ_INTX
  usb: hcd-pci: Use PCI_IRQ_INTX
  tty: 8250_pci: Use PCI_IRQ_INTX
  platform: intel_ips: Use PCI_IRQ_INTX
  ntb: Use PCI_IRQ_INTX
  mfd: intel-lpss-pci: Use PCI_IRQ_INTX
  drm: amdgpu: Use PCI_IRQ_INTX
  infiniband: qib: Use PCI_IRQ_INTX
  infiniband: vmw_pvrdma: Use PCI_IRQ_INTX
  misc: vmci_guest: Use PCI_IRQ_ALL_TYPES
  net: xgbe: Use PCI_IRQ_INTX
  net: aquantia atlantic: Use PCI_IRQ_INTX
  net: atheros: alx: Use PCI_IRQ_INTX
  net: realtek: r8169: Use PCI_IRQ_INTX
  net: wangxun: Use PCI_IRQ_INTX
  net: wireless: ath10k: Use references to INTX instead of LEGACY
  net wireless; realtec: Use PCI_IRQ_INTX
  scsi: arcmsr: Use PCI_IRQ_INTX
  scsi: hpsa: Use PCI_IRQ_INTX
  scsi: ipr: Use PCI_IRQ_INTX
  scsi: megaraid: Use PCI_IRQ_INTX
  scsi: mpt3sas: Use PCI_IRQ_INTX
  scsi: pmcraid: Use PCI_IRQ_INTX
  scsi: vmw_pvscsi: Do not use PCI_IRQ_LEGACY
  PCI: Remove PCI_IRQ_LEGACY

 Documentation/PCI/msi-howto.rst               |  2 +-
 Documentation/PCI/pci.rst                     |  2 +-
 .../translations/zh_CN/PCI/msi-howto.rst      |  2 +-
 Documentation/translations/zh_CN/PCI/pci.rst  |  2 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_irq.c       |  2 +-
 drivers/infiniband/hw/qib/qib_iba7220.c       |  2 +-
 drivers/infiniband/hw/qib/qib_iba7322.c       |  5 ++-
 drivers/infiniband/hw/qib/qib_pcie.c          |  2 +-
 .../infiniband/hw/vmw_pvrdma/pvrdma_main.c    |  2 +-
 drivers/mfd/intel-lpss-pci.c                  |  2 +-
 drivers/misc/vmw_vmci/vmci_guest.c            |  3 +-
 drivers/net/ethernet/amd/xgbe/xgbe-pci.c      |  2 +-
 .../net/ethernet/aquantia/atlantic/aq_cfg.h   |  2 +-
 .../net/ethernet/aquantia/atlantic/aq_hw.h    |  2 +-
 .../net/ethernet/aquantia/atlantic/aq_nic.c   |  2 +-
 .../ethernet/aquantia/atlantic/aq_pci_func.c  |  9 ++---
 .../aquantia/atlantic/hw_atl/hw_atl_a0.c      |  2 +-
 .../aquantia/atlantic/hw_atl/hw_atl_b0.c      |  2 +-
 .../aquantia/atlantic/hw_atl2/hw_atl2.c       |  2 +-
 drivers/net/ethernet/atheros/alx/main.c       |  2 +-
 drivers/net/ethernet/realtek/r8169_main.c     |  2 +-
 drivers/net/ethernet/wangxun/libwx/wx_lib.c   |  8 ++---
 drivers/net/wireless/ath/ath10k/ahb.c         | 18 +++++-----
 drivers/net/wireless/ath/ath10k/pci.c         | 36 +++++++++----------
 drivers/net/wireless/ath/ath10k/pci.h         |  6 ++--
 drivers/net/wireless/realtek/rtw88/pci.c      |  2 +-
 drivers/net/wireless/realtek/rtw89/pci.c      |  2 +-
 drivers/ntb/hw/idt/ntb_hw_idt.c               |  2 +-
 drivers/pci/msi/api.c                         |  8 ++---
 drivers/pci/pcie/portdrv.c                    |  8 ++---
 drivers/platform/x86/intel_ips.c              |  2 +-
 drivers/scsi/arcmsr/arcmsr_hba.c              |  2 +-
 drivers/scsi/hpsa.c                           |  2 +-
 drivers/scsi/ipr.c                            |  2 +-
 drivers/scsi/megaraid/megaraid_sas_base.c     |  4 +--
 drivers/scsi/mpt3sas/mpt3sas_base.c           |  2 +-
 drivers/scsi/pmcraid.c                        |  2 +-
 drivers/scsi/vmw_pvscsi.c                     |  2 +-
 drivers/tty/serial/8250/8250_pci.c            |  2 +-
 drivers/usb/core/hcd-pci.c                    |  3 +-
 include/linux/pci.h                           |  7 ++--
 sound/soc/intel/avs/core.c                    |  2 +-
 42 files changed, 84 insertions(+), 91 deletions(-)

Comments

Hans de Goede March 25, 2024, 11:13 a.m. UTC | #1
Hi,

On 3/25/24 8:09 AM, Damien Le Moal wrote:
> Use the macro PCI_IRQ_INTX instead of the deprecated PCI_IRQ_LEGACY
> macro.
> 
> Signed-off-by: Damien Le Moal <dlemoal@kernel.org>

Thanks, patch looks good to me, feel free to merge
this through whatever tree is convenient (or let me
know if you want me to pick up just this one patch
from the series).

Acked-by: Hans de Goede <hdegoede@redhat.com>

Regards,

Hans




> ---
>  drivers/platform/x86/intel_ips.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/platform/x86/intel_ips.c b/drivers/platform/x86/intel_ips.c
> index ba38649cc142..73ec4460a151 100644
> --- a/drivers/platform/x86/intel_ips.c
> +++ b/drivers/platform/x86/intel_ips.c
> @@ -1505,7 +1505,7 @@ static int ips_probe(struct pci_dev *dev, const struct pci_device_id *id)
>  	 * IRQ handler for ME interaction
>  	 * Note: don't use MSI here as the PCH has bugs.
>  	 */
> -	ret = pci_alloc_irq_vectors(dev, 1, 1, PCI_IRQ_LEGACY);
> +	ret = pci_alloc_irq_vectors(dev, 1, 1, PCI_IRQ_INTX);
>  	if (ret < 0)
>  		return ret;
>
Deucher, Alexander March 25, 2024, 2:29 p.m. UTC | #2
[Public]

> -----Original Message-----
> From: amd-gfx <amd-gfx-bounces@lists.freedesktop.org> On Behalf Of
> Damien Le Moal
> Sent: Monday, March 25, 2024 3:09 AM
> To: linux-pci@vger.kernel.org; Bjorn Helgaas <bhelgaas@google.com>;
> Manivannan Sadhasivami <manivannan.sadhasivam@linaro.org>; linux-
> scsi@vger.kernel.org; Martin K . Petersen <martin.petersen@oracle.com>;
> Jaroslav Kysela <perex@perex.cz>; linux-sound@vger.kernel.org; Greg Kroah-
> Hartman <gregkh@linuxfoundation.org>; linux-usb@vger.kernel.org; linux-
> serial@vger.kernel.org; Hans de Goede <hdegoede@redhat.com>; platform-
> driver-x86@vger.kernel.org; ntb@lists.linux.dev; Lee Jones <lee@kernel.org>;
> David Airlie <airlied@gmail.com>; amd-gfx@lists.freedesktop.org; Jason
> Gunthorpe <jgg@ziepe.ca>; linux-rdma@vger.kernel.org; David S . Miller
> <davem@davemloft.net>; Eric Dumazet <edumazet@google.com>;
> netdev@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: [PATCH 10/28] drm: amdgpu: Use PCI_IRQ_INTX
>
> Use the macro PCI_IRQ_INTX instead of the deprecated PCI_IRQ_LEGACY
> macro.
>
> Signed-off-by: Damien Le Moal <dlemoal@kernel.org>

Feel free to take it through whatever tree makes sense.  If you want me to pick it up, let me know.
Acked-by: Alex Deucher <alexander.deucher@amd.com>

> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_irq.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_irq.c
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_irq.c
> index 7e6d09730e6d..d18113017ee7 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_irq.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_irq.c
> @@ -279,7 +279,7 @@ int amdgpu_irq_init(struct amdgpu_device *adev)
>       adev->irq.msi_enabled = false;
>
>       if (!amdgpu_msi_ok(adev))
> -             flags = PCI_IRQ_LEGACY;
> +             flags = PCI_IRQ_INTX;
>       else
>               flags = PCI_IRQ_ALL_TYPES;
>
> --
> 2.44.0
Bjorn Helgaas March 25, 2024, 5:59 p.m. UTC | #3
On Mon, Mar 25, 2024 at 04:09:11PM +0900, Damien Le Moal wrote:
> This patch series removes the use of the depracated PCI_IRQ_LEGACY macro
> and replace it with PCI_IRQ_INTX. No functional change.
> 
> Damien Le Moal (28):
>   PCI: msi: Use PCI_IRQ_INTX
>   PCI: portdrv: Use PCI_IRQ_INTX
>   PCI: documentation: Use PCI_IRQ_INTX
>   sound: intel: Use PCI_IRQ_INTX
>   usb: hcd-pci: Use PCI_IRQ_INTX
>   tty: 8250_pci: Use PCI_IRQ_INTX
>   platform: intel_ips: Use PCI_IRQ_INTX
>   ntb: Use PCI_IRQ_INTX
>   mfd: intel-lpss-pci: Use PCI_IRQ_INTX
>   drm: amdgpu: Use PCI_IRQ_INTX
>   infiniband: qib: Use PCI_IRQ_INTX
>   infiniband: vmw_pvrdma: Use PCI_IRQ_INTX
>   misc: vmci_guest: Use PCI_IRQ_ALL_TYPES
>   net: xgbe: Use PCI_IRQ_INTX
>   net: aquantia atlantic: Use PCI_IRQ_INTX
>   net: atheros: alx: Use PCI_IRQ_INTX
>   net: realtek: r8169: Use PCI_IRQ_INTX
>   net: wangxun: Use PCI_IRQ_INTX
>   net: wireless: ath10k: Use references to INTX instead of LEGACY
>   net wireless; realtec: Use PCI_IRQ_INTX
>   scsi: arcmsr: Use PCI_IRQ_INTX
>   scsi: hpsa: Use PCI_IRQ_INTX
>   scsi: ipr: Use PCI_IRQ_INTX
>   scsi: megaraid: Use PCI_IRQ_INTX
>   scsi: mpt3sas: Use PCI_IRQ_INTX
>   scsi: pmcraid: Use PCI_IRQ_INTX
>   scsi: vmw_pvscsi: Do not use PCI_IRQ_LEGACY
>   PCI: Remove PCI_IRQ_LEGACY
> 
>  Documentation/PCI/msi-howto.rst               |  2 +-
>  Documentation/PCI/pci.rst                     |  2 +-
>  .../translations/zh_CN/PCI/msi-howto.rst      |  2 +-
>  Documentation/translations/zh_CN/PCI/pci.rst  |  2 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_irq.c       |  2 +-
>  drivers/infiniband/hw/qib/qib_iba7220.c       |  2 +-
>  drivers/infiniband/hw/qib/qib_iba7322.c       |  5 ++-
>  drivers/infiniband/hw/qib/qib_pcie.c          |  2 +-
>  .../infiniband/hw/vmw_pvrdma/pvrdma_main.c    |  2 +-
>  drivers/mfd/intel-lpss-pci.c                  |  2 +-
>  drivers/misc/vmw_vmci/vmci_guest.c            |  3 +-
>  drivers/net/ethernet/amd/xgbe/xgbe-pci.c      |  2 +-
>  .../net/ethernet/aquantia/atlantic/aq_cfg.h   |  2 +-
>  .../net/ethernet/aquantia/atlantic/aq_hw.h    |  2 +-
>  .../net/ethernet/aquantia/atlantic/aq_nic.c   |  2 +-
>  .../ethernet/aquantia/atlantic/aq_pci_func.c  |  9 ++---
>  .../aquantia/atlantic/hw_atl/hw_atl_a0.c      |  2 +-
>  .../aquantia/atlantic/hw_atl/hw_atl_b0.c      |  2 +-
>  .../aquantia/atlantic/hw_atl2/hw_atl2.c       |  2 +-
>  drivers/net/ethernet/atheros/alx/main.c       |  2 +-
>  drivers/net/ethernet/realtek/r8169_main.c     |  2 +-
>  drivers/net/ethernet/wangxun/libwx/wx_lib.c   |  8 ++---
>  drivers/net/wireless/ath/ath10k/ahb.c         | 18 +++++-----
>  drivers/net/wireless/ath/ath10k/pci.c         | 36 +++++++++----------
>  drivers/net/wireless/ath/ath10k/pci.h         |  6 ++--
>  drivers/net/wireless/realtek/rtw88/pci.c      |  2 +-
>  drivers/net/wireless/realtek/rtw89/pci.c      |  2 +-
>  drivers/ntb/hw/idt/ntb_hw_idt.c               |  2 +-
>  drivers/pci/msi/api.c                         |  8 ++---
>  drivers/pci/pcie/portdrv.c                    |  8 ++---
>  drivers/platform/x86/intel_ips.c              |  2 +-
>  drivers/scsi/arcmsr/arcmsr_hba.c              |  2 +-
>  drivers/scsi/hpsa.c                           |  2 +-
>  drivers/scsi/ipr.c                            |  2 +-
>  drivers/scsi/megaraid/megaraid_sas_base.c     |  4 +--
>  drivers/scsi/mpt3sas/mpt3sas_base.c           |  2 +-
>  drivers/scsi/pmcraid.c                        |  2 +-
>  drivers/scsi/vmw_pvscsi.c                     |  2 +-
>  drivers/tty/serial/8250/8250_pci.c            |  2 +-
>  drivers/usb/core/hcd-pci.c                    |  3 +-
>  include/linux/pci.h                           |  7 ++--
>  sound/soc/intel/avs/core.c                    |  2 +-
>  42 files changed, 84 insertions(+), 91 deletions(-)

I applied all these to pci/enumeration for v6.10, thanks!

I added acks and reviewed-by and will update if we receive more, and
adjusted subject lines to add "... instead of PCI_IRQ_LEGACY" and in
some cases to match history of the file.

Bjorn
Martin K. Petersen March 25, 2024, 10:41 p.m. UTC | #4
Damien,

> This patch series removes the use of the depracated PCI_IRQ_LEGACY macro
> and replace it with PCI_IRQ_INTX. No functional change.

SCSI changes look good to me.

Acked-by: Martin K. Petersen <martin.petersen@oracle.com>