Re: [PATCH] ASoC: rt722-sdca: add FU06 Playback Switch for speaker mute control

[Charles Keepax <ckeepax@opensource.cirrus.com>]

On Mon, Apr 27, 2026 at 04:53:20PM +0200, Pierre-Louis Bossart wrote:
> On 4/24/26 17:53, Charles Keepax wrote:
> > On Fri, Apr 24, 2026 at 03:09:10PM +0200, Pierre-Louis Bossart wrote:
> >> On 4/24/26 05:52, Aaron Ma wrote:
> >>> On Thu, Apr 23, 2026 at 9:01 PM Pierre-Louis Bossart
> >>> <pierre-louis.bossart@linux.dev> wrote:
> >>>> On 4/23/26 12:13, Aaron Ma wrote:
> Yeah I can't say I have a good view either...
> 
> In theory at the Function level there is a Commit Group mask,
> a Function can be a member of multiple groups. The fun part
> is this statement in the spec
> 
> "This Control selects the Commit Group(s) which contain every
> dual-ranked Control within this Function."
> 
> but then the SoundWire spec says
> 
> "Every Dual-Ranked Register is associated with one or more of
> 7 Commit Groups"
> 
> interesting wording, I am in the dark on how we would know
> which controls are part of which Commit Group.

There is "Atomic Behaviors Using Dual-Ranked Controls and Commit"
10.3.8 (in 1.0), 13.4 (1.1), which gives some examples of how one
could group stuff. Although its unclear to me how much these are
"recommendations" and how much they are an exhaustive list.

> And then IIRC there was a desire to update volume controls on
> multiple devices, e.g. if there are multiple amps the volumes
> would be updated across devices. Not sure how to achieve
> this...

phew... yeah I hadn't even remotely considered commit groups
spanning functions or indeed even devices. That is indeed some
scary stuff.

Thanks,
Charles