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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <f5f19a27-aa89-4e8c-8e11-6fb85588d015@linux.dev>
Date: Fri, 7 Feb 2025 11:14:48 -0600
From: Pierre-Louis Bossart <pierre-louis.bossart@...ux.dev>
To: Charles Keepax <ckeepax@...nsource.cirrus.com>, broonie@...nel.org
Cc: lgirdwood@...il.com, yung-chuan.liao@...ux.intel.com,
 peter.ujfalusi@...ux.intel.com, linux-sound@...r.kernel.org,
 linux-kernel@...r.kernel.org, patches@...nsource.cirrus.com
Subject: Re: [PATCH 00/10] Add SDCA DisCo parsing support

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@...ux.dev>

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(-)
>