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

[Sathyakam M]

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

> -----Original Message-----
> From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski at linaro.org]
> Sent: 21 November 2022 15:01
> To: Sathyakam M <sathya at samsung.com>
> Cc: Kishon Vijay Abraham I <kishon at kernel.org>; Krzysztof Kozlowski
> <krzysztof.kozlowski+dt at linaro.org>; Pankaj Kumar Dubey
> <pankaj.dubey at samsung.com>; Alim Akhtar <alim.akhtar at samsung.com>;
> Aswani Reddy <aswani.reddy at samsung.com>; Andi Shyti
> <andi at etezian.org>; Hans Verkuil <hverkuil-cisco at xs4all.nl>; Ming Qian
> <ming.qian at nxp.com>; Jacopo Mondi <jacopo at jmondi.org>; Will Deacon
> <will at kernel.org>; Shawn Guo <shawnguo at kernel.org>; Marcel Ziswiler
> <marcel.ziswiler at toradex.com>; linux-arm-kernel at lists.infradead.org; linux-
> kernel at vger.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 at samsung.com>,Vinod Koul <vkoul at kernel.org>
> > Cc: Kishon Vijay Abraham I <kishon at kernel.org>,Rob Herring
> > <robh+dt at kernel.org>
> > Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt at linaro.org>
> > Cc: Pankaj Kumar Dubey <pankaj.dubey at samsung.com>,Mauro Carvalho
> > Chehab <mchehab at kernel.org>
> > Cc: Alim Akhtar <alim.akhtar at samsung.com>,Shashank Prashar
> > <s.prashar at samsung.com>
> > Cc: Aswani Reddy <aswani.reddy at samsung.com>,Chandrasekar R
> > <rcsekar at samsung.com>
> > Cc: Andi Shyti <andi at etezian.org>,Laurent Pinchart
> > <laurent.pinchart at ideasonboard.com>
> > Cc: Hans Verkuil <hverkuil-cisco at xs4all.nl>,Jernej Skrabec
> > <jernej.skrabec at gmail.com>
> > Cc: Ming Qian <ming.qian at nxp.com>,Dmitry Osipenko
> <digetx at gmail.com>
> > Cc: Jacopo Mondi <jacopo at jmondi.org>,Catalin Marinas
> > <catalin.marinas at arm.com>
> > Cc: Will Deacon <will at kernel.org>,Bjorn Andersson
> > <andersson at kernel.org>
> > Cc: Shawn Guo <shawnguo at kernel.org>,Arnd Bergmann
> <arnd at arndb.de>
> > Cc: Marcel Ziswiler <marcel.ziswiler at toradex.com>,Mark Brown
> > <broonie at kernel.org>
> > Cc:
> > linux-arm-kernel at lists.infradead.org,linux-phy at lists.infradead.org,dev
> > icetree at vger.kernel.org
> > Cc: linux-kernel at vger.kernel.org,linux-media at vger.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