Re: [PATCH 00/10] Add SDCA DisCo parsing support

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

On 2/5/25 05:37, Charles Keepax wrote:
> The MIPI SoundWire Device Class for Audio (SDCA) specification defines
> most details of the hardware in ACPI using the MIPI Discovery and
> Configuration (DisCo) specification. This patch chain adds support for
> parsing most of this information into the kernel such that future work
> can make use of it to construct CODEC devices and soundcards.
> 
> The most notable outstanding work here, is parsing the separate
> properties for the Control Numbers (roughly equivalent to channels)
> within an individual Control. The separate Control Numbers are
> supported but currently only the scheme were a single default etc. is
> supplied for all.  This should not be super hard to add in the future
> but isn't currently required by any of the hardware I am working to
> support.
> 
> Thanks,
> Charles
> 
> Charles Keepax (8):
>   ASoC: SDCA: Minor formatting and naming tweaks
>   ASoC: SDCA: Add support for Entity 0
>   ASoC: SDCA: Add SDCA Control parsing
>   ASoC: SDCA: Add parsing for Control range structures
>   ASoC: SDCA: Add Channel Cluster parsing
>   ASoC: SDCA: Add support for IT/OT Entity properties
>   ASoC: SDCA: Add support for clock Entity properties
>   ASoC: SDCA: Add support for PDE Entity properties
> 
> Pierre-Louis Bossart (2):
>   ASoC: SDCA: Add code to parse Function information
>   ASoC: SDCA: Parse initialization write table

Thanks Charles for extending my initial draft, LGTM.

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

For the record all the definitions in this patchset are extracted from the SDCA public version available at https://www.mipi.org/mipi-sdca-v1-0-download

> 
>  include/sound/sdca.h            |   22 +-
>  include/sound/sdca_function.h   | 1089 +++++++++++++++++++++++-
>  sound/soc/sdca/sdca_device.c    |    3 +-
>  sound/soc/sdca/sdca_functions.c | 1407 ++++++++++++++++++++++++++++++-
>  4 files changed, 2480 insertions(+), 41 deletions(-)
>