All of lore.kernel.org
 help / color / mirror / Atom feed
From: Maxime Ripard <maxime@cerno.tech>
To: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
Cc: linux-media@vger.kernel.org, devicetree@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, devel@driverdev.osuosl.org,
	linux-sunxi@googlegroups.com,
	Mauro Carvalho Chehab <mchehab@kernel.org>,
	Rob Herring <robh+dt@kernel.org>, Chen-Yu Tsai <wens@csie.org>,
	Yong Deng <yong.deng@magewell.com>,
	Kishon Vijay Abraham I <kishon@ti.com>,
	Vinod Koul <vkoul@kernel.org>,
	Helen Koike <helen.koike@collabora.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Philipp Zabel <p.zabel@pengutronix.de>,
	Hans Verkuil <hans.verkuil@cisco.com>,
	Sakari Ailus <sakari.ailus@linux.intel.com>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	Hans Verkuil <hverkuil@xs4all.nl>,
	kevin.lhopital@hotmail.com
Subject: Re: [PATCH 10/14] ARM: dts: sun8i: v3s: Add MIPI D-PHY and MIPI CSI-2 interface nodes
Date: Mon, 26 Oct 2020 17:55:36 +0100	[thread overview]
Message-ID: <20201026165536.glmo727mzixtrtq5@gilmour.lan> (raw)
In-Reply-To: <20201023174546.504028-11-paul.kocialkowski@bootlin.com>

[-- Attachment #1: Type: text/plain, Size: 627 bytes --]

On Fri, Oct 23, 2020 at 07:45:42PM +0200, Paul Kocialkowski wrote:
> MIPI CSI-2 is supported on the V3s with an A31 controller, which seems
> to be used on all Allwinner chips supporting it, except for the A83T.
> The controller is connected to CSI0 through fwnode endpoints.
> The mipi_csi2_in port node is provided to connect MIPI CSI-2 sensors.
> 
> The D-PHY part is the same that already drives DSI, but used in Rx mode.
> 
> Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>

Especially since you haven't tested CSI0 without MIPI-CSI, you can
squash that patch into the previous one.

Maxime

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: Maxime Ripard <maxime@cerno.tech>
To: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
Cc: devel@driverdev.osuosl.org, devicetree@vger.kernel.org,
	Philipp Zabel <p.zabel@pengutronix.de>,
	Kishon Vijay Abraham I <kishon@ti.com>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Helen Koike <helen.koike@collabora.com>,
	linux-kernel@vger.kernel.org, Chen-Yu Tsai <wens@csie.org>,
	Hans Verkuil <hverkuil@xs4all.nl>,
	linux-sunxi@googlegroups.com, Rob Herring <robh+dt@kernel.org>,
	Vinod Koul <vkoul@kernel.org>, Yong Deng <yong.deng@magewell.com>,
	Sakari Ailus <sakari.ailus@linux.intel.com>,
	Hans Verkuil <hans.verkuil@cisco.com>,
	Mauro Carvalho Chehab <mchehab@kernel.org>,
	kevin.lhopital@hotmail.com, linux-arm-kernel@lists.infradead.org,
	linux-media@vger.kernel.org
Subject: Re: [PATCH 10/14] ARM: dts: sun8i: v3s: Add MIPI D-PHY and MIPI CSI-2 interface nodes
Date: Mon, 26 Oct 2020 17:55:36 +0100	[thread overview]
Message-ID: <20201026165536.glmo727mzixtrtq5@gilmour.lan> (raw)
In-Reply-To: <20201023174546.504028-11-paul.kocialkowski@bootlin.com>


[-- Attachment #1.1: Type: text/plain, Size: 627 bytes --]

On Fri, Oct 23, 2020 at 07:45:42PM +0200, Paul Kocialkowski wrote:
> MIPI CSI-2 is supported on the V3s with an A31 controller, which seems
> to be used on all Allwinner chips supporting it, except for the A83T.
> The controller is connected to CSI0 through fwnode endpoints.
> The mipi_csi2_in port node is provided to connect MIPI CSI-2 sensors.
> 
> The D-PHY part is the same that already drives DSI, but used in Rx mode.
> 
> Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>

Especially since you haven't tested CSI0 without MIPI-CSI, you can
squash that patch into the previous one.

Maxime

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[-- Attachment #2: Type: text/plain, Size: 169 bytes --]

_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

WARNING: multiple messages have this Message-ID (diff)
From: Maxime Ripard <maxime@cerno.tech>
To: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
Cc: devel@driverdev.osuosl.org, devicetree@vger.kernel.org,
	Philipp Zabel <p.zabel@pengutronix.de>,
	Kishon Vijay Abraham I <kishon@ti.com>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Helen Koike <helen.koike@collabora.com>,
	linux-kernel@vger.kernel.org, Chen-Yu Tsai <wens@csie.org>,
	Hans Verkuil <hverkuil@xs4all.nl>,
	linux-sunxi@googlegroups.com, Rob Herring <robh+dt@kernel.org>,
	Vinod Koul <vkoul@kernel.org>, Yong Deng <yong.deng@magewell.com>,
	Sakari Ailus <sakari.ailus@linux.intel.com>,
	Hans Verkuil <hans.verkuil@cisco.com>,
	Mauro Carvalho Chehab <mchehab@kernel.org>,
	kevin.lhopital@hotmail.com, linux-arm-kernel@lists.infradead.org,
	linux-media@vger.kernel.org
Subject: Re: [PATCH 10/14] ARM: dts: sun8i: v3s: Add MIPI D-PHY and MIPI CSI-2 interface nodes
Date: Mon, 26 Oct 2020 17:55:36 +0100	[thread overview]
Message-ID: <20201026165536.glmo727mzixtrtq5@gilmour.lan> (raw)
In-Reply-To: <20201023174546.504028-11-paul.kocialkowski@bootlin.com>


[-- Attachment #1.1: Type: text/plain, Size: 627 bytes --]

On Fri, Oct 23, 2020 at 07:45:42PM +0200, Paul Kocialkowski wrote:
> MIPI CSI-2 is supported on the V3s with an A31 controller, which seems
> to be used on all Allwinner chips supporting it, except for the A83T.
> The controller is connected to CSI0 through fwnode endpoints.
> The mipi_csi2_in port node is provided to connect MIPI CSI-2 sensors.
> 
> The D-PHY part is the same that already drives DSI, but used in Rx mode.
> 
> Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>

Especially since you haven't tested CSI0 without MIPI-CSI, you can
squash that patch into the previous one.

Maxime

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[-- Attachment #2: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2020-10-26 16:55 UTC|newest]

Thread overview: 198+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-23 17:45 [PATCH 00/14] Allwinner MIPI CSI-2 support for A31/V3s/A83T Paul Kocialkowski
2020-10-23 17:45 ` Paul Kocialkowski
2020-10-23 17:45 ` Paul Kocialkowski
2020-10-23 17:45 ` [PATCH 01/14] phy: Distinguish between Rx and Tx for MIPI D-PHY with submodes Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 18:18   ` [linux-sunxi] " Jernej Škrabec
2020-10-23 18:18     ` Jernej Škrabec
2020-10-23 18:18     ` Jernej Škrabec
2020-10-24  8:31     ` Paul Kocialkowski
2020-10-24  8:31       ` Paul Kocialkowski
2020-10-24  8:31       ` Paul Kocialkowski
2020-10-30 22:44   ` Helen Koike
2020-10-30 22:44     ` Helen Koike
2020-10-30 22:44     ` Helen Koike
2020-10-23 17:45 ` [PATCH 02/14] phy: allwinner: phy-sun6i-mipi-dphy: Support D-PHY Rx mode for MIPI CSI-2 Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-26 15:38   ` Maxime Ripard
2020-10-26 15:38     ` Maxime Ripard
2020-10-26 15:38     ` Maxime Ripard
2020-10-27  9:23     ` Paul Kocialkowski
2020-10-27  9:23       ` Paul Kocialkowski
2020-10-27  9:23       ` Paul Kocialkowski
2020-10-27 18:28       ` Maxime Ripard
2020-10-27 18:28         ` Maxime Ripard
2020-10-27 18:28         ` Maxime Ripard
2020-11-04 10:53         ` Paul Kocialkowski
2020-11-04 10:53           ` Paul Kocialkowski
2020-11-04 10:53           ` Paul Kocialkowski
2020-10-30 22:44   ` Helen Koike
2020-10-30 22:44     ` Helen Koike
2020-10-30 22:44     ` Helen Koike
2020-11-04 10:54     ` Paul Kocialkowski
2020-11-04 10:54       ` Paul Kocialkowski
2020-11-04 10:54       ` Paul Kocialkowski
2020-10-23 17:45 ` [PATCH 03/14] media: sun6i-csi: Support an optional dedicated memory pool Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-26 15:41   ` Maxime Ripard
2020-10-26 15:41     ` Maxime Ripard
2020-10-26 15:41     ` Maxime Ripard
2020-10-27  9:26     ` Paul Kocialkowski
2020-10-27  9:26       ` Paul Kocialkowski
2020-10-27  9:26       ` Paul Kocialkowski
2020-10-23 17:45 ` [PATCH 04/14] media: sun6i-csi: Fix the image storage bpp for 10/12-bit Bayer formats Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-30 22:45   ` Helen Koike
2020-10-30 22:45     ` Helen Koike
2020-10-30 22:45     ` Helen Koike
2020-11-04 10:56     ` Paul Kocialkowski
2020-11-04 10:56       ` Paul Kocialkowski
2020-11-04 10:56       ` Paul Kocialkowski
2020-10-23 17:45 ` [PATCH 05/14] media: sun6i-csi: Only configure the interface data width for parallel Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-26 16:00   ` Maxime Ripard
2020-10-26 16:00     ` Maxime Ripard
2020-10-26 16:00     ` Maxime Ripard
2020-10-27  9:31     ` Paul Kocialkowski
2020-10-27  9:31       ` Paul Kocialkowski
2020-10-27  9:31       ` Paul Kocialkowski
2020-10-27 18:31       ` Maxime Ripard
2020-10-27 18:31         ` Maxime Ripard
2020-10-27 18:31         ` Maxime Ripard
2020-10-23 17:45 ` [PATCH 06/14] media: sun6i-csi: Support feeding from the MIPI CSI-2 controller Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45 ` [PATCH 07/14] dt-bindings: media: i2c: Add A31 MIPI CSI-2 bindings documentation Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-26 16:14   ` Maxime Ripard
2020-10-26 16:14     ` Maxime Ripard
2020-10-26 16:14     ` Maxime Ripard
2020-10-27  9:52     ` Paul Kocialkowski
2020-10-27  9:52       ` Paul Kocialkowski
2020-10-27  9:52       ` Paul Kocialkowski
2020-10-27 18:44       ` Maxime Ripard
2020-10-27 18:44         ` Maxime Ripard
2020-10-27 18:44         ` Maxime Ripard
2020-11-04 10:48         ` Paul Kocialkowski
2020-11-04 10:48           ` Paul Kocialkowski
2020-11-04 10:48           ` Paul Kocialkowski
2020-11-04 16:53           ` Maxime Ripard
2020-11-04 16:53             ` Maxime Ripard
2020-11-04 16:53             ` Maxime Ripard
2020-10-30 16:33   ` Rob Herring
2020-10-30 16:33     ` Rob Herring
2020-10-30 16:33     ` Rob Herring
2020-10-30 16:56   ` Sakari Ailus
2020-10-30 16:56     ` Sakari Ailus
2020-10-30 16:56     ` Sakari Ailus
2020-10-23 17:45 ` [PATCH 08/14] media: sunxi: Add support for the A31 MIPI CSI-2 controller Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-26  8:39   ` Dan Carpenter
2020-10-26  8:39     ` Dan Carpenter
2020-10-26  8:39     ` Dan Carpenter
2020-10-26 16:54   ` Maxime Ripard
2020-10-26 16:54     ` Maxime Ripard
2020-10-26 16:54     ` Maxime Ripard
2020-11-04 11:34     ` Paul Kocialkowski
2020-11-04 11:34       ` Paul Kocialkowski
2020-11-04 11:34       ` Paul Kocialkowski
2020-11-04 18:56       ` Maxime Ripard
2020-11-04 18:56         ` Maxime Ripard
2020-11-04 18:56         ` Maxime Ripard
2020-11-05 14:52         ` Paul Kocialkowski
2020-11-05 14:52           ` Paul Kocialkowski
2020-11-05 14:52           ` Paul Kocialkowski
2020-10-30 22:45   ` Helen Koike
2020-10-30 22:45     ` Helen Koike
2020-10-30 22:45     ` Helen Koike
2020-11-02  9:21     ` Maxime Ripard
2020-11-02  9:21       ` Maxime Ripard
2020-11-02  9:21       ` Maxime Ripard
2020-11-04 11:17       ` Paul Kocialkowski
2020-11-04 11:17         ` Paul Kocialkowski
2020-11-04 11:17         ` Paul Kocialkowski
2020-11-04 16:38         ` Helen Koike
2020-11-04 16:38           ` Helen Koike
2020-11-04 16:38           ` Helen Koike
2020-11-04 18:45           ` Maxime Ripard
2020-11-04 18:45             ` Maxime Ripard
2020-11-04 18:45             ` Maxime Ripard
2020-11-05 14:14             ` Helen Koike
2020-11-05 14:14               ` Helen Koike
2020-11-05 14:14               ` Helen Koike
2020-11-05  8:45   ` Sakari Ailus
2020-11-05  8:45     ` Sakari Ailus
2020-11-05  8:45     ` Sakari Ailus
2020-11-05 14:55     ` Paul Kocialkowski
2020-11-05 14:55       ` Paul Kocialkowski
2020-11-05 14:55       ` Paul Kocialkowski
2020-10-23 17:45 ` [PATCH 09/14] ARM: dts: sun8i: v3s: Add CSI0 camera interface node Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45 ` [PATCH 10/14] ARM: dts: sun8i: v3s: Add MIPI D-PHY and MIPI CSI-2 interface nodes Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-26 16:55   ` Maxime Ripard [this message]
2020-10-26 16:55     ` Maxime Ripard
2020-10-26 16:55     ` Maxime Ripard
2020-10-23 17:45 ` [PATCH 11/14] dt-bindings: media: i2c: Add A83T MIPI CSI-2 bindings documentation Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-26 16:56   ` Maxime Ripard
2020-10-26 16:56     ` Maxime Ripard
2020-10-26 16:56     ` Maxime Ripard
2020-11-04 10:33     ` Paul Kocialkowski
2020-11-04 10:33       ` Paul Kocialkowski
2020-11-04 10:33       ` Paul Kocialkowski
2020-11-05  8:48   ` Sakari Ailus
2020-11-05  8:48     ` Sakari Ailus
2020-11-05  8:48     ` Sakari Ailus
2020-10-23 17:45 ` [PATCH 12/14] media: sunxi: Add support for the A83T MIPI CSI-2 controller Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-26  8:53   ` Dan Carpenter
2020-10-26  8:53     ` Dan Carpenter
2020-10-26  8:53     ` Dan Carpenter
2020-10-26 17:00   ` Maxime Ripard
2020-10-26 17:00     ` Maxime Ripard
2020-10-26 17:00     ` Maxime Ripard
2020-11-04 10:37     ` Paul Kocialkowski
2020-11-04 10:37       ` Paul Kocialkowski
2020-11-04 10:37       ` Paul Kocialkowski
2020-10-23 17:45 ` [PATCH 13/14] ARM: dts: sun8i: a83t: Add MIPI CSI-2 controller node Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45 ` [PATCH 14/14] media: sunxi: sun8i-a83t-mipi-csi2: Avoid using the (unsolicited) interrupt Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-23 17:45   ` Paul Kocialkowski
2020-10-26 16:57   ` Maxime Ripard
2020-10-26 16:57     ` Maxime Ripard
2020-10-26 16:57     ` Maxime Ripard
2020-10-26 17:20 ` [PATCH 00/14] Allwinner MIPI CSI-2 support for A31/V3s/A83T Maxime Ripard
2020-10-26 17:20   ` Maxime Ripard
2020-10-26 17:20   ` Maxime Ripard
2020-10-30 22:44 ` Helen Koike
2020-10-30 22:44   ` Helen Koike
2020-10-30 22:44   ` Helen Koike
2020-11-02  9:17   ` Maxime Ripard
2020-11-02  9:17     ` Maxime Ripard
2020-11-02  9:17     ` Maxime Ripard
2020-11-04 11:11   ` Paul Kocialkowski
2020-11-04 11:11     ` Paul Kocialkowski
2020-11-04 11:11     ` Paul Kocialkowski
2020-11-04 11:14     ` Paul Kocialkowski
2020-11-04 11:14       ` Paul Kocialkowski
2020-11-04 11:14       ` Paul Kocialkowski
2020-11-04 16:36     ` Helen Koike
2020-11-04 16:36       ` Helen Koike
2020-11-04 16:36       ` Helen Koike
2020-11-05 14:58       ` Paul Kocialkowski
2020-11-05 14:58         ` Paul Kocialkowski
2020-11-05 14:58         ` Paul Kocialkowski

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20201026165536.glmo727mzixtrtq5@gilmour.lan \
    --to=maxime@cerno.tech \
    --cc=devel@driverdev.osuosl.org \
    --cc=devicetree@vger.kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=hans.verkuil@cisco.com \
    --cc=helen.koike@collabora.com \
    --cc=hverkuil@xs4all.nl \
    --cc=kevin.lhopital@hotmail.com \
    --cc=kishon@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-sunxi@googlegroups.com \
    --cc=mchehab@kernel.org \
    --cc=p.zabel@pengutronix.de \
    --cc=paul.kocialkowski@bootlin.com \
    --cc=robh+dt@kernel.org \
    --cc=sakari.ailus@linux.intel.com \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=vkoul@kernel.org \
    --cc=wens@csie.org \
    --cc=yong.deng@magewell.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.