diff mbox

ARM: dts: am437x-gp-evm: add DT nodes for ov2659 sensor

Message ID 1426203501-32115-1-git-send-email-prabhakar.csengg@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Lad, Prabhakar March 12, 2015, 11:38 p.m. UTC
From: "Lad, Prabhakar" <prabhakar.csengg@gmail.com>

this patch does the following:
1: adds DT node for fixed oscillator.
2: adds DT node entries for ov2659 sensor
3: adds remote-endpoint entry for VPFE.

Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
---
 Note this patch depends on
 https://patchwork.kernel.org/patch/6000161/
 
 arch/arm/boot/dts/am437x-gp-evm.dts | 42 +++++++++++++++++++++++++++++++++++--
 1 file changed, 40 insertions(+), 2 deletions(-)

Comments

Tony Lindgren March 16, 2015, 10:17 p.m. UTC | #1
* Lad Prabhakar <prabhakar.csengg@gmail.com> [150312 16:38]:
> From: "Lad, Prabhakar" <prabhakar.csengg@gmail.com>
> 
> this patch does the following:
> 1: adds DT node for fixed oscillator.
> 2: adds DT node entries for ov2659 sensor
> 3: adds remote-endpoint entry for VPFE.
> 
> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>

Applying into omap-for-v4.1/dt thanks.

Tony

> ---
>  Note this patch depends on
>  https://patchwork.kernel.org/patch/6000161/
>  
>  arch/arm/boot/dts/am437x-gp-evm.dts | 42 +++++++++++++++++++++++++++++++++++--
>  1 file changed, 40 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/am437x-gp-evm.dts b/arch/arm/boot/dts/am437x-gp-evm.dts
> index f84d971..195f452 100644
> --- a/arch/arm/boot/dts/am437x-gp-evm.dts
> +++ b/arch/arm/boot/dts/am437x-gp-evm.dts
> @@ -106,6 +106,14 @@
>  			};
>  		};
>  	};
> +
> +	/* fixed 12MHz oscillator */
> +	refclk: oscillator {
> +		#clock-cells = <0>;
> +		compatible = "fixed-clock";
> +		clock-frequency = <12000000>;
> +	};
> +
>  };
>  
>  &am43xx_pinmux {
> @@ -404,6 +412,21 @@
>  			regulator-always-on;
>  		};
>  	};
> +
> +	ov2659@30 {
> +		compatible = "ovti,ov2659";
> +		reg = <0x30>;
> +
> +		clocks = <&refclk 0>;
> +		clock-names = "xvclk";
> +
> +		port {
> +			ov2659_0: endpoint {
> +				remote-endpoint = <&vpfe1_ep>;
> +				link-frequencies = /bits/ 64 <70000000>;
> +			};
> +		};
> +	};
>  };
>  
>  &i2c1 {
> @@ -423,6 +446,21 @@
>  		touchscreen-size-x = <1024>;
>  		touchscreen-size-y = <600>;
>  	};
> +
> +	ov2659@30 {
> +		compatible = "ovti,ov2659";
> +		reg = <0x30>;
> +
> +		clocks = <&refclk 0>;
> +		clock-names = "xvclk";
> +
> +		port {
> +			ov2659_1: endpoint {
> +				remote-endpoint = <&vpfe0_ep>;
> +				link-frequencies = /bits/ 64 <70000000>;
> +			};
> +		};
> +	};
>  };
>  
>  &epwmss0 {
> @@ -626,7 +664,7 @@
>  
>  	port {
>  		vpfe0_ep: endpoint {
> -			/* remote-endpoint = <&sensor>; add once we have it */
> +			remote-endpoint = <&ov2659_1>;
>  			ti,am437x-vpfe-interface = <0>;
>  			bus-width = <8>;
>  			hsync-active = <0>;
> @@ -643,7 +681,7 @@
>  
>  	port {
>  		vpfe1_ep: endpoint {
> -			/* remote-endpoint = <&sensor>; add once we have it */
> +			remote-endpoint = <&ov2659_0>;
>  			ti,am437x-vpfe-interface = <0>;
>  			bus-width = <8>;
>  			hsync-active = <0>;
> -- 
> 2.1.0
>
Lad, Prabhakar March 17, 2015, 1:20 a.m. UTC | #2
Hi Tony,

On Mon, Mar 16, 2015 at 10:17 PM, Tony Lindgren <tony@atomide.com> wrote:
> * Lad Prabhakar <prabhakar.csengg@gmail.com> [150312 16:38]:
>> From: "Lad, Prabhakar" <prabhakar.csengg@gmail.com>
>>
>> this patch does the following:
>> 1: adds DT node for fixed oscillator.
>> 2: adds DT node entries for ov2659 sensor
>> 3: adds remote-endpoint entry for VPFE.
>>
>> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
>
> Applying into omap-for-v4.1/dt thanks.
>
I would like to get this one in via media tree to avoid dependency
as I am still waiting for Acks from DT maintainers for the sensor
driver.

If I can get your Ack on this I'll queue it up along with sensor
driver via media tree.

Cheers,
--Prabhakar Lad

> Tony
>
>> ---
>>  Note this patch depends on
>>  https://patchwork.kernel.org/patch/6000161/
>>
>>  arch/arm/boot/dts/am437x-gp-evm.dts | 42 +++++++++++++++++++++++++++++++++++--
>>  1 file changed, 40 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/am437x-gp-evm.dts b/arch/arm/boot/dts/am437x-gp-evm.dts
>> index f84d971..195f452 100644
>> --- a/arch/arm/boot/dts/am437x-gp-evm.dts
>> +++ b/arch/arm/boot/dts/am437x-gp-evm.dts
>> @@ -106,6 +106,14 @@
>>                       };
>>               };
>>       };
>> +
>> +     /* fixed 12MHz oscillator */
>> +     refclk: oscillator {
>> +             #clock-cells = <0>;
>> +             compatible = "fixed-clock";
>> +             clock-frequency = <12000000>;
>> +     };
>> +
>>  };
>>
>>  &am43xx_pinmux {
>> @@ -404,6 +412,21 @@
>>                       regulator-always-on;
>>               };
>>       };
>> +
>> +     ov2659@30 {
>> +             compatible = "ovti,ov2659";
>> +             reg = <0x30>;
>> +
>> +             clocks = <&refclk 0>;
>> +             clock-names = "xvclk";
>> +
>> +             port {
>> +                     ov2659_0: endpoint {
>> +                             remote-endpoint = <&vpfe1_ep>;
>> +                             link-frequencies = /bits/ 64 <70000000>;
>> +                     };
>> +             };
>> +     };
>>  };
>>
>>  &i2c1 {
>> @@ -423,6 +446,21 @@
>>               touchscreen-size-x = <1024>;
>>               touchscreen-size-y = <600>;
>>       };
>> +
>> +     ov2659@30 {
>> +             compatible = "ovti,ov2659";
>> +             reg = <0x30>;
>> +
>> +             clocks = <&refclk 0>;
>> +             clock-names = "xvclk";
>> +
>> +             port {
>> +                     ov2659_1: endpoint {
>> +                             remote-endpoint = <&vpfe0_ep>;
>> +                             link-frequencies = /bits/ 64 <70000000>;
>> +                     };
>> +             };
>> +     };
>>  };
>>
>>  &epwmss0 {
>> @@ -626,7 +664,7 @@
>>
>>       port {
>>               vpfe0_ep: endpoint {
>> -                     /* remote-endpoint = <&sensor>; add once we have it */
>> +                     remote-endpoint = <&ov2659_1>;
>>                       ti,am437x-vpfe-interface = <0>;
>>                       bus-width = <8>;
>>                       hsync-active = <0>;
>> @@ -643,7 +681,7 @@
>>
>>       port {
>>               vpfe1_ep: endpoint {
>> -                     /* remote-endpoint = <&sensor>; add once we have it */
>> +                     remote-endpoint = <&ov2659_0>;
>>                       ti,am437x-vpfe-interface = <0>;
>>                       bus-width = <8>;
>>                       hsync-active = <0>;
>> --
>> 2.1.0
>>
Tony Lindgren March 17, 2015, 2:23 a.m. UTC | #3
* Lad, Prabhakar <prabhakar.csengg@gmail.com> [150316 18:20]:
> Hi Tony,
> 
> On Mon, Mar 16, 2015 at 10:17 PM, Tony Lindgren <tony@atomide.com> wrote:
> > * Lad Prabhakar <prabhakar.csengg@gmail.com> [150312 16:38]:
> >> From: "Lad, Prabhakar" <prabhakar.csengg@gmail.com>
> >>
> >> this patch does the following:
> >> 1: adds DT node for fixed oscillator.
> >> 2: adds DT node entries for ov2659 sensor
> >> 3: adds remote-endpoint entry for VPFE.
> >>
> >> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
> >
> > Applying into omap-for-v4.1/dt thanks.
> >
> I would like to get this one in via media tree to avoid dependency
> as I am still waiting for Acks from DT maintainers for the sensor
> driver.

OK dropping it.
 
> If I can get your Ack on this I'll queue it up along with sensor
> driver via media tree.

Sorry the chances are too big for pointless merge conflicts with
these files with constant patching going on.

Please just resend this patch alone again to me later on once the
driver changes are merged into Linux next and on their way to the
mainline kernel.

Regards,

Tony
Lad, Prabhakar April 10, 2015, 8:22 p.m. UTC | #4
Hi Tony,

On Tue, Mar 17, 2015 at 2:23 AM, Tony Lindgren <tony@atomide.com> wrote:
> * Lad, Prabhakar <prabhakar.csengg@gmail.com> [150316 18:20]:
>> Hi Tony,
>>
>> On Mon, Mar 16, 2015 at 10:17 PM, Tony Lindgren <tony@atomide.com> wrote:
>> > * Lad Prabhakar <prabhakar.csengg@gmail.com> [150312 16:38]:
>> >> From: "Lad, Prabhakar" <prabhakar.csengg@gmail.com>
>> >>
>> >> this patch does the following:
>> >> 1: adds DT node for fixed oscillator.
>> >> 2: adds DT node entries for ov2659 sensor
>> >> 3: adds remote-endpoint entry for VPFE.
>> >>
>> >> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
>> >
>> > Applying into omap-for-v4.1/dt thanks.
>> >
>> I would like to get this one in via media tree to avoid dependency
>> as I am still waiting for Acks from DT maintainers for the sensor
>> driver.
>
> OK dropping it.
>
>> If I can get your Ack on this I'll queue it up along with sensor
>> driver via media tree.
>
> Sorry the chances are too big for pointless merge conflicts with
> these files with constant patching going on.
>
> Please just resend this patch alone again to me later on once the
> driver changes are merged into Linux next and on their way to the
> mainline kernel.
>
the sensor driver is now present in linux-next and the same patch
applies cleanly, can you pick up the same or do you want me to
resend the patch ?

Cheers,
--Prabhakar Lad
Lad, Prabhakar May 1, 2015, 8:55 a.m. UTC | #5
On Fri, Apr 10, 2015 at 9:22 PM, Lad, Prabhakar
<prabhakar.csengg@gmail.com> wrote:
> Hi Tony,
>
> On Tue, Mar 17, 2015 at 2:23 AM, Tony Lindgren <tony@atomide.com> wrote:
>> * Lad, Prabhakar <prabhakar.csengg@gmail.com> [150316 18:20]:
>>> Hi Tony,
>>>
>>> On Mon, Mar 16, 2015 at 10:17 PM, Tony Lindgren <tony@atomide.com> wrote:
>>> > * Lad Prabhakar <prabhakar.csengg@gmail.com> [150312 16:38]:
>>> >> From: "Lad, Prabhakar" <prabhakar.csengg@gmail.com>
>>> >>
>>> >> this patch does the following:
>>> >> 1: adds DT node for fixed oscillator.
>>> >> 2: adds DT node entries for ov2659 sensor
>>> >> 3: adds remote-endpoint entry for VPFE.
>>> >>
>>> >> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
>>> >
>>> > Applying into omap-for-v4.1/dt thanks.
>>> >
>>> I would like to get this one in via media tree to avoid dependency
>>> as I am still waiting for Acks from DT maintainers for the sensor
>>> driver.
>>
>> OK dropping it.
>>
>>> If I can get your Ack on this I'll queue it up along with sensor
>>> driver via media tree.
>>
>> Sorry the chances are too big for pointless merge conflicts with
>> these files with constant patching going on.
>>
>> Please just resend this patch alone again to me later on once the
>> driver changes are merged into Linux next and on their way to the
>> mainline kernel.
>>
> the sensor driver is now present in linux-next and the same patch
> applies cleanly, can you pick up the same or do you want me to
> resend the patch ?
>
Ping...

Cheers,
--Prabhakar Lad
Tony Lindgren May 20, 2015, 6:14 p.m. UTC | #6
* Tony Lindgren <tony@atomide.com> [150316 15:22]:
> * Lad Prabhakar <prabhakar.csengg@gmail.com> [150312 16:38]:
> > From: "Lad, Prabhakar" <prabhakar.csengg@gmail.com>
> > 
> > this patch does the following:
> > 1: adds DT node for fixed oscillator.
> > 2: adds DT node entries for ov2659 sensor
> > 3: adds remote-endpoint entry for VPFE.
> > 
> > Signed-off-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
> 
> Applying into omap-for-v4.1/dt thanks.

Hmm looks like I somehow dropped this one, sorry. Applying into
omap-for-v4.2/dt now thanks.

Regards,

Tony
diff mbox

Patch

diff --git a/arch/arm/boot/dts/am437x-gp-evm.dts b/arch/arm/boot/dts/am437x-gp-evm.dts
index f84d971..195f452 100644
--- a/arch/arm/boot/dts/am437x-gp-evm.dts
+++ b/arch/arm/boot/dts/am437x-gp-evm.dts
@@ -106,6 +106,14 @@ 
 			};
 		};
 	};
+
+	/* fixed 12MHz oscillator */
+	refclk: oscillator {
+		#clock-cells = <0>;
+		compatible = "fixed-clock";
+		clock-frequency = <12000000>;
+	};
+
 };
 
 &am43xx_pinmux {
@@ -404,6 +412,21 @@ 
 			regulator-always-on;
 		};
 	};
+
+	ov2659@30 {
+		compatible = "ovti,ov2659";
+		reg = <0x30>;
+
+		clocks = <&refclk 0>;
+		clock-names = "xvclk";
+
+		port {
+			ov2659_0: endpoint {
+				remote-endpoint = <&vpfe1_ep>;
+				link-frequencies = /bits/ 64 <70000000>;
+			};
+		};
+	};
 };
 
 &i2c1 {
@@ -423,6 +446,21 @@ 
 		touchscreen-size-x = <1024>;
 		touchscreen-size-y = <600>;
 	};
+
+	ov2659@30 {
+		compatible = "ovti,ov2659";
+		reg = <0x30>;
+
+		clocks = <&refclk 0>;
+		clock-names = "xvclk";
+
+		port {
+			ov2659_1: endpoint {
+				remote-endpoint = <&vpfe0_ep>;
+				link-frequencies = /bits/ 64 <70000000>;
+			};
+		};
+	};
 };
 
 &epwmss0 {
@@ -626,7 +664,7 @@ 
 
 	port {
 		vpfe0_ep: endpoint {
-			/* remote-endpoint = <&sensor>; add once we have it */
+			remote-endpoint = <&ov2659_1>;
 			ti,am437x-vpfe-interface = <0>;
 			bus-width = <8>;
 			hsync-active = <0>;
@@ -643,7 +681,7 @@ 
 
 	port {
 		vpfe1_ep: endpoint {
-			/* remote-endpoint = <&sensor>; add once we have it */
+			remote-endpoint = <&ov2659_0>;
 			ti,am437x-vpfe-interface = <0>;
 			bus-width = <8>;
 			hsync-active = <0>;