linux-kernel - Re: 回复: [v1] staging: media: starfive: Renamed capture_raw to capture

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: <443e9909-7036-48a4-a1ea-c3908263282e@xs4all.nl>
Date: Thu, 21 Mar 2024 13:49:26 +0100
From: Hans Verkuil <hverkuil-cisco@...all.nl>
To: Changhuang Liang <changhuang.liang@...rfivetech.com>,
 Mauro Carvalho Chehab <mchehab@...nel.org>,
 Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc: Laurent Pinchart <laurent.pinchart@...asonboard.com>,
 Jack Zhu <jack.zhu@...rfivetech.com>,
 "linux-media@...r.kernel.org" <linux-media@...r.kernel.org>,
 "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
 "linux-staging@...ts.linux.dev" <linux-staging@...ts.linux.dev>
Subject: Re: 回复: [v1] staging: media: starfive: Renamed capture_raw to capture_dump

On 21/03/2024 1:17 pm, Changhuang Liang wrote:
> Hi, Hans
> 
> Thanks for your comments.
> 
>> On 01/03/2024 8:00 am, Changhuang Liang wrote:
>>> The pixel formats captured by the capture_raw video device depends on
>>> what pixel formats come from the source device. It is actually dump
>>> the source device data. So renamed it to capture_dump.
>>
>> I don't think 'dump' is a great name. I think what you really want to say is
>> 'unprocessed'. And usually that is indeed called 'raw capture'.
>>
> 
> StarFive JH7110 SoC also supports the output of the raw format data from ISP port, see:
> 
> https://lore.kernel.org/all/20240306093334.9321-1-changhuang.liang@starfivetech.com/
> 
> So I think it is better to call the ISP output port "capture_raw".

If it is processed by the ISP, how can it be 'raw'? Or do you mean that this uses Bayer pixelformat?

The normal convention AFAIK is that the term 'raw capture' generally means that
the sensor bayer data is captured without being processed (or only minimal processing).

Looking at:

https://lore.kernel.org/all/20240306093334.9321-6-changhuang.liang@starfivetech.com/

I think you indeed mean that you can get the processed ISP data in bayer format
through a separate video device. So I would call that port 'capture_bayer' and keep the
'capture_raw' name for the port from the CSI receiver.

Regards,

	Hans

> 
> On the other hand, Old "capture_raw" video devices can also support YUV color space, 
> mainly depending on the camera output format. There may be a confusion called it "capture_raw".
> 
> Regards,
> Changhuang
> 
>