On 04/02/2024 09:47, Laurent Pinchart wrote: > Hi Ricardo, > > Thank you for the patch. > > On Sat, Feb 03, 2024 at 12:31:26PM -0300, Ricardo B. Marliere wrote: >> Now that the driver core can properly handle constant struct bus_type, >> move the cec_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@xxxxxxxxxxxxxxxxxxx> >> Suggested-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> >> Signed-off-by: Ricardo B. Marliere <ricardo@xxxxxxxxxxxx> > > Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx> > > Hans, I assume you'll pick this patch. Could you also pick 2/2 from the > series ? Of course! Hans > >> --- >> drivers/media/cec/core/cec-core.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/media/cec/core/cec-core.c b/drivers/media/cec/core/cec-core.c >> index 7e153c5cad04..5a54db839e5d 100644 >> --- a/drivers/media/cec/core/cec-core.c >> +++ b/drivers/media/cec/core/cec-core.c >> @@ -93,7 +93,7 @@ static void cec_devnode_release(struct device *cd) >> cec_delete_adapter(to_cec_adapter(devnode)); >> } >> >> -static struct bus_type cec_bus_type = { >> +static const struct bus_type cec_bus_type = { >> .name = CEC_NAME, >> }; >> >