mbox series

[v2,0/5] powerpc: struct bus_type cleanup

Message ID 20240212-bus_cleanup-powerpc2-v2-0-8441b3f77827@marliere.net (mailing list archive)
Headers show
Series powerpc: struct bus_type cleanup | expand

Message

Ricardo B. Marliere Feb. 12, 2024, 8:04 p.m. UTC
This series is part of an effort to cleanup the users of the driver
core, as can be seen in many recent patches authored by Greg across the
tree (e.g. [1]). Patch 1/5 is a prerequisite to 2/5, but the others have
no dependency. They were built using bootlin's without warnings using
powerpc64le-power8--glibc--stable-2023.11-1 toolchain.

---
[1]: https://lore.kernel.org/lkml/?q=f%3Agregkh%40linuxfoundation.org+s%3A%22make%22+and+s%3A%22const%22

Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Ricardo B. Marliere <ricardo@marliere.net>

---
Changes in v2:
- Added a new patch to make macio_bus_type const.
- Improved changelogs to remove the word "Now".
- Fixed a build error: https://lore.kernel.org/oe-kbuild-all/202402102142.uphiKeqw-lkp@intel.com/
- Link to v1: https://lore.kernel.org/r/20240209-bus_cleanup-powerpc2-v1-0-79a56dcaebb1@marliere.net

---
Ricardo B. Marliere (5):
      powerpc: vio: move device attributes into a new ifdef
      powerpc: vio: make vio_bus_type const
      powerpc: mpic: make mpic_subsys const
      powerpc: pmac: make macio_bus_type const
      powerpc: ibmebus: make ibmebus_bus_type const

 arch/powerpc/include/asm/ibmebus.h       |  2 +-
 arch/powerpc/include/asm/macio.h         |  2 +-
 arch/powerpc/include/asm/mpic.h          |  2 +-
 arch/powerpc/include/asm/vio.h           |  2 +-
 arch/powerpc/platforms/pseries/ibmebus.c |  4 +--
 arch/powerpc/platforms/pseries/vio.c     | 61 ++++++++++++++++++--------------
 arch/powerpc/sysdev/mpic.c               |  2 +-
 drivers/macintosh/macio_asic.c           |  2 +-
 8 files changed, 43 insertions(+), 34 deletions(-)
---
base-commit: 41bccc98fb7931d63d03f326a746ac4d429c1dd3
change-id: 20240209-bus_cleanup-powerpc2-498426fccb98

Best regards,

Comments

Christophe Leroy Feb. 13, 2024, 8:05 a.m. UTC | #1
Hi

Le 12/02/2024 à 21:04, Ricardo B. Marliere a écrit :
> This series is part of an effort to cleanup the users of the driver
> core, as can be seen in many recent patches authored by Greg across the
> tree (e.g. [1]). Patch 1/5 is a prerequisite to 2/5, but the others have
> no dependency. They were built using bootlin's without warnings using
> powerpc64le-power8--glibc--stable-2023.11-1 toolchain.
> 
> ---
> [1]: https://lore.kernel.org/lkml/?q=f%3Agregkh%40linuxfoundation.org+s%3A%22make%22+and+s%3A%22const%22
> 
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Signed-off-by: Ricardo B. Marliere <ricardo@marliere.net>
> 
> ---
> Changes in v2:
> - Added a new patch to make macio_bus_type const.
> - Improved changelogs to remove the word "Now".
> - Fixed a build error: https://lore.kernel.org/oe-kbuild-all/202402102142.uphiKeqw-lkp@intel.com/
> - Link to v1: https://lore.kernel.org/r/20240209-bus_cleanup-powerpc2-v1-0-79a56dcaebb1@marliere.net

I see another series with the same name, does this v2 also superseeds it 
? https://patchwork.ozlabs.org/project/linuxppc-dev/list/?series=393570

Christophe

> 
> ---
> Ricardo B. Marliere (5):
>        powerpc: vio: move device attributes into a new ifdef
>        powerpc: vio: make vio_bus_type const
>        powerpc: mpic: make mpic_subsys const
>        powerpc: pmac: make macio_bus_type const
>        powerpc: ibmebus: make ibmebus_bus_type const
> 
>   arch/powerpc/include/asm/ibmebus.h       |  2 +-
>   arch/powerpc/include/asm/macio.h         |  2 +-
>   arch/powerpc/include/asm/mpic.h          |  2 +-
>   arch/powerpc/include/asm/vio.h           |  2 +-
>   arch/powerpc/platforms/pseries/ibmebus.c |  4 +--
>   arch/powerpc/platforms/pseries/vio.c     | 61 ++++++++++++++++++--------------
>   arch/powerpc/sysdev/mpic.c               |  2 +-
>   drivers/macintosh/macio_asic.c           |  2 +-
>   8 files changed, 43 insertions(+), 34 deletions(-)
> ---
> base-commit: 41bccc98fb7931d63d03f326a746ac4d429c1dd3
> change-id: 20240209-bus_cleanup-powerpc2-498426fccb98
> 
> Best regards,
Ricardo B. Marliere Feb. 13, 2024, 9:36 a.m. UTC | #2
Hi Christophe,

On 13 Feb 08:05, Christophe Leroy wrote:
> Hi
> 
> Le 12/02/2024 à 21:04, Ricardo B. Marliere a écrit :
> > This series is part of an effort to cleanup the users of the driver
> > core, as can be seen in many recent patches authored by Greg across the
> > tree (e.g. [1]). Patch 1/5 is a prerequisite to 2/5, but the others have
> > no dependency. They were built using bootlin's without warnings using
> > powerpc64le-power8--glibc--stable-2023.11-1 toolchain.
> > 
> > ---
> > [1]: https://lore.kernel.org/lkml/?q=f%3Agregkh%40linuxfoundation.org+s%3A%22make%22+and+s%3A%22const%22
> > 
> > Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > Signed-off-by: Ricardo B. Marliere <ricardo@marliere.net>
> > 
> > ---
> > Changes in v2:
> > - Added a new patch to make macio_bus_type const.
> > - Improved changelogs to remove the word "Now".
> > - Fixed a build error: https://lore.kernel.org/oe-kbuild-all/202402102142.uphiKeqw-lkp@intel.com/
> > - Link to v1: https://lore.kernel.org/r/20240209-bus_cleanup-powerpc2-v1-0-79a56dcaebb1@marliere.net
> 
> I see another series with the same name, does this v2 also superseeds it 
> ? https://patchwork.ozlabs.org/project/linuxppc-dev/list/?series=393570

Yes, please disregard the other one!

Thank you,
-	Ricardo.


> 
> Christophe
> 
> > 
> > ---
> > Ricardo B. Marliere (5):
> >        powerpc: vio: move device attributes into a new ifdef
> >        powerpc: vio: make vio_bus_type const
> >        powerpc: mpic: make mpic_subsys const
> >        powerpc: pmac: make macio_bus_type const
> >        powerpc: ibmebus: make ibmebus_bus_type const
> > 
> >   arch/powerpc/include/asm/ibmebus.h       |  2 +-
> >   arch/powerpc/include/asm/macio.h         |  2 +-
> >   arch/powerpc/include/asm/mpic.h          |  2 +-
> >   arch/powerpc/include/asm/vio.h           |  2 +-
> >   arch/powerpc/platforms/pseries/ibmebus.c |  4 +--
> >   arch/powerpc/platforms/pseries/vio.c     | 61 ++++++++++++++++++--------------
> >   arch/powerpc/sysdev/mpic.c               |  2 +-
> >   drivers/macintosh/macio_asic.c           |  2 +-
> >   8 files changed, 43 insertions(+), 34 deletions(-)
> > ---
> > base-commit: 41bccc98fb7931d63d03f326a746ac4d429c1dd3
> > change-id: 20240209-bus_cleanup-powerpc2-498426fccb98
> > 
> > Best regards,
Greg KH Feb. 13, 2024, 11:07 a.m. UTC | #3
On Mon, Feb 12, 2024 at 05:04:58PM -0300, Ricardo B. Marliere wrote:
> This series is part of an effort to cleanup the users of the driver
> core, as can be seen in many recent patches authored by Greg across the
> tree (e.g. [1]). Patch 1/5 is a prerequisite to 2/5, but the others have
> no dependency. They were built using bootlin's without warnings using
> powerpc64le-power8--glibc--stable-2023.11-1 toolchain.
> 
> ---
> [1]: https://lore.kernel.org/lkml/?q=f%3Agregkh%40linuxfoundation.org+s%3A%22make%22+and+s%3A%22const%22
> 
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Signed-off-by: Ricardo B. Marliere <ricardo@marliere.net>

Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Michael Ellerman Feb. 15, 2024, 1 p.m. UTC | #4
On Mon, 12 Feb 2024 17:04:58 -0300, Ricardo B. Marliere wrote:
> This series is part of an effort to cleanup the users of the driver
> core, as can be seen in many recent patches authored by Greg across the
> tree (e.g. [1]). Patch 1/5 is a prerequisite to 2/5, but the others have
> no dependency. They were built using bootlin's without warnings using
> powerpc64le-power8--glibc--stable-2023.11-1 toolchain.
> 

Applied to powerpc/next.

[1/5] powerpc: vio: move device attributes into a new ifdef
      https://git.kernel.org/powerpc/c/e15d01277a8bdacf8ac485049d21d450153fa47e
[2/5] powerpc: vio: make vio_bus_type const
      https://git.kernel.org/powerpc/c/565206aaa6528b30df9294e9aafac429e4bc94eb
[3/5] powerpc: mpic: make mpic_subsys const
      https://git.kernel.org/powerpc/c/8e3d0b8d99d708e8262e76313e0436339add80ec
[4/5] powerpc: pmac: make macio_bus_type const
      https://git.kernel.org/powerpc/c/112202f34e56cd475e26b2a461dd856ca7570ef9
[5/5] powerpc: ibmebus: make ibmebus_bus_type const
      https://git.kernel.org/powerpc/c/14ce0dbb562713bc058ad16d281db355757e6ec0

cheers