Received-SPF: pass (google.com: domain of linux-kernel+bounces-5762-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
From: "Ertman, David M" <david.m.ertman@intel.com>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, "rafael@kernel.org"
	<rafael@kernel.org>, "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>
CC: "Weiny, Ira" <ira.weiny@intel.com>, William Breathitt Gray
	<william.gray@linaro.org>, David Hildenbrand <david@redhat.com>, "Oscar
 Salvador" <osalvador@suse.de>, Kevin Hilman <khilman@kernel.org>, Ulf Hansson
	<ulf.hansson@linaro.org>, "Brown, Len" <len.brown@intel.com>
Subject: RE: [PATCH] driver core: mark remaining local bus_type variables as
 const
Thread-Topic: [PATCH] driver core: mark remaining local bus_type variables as
 const
Thread-Index: AQHaMpEHnG+TLmy1q0aP6ABzV9Z91bCw5b+Q
Date: Tue, 19 Dec 2023 18:02:33 +0000
Message-ID: <MW5PR11MB5811AB2DDE57E309A329F5BEDD97A@MW5PR11MB5811.namprd11.prod.outlook.com>
References: <2023121908-paver-follow-cc21@gregkh>
In-Reply-To: <2023121908-paver-follow-cc21@gregkh>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?g2dN37qotdC6dqlwybN79TCu9hRin3ivKXuBTyGeT5CYHmqT6nxrMCwjBKjF?=
 =?us-ascii?Q?wDrIC1hnazpPBki4VOd4eAbn/5GucLTDEzdMmqq7QbUQ57qT62o8IPDo3h5H?=
 =?us-ascii?Q?7G+unIr4uIedHLiZAYI8vgVvBwuoWFgL0TOjNLJHOQNrJc7SjMSFJKNbDu3G?=
 =?us-ascii?Q?0bUqWQXdd5E0JUdz/Hm3FlgrKaG0XjR36z6uXiejnbPWJ/zGUEvfrLnjHDNg?=
 =?us-ascii?Q?OFyVrrvWXxUxiwMMKfwspWKItFmJfwHxSNFKj1WY7RuU1ql5GRAsZ679cpZc?=
 =?us-ascii?Q?fIQIjm4eSZi1hjkxFMjiNHZddsfhTiLckgSUIN/+NjK+fARx53+qVYTxyuYl?=
 =?us-ascii?Q?qWizLMNnrrNgw8TQIWqskJjsISqi+LfQpWraPeGo3cyL9IdUEoqvsg77O2RJ?=
 =?us-ascii?Q?CykVFcJaGprdqmflgD55pNVUHvlEcbJZefAxH4Zvu1RQtwoFFISAabjEFNti?=
 =?us-ascii?Q?doW6R5yiSzMK2QZJw88bhiOIuroudyvJqX8FOqrudchHszrv3NupUsfaAi4V?=
 =?us-ascii?Q?T73X/kCH8RYbuu3+6sbbbLgH7PGvnK1Poh7Ywb7QhjnVDQDBFOxswk55SVpP?=
 =?us-ascii?Q?Ib8J7XRcKitHzk9MD7felMmACOWkx0IeMqWD0lvttIZn5Zgc79pa5A3fnsns?=
 =?us-ascii?Q?xwZYNabZ4kgu31IiuNB/INqNhx/lvvjmOqYBp6S9qCr3IXfrgIkXSC+KN/FT?=
 =?us-ascii?Q?oV2DvB3vAracSZD8uf8/FI3t44J8KIO0am8U4LFzKXf/yeuO6bZNVmMiUhpt?=
 =?us-ascii?Q?o1kE1dYvgvYER8cIhch9hBEfRAEzXg+cQXfiwmBFSZD2/bsWbbW83y4Mir+0?=
 =?us-ascii?Q?xX/5cyA0tn6+YE+sf+cvnuR5+NpxL5aLaYlm6D5YAmBARbRIyP15d9w4txW4?=
 =?us-ascii?Q?565n09uI0rbMWsktJgvwROWzFgjJtd3mDrxQN1hV3F/DX26V+TiXRPqP6wBN?=
 =?us-ascii?Q?+6skj6ZmMfN02PJmo6PJ3AX0XIKo/KbjSxgIaxGLvpa4QysBpLeVm0qObwhE?=
 =?us-ascii?Q?152ABYVvaa2yztKjU2KS9TfPRDxS+CZRqvXusDtuz6H1MBvRVmXsqJbUk4Kf?=
 =?us-ascii?Q?YUOs2PSkBGt5v7O8CgmbQ9ow5xdGcIQura35t/P8oSv1CodM1wY9jaKUfXqh?=
 =?us-ascii?Q?DxmidLRP3/EMspAYLmQnkNEX/G+IE8C5XVr6ZhpwZu0FbmOLVoICF3SEogRw?=
 =?us-ascii?Q?X3Kwle6jOkpNHzGdYC3a8L0YEhnZ/QM3mQWtmrdfEcn6vW2yKrzh46eNMB0Y?=
 =?us-ascii?Q?KQbnt3XNbvVfFsI1PTMYwtEeAAFReAIZp1JY64sO37F81HZMUJtmoFbcSWel?=
 =?us-ascii?Q?ylYuUeIYIj/Awg7lPgd7IBdjrh/C6zTShc+6KG6biEF0/IARMmXqagUyu+zC?=
 =?us-ascii?Q?iVTJrore3iBcpNm3+UfDZPM2Im9sl/ZmK0d7BHhavTlYKzVp0mk8zvjno5R2?=
 =?us-ascii?Q?pN9CZ3RErw6+mcDxHfOvXbAAdZ4ZIIpLcUuhbHn3FZGwsHCUoD5ChM1EcBWp?=
 =?us-ascii?Q?/ZXYytRJrDoOTcE5lp3J0/9Yer//t3u2Hz7r74VZ1WJzsdB/jnH/oAcEUu+d?=
 =?us-ascii?Q?LLNPtHr7N1lJy2W5VguZMvzzz7kS3a/CJKjRjT90?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: MW5PR11MB5811.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a0d707c4-f766-4176-565b-08dc00bcad3b
X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Dec 2023 18:02:33.9892
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: PQYueaq/OrUnmFGJPJJwIhVYXfLnoT37Q88NpuyJuhL+s+uEW+J9QE4+j4dyMul76MwvANyByO5V4gzvzOlVshfsIW31lvYwz4MkXyLhY0I=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: LV2PR11MB6000
X-OriginatorOrg: intel.com

> -----Original Message-----
> From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> Sent: Tuesday, December 19, 2023 7:35 AM
> To: rafael@kernel.org; linux-kernel@vger.kernel.org
> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>; Ertman, David M
> <david.m.ertman@intel.com>; Weiny, Ira <ira.weiny@intel.com>; William
> Breathitt Gray <william.gray@linaro.org>; David Hildenbrand
> <david@redhat.com>; Oscar Salvador <osalvador@suse.de>; Kevin Hilman
> <khilman@kernel.org>; Ulf Hansson <ulf.hansson@linaro.org>; Brown, Len
> <len.brown@intel.com>
> Subject: [PATCH] driver core: mark remaining local bus_type variables as
> const
>=20
> Now that the driver core can properly handle constant struct bus_type,
> change the local driver core bus_type variables to be a constant
> structure as well, placing them into read-only memory which can not be
> modified at runtime.
>=20
> Cc: Dave Ertman <david.m.ertman@intel.com>
> Cc: Ira Weiny <ira.weiny@intel.com>
> Cc: "Rafael J. Wysocki" <rafael@kernel.org>
> Cc: William Breathitt Gray <william.gray@linaro.org>
> Cc: David Hildenbrand <david@redhat.com>
> Cc: Oscar Salvador <osalvador@suse.de>
> Cc: Kevin Hilman <khilman@kernel.org>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Cc: Len Brown <len.brown@intel.com>
> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> ---
>  drivers/base/auxiliary.c    | 2 +-
>  drivers/base/isa.c          | 2 +-
>  drivers/base/memory.c       | 2 +-
>  drivers/base/node.c         | 2 +-
>  drivers/base/power/domain.c | 2 +-
>  drivers/base/soc.c          | 2 +-
>  6 files changed, 6 insertions(+), 6 deletions(-)
>=20
> diff --git a/drivers/base/auxiliary.c b/drivers/base/auxiliary.c
> index 4d4c2c8d26c4..d3a2c40c2f12 100644
> --- a/drivers/base/auxiliary.c
> +++ b/drivers/base/auxiliary.c
> @@ -244,7 +244,7 @@ static void auxiliary_bus_shutdown(struct device
> *dev)
>  		auxdrv->shutdown(auxdev);
>  }
>=20
> -static struct bus_type auxiliary_bus_type =3D {
> +static const struct bus_type auxiliary_bus_type =3D {
>  	.name =3D "auxiliary",
>  	.probe =3D auxiliary_bus_probe,
>  	.remove =3D auxiliary_bus_remove,
> diff --git a/drivers/base/isa.c b/drivers/base/isa.c
> index 675ad3139224..e23d0b49a793 100644
> --- a/drivers/base/isa.c
> +++ b/drivers/base/isa.c

LGTM - ACK

DaveE