* [PATCH v2 1/5] powerpc: vio: move device attributes into a new ifdef
2024-02-12 20:04 [PATCH v2 0/5] powerpc: struct bus_type cleanup Ricardo B. Marliere
@ 2024-02-12 20:04 ` Ricardo B. Marliere
2024-02-12 20:05 ` [PATCH v2 2/5] powerpc: vio: make vio_bus_type const Ricardo B. Marliere
` (6 subsequent siblings)
7 siblings, 0 replies; 10+ messages in thread
From: Ricardo B. Marliere @ 2024-02-12 20:04 UTC (permalink / raw)
To: Michael Ellerman, Nicholas Piggin, Christophe Leroy,
Aneesh Kumar K.V, Naveen N. Rao
Cc: Greg Kroah-Hartman, Ricardo B. Marliere, linuxppc-dev,
linux-kernel
In order to make the distinction of the vio_bus_type variable based on
CONFIG_PPC_SMLPAR more explicit, move the required structs into a new
ifdef block. This is needed in order to make vio_bus_type const and
because the distinction is made explicit, there is no need to set the
fields within the vio_cmo_sysfs_init function.
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Ricardo B. Marliere <ricardo@marliere.net>
---
arch/powerpc/platforms/pseries/vio.c | 59 +++++++++++++++++++++---------------
1 file changed, 34 insertions(+), 25 deletions(-)
diff --git a/arch/powerpc/platforms/pseries/vio.c b/arch/powerpc/platforms/pseries/vio.c
index 2dc9cbc4bcd8..6c58824190a2 100644
--- a/arch/powerpc/platforms/pseries/vio.c
+++ b/arch/powerpc/platforms/pseries/vio.c
@@ -991,18 +991,6 @@ static DEVICE_ATTR_RO(cmo_allocated);
static DEVICE_ATTR_RW(cmo_desired);
static DEVICE_ATTR_RW(cmo_allocs_failed);
-static struct attribute *vio_cmo_dev_attrs[] = {
- &dev_attr_name.attr,
- &dev_attr_devspec.attr,
- &dev_attr_modalias.attr,
- &dev_attr_cmo_entitled.attr,
- &dev_attr_cmo_allocated.attr,
- &dev_attr_cmo_desired.attr,
- &dev_attr_cmo_allocs_failed.attr,
- NULL,
-};
-ATTRIBUTE_GROUPS(vio_cmo_dev);
-
/* sysfs bus functions and data structures for CMO */
#define viobus_cmo_rd_attr(name) \
@@ -1062,11 +1050,7 @@ static struct attribute *vio_bus_attrs[] = {
};
ATTRIBUTE_GROUPS(vio_bus);
-static void __init vio_cmo_sysfs_init(void)
-{
- vio_bus_type.dev_groups = vio_cmo_dev_groups;
- vio_bus_type.bus_groups = vio_bus_groups;
-}
+static void __init vio_cmo_sysfs_init(void) { }
#else /* CONFIG_PPC_SMLPAR */
int vio_cmo_entitlement_update(size_t new_entitlement) { return 0; }
void vio_cmo_set_dev_desired(struct vio_dev *viodev, size_t desired) {}
@@ -1584,14 +1568,6 @@ static ssize_t modalias_show(struct device *dev, struct device_attribute *attr,
}
static DEVICE_ATTR_RO(modalias);
-static struct attribute *vio_dev_attrs[] = {
- &dev_attr_name.attr,
- &dev_attr_devspec.attr,
- &dev_attr_modalias.attr,
- NULL,
-};
-ATTRIBUTE_GROUPS(vio_dev);
-
void vio_unregister_device(struct vio_dev *viodev)
{
device_unregister(&viodev->dev);
@@ -1626,6 +1602,38 @@ static int vio_hotplug(const struct device *dev, struct kobj_uevent_env *env)
return 0;
}
+#ifdef CONFIG_PPC_SMLPAR
+static struct attribute *vio_cmo_dev_attrs[] = {
+ &dev_attr_name.attr,
+ &dev_attr_devspec.attr,
+ &dev_attr_modalias.attr,
+ &dev_attr_cmo_entitled.attr,
+ &dev_attr_cmo_allocated.attr,
+ &dev_attr_cmo_desired.attr,
+ &dev_attr_cmo_allocs_failed.attr,
+ NULL,
+};
+ATTRIBUTE_GROUPS(vio_cmo_dev);
+
+struct bus_type vio_bus_type = {
+ .name = "vio",
+ .dev_groups = vio_cmo_dev_groups,
+ .bus_groups = vio_bus_groups,
+ .uevent = vio_hotplug,
+ .match = vio_bus_match,
+ .probe = vio_bus_probe,
+ .remove = vio_bus_remove,
+ .shutdown = vio_bus_shutdown,
+};
+#else /* CONFIG_PPC_SMLPAR */
+static struct attribute *vio_dev_attrs[] = {
+ &dev_attr_name.attr,
+ &dev_attr_devspec.attr,
+ &dev_attr_modalias.attr,
+ NULL,
+};
+ATTRIBUTE_GROUPS(vio_dev);
+
struct bus_type vio_bus_type = {
.name = "vio",
.dev_groups = vio_dev_groups,
@@ -1635,6 +1643,7 @@ struct bus_type vio_bus_type = {
.remove = vio_bus_remove,
.shutdown = vio_bus_shutdown,
};
+#endif /* CONFIG_PPC_SMLPAR */
/**
* vio_get_attribute: - get attribute for virtual device
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH v2 2/5] powerpc: vio: make vio_bus_type const
2024-02-12 20:04 [PATCH v2 0/5] powerpc: struct bus_type cleanup Ricardo B. Marliere
2024-02-12 20:04 ` [PATCH v2 1/5] powerpc: vio: move device attributes into a new ifdef Ricardo B. Marliere
@ 2024-02-12 20:05 ` Ricardo B. Marliere
2024-02-12 20:05 ` [PATCH v2 3/5] powerpc: mpic: make mpic_subsys const Ricardo B. Marliere
` (5 subsequent siblings)
7 siblings, 0 replies; 10+ messages in thread
From: Ricardo B. Marliere @ 2024-02-12 20:05 UTC (permalink / raw)
To: Michael Ellerman, Nicholas Piggin, Christophe Leroy,
Aneesh Kumar K.V, Naveen N. Rao
Cc: Greg Kroah-Hartman, Ricardo B. Marliere, linuxppc-dev,
linux-kernel
Since commit d492cc2573a0 ("driver core: device.h: make struct
bus_type a const *"), the driver core can properly handle constant
struct bus_type, move the vio_bus_type variable to be a constant
structure as well, placing it into read-only memory which can not be
modified at runtime.
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Suggested-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Ricardo B. Marliere <ricardo@marliere.net>
---
arch/powerpc/include/asm/vio.h | 2 +-
arch/powerpc/platforms/pseries/vio.c | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/powerpc/include/asm/vio.h b/arch/powerpc/include/asm/vio.h
index cc9b787627ad..6faf2a931755 100644
--- a/arch/powerpc/include/asm/vio.h
+++ b/arch/powerpc/include/asm/vio.h
@@ -39,7 +39,7 @@
*/
#define VIO_CMO_MIN_ENT 1562624
-extern struct bus_type vio_bus_type;
+extern const struct bus_type vio_bus_type;
struct iommu_table;
diff --git a/arch/powerpc/platforms/pseries/vio.c b/arch/powerpc/platforms/pseries/vio.c
index 6c58824190a2..90ff85c879bf 100644
--- a/arch/powerpc/platforms/pseries/vio.c
+++ b/arch/powerpc/platforms/pseries/vio.c
@@ -1615,7 +1615,7 @@ static struct attribute *vio_cmo_dev_attrs[] = {
};
ATTRIBUTE_GROUPS(vio_cmo_dev);
-struct bus_type vio_bus_type = {
+const struct bus_type vio_bus_type = {
.name = "vio",
.dev_groups = vio_cmo_dev_groups,
.bus_groups = vio_bus_groups,
@@ -1634,7 +1634,7 @@ static struct attribute *vio_dev_attrs[] = {
};
ATTRIBUTE_GROUPS(vio_dev);
-struct bus_type vio_bus_type = {
+const struct bus_type vio_bus_type = {
.name = "vio",
.dev_groups = vio_dev_groups,
.uevent = vio_hotplug,
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH v2 3/5] powerpc: mpic: make mpic_subsys const
2024-02-12 20:04 [PATCH v2 0/5] powerpc: struct bus_type cleanup Ricardo B. Marliere
2024-02-12 20:04 ` [PATCH v2 1/5] powerpc: vio: move device attributes into a new ifdef Ricardo B. Marliere
2024-02-12 20:05 ` [PATCH v2 2/5] powerpc: vio: make vio_bus_type const Ricardo B. Marliere
@ 2024-02-12 20:05 ` Ricardo B. Marliere
2024-02-12 20:05 ` [PATCH v2 4/5] powerpc: pmac: make macio_bus_type const Ricardo B. Marliere
` (4 subsequent siblings)
7 siblings, 0 replies; 10+ messages in thread
From: Ricardo B. Marliere @ 2024-02-12 20:05 UTC (permalink / raw)
To: Michael Ellerman, Nicholas Piggin, Christophe Leroy,
Aneesh Kumar K.V, Naveen N. Rao
Cc: Greg Kroah-Hartman, Ricardo B. Marliere, linuxppc-dev,
linux-kernel
Since commit d492cc2573a0 ("driver core: device.h: make struct
bus_type a const *"), the driver core can properly handle constant
struct bus_type, move the mpic_subsys variable to be a constant
structure as well, placing it into read-only memory which can not be
modified at runtime.
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Suggested-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Ricardo B. Marliere <ricardo@marliere.net>
---
arch/powerpc/include/asm/mpic.h | 2 +-
arch/powerpc/sysdev/mpic.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/powerpc/include/asm/mpic.h b/arch/powerpc/include/asm/mpic.h
index 58353c5bd3fb..0c03a98986cd 100644
--- a/arch/powerpc/include/asm/mpic.h
+++ b/arch/powerpc/include/asm/mpic.h
@@ -336,7 +336,7 @@ struct mpic
#endif
};
-extern struct bus_type mpic_subsys;
+extern const struct bus_type mpic_subsys;
/*
* MPIC flags (passed to mpic_alloc)
diff --git a/arch/powerpc/sysdev/mpic.c b/arch/powerpc/sysdev/mpic.c
index dabbdd356664..d94cf36b0f65 100644
--- a/arch/powerpc/sysdev/mpic.c
+++ b/arch/powerpc/sysdev/mpic.c
@@ -49,7 +49,7 @@
#define DBG(fmt...)
#endif
-struct bus_type mpic_subsys = {
+const struct bus_type mpic_subsys = {
.name = "mpic",
.dev_name = "mpic",
};
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH v2 4/5] powerpc: pmac: make macio_bus_type const
2024-02-12 20:04 [PATCH v2 0/5] powerpc: struct bus_type cleanup Ricardo B. Marliere
` (2 preceding siblings ...)
2024-02-12 20:05 ` [PATCH v2 3/5] powerpc: mpic: make mpic_subsys const Ricardo B. Marliere
@ 2024-02-12 20:05 ` Ricardo B. Marliere
2024-02-12 20:05 ` [PATCH v2 5/5] powerpc: ibmebus: make ibmebus_bus_type const Ricardo B. Marliere
` (3 subsequent siblings)
7 siblings, 0 replies; 10+ messages in thread
From: Ricardo B. Marliere @ 2024-02-12 20:05 UTC (permalink / raw)
To: Michael Ellerman, Nicholas Piggin, Christophe Leroy,
Aneesh Kumar K.V, Naveen N. Rao
Cc: Greg Kroah-Hartman, Ricardo B. Marliere, linuxppc-dev,
linux-kernel
Since commit d492cc2573a0 ("driver core: device.h: make struct
bus_type a const *"), the driver core can properly handle constant
struct bus_type, move the macio_bus_type variable to be a constant
structure as well, placing it into read-only memory which can not be
modified at runtime.
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Suggested-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Ricardo B. Marliere <ricardo@marliere.net>
---
arch/powerpc/include/asm/macio.h | 2 +-
drivers/macintosh/macio_asic.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/powerpc/include/asm/macio.h b/arch/powerpc/include/asm/macio.h
index 3a07c62973aa..ab9608e63e40 100644
--- a/arch/powerpc/include/asm/macio.h
+++ b/arch/powerpc/include/asm/macio.h
@@ -6,7 +6,7 @@
#include <linux/of.h>
#include <linux/platform_device.h>
-extern struct bus_type macio_bus_type;
+extern const struct bus_type macio_bus_type;
/* MacIO device driver is defined later */
struct macio_driver;
diff --git a/drivers/macintosh/macio_asic.c b/drivers/macintosh/macio_asic.c
index a5ee8f736a8e..565f1e21ff7d 100644
--- a/drivers/macintosh/macio_asic.c
+++ b/drivers/macintosh/macio_asic.c
@@ -136,7 +136,7 @@ static int macio_device_modalias(const struct device *dev, struct kobj_uevent_en
extern const struct attribute_group *macio_dev_groups[];
-struct bus_type macio_bus_type = {
+const struct bus_type macio_bus_type = {
.name = "macio",
.match = macio_bus_match,
.uevent = macio_device_modalias,
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH v2 5/5] powerpc: ibmebus: make ibmebus_bus_type const
2024-02-12 20:04 [PATCH v2 0/5] powerpc: struct bus_type cleanup Ricardo B. Marliere
` (3 preceding siblings ...)
2024-02-12 20:05 ` [PATCH v2 4/5] powerpc: pmac: make macio_bus_type const Ricardo B. Marliere
@ 2024-02-12 20:05 ` Ricardo B. Marliere
2024-02-13 8:05 ` [PATCH v2 0/5] powerpc: struct bus_type cleanup Christophe Leroy
` (2 subsequent siblings)
7 siblings, 0 replies; 10+ messages in thread
From: Ricardo B. Marliere @ 2024-02-12 20:05 UTC (permalink / raw)
To: Michael Ellerman, Nicholas Piggin, Christophe Leroy,
Aneesh Kumar K.V, Naveen N. Rao
Cc: Greg Kroah-Hartman, Ricardo B. Marliere, linuxppc-dev,
linux-kernel
Since commit d492cc2573a0 ("driver core: device.h: make struct
bus_type a const *"), the driver core can properly handle constant
struct bus_type, move the ibmebus_bus_type variable to be a constant
structure as well, placing it into read-only memory which can not be
modified at runtime.
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Suggested-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Ricardo B. Marliere <ricardo@marliere.net>
---
arch/powerpc/include/asm/ibmebus.h | 2 +-
arch/powerpc/platforms/pseries/ibmebus.c | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/powerpc/include/asm/ibmebus.h b/arch/powerpc/include/asm/ibmebus.h
index 6f33253a364a..46fe406f461c 100644
--- a/arch/powerpc/include/asm/ibmebus.h
+++ b/arch/powerpc/include/asm/ibmebus.h
@@ -48,7 +48,7 @@
struct platform_driver;
-extern struct bus_type ibmebus_bus_type;
+extern const struct bus_type ibmebus_bus_type;
int ibmebus_register_driver(struct platform_driver *drv);
void ibmebus_unregister_driver(struct platform_driver *drv);
diff --git a/arch/powerpc/platforms/pseries/ibmebus.c b/arch/powerpc/platforms/pseries/ibmebus.c
index 998e3aff2457..b401282727a4 100644
--- a/arch/powerpc/platforms/pseries/ibmebus.c
+++ b/arch/powerpc/platforms/pseries/ibmebus.c
@@ -55,7 +55,7 @@ static struct device ibmebus_bus_device = { /* fake "parent" device */
.init_name = "ibmebus",
};
-struct bus_type ibmebus_bus_type;
+const struct bus_type ibmebus_bus_type;
/* These devices will automatically be added to the bus during init */
static const struct of_device_id ibmebus_matches[] __initconst = {
@@ -432,7 +432,7 @@ static int ibmebus_bus_modalias(const struct device *dev, struct kobj_uevent_env
return of_device_uevent_modalias(dev, env);
}
-struct bus_type ibmebus_bus_type = {
+const struct bus_type ibmebus_bus_type = {
.name = "ibmebus",
.uevent = ibmebus_bus_modalias,
.bus_groups = ibmbus_bus_groups,
--
2.43.0
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [PATCH v2 0/5] powerpc: struct bus_type cleanup
2024-02-12 20:04 [PATCH v2 0/5] powerpc: struct bus_type cleanup Ricardo B. Marliere
` (4 preceding siblings ...)
2024-02-12 20:05 ` [PATCH v2 5/5] powerpc: ibmebus: make ibmebus_bus_type const Ricardo B. Marliere
@ 2024-02-13 8:05 ` Christophe Leroy
2024-02-13 9:36 ` Ricardo B. Marliere
2024-02-13 11:07 ` Greg Kroah-Hartman
2024-02-15 13:00 ` Michael Ellerman
7 siblings, 1 reply; 10+ messages in thread
From: Christophe Leroy @ 2024-02-13 8:05 UTC (permalink / raw)
To: Ricardo B. Marliere, Michael Ellerman, Nicholas Piggin,
Aneesh Kumar K.V, Naveen N. Rao
Cc: Greg Kroah-Hartman, linuxppc-dev@lists.ozlabs.org,
linux-kernel@vger.kernel.org
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,
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH v2 0/5] powerpc: struct bus_type cleanup
2024-02-13 8:05 ` [PATCH v2 0/5] powerpc: struct bus_type cleanup Christophe Leroy
@ 2024-02-13 9:36 ` Ricardo B. Marliere
0 siblings, 0 replies; 10+ messages in thread
From: Ricardo B. Marliere @ 2024-02-13 9:36 UTC (permalink / raw)
To: Christophe Leroy
Cc: linux-kernel@vger.kernel.org, Nicholas Piggin, Aneesh Kumar K.V,
Greg Kroah-Hartman, Naveen N. Rao, linuxppc-dev@lists.ozlabs.org
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,
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2 0/5] powerpc: struct bus_type cleanup
2024-02-12 20:04 [PATCH v2 0/5] powerpc: struct bus_type cleanup Ricardo B. Marliere
` (5 preceding siblings ...)
2024-02-13 8:05 ` [PATCH v2 0/5] powerpc: struct bus_type cleanup Christophe Leroy
@ 2024-02-13 11:07 ` Greg Kroah-Hartman
2024-02-15 13:00 ` Michael Ellerman
7 siblings, 0 replies; 10+ messages in thread
From: Greg Kroah-Hartman @ 2024-02-13 11:07 UTC (permalink / raw)
To: Ricardo B. Marliere
Cc: linux-kernel, Aneesh Kumar K.V, Nicholas Piggin, Naveen N. Rao,
linuxppc-dev
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>
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH v2 0/5] powerpc: struct bus_type cleanup
2024-02-12 20:04 [PATCH v2 0/5] powerpc: struct bus_type cleanup Ricardo B. Marliere
` (6 preceding siblings ...)
2024-02-13 11:07 ` Greg Kroah-Hartman
@ 2024-02-15 13:00 ` Michael Ellerman
7 siblings, 0 replies; 10+ messages in thread
From: Michael Ellerman @ 2024-02-15 13:00 UTC (permalink / raw)
To: Nicholas Piggin, Christophe Leroy, Aneesh Kumar K.V,
Naveen N. Rao, Ricardo B. Marliere
Cc: Greg Kroah-Hartman, linuxppc-dev, linux-kernel
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
^ permalink raw reply [flat|nested] 10+ messages in thread