linux-kernel - RE: [PATCH 0/5] media: add FSD MIPI-CSI2 Rx controller driver

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: <00c201d8ff18$01257680$03706380$@samsung.com>
Date:   Wed, 23 Nov 2022 14:15:54 +0530
From:   "Sathyakam M" <sathya@...sung.com>
To:     "'Krzysztof Kozlowski'" <krzysztof.kozlowski@...aro.org>
Cc:     "'Kishon Vijay Abraham I'" <kishon@...nel.org>,
        "'Krzysztof Kozlowski'" <krzysztof.kozlowski+dt@...aro.org>,
        "'Pankaj Kumar Dubey'" <pankaj.dubey@...sung.com>,
        "'Alim Akhtar'" <alim.akhtar@...sung.com>,
        "'Aswani Reddy'" <aswani.reddy@...sung.com>,
        "'Andi Shyti'" <andi@...zian.org>,
        "'Hans Verkuil'" <hverkuil-cisco@...all.nl>,
        "'Ming Qian'" <ming.qian@....com>,
        "'Jacopo Mondi'" <jacopo@...ndi.org>,
        "'Will Deacon'" <will@...nel.org>,
        "'Shawn Guo'" <shawnguo@...nel.org>,
        "'Marcel Ziswiler'" <marcel.ziswiler@...adex.com>,
        <linux-arm-kernel@...ts.infradead.org>,
        <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH 0/5] media: add FSD MIPI-CSI2 Rx controller driver

Hi Krzysztof,
Thank you for taking your time to review the patches.

> -----Original Message-----
> From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@...aro.org]
> Sent: 21 November 2022 15:01
> To: Sathyakam M <sathya@...sung.com>
> Cc: Kishon Vijay Abraham I <kishon@...nel.org>; Krzysztof Kozlowski
> <krzysztof.kozlowski+dt@...aro.org>; Pankaj Kumar Dubey
> <pankaj.dubey@...sung.com>; Alim Akhtar <alim.akhtar@...sung.com>;
> Aswani Reddy <aswani.reddy@...sung.com>; Andi Shyti
> <andi@...zian.org>; Hans Verkuil <hverkuil-cisco@...all.nl>; Ming Qian
> <ming.qian@....com>; Jacopo Mondi <jacopo@...ndi.org>; Will Deacon
> <will@...nel.org>; Shawn Guo <shawnguo@...nel.org>; Marcel Ziswiler
> <marcel.ziswiler@...adex.com>; linux-arm-kernel@...ts.infradead.org; linux-
> kernel@...r.kernel.org
> Subject: Re: [PATCH 0/5] media: add FSD MIPI-CSI2 Rx controller driver
> 
> On 20/11/2022 19:52, Sathyakam M wrote:
> > This patch series adds the V4L2 media driver for Tesla Full Self
> > Driving (FSD) SoC MIPI-CSI2 Rx controller.
> >
> > The controller is compliant to MIPI CSI2 v1.3 specification.
> > It receives images from camera sensors over the D-PHY interface.
> >
> > The D-PHY interface is compliant to MIPI D-PHY v1.2 specification. It
> > supports up to maximum of 4 data lanes.
> >
> > V4L2 user control is provided to set the desired number of data lanes.
> >
> > Maximum of up to 4 streams can be received simultaneously by one Rx
> > port. (Each streams identified by designated Virtual Channel
> > Identifier).
> >
> > The controller instances consist of port property describing the endpoints.
> > When combined with remote-endpoint property the driver instantiates
> > video device nodes using V4L2 async subdev methods.
> >
> > Streaming formats supported:
> > - YUV420
> > - RGB565, RGB666, RGB888
> > - RAW8, RAW10, RAW12
> > - JPEG
> >
> > The driver implementation is verified on FSD evaluation board.
> >
> > Thanks, Sathyakam
> >
> > Cc: Sathyakam M <sathya@...sung.com>,Vinod Koul <vkoul@...nel.org>
> > Cc: Kishon Vijay Abraham I <kishon@...nel.org>,Rob Herring
> > <robh+dt@...nel.org>
> > Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>
> > Cc: Pankaj Kumar Dubey <pankaj.dubey@...sung.com>,Mauro Carvalho
> > Chehab <mchehab@...nel.org>
> > Cc: Alim Akhtar <alim.akhtar@...sung.com>,Shashank Prashar
> > <s.prashar@...sung.com>
> > Cc: Aswani Reddy <aswani.reddy@...sung.com>,Chandrasekar R
> > <rcsekar@...sung.com>
> > Cc: Andi Shyti <andi@...zian.org>,Laurent Pinchart
> > <laurent.pinchart@...asonboard.com>
> > Cc: Hans Verkuil <hverkuil-cisco@...all.nl>,Jernej Skrabec
> > <jernej.skrabec@...il.com>
> > Cc: Ming Qian <ming.qian@....com>,Dmitry Osipenko
> <digetx@...il.com>
> > Cc: Jacopo Mondi <jacopo@...ndi.org>,Catalin Marinas
> > <catalin.marinas@....com>
> > Cc: Will Deacon <will@...nel.org>,Bjorn Andersson
> > <andersson@...nel.org>
> > Cc: Shawn Guo <shawnguo@...nel.org>,Arnd Bergmann
> <arnd@...db.de>
> > Cc: Marcel Ziswiler <marcel.ziswiler@...adex.com>,Mark Brown
> > <broonie@...nel.org>
> > Cc:
> > linux-arm-kernel@...ts.infradead.org,linux-phy@...ts.infradead.org,dev
> > icetree@...r.kernel.org
> > Cc: linux-kernel@...r.kernel.org,linux-media@...r.kernel.org
> >
> > Sathyakam M (5):
> >   dt-bindings: phy: fsd: MIPI CSI2 Rx controller syscon node
> >   media: dt-bindings: fsd: MIPI CSI2 Rx controller device nodes
> >   arm64: dts: fsd: add device tree nodes for MIPI CSI2 Rx controller
> >   media: fsd: add MIPI CSI2 Rx controller driver
> 
> I did not get your patch #4. Don't Cc selectively.
Sorry for that. There was problem with sending the bundled patch set. I see that you got "[PATCH 4/5] media: fsd: add MIPI CSI2 Rx controller driver".
I will address the comments for the same.
> 
> Best regards,
> Krzysztof