On Thu, Jun 27, 2024 at 09:46:29AM +0200, Krzysztof Kozlowski wrote: > On 27/06/2024 05:52, Kuninori Morimoto wrote: > > Current simple-audio-mux selects MUX by "Input 1" or "Input 2", > > it is not user friendly. Adds new "mux-names" property and enable to > > select MUX by own names. > > + mux-names: > > + description: | > > + Name of multiplexers. default is "Input 1", "Input 2" > I have troubles with this binding... It seems driver expects only one > GPIO, but the binding allows any number. Similarly mux-names... > Anyway, this does not look like hardware description but rather > configuration of driver. What's wrong with input 1 or mux 1 or whatever > is there for default? I would expect that when this driver is deployed it would mainly be for selecting between things like microphone inputs that are defined and labelled as part of the system hardware design. Using numbered inputs would *work* but it's not great for usability so I do see a use case for labels. Possibly we could figure something out by walking the graph but that seems quite hard, possibly unreasonably so.
Attachment:
signature.asc
Description: PGP signature
- Follow-Ups:
- Re: [PATCH 2/2] ASoC: dt-bindings: simple-audio-mux: add mux-names property
- From: Kuninori Morimoto
- Re: [PATCH 2/2] ASoC: dt-bindings: simple-audio-mux: add mux-names property
- References:
- [PATCH 0/2] ASoC: simple-audio-mux: add mux-name
- From: Kuninori Morimoto
- [PATCH 2/2] ASoC: dt-bindings: simple-audio-mux: add mux-names property
- From: Kuninori Morimoto
- Re: [PATCH 2/2] ASoC: dt-bindings: simple-audio-mux: add mux-names property
- From: Krzysztof Kozlowski
- [PATCH 0/2] ASoC: simple-audio-mux: add mux-name
- Prev by Date: Re: [PATCH 2/2] ASoC: rt712-sdca: add the function for version B
- Next by Date: [PATCH 0/6] ASoC: codecs: wsa88xx: add support for static port mapping.
- Previous by thread: Re: [PATCH 2/2] ASoC: dt-bindings: simple-audio-mux: add mux-names property
- Next by thread: Re: [PATCH 2/2] ASoC: dt-bindings: simple-audio-mux: add mux-names property
- Index(es):