Hi Srinivas, srinivas.kandagatla@xxxxxxxxxx wrote on Fri, 10 Mar 2023 11:03:45 +0000: > On 08/03/2023 15:31, Miquel Raynal wrote: > > + > > +static int __init sl28vpd_init(void) > > +{ > > + return nvmem_layout_register(&sl28vpd_layout); > > +} > > + > > +static void __exit sl28vpd_exit(void) > > +{ > > + nvmem_layout_unregister(&sl28vpd_layout); > > +} > > + > > +module_init(sl28vpd_init); > > +module_exit(sl28vpd_exit); > Looking at this pattern in two drivers, we could come up with some helper macro like. > > #define module_nvmem_layout_driver(__layout_driver) \ > module_driver(__layout_driver, nvmem_layout_register, \ > nvmem_layout_unregister) Nice idea! I will make a patch; is it okay do it on top? Thanks, Miquèl