linux-kernel - Re: [PATCH v3] media: uvcvideo: Enable Intel RealSense metadata for devices.

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <ZEEhtUH/9Nqm/+n0@valkosipuli.retiisi.eu>
Date:   Thu, 20 Apr 2023 14:27:49 +0300
From:   Sakari Ailus <sakari.ailus@....fi>
To:     Dmitry Perchanov <dmitry.perchanov@...el.com>
Cc:     linux-media@...r.kernel.org, laurent.pinchart@...asonboard.com,
        mchehab@...nel.org, linux-kernel@...r.kernel.org,
        evgeni.raikhel@...el.com, demisrael@...il.com
Subject: Re: [PATCH v3] media: uvcvideo: Enable Intel RealSense metadata for
 devices.

Hi Dmitry,

On Thu, Apr 20, 2023 at 02:26:53PM +0300, Dmitry Perchanov wrote:
> Hi Sakari,
> 
> On Thu, 2023-04-20 at 12:24 +0300, Sakari Ailus wrote:
> > Hi Dmitry,
> > 
> > On Thu, Apr 20, 2023 at 12:06:55PM +0300, Dmitry Perchanov wrote:
> > > Intel RealSense UVC Depth cameras produce metadata in a
> > > vendor-specific format that is already supported by the uvcvideo driver.
> > > Enable handling of this metadata for 7 additional RealSense devices.
> > > 
> > > Co-developed-by: Yu MENG <yu1.meng@...el.com>
> > > Co-developed-by: Evgeni Raikhel <evgeni.raikhel@...el.com>
> > > Signed-off-by: Dmitry Perchanov <dmitry.perchanov@...el.com>
> > 
> > Thanks for the update.
> > 
> > Is the D4XX format on these devices the same that's documented in
> > Documentation/userspace-api/media/v4l/pixfmt-meta-d4xx.rst ? I.e. there are
> > no additional fields or differences, and the version of the format is the
> > same?
> We verified the document and found it generally correct.
> Although, we expanded number of fields and I can update this document.
> > 
> > The format documentation doesn't say which version number even the current
> > devices use (just refers to "xxx").
> The document compatible with all D4xx/D5xx devices so far.
> We dropped all obsolete incompatible cameras from support.
> > 
> 
> Should I submit another patch with extended structures?

Please do.

If some of the fields are only available on certain devices, it'd be good
to document which fields and which devices, or whether it's struct version
dependent (and then what that depends on).

-- 
Regards,

Sakari Ailus