Re: [PATCH 7/7] hwmon: adm1275: Support module auto-loading

[Matti Vaittinen]

On 16/06/2026 17:04, Guenter Roeck wrote:
> On 6/15/26 23:47, Matti Vaittinen wrote:
> > From: Matti Vaittinen <mazziesaccount@xxxxxxxxx>
> >
> > Populating the spi_device_id -table is not enough to make the
> > driver module automatically load when device-tree node for the bd12780
> > is parsed at boot.
> >
> > Adding the of_device_id tables causes the driver module to be
> > automatically load at boot. Testing has been done with rather old Debian
> > system.
> >
> > When inspecting the generated module-aliases with the insmod, following
> > entries seem to be the difference:
> >
> > alias:          of:N*T*Crohm,bd12780C*
> > alias:          of:N*T*Crohm,bd12780
> >
> > I suspect these are required for the module loading to work.
> >
> > Signed-off-by: Matti Vaittinen <mazziesaccount@xxxxxxxxx>
> >
> > ---
> >
> > I did not add of_device_ids for other supported ICs as I can't verify it
> > doesn't cause side-effects. Please let me know if you think those IDs
> > should be added as well. I would be glad if I got more educated opinion
> > on adding the of-IDs :) (I can squash this to 3/7 and 6/7 in next
> > revision, and add own patch for adding of-IDs for other ICs if
> > required).
>
> I don't know what those side effects might be. I am much more concerned
> about side effects of having some of the devices in adm1275_of_match
> and some in adm1275_id. So, yes, please add a patch to provide
> adm1275_of_match for all chips supported by the driver.
It's nice to have an opinion on this as I was really unsure what is the 
right way forward. Thanks for all the help this far. I'll do that in v2.

Yours,
	-- Matti

--
Matti Vaittinen
Linux kernel developer at ROHM Semiconductors
Oulu Finland

~~ When things go utterly wrong vim users can always type :help! ~~