powerpc: pmac: make macio_bus_type const
authorRicardo B. Marliere <ricardo@marliere.net>
Mon, 12 Feb 2024 20:05:02 +0000 (17:05 -0300)
committerMichael Ellerman <mpe@ellerman.id.au>
Wed, 14 Feb 2024 13:14:06 +0000 (00:14 +1100)
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.

Suggested-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: "Ricardo B. Marliere" <ricardo@marliere.net>
Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://msgid.link/20240212-bus_cleanup-powerpc2-v2-4-8441b3f77827@marliere.net
arch/powerpc/include/asm/macio.h
drivers/macintosh/macio_asic.c

index 3a07c62973aab6d7625d50051066ce2cc0929b3f..ab9608e63e40ae9b1035382cb06a0c431e1b1de3 100644 (file)
@@ -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;
index a5ee8f736a8e006af30d372aa6e6343577206428..565f1e21ff7dc0802c8e378a02bc723af84cb58b 100644 (file)
@@ -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,