[alsa-devel] Fwd: Re: no reset_resume for driver snd-usb-audio for logitech headset H600

[baumber at hotmail.com]

Hello,

there must be a misunderstanding;

There is a functional change, because with the line/patch ".reset_resume = usb_audio_resume," the mixer of the usb headset was not removed during suspend/resume and so the master channel in KDE Mixer was not changed (Headset), which is the correct behaviour.

Without the line ".reset_resume = usb_audio_resume," in /sound/usb/card.c, the mixer was removed during suspend/resume and re-added, and so the master channel changed in KDE Mixer to the default onboard audio.

I have tested kernel 3.13rc8, and as you mentioned, the "reset_resume"-messages are gone.
But the behaviour of removing the mixer of the usb_audio_headset occurs, which is, in my opinion, not the correct behaviour, because the chosen master channel/mixer should stay after suspend/resume.

Could you please take a look at the issue again.

Thanks, Bernhard


On 2014-01-13 10:55, Takashi Iwai wrote:

> At Sat, 11 Jan 2014 11:08:39 +0100,
> The "reset_resume" kernel message was known to be harmless, and now
> it's degraded as a debug message.  That's the only change, so there is
> no functional change.
>
> That being said, you just need to ignore the kernel messages in the
> old kernels.  Or, feel free to submit the commit [0a56b4fa6844: USB:
> change dev_warn about missing reset-resume to dev_dbg] to stable
> kernel.
>
>
> Takashi
>

> <baumber at hotmail.com> wrote:
>>
>> FWD:
>>
>> Any news to the snd-usb-audio issue?
>>
>> Thanks, Bernhard
>>
>> Old Message:
>> -------------
>> I have tested the patch with kernel 3.12.0 with suspend and hibernate, and the problem is gone.
>>
>> No messages "no reset_resume for driver snd-usb-audio?" or other negative effects/messages.
>>
>> After resume, the usb_audio_device (Headset) is the master channel in KDE Mixer, as it should be.
>>
>> => So the problem is solved for me.
>>
>> Can you please add the patch to the latest kernels and LTS-kernels or whitelist the workaround for my device (ID 046d:0a29 Logitech, Inc. H600 [Wireless Headset])
>

>>
>> Thanks, Bernhard
>>
>> On 2013-10-20 23:35, Takashi Iwai wrote:
>>> At Sun, 20 Oct 2013 23:18:13 +0200,
>>> <baumber at hotmail.com> wrote:
>>>>
>>>> On 2013-10-09 11:16 (CEST,UTC+2), Takashi Iwai wrote:
>>>>>>>> diff --git a/sound/usb/card.c b/sound/usb/card.c
>>>>>>>> index 9d9de8d..c591a83 100644
>>>>>>>> --- a/sound/usb/card.c
>>>>>>>> +++ b/sound/usb/card.c
>>>>>>>> @@ -747,6 +747,7 @@ static struct usb_driver usb_audio_driver = {
>>>>>>>>     	.disconnect =	usb_audio_disconnect,
>>>>>>>>     	.suspend =	usb_audio_suspend,
>>>>>>>>     	.resume =	usb_audio_resume,
>>>>>>>> +	.reset_resume =	usb_audio_resume,
>>>>>>>>     	.id_table =	usb_audio_ids,
>>>>>>>>     	.supports_autosuspend = 1,
>>>>>>>>     };
>>>>
>>>> Shall I add the line in b/sound/usb/card.c and try it? Which kernel version shall I use?
>>>
>>> It doesn't matter which kernel version is.
>>> The error message should disappear, but I don't know whether any other
>>> positive effect you get.  As Clemens pointed, there can be some
>>> negative effects by this, so it cannot be taken as is.  But at least,
>>> it'd be good to know the result...
>>>
>>>
>>> thanks,
>>>
>>> Takashi
>>
>>
>>
>>
>>
>>
>
>