Re: [PATCH 3/3] soc: ti: Add TDA54 SCM configuration syscon support
From: Nishanth Menon
Date: Wed Oct 21 2026 - 08:08:50 EST
On 12:06-20261021, Moteen Shah wrote:
> I was going by the reviews from Sashiko[0], I had similar concern before
> sharing this patch upstream.
> It would be really great if I can have some suggestions on this.
>
> 1. Have it inbuilt
> 2. Remove the module_exit, make the module unremovable
> 3. Look towards a core syscon fixup?
>
> Option 3 would be to add of_syscon_unregister_regmap(np) and subsequent
> consumer driver changes.
>
> [0]https://sashiko.dev/#/patchset/20261019080736.944127-1-m-shah%40ti.com
My preference for drivers/soc/ti is to have drivers as modules.
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
https://ti.com/opensource