[PATCH 0/3] Remove regmap_config struct from allocated data

Link Mauve posted 3 patches 1 month, 4 weeks ago
drivers/media/dvb-frontends/m88ds3103.c      |  9 +++++----
drivers/media/dvb-frontends/m88ds3103_priv.h |  1 -
drivers/media/dvb-frontends/rtl2832.c        | 18 ++++++++++--------
drivers/media/dvb-frontends/rtl2832_priv.h   |  1 -
drivers/media/dvb-frontends/ts2020.c         | 15 ++++++++-------
5 files changed, 23 insertions(+), 21 deletions(-)
[PATCH 0/3] Remove regmap_config struct from allocated data
Posted by Link Mauve 1 month, 4 weeks ago
This is a follow-up for eeafd9e12144fb6249e1510f9db8a4aaf8155a0a, after
which I found a bunch of regmap_config structs defined inside of other
structs, even though they either completely static (in which case I put
them in .rodata), or needed only a single dynamic argument (in which
case I put them on the stack of the function using it).

I found only these three instances in the media subsystem, which removes
192 bytes from the allocation of each of these driver instances.

Link Mauve (3):
  media: rtl2832: make regmap_config static const
  media: m88ds3103: make regmap_config static const
  media: ts2020: put regmap_config on the stack

 drivers/media/dvb-frontends/m88ds3103.c      |  9 +++++----
 drivers/media/dvb-frontends/m88ds3103_priv.h |  1 -
 drivers/media/dvb-frontends/rtl2832.c        | 18 ++++++++++--------
 drivers/media/dvb-frontends/rtl2832_priv.h   |  1 -
 drivers/media/dvb-frontends/ts2020.c         | 15 ++++++++-------
 5 files changed, 23 insertions(+), 21 deletions(-)


base-commit: 9a4cdc958dd79fc6c3b20b51a10debec6ca09fec
-- 
2.55.0

Re: [PATCH 0/3] Remove regmap_config struct from allocated data
Posted by Link Mauve 1 week, 4 days ago
Hi,

Is there any interest in this series?  I know it doesn’t improve much,
but it should be easy enough to review.

On Tue, Aug 04, 2026 at 01:46:20PM +0200, Link Mauve wrote:
> This is a follow-up for eeafd9e12144fb6249e1510f9db8a4aaf8155a0a, after
> which I found a bunch of regmap_config structs defined inside of other
> structs, even though they either completely static (in which case I put
> them in .rodata), or needed only a single dynamic argument (in which
> case I put them on the stack of the function using it).
> 
> I found only these three instances in the media subsystem, which removes
> 192 bytes from the allocation of each of these driver instances.
> 
> Link Mauve (3):
>   media: rtl2832: make regmap_config static const
>   media: m88ds3103: make regmap_config static const
>   media: ts2020: put regmap_config on the stack
> 
>  drivers/media/dvb-frontends/m88ds3103.c      |  9 +++++----
>  drivers/media/dvb-frontends/m88ds3103_priv.h |  1 -
>  drivers/media/dvb-frontends/rtl2832.c        | 18 ++++++++++--------
>  drivers/media/dvb-frontends/rtl2832_priv.h   |  1 -
>  drivers/media/dvb-frontends/ts2020.c         | 15 ++++++++-------
>  5 files changed, 23 insertions(+), 21 deletions(-)
> 
> 
> base-commit: 9a4cdc958dd79fc6c3b20b51a10debec6ca09fec
> -- 
> 2.55.0
> 

-- 
Link Mauve