linux-kernel - Re: [PATCH] usb: gadget: uvc: Add H264 frame format support

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: <5dc07ab0-0332-4c4b-8627-fdb2662b0ef2@quicinc.com>
Date: Thu, 11 Jul 2024 13:43:47 +0530
From: AKASH KUMAR <quic_akakum@...cinc.com>
To: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
CC: Thinh Nguyen <Thinh.Nguyen@...opsys.com>, Jing Leng <jleng@...arella.com>,
        Felipe Balbi <balbi@...nel.org>, Jack Pham <quic_jackp@...cinc.com>,
        <kernel@...cinc.com>, Wesley Cheng <quic_wcheng@...cinc.com>,
        "Laurent
 Pinchart" <laurent.pinchart@...asonboard.com>,
        Daniel Scally
	<dan.scally@...asonboard.com>,
        Vijayavardhan Vennapusa
	<quic_vvreddy@...cinc.com>,
        Krishna Kurapati <quic_kriskura@...cinc.com>,
        <linux-usb@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] usb: gadget: uvc: Add H264 frame format support

Sure greg adding the details in documentation and yeah it is needed as i 
have changed uvc frame structure
from single to separate for mjpeg, uncompressed and h264.
Updating the details in next patchset.
> On Mon, Jul 08, 2024 at 09:43:28AM +0530, Akash Kumar wrote:
>> Add support for framebased frame format which can be used to support
>> multiple formats like H264 or H265 other than mjpeg and YUV frames.
>>
>> Framebased format is set to H264 by default, which can be updated to
>> other formats by updating the GUID through guid configfs attribute.
>>
>> Signed-off-by: Akash Kumar <quic_akakum@...cinc.com>
>> ---
>>   drivers/usb/gadget/function/uvc_configfs.c | 570 ++++++++++++++++++---
>>   drivers/usb/gadget/function/uvc_configfs.h |  34 +-
>>   drivers/usb/gadget/function/uvc_v4l2.c     |  80 ++-
>>   include/uapi/linux/usb/video.h             |  62 +++
>>   4 files changed, 638 insertions(+), 108 deletions(-)
> DOn't you need to add the new configfs entries to Documentation/ABI/?
>
> And how is anyone going to know about this new api or format if there
> are not new files?  WHere is it now documented?
>
> thanks,
>
> greg k-h
Thanks,
Akash