[v3] regcache: Split out ->populate() and use it

[PATCH v3 0/5] regcache: Split out ->populate() and use it

Posted by Andy Shevchenko 3 months, 1 week ago

This is a refactoring series to decouple cache initialisation and population.
On its own it has no functional impact but will be used in the further
development. Besides that I found this split useful on its own (from the design
perspective). That's why I decided to send it out as is separately from a bigger
(and ongoing) work.

Changelog v3:
- added missing space (Mark)
- collected tags (Sander)

Changelog v2:
v2: https://lore.kernel.org/r/20251030173915.3886882-1-andriy.shevchenko@linux.intel.com
- rebased on top of the recent regcache changes (Mark)
- a new clean up patch induced by the previous

v1: https://lore.kernel.org/r/20251029073131.3004660-1-andriy.shevchenko@linux.intel.com

Andy Shevchenko (5):
  regcache: Add ->populate() callback to separate from ->init()
  regcache: rbtree: Split ->populate() from ->init()
  regcache: flat: Remove unneeded check and error message for -ENOMEM
  regcache: flat: Split ->populate() from ->init()
  regcache: maple: Split ->populate() from ->init()

 drivers/base/regmap/internal.h        |  1 +
 drivers/base/regmap/regcache-flat.c   | 37 +++++++++++----------
 drivers/base/regmap/regcache-maple.c  | 47 ++++++++++++---------------
 drivers/base/regmap/regcache-rbtree.c | 31 ++++++++++--------
 drivers/base/regmap/regcache.c        | 16 +++++++++
 5 files changed, 74 insertions(+), 58 deletions(-)

-- 
2.50.1

Re: [PATCH v3 0/5] regcache: Split out ->populate() and use it

Posted by Mark Brown 3 months, 1 week ago

On Fri, 31 Oct 2025 09:03:15 +0100, Andy Shevchenko wrote:
> This is a refactoring series to decouple cache initialisation and population.
> On its own it has no functional impact but will be used in the further
> development. Besides that I found this split useful on its own (from the design
> perspective). That's why I decided to send it out as is separately from a bigger
> (and ongoing) work.
> 
> Changelog v3:
> - added missing space (Mark)
> - collected tags (Sander)
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regmap.git for-next

Thanks!

[1/5] regcache: Add ->populate() callback to separate from ->init()
      commit: 94a3a95f03154d8d4c6206950a7f6ef9a30baec6
[2/5] regcache: rbtree: Split ->populate() from ->init()
      commit: bda6f8749c8e0b10f083dc7a1edf169f349fb776
[3/5] regcache: flat: Remove unneeded check and error message for -ENOMEM
      commit: 27fef3048fe95934f6f2f87341eb33ef6581a075
[4/5] regcache: flat: Split ->populate() from ->init()
      commit: 44c1a444b030647803d900e60f5a8af31a782f0e
[5/5] regcache: maple: Split ->populate() from ->init()
      commit: ed5d499b5c9cc11dd3edae1a7a55db7dfa4f1bdc

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark