Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 07 Nov 2022 13:13:28 +0100
Message-ID: <87h6zbt11z.wl-tiwai@suse.de>
From:   Takashi Iwai <tiwai@suse.de>
To:     =?ISO-2022-JP?B?GyRCZ2hENhsoQg==?= <aichao@kylinos.cn>
Cc:     <perex@perex.cz>, <tiwai@suse.com>, <alsa-devel@alsa-project.org>,
        <linux-kernel@vger.kernel.org>
Subject: Re: =?GB2312?B?u9i4tDo=?= Re: =?GB2312?B?u9i4tDo=?= Re:
 =?GB2312?B?u9i4tDo=?= Re: [PATCH] ALSA: usb-audio: fix Hamedal C20 usb
 camero disconnect issue
In-Reply-To: <41c727u0jt-41c727u0ju@nsmail6.0>
User-Agent: Wanderlust/2.15.9 (Almost Unreal) Emacs/27.2 Mule/6.0
MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue")
Content-Type: text/plain; charset=ISO-2022-JP
Precedence: bulk

On Mon, 07 Nov 2022 10:01:08 +0100,
艾超 wrote:
> 
>     I tested the hotplug 、record and catch the picture,  this interface reset
> doesn't cause any problem.
> 
> In snd_usb_endpoint_configure,  It set false to interface to deselect
> altsetting at first.
> 
> But , if skip the interface clear (ex.  ep->cur_rate = 0;), there is no sound
> output.

OK, then just skipping the interface reset at closing PCM appears to
be the proper fix.  Please go ahead and submit the patch for a new
quirk with this type.  Thanks!


Takashi