Re: [PATCH 0/4] Add SDCA register map support

[Pierre-Louis Bossart <pierre-louis.bossart@linux.dev>]

On 2/17/25 08:01, Charles Keepax wrote:
> This series is the next step of adding SDCA support. Here we add
> helper functions to allow drivers to easily use the SDCA DisCo
> information to create a register map for the device.

Can you remind me where we ended-up in the discussion on one regmap per physical device or one regmap per function?

The DisCo definition are all function-centric, but the physical SoundWire bus access for all read/writes is shared by all functions so having a single regmap isn't silly either.

> The basic idea here is the code takes the list of SDCA controls parsed
> from DisCo and uses primarily the Access Mode to determine if the
> register should be marked as readable/writable etc. Further more
> some additional concepts such as DisCo Constants and Defaults are
> handled. There is some potential confusion, as DisCo Constants are
> handled as an entry in the regmap defaults table, whereas a DisCo
> Default is simply handled as a write to the register. Alas the naming
> confusion is an unavoidable result of the slight impedance mismatch
> between the two systems.
> 
> Thanks,
> Charles
> 
> Charles Keepax (4):
>   regcache: Add support for sorting defaults arrays
>   ASoC: SDCA: Add generic regmap SDCA helpers
>   ASoC: SDCA: Add regmap helpers for parsing for DisCo Constant values
>   ASoC: SDCA: Add helper to write out defaults and fixed values

The code looks fine, thanks!

Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.dev>

> 
>  drivers/base/regmap/regcache.c |  31 ++++
>  include/linux/regmap.h         |   7 +
>  include/sound/sdca_regmap.h    |  31 ++++
>  sound/soc/sdca/Makefile        |   2 +-
>  sound/soc/sdca/sdca_regmap.c   | 321 +++++++++++++++++++++++++++++++++
>  5 files changed, 391 insertions(+), 1 deletion(-)
>  create mode 100644 include/sound/sdca_regmap.h
>  create mode 100644 sound/soc/sdca/sdca_regmap.c
>