Messages in this thread | | | From | "Sathyakam M" <> | Subject | RE: [PATCH 0/5] media: add FSD MIPI-CSI2 Rx controller driver | Date | Wed, 23 Nov 2022 14:15:54 +0530 |
| |
Hi Krzysztof, Thank you for taking your time to review the patches.
> -----Original Message----- > From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@linaro.org] > Sent: 21 November 2022 15:01 > To: Sathyakam M <sathya@samsung.com> > Cc: Kishon Vijay Abraham I <kishon@kernel.org>; Krzysztof Kozlowski > <krzysztof.kozlowski+dt@linaro.org>; Pankaj Kumar Dubey > <pankaj.dubey@samsung.com>; Alim Akhtar <alim.akhtar@samsung.com>; > Aswani Reddy <aswani.reddy@samsung.com>; Andi Shyti > <andi@etezian.org>; Hans Verkuil <hverkuil-cisco@xs4all.nl>; Ming Qian > <ming.qian@nxp.com>; Jacopo Mondi <jacopo@jmondi.org>; Will Deacon > <will@kernel.org>; Shawn Guo <shawnguo@kernel.org>; Marcel Ziswiler > <marcel.ziswiler@toradex.com>; linux-arm-kernel@lists.infradead.org; linux- > kernel@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@samsung.com>,Vinod Koul <vkoul@kernel.org> > > Cc: Kishon Vijay Abraham I <kishon@kernel.org>,Rob Herring > > <robh+dt@kernel.org> > > Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org> > > Cc: Pankaj Kumar Dubey <pankaj.dubey@samsung.com>,Mauro Carvalho > > Chehab <mchehab@kernel.org> > > Cc: Alim Akhtar <alim.akhtar@samsung.com>,Shashank Prashar > > <s.prashar@samsung.com> > > Cc: Aswani Reddy <aswani.reddy@samsung.com>,Chandrasekar R > > <rcsekar@samsung.com> > > Cc: Andi Shyti <andi@etezian.org>,Laurent Pinchart > > <laurent.pinchart@ideasonboard.com> > > Cc: Hans Verkuil <hverkuil-cisco@xs4all.nl>,Jernej Skrabec > > <jernej.skrabec@gmail.com> > > Cc: Ming Qian <ming.qian@nxp.com>,Dmitry Osipenko > <digetx@gmail.com> > > Cc: Jacopo Mondi <jacopo@jmondi.org>,Catalin Marinas > > <catalin.marinas@arm.com> > > Cc: Will Deacon <will@kernel.org>,Bjorn Andersson > > <andersson@kernel.org> > > Cc: Shawn Guo <shawnguo@kernel.org>,Arnd Bergmann > <arnd@arndb.de> > > Cc: Marcel Ziswiler <marcel.ziswiler@toradex.com>,Mark Brown > > <broonie@kernel.org> > > Cc: > > linux-arm-kernel@lists.infradead.org,linux-phy@lists.infradead.org,dev > > icetree@vger.kernel.org > > Cc: linux-kernel@vger.kernel.org,linux-media@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
| |