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] [day] [month] [year] [list]
Message-ID: <2024073117-velocity-buckle-ab6b@gregkh>
Date: Wed, 31 Jul 2024 11:48:31 +0200
From: "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>
To: 胡连勤 <hulianqin@...o.com>
Cc: "akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"opensource.kernel" <opensource.kernel@...o.com>
Subject: Re: 答复: 答复: [PATCH v2] usb: gadget: uvc: Fixed the abnormal
 enumeration problem of mobile phone as UVC camera.

On Wed, Jul 31, 2024 at 09:39:16AM +0000, 胡连勤 wrote:
> When the phone is connected to the computer to use the webcam function,
> the phone needs to be enumerated as a uvc camera function.
> 
> Because uvc->func.bind_deactivated is configured as true in the f_uvc
> driver uvc_alloc function, the usb_gadget_deactivate function is called
> during the execution of the configfs_composite_bind function to
> set gadget->deactivated to true, which in turn causes the
> usb_gadget_connect_locked function to fail to call the corresponding
> controller pullup operation (such as: dwc3_gadget_pullup,
> mtu3_gadget_pullup), and the USB cannot be enumerated
> normally under the uvc function combination.
> 
> After applying this patch, we measured that under the uvc function,
> the dwc3 controller and the mtu3 controller can be enumerated normally,
> and the webcam function is normal.
> 
> Therefore, modify the f_uvc driver to remove the operation of setting
> uvc->func.bind_deactivated to true.
> 
> Signed-off-by: Lianqin Hu <hulianqin@...o.com>
> ---
> v2:
>   - Modify "Fixes:" tag on the commit
>   - Modify description information.

I see no "Fixes:" tag here :(


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ