linux-kernel - Re: [PATCH v24 10/34] ASoC: usb: Create SOC USB SND jack kcontrol

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <5c882055-85d5-41ba-98bf-da375a490c5b@linux.intel.com>
Date: Tue, 6 Aug 2024 16:51:06 +0200
From: Amadeusz Sławiński
 <amadeuszx.slawinski@...ux.intel.com>
To: Wesley Cheng <quic_wcheng@...cinc.com>,
 Pierre-Louis Bossart <pierre-louis.bossart@...ux.intel.com>,
 srinivas.kandagatla@...aro.org, mathias.nyman@...el.com, perex@...ex.cz,
 conor+dt@...nel.org, corbet@....net, broonie@...nel.org,
 lgirdwood@...il.com, krzk+dt@...nel.org, Thinh.Nguyen@...opsys.com,
 bgoswami@...cinc.com, tiwai@...e.com, gregkh@...uxfoundation.org,
 robh@...nel.org
Cc: linux-kernel@...r.kernel.org, devicetree@...r.kernel.org,
 linux-sound@...r.kernel.org, linux-usb@...r.kernel.org,
 linux-arm-msm@...r.kernel.org, linux-doc@...r.kernel.org,
 alsa-devel@...a-project.org
Subject: Re: [PATCH v24 10/34] ASoC: usb: Create SOC USB SND jack kcontrol

On 8/2/2024 12:43 AM, Wesley Cheng wrote:
> Hi Pierre,
> 
> On 8/1/2024 1:07 AM, Pierre-Louis Bossart wrote:
>>
>>
>>> +static inline int snd_soc_usb_setup_offload_jack(struct snd_soc_component *component,
>>> +						 struct snd_soc_jack *jack)
>>> +{
>>> +	return -ENODEV;
>>> +}
>>> +
>>> +static inline int snd_soc_usb_disable_offload_jack(struct snd_soc_component *component)
>>> +{
>>> +	return -ENODEV;
>>> +}
>> usually fallback functions return 0, is the error code intentional?
> ACK.
>>
>>> +int snd_soc_usb_setup_offload_jack(struct snd_soc_component *component,
>>> +					struct snd_soc_jack *jack)
>>> +{
>>> +	int ret;
>>> +
>>> +	ret = snd_soc_card_jack_new(component->card, "USB Offload Playback Jack",
>> do we need the reference to Playback?
> No, will remove.
>>> +					SND_JACK_HEADPHONE, jack);
>> wondering if there would be any merit in defining a new type of jack,
>> e.g. SND_JACK_USB since here the purpose is to notify that there's a USB
>> device connected. The type of device does not really matter, does it?
>>
> Not as of now, but I think we discussed in the past that maybe depending on if playback and capture is supported, we can notify SND_JACK_HEADSET?  That is something I will need to change depending on how we want to handle the comments on patch#9
> 

I agree with Pierre that SND_JACK_HEADPHONE is too specific in this 
case, adding SND_JACK_USB sounds like good solution, as there are more 
device types than headset and headphones. Alternatively you could also 
consider defining some type of USB Audio Class mapping to existing 
SND_JACK types. (Look for UAC_INPUT_TERMINAL_*, UAC_OUTPUT_TERMINAL_* & 
UAC_BIDIR_TERMINAL_*.)