[PATCH 0/4] PCI: pci_printk() removal (+ related cleanups)

Ilpo Järvinen posted 4 patches 1 year, 1 month ago
drivers/pci/hotplug/shpchp.h      | 18 +-----------------
drivers/pci/hotplug/shpchp_core.c | 13 +------------
drivers/pci/hotplug/shpchp_hpc.c  |  2 +-
drivers/pci/pcie/aer.c            | 10 +++++++---
include/linux/pci.h               |  3 ---
5 files changed, 10 insertions(+), 36 deletions(-)
[PATCH 0/4] PCI: pci_printk() removal (+ related cleanups)
Posted by Ilpo Järvinen 1 year, 1 month ago
include/linux/pci.h provides pci_printk() which is a low-level
interface with level that is only useful for AER due to error severity
variations.

This series cleans up shpchp logging wrappers to avoid using low-level
pci_printk() unnecessarily and replaces pci_printk() with aer_printk().

Ilpo Järvinen (4):
  PCI: shpchp: Remove logging from module init/exit functions
  PCI: shpchp: Change dbg() -> ctrl_dbg()
  PCI: shpchp: Cleanup logging and debug wrappers
  PCI: Descope pci_printk() to aer_printk()

 drivers/pci/hotplug/shpchp.h      | 18 +-----------------
 drivers/pci/hotplug/shpchp_core.c | 13 +------------
 drivers/pci/hotplug/shpchp_hpc.c  |  2 +-
 drivers/pci/pcie/aer.c            | 10 +++++++---
 include/linux/pci.h               |  3 ---
 5 files changed, 10 insertions(+), 36 deletions(-)

-- 
2.39.5

Re: [PATCH 0/4] PCI: pci_printk() removal (+ related cleanups)
Posted by Bjorn Helgaas 12 months ago
On Mon, Dec 16, 2024 at 06:10:08PM +0200, Ilpo Järvinen wrote:
> include/linux/pci.h provides pci_printk() which is a low-level
> interface with level that is only useful for AER due to error severity
> variations.
> 
> This series cleans up shpchp logging wrappers to avoid using low-level
> pci_printk() unnecessarily and replaces pci_printk() with aer_printk().
> 
> Ilpo Järvinen (4):
>   PCI: shpchp: Remove logging from module init/exit functions
>   PCI: shpchp: Change dbg() -> ctrl_dbg()

I applied the above to pci/hotplug for v6.15, thanks!

>   PCI: shpchp: Cleanup logging and debug wrappers
>   PCI: Descope pci_printk() to aer_printk()
> 
>  drivers/pci/hotplug/shpchp.h      | 18 +-----------------
>  drivers/pci/hotplug/shpchp_core.c | 13 +------------
>  drivers/pci/hotplug/shpchp_hpc.c  |  2 +-
>  drivers/pci/pcie/aer.c            | 10 +++++++---
>  include/linux/pci.h               |  3 ---
>  5 files changed, 10 insertions(+), 36 deletions(-)
> 
> -- 
> 2.39.5
>