Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@bp.renesas.com>
To:     Geert Uytterhoeven <geert@linux-m68k.org>,
        Prabhakar <prabhakar.csengg@gmail.com>,
        Magnus Damm <magnus.damm@gmail.com>,
        Rob Herring <robh+dt@kernel.org>
CC:     Biju Das <biju.das.jz@bp.renesas.com>,
        "linux-renesas-soc@vger.kernel.org" 
        <linux-renesas-soc@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH v4 1/2] ARM: dts: r8a7742-iwg21d-q7-dbcm-ca: Separate out
 ov5640 nodes
Thread-Topic: [PATCH v4 1/2] ARM: dts: r8a7742-iwg21d-q7-dbcm-ca: Separate out
 ov5640 nodes
Thread-Index: AQHWw/9UPOaFqxmSLkudy+21evfaLqnarTJg
Date:   Thu, 26 Nov 2020 17:42:27 +0000
Message-ID: <OSBPR01MB50483C8B0A3AC24C0987F14EAAF90@OSBPR01MB5048.jpnprd01.prod.outlook.com>
References: <20201126134031.4115211-1-geert@linux-m68k.org>
 <20201126134031.4115211-2-geert@linux-m68k.org>
In-Reply-To: <20201126134031.4115211-2-geert@linux-m68k.org>
Accept-Language: en-GB, en-US
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: c431e460-1141-47e8-bf9b-08d89232a447
X-MS-Exchange-CrossTenant-originalarrivaltime: 26 Nov 2020 17:42:27.4121
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 53d82571-da19-47e4-9cb4-625a166a4a2a
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: H4JPGY9pAjshTF8pRcezA1yFXizRVTw8KDHK0x+iarf6K6rhKvzgPg5u7vtiF1xvLnXRn4C0pHChP0qGHHVadw4IaEpHLQ80MzvrADD0a3VzDz9Vx+0Qm7ZvwYY+j9bt
X-MS-Exchange-Transport-CrossTenantHeadersStamped: OS0PR01MB5859
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Geert,

> -----Original Message-----
> From: Geert Uytterhoeven <geert@linux-m68k.org>
> Sent: 26 November 2020 13:41
> To: Prabhakar <prabhakar.csengg@gmail.com>; Magnus Damm <magnus.damm@gmai=
l.com>; Rob Herring
> <robh+dt@kernel.org>
> Cc: Biju Das <biju.das.jz@bp.renesas.com>; linux-renesas-soc@vger.kernel.=
org;
> devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; Prabhakar Mahad=
ev Lad <prabhakar.mahadev-
> lad.rj@bp.renesas.com>; Geert Uytterhoeven <geert@linux-m68k.org>
> Subject: [PATCH v4 1/2] ARM: dts: r8a7742-iwg21d-q7-dbcm-ca: Separate out=
 ov5640 nodes
>=20
> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
>=20
> The camera daughter board can also be connected to 8-bit ov7725 sensors,
> so in preparation for configurable option to choose depending on the
> camera's connected separate out the ov5640 nodes in a dtsi file.
>=20
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> Reviewed-by: Biju Das <biju.das.jz@bp.renesas.com>
> [geert: describe a single camera in the .dtsi, include multiple times]
> Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
> ---
> v4:
>   - Describe a single camera in the .dtsi file,
>   - Include the .dtsi multiple times,
>=20
>  .../boot/dts/r8a7742-iwg21d-q7-dbcm-ca.dts    | 151 +++++++++---------
>  .../r8a7742-iwg21d-q7-dbcm-ov5640-single.dtsi |  30 ++++
>  2 files changed, 103 insertions(+), 78 deletions(-)
>  create mode 100644 arch/arm/boot/dts/r8a7742-iwg21d-q7-dbcm-ov5640-singl=
e.dtsi
>=20
That looks much cleaner thank you.

Cheers,
Prabhakar

> diff --git a/arch/arm/boot/dts/r8a7742-iwg21d-q7-dbcm-ca.dts b/arch/arm/b=
oot/dts/r8a7742-iwg21d-q7-
> dbcm-ca.dts
> index 98c3fbd89fa6c5c7..00634eb58ce39da5 100644
> --- a/arch/arm/boot/dts/r8a7742-iwg21d-q7-dbcm-ca.dts
> +++ b/arch/arm/boot/dts/r8a7742-iwg21d-q7-dbcm-ca.dts
> @@ -91,67 +91,12 @@ &hscif0 {
>  	status =3D "okay";
>  };
>=20
> -&i2c0 {
> -	ov5640@3c {
> -		compatible =3D "ovti,ov5640";
> -		reg =3D <0x3c>;
> -		clocks =3D <&mclk_cam1>;
> -		clock-names =3D "xclk";
> -
> -		port {
> -			ov5640_0: endpoint {
> -				bus-width =3D <8>;
> -				data-shift =3D <2>;
> -				bus-type =3D <6>;
> -				pclk-sample =3D <1>;
> -				remote-endpoint =3D <&vin0ep>;
> -			};
> -		};
> -	};
> -};
> -
>  &i2c1 {
>  	pinctrl-0 =3D <&i2c1_pins>;
>  	pinctrl-names =3D "default";
>=20
>  	status =3D "okay";
>  	clock-frequency =3D <400000>;
> -
> -	ov5640@3c {
> -		compatible =3D "ovti,ov5640";
> -		reg =3D <0x3c>;
> -		clocks =3D <&mclk_cam2>;
> -		clock-names =3D "xclk";
> -
> -		port {
> -			ov5640_1: endpoint {
> -				bus-width =3D <8>;
> -				data-shift =3D <2>;
> -				bus-type =3D <6>;
> -				pclk-sample =3D <1>;
> -				remote-endpoint =3D <&vin1ep>;
> -			};
> -		};
> -	};
> -};
> -
> -&i2c2 {
> -	ov5640@3c {
> -		compatible =3D "ovti,ov5640";
> -		reg =3D <0x3c>;
> -		clocks =3D <&mclk_cam3>;
> -		clock-names =3D "xclk";
> -
> -		port {
> -			ov5640_2: endpoint {
> -				bus-width =3D <8>;
> -				data-shift =3D <2>;
> -				bus-type =3D <6>;
> -				pclk-sample =3D <1>;
> -				remote-endpoint =3D <&vin2ep>;
> -			};
> -		};
> -	};
>  };
>=20
>  &i2c3 {
> @@ -160,23 +105,6 @@ &i2c3 {
>=20
>  	status =3D "okay";
>  	clock-frequency =3D <400000>;
> -
> -	ov5640@3c {
> -		compatible =3D "ovti,ov5640";
> -		reg =3D <0x3c>;
> -		clocks =3D <&mclk_cam4>;
> -		clock-names =3D "xclk";
> -
> -		port {
> -			ov5640_3: endpoint {
> -				bus-width =3D <8>;
> -				data-shift =3D <2>;
> -				bus-type =3D <6>;
> -				pclk-sample =3D <1>;
> -				remote-endpoint =3D <&vin3ep>;
> -			};
> -		};
> -	};
>  };
>=20
>  &pfc {
> @@ -267,6 +195,21 @@ &scifb1 {
>  	cts-gpios =3D <&gpio4 17 GPIO_ACTIVE_LOW>;
>  };
>=20
> +/*
> + * Below configuration ties VINx endpoints to ov5640/ov7725 camera endpo=
ints
> + *
> + * Uncomment the #include statements to change configuration
> + */
> +
> +/* 8bit CMOS Camera 1 (J13) */
> +#define CAM_PARENT_I2C		i2c0
> +#define MCLK_CAM		mclk_cam1
> +#define CAM_EP			cam0ep
> +#define VIN_EP			vin0ep
> +#undef CAM_ENABLED
> +#include "r8a7742-iwg21d-q7-dbcm-ov5640-single.dtsi"
> +
> +#ifdef CAM_ENABLED
>  &vin0 {
>  	/*
>  	 * Set SW2 switch on the SOM to 'ON'
> @@ -278,13 +221,28 @@ &vin0 {
>=20
>  	port {
>  		vin0ep: endpoint {
> -			remote-endpoint =3D <&ov5640_0>;
> +			remote-endpoint =3D <&cam0ep>;
>  			bus-width =3D <8>;
>  			bus-type =3D <6>;
>  		};
>  	};
>  };
> -
> +#endif /* CAM_ENABLED */
> +
> +#undef CAM_PARENT_I2C
> +#undef MCLK_CAM
> +#undef CAM_EP
> +#undef VIN_EP
> +
> +/* 8bit CMOS Camera 2 (J14) */
> +#define CAM_PARENT_I2C		i2c1
> +#define MCLK_CAM		mclk_cam2
> +#define CAM_EP			cam1ep
> +#define VIN_EP			vin1ep
> +#undef CAM_ENABLED
> +#include "r8a7742-iwg21d-q7-dbcm-ov5640-single.dtsi"
> +
> +#ifdef CAM_ENABLED
>  &vin1 {
>  	/* Set SW1 switch on the SOM to 'ON' */
>  	status =3D "okay";
> @@ -293,13 +251,29 @@ &vin1 {
>=20
>  	port {
>  		vin1ep: endpoint {
> -			remote-endpoint =3D <&ov5640_1>;
> +			remote-endpoint =3D <&cam1ep>;
>  			bus-width =3D <8>;
>  			bus-type =3D <6>;
>  		};
>  	};
>  };
>=20
> +#endif /* CAM_ENABLED */
> +
> +#undef CAM_PARENT_I2C
> +#undef MCLK_CAM
> +#undef CAM_EP
> +#undef VIN_EP
> +
> +/* 8bit CMOS Camera 3 (J12) */
> +#define CAM_PARENT_I2C		i2c2
> +#define MCLK_CAM		mclk_cam3
> +#define CAM_EP			cam2ep
> +#define VIN_EP			vin2ep
> +#undef CAM_ENABLED
> +#include "r8a7742-iwg21d-q7-dbcm-ov5640-single.dtsi"
> +
> +#ifdef CAM_ENABLED
>  &vin2 {
>  	status =3D "okay";
>  	pinctrl-0 =3D <&vin2_pins>;
> @@ -307,14 +281,29 @@ &vin2 {
>=20
>  	port {
>  		vin2ep: endpoint {
> -			remote-endpoint =3D <&ov5640_2>;
> +			remote-endpoint =3D <&cam2ep>;
>  			bus-width =3D <8>;
>  			data-shift =3D <8>;
>  			bus-type =3D <6>;
>  		};
>  	};
>  };
> -
> +#endif /* CAM_ENABLED */
> +
> +#undef CAM_PARENT_I2C
> +#undef MCLK_CAM
> +#undef CAM_EP
> +#undef VIN_EP
> +
> +/* 8bit CMOS Camera 4 (J11) */
> +#define CAM_PARENT_I2C		i2c3
> +#define MCLK_CAM		mclk_cam4
> +#define CAM_EP			cam3ep
> +#define VIN_EP			vin3ep
> +#undef CAM_ENABLED
> +#include "r8a7742-iwg21d-q7-dbcm-ov5640-single.dtsi"
> +
> +#ifdef CAM_ENABLED
>  &vin3 {
>  	status =3D "okay";
>  	pinctrl-0 =3D <&vin3_pins>;
> @@ -322,9 +311,15 @@ &vin3 {
>=20
>  	port {
>  		vin3ep: endpoint {
> -			remote-endpoint =3D <&ov5640_3>;
> +			remote-endpoint =3D <&cam3ep>;
>  			bus-width =3D <8>;
>  			bus-type =3D <6>;
>  		};
>  	};
>  };
> +#endif /* CAM_ENABLED */
> +
> +#undef CAM_PARENT_I2C
> +#undef MCLK_CAM
> +#undef CAM_EP
> +#undef VIN_EP
> diff --git a/arch/arm/boot/dts/r8a7742-iwg21d-q7-dbcm-ov5640-single.dtsi =
b/arch/arm/boot/dts/r8a7742-
> iwg21d-q7-dbcm-ov5640-single.dtsi
> new file mode 100644
> index 0000000000000000..408016620f5b1a04
> --- /dev/null
> +++ b/arch/arm/boot/dts/r8a7742-iwg21d-q7-dbcm-ov5640-single.dtsi
> @@ -0,0 +1,30 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * This include file ties a VIN interface with a single ov5640 sensor on
> + * the iWave-RZ/G1H Qseven board development platform connected with the
> + * camera daughter board.
> + *
> + * Copyright (C) 2020 Renesas Electronics Corp.
> + */
> +
> +#define CAM_ENABLED	1
> +
> +&CAM_PARENT_I2C {
> +	ov5640@3c {
> +		compatible =3D "ovti,ov5640";
> +		reg =3D <0x3c>;
> +		clocks =3D <&MCLK_CAM>;
> +		clock-names =3D "xclk";
> +		status =3D "okay";
> +
> +		port {
> +			CAM_EP: endpoint {
> +				bus-width =3D <8>;
> +				data-shift =3D <2>;
> +				bus-type =3D <6>;
> +				pclk-sample =3D <1>;
> +				remote-endpoint =3D <&VIN_EP>;
> +			};
> +		};
> +	};
> +};
> --
> 2.25.1