Re: [PATCH v21 26/39] ASoC: qcom: qdsp6: Add support to track available USB PCM devices

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Pierre,

On 5/9/2024 6:02 AM, Pierre-Louis Bossart wrote:


On 5/8/24 18:57, Wesley Cheng wrote:
Hi Pierre,

On 5/7/2024 2:23 PM, Pierre-Louis Bossart wrote:

@@ -113,6 +120,12 @@ static int q6usb_alsa_connection_cb(struct
snd_soc_usb *usb,
       if (connected) {
           /* We only track the latest USB headset plugged in */
           data->active_usb_chip_idx = sdev->card_idx;
+
+        set_bit(sdev->card_idx, &data->available_card_slot);
+        data->status[sdev->card_idx].sdev = sdev;

Not following the 'only track the latest USB headset plugged in', I
don't see anything that discard the previously latest headset...

If you plug headset1, then headset2, how is headset1 marked as not
available for USB offload?


It won't mark headset1 as not available for offload, because offload
could happen on either depending on what is selected (from the kcontrol
as well).

Right, so the wording 'only track the latest USB headset plugged in' is
incorrect or obsolete, isn't it?

Sure, I can reword it.  Will specify that it
"selects the latest USB headset plugged in for offloading"

Thanks
Wesley Cheng




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux