diff mbox

[PATCHv5,11/11] arm: sti: update sti-cec for CEC notifier support

Message ID 20170329141543.32935-12-hverkuil@xs4all.nl (mailing list archive)
State Not Applicable
Headers show

Commit Message

Hans Verkuil March 29, 2017, 2:15 p.m. UTC 
From: Benjamin Gaignard <benjamin.gaignard@linaro.org>

To use CEC notifier sti CEC driver needs to get phandle
of the hdmi device.

Signed-off-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
CC: devicetree@vger.kernel.org
---
 arch/arm/boot/dts/stih407-family.dtsi | 12 ------------
 arch/arm/boot/dts/stih410.dtsi        | 13 +++++++++++++
 2 files changed, 13 insertions(+), 12 deletions(-)

Comments

Benjamin Gaignard March 30, 2017, 7:41 a.m. UTC | #1 
+ Patrice for sti DT

2017-03-29 16:15 GMT+02:00 Hans Verkuil <hverkuil@xs4all.nl>:
> From: Benjamin Gaignard <benjamin.gaignard@linaro.org>
>
> To use CEC notifier sti CEC driver needs to get phandle
> of the hdmi device.
>
> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>
> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
> CC: devicetree@vger.kernel.org
> ---
>  arch/arm/boot/dts/stih407-family.dtsi | 12 ------------
>  arch/arm/boot/dts/stih410.dtsi        | 13 +++++++++++++
>  2 files changed, 13 insertions(+), 12 deletions(-)
>
> diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi
> index d753ac36788f..044184580326 100644
> --- a/arch/arm/boot/dts/stih407-family.dtsi
> +++ b/arch/arm/boot/dts/stih407-family.dtsi
> @@ -742,18 +742,6 @@
>                                  <&clk_s_c0_flexgen CLK_ETH_PHY>;
>                 };
>
> -               cec: sti-cec@094a087c {
> -                       compatible = "st,stih-cec";
> -                       reg = <0x94a087c 0x64>;
> -                       clocks = <&clk_sysin>;
> -                       clock-names = "cec-clk";
> -                       interrupts = <GIC_SPI 140 IRQ_TYPE_NONE>;
> -                       interrupt-names = "cec-irq";
> -                       pinctrl-names = "default";
> -                       pinctrl-0 = <&pinctrl_cec0_default>;
> -                       resets = <&softreset STIH407_LPM_SOFTRESET>;
> -               };
> -
>                 rng10: rng@08a89000 {
>                         compatible      = "st,rng";
>                         reg             = <0x08a89000 0x1000>;
> diff --git a/arch/arm/boot/dts/stih410.dtsi b/arch/arm/boot/dts/stih410.dtsi
> index 3c9672c5b09f..21fe72b183d8 100644
> --- a/arch/arm/boot/dts/stih410.dtsi
> +++ b/arch/arm/boot/dts/stih410.dtsi
> @@ -281,5 +281,18 @@
>                                  <&clk_s_c0_flexgen CLK_ST231_DMU>,
>                                  <&clk_s_c0_flexgen CLK_FLASH_PROMIP>;
>                 };
> +
> +               sti-cec@094a087c {
> +                       compatible = "st,stih-cec";
> +                       reg = <0x94a087c 0x64>;
> +                       clocks = <&clk_sysin>;
> +                       clock-names = "cec-clk";
> +                       interrupts = <GIC_SPI 140 IRQ_TYPE_NONE>;
> +                       interrupt-names = "cec-irq";
> +                       pinctrl-names = "default";
> +                       pinctrl-0 = <&pinctrl_cec0_default>;
> +                       resets = <&softreset STIH407_LPM_SOFTRESET>;
> +                       hdmi-phandle = <&sti_hdmi>;
> +               };
>         };
>  };
> --
> 2.11.0
>
Patrice CHOTARD March 30, 2017, 9:30 a.m. UTC | #2 
Hi Benjamin

On 03/30/2017 09:41 AM, Benjamin Gaignard wrote:
> + Patrice for sti DT


In order to be coherent with all previous STi DT patches,

can you update the commit message with "ARM: dts: STiH410: update 
sti-cec for CEC notifier support"

Thanks

Patrice

>

> 2017-03-29 16:15 GMT+02:00 Hans Verkuil <hverkuil@xs4all.nl>:

>> From: Benjamin Gaignard <benjamin.gaignard@linaro.org>

>>

>> To use CEC notifier sti CEC driver needs to get phandle

>> of the hdmi device.

>>

>> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>

>> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>

>> CC: devicetree@vger.kernel.org

>> ---

>>  arch/arm/boot/dts/stih407-family.dtsi | 12 ------------

>>  arch/arm/boot/dts/stih410.dtsi        | 13 +++++++++++++

>>  2 files changed, 13 insertions(+), 12 deletions(-)

>>

>> diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi

>> index d753ac36788f..044184580326 100644

>> --- a/arch/arm/boot/dts/stih407-family.dtsi

>> +++ b/arch/arm/boot/dts/stih407-family.dtsi

>> @@ -742,18 +742,6 @@

>>                                  <&clk_s_c0_flexgen CLK_ETH_PHY>;

>>                 };

>>

>> -               cec: sti-cec@094a087c {

>> -                       compatible = "st,stih-cec";

>> -                       reg = <0x94a087c 0x64>;

>> -                       clocks = <&clk_sysin>;

>> -                       clock-names = "cec-clk";

>> -                       interrupts = <GIC_SPI 140 IRQ_TYPE_NONE>;

>> -                       interrupt-names = "cec-irq";

>> -                       pinctrl-names = "default";

>> -                       pinctrl-0 = <&pinctrl_cec0_default>;

>> -                       resets = <&softreset STIH407_LPM_SOFTRESET>;

>> -               };

>> -

>>                 rng10: rng@08a89000 {

>>                         compatible      = "st,rng";

>>                         reg             = <0x08a89000 0x1000>;

>> diff --git a/arch/arm/boot/dts/stih410.dtsi b/arch/arm/boot/dts/stih410.dtsi

>> index 3c9672c5b09f..21fe72b183d8 100644

>> --- a/arch/arm/boot/dts/stih410.dtsi

>> +++ b/arch/arm/boot/dts/stih410.dtsi

>> @@ -281,5 +281,18 @@

>>                                  <&clk_s_c0_flexgen CLK_ST231_DMU>,

>>                                  <&clk_s_c0_flexgen CLK_FLASH_PROMIP>;

>>                 };

>> +

>> +               sti-cec@094a087c {

>> +                       compatible = "st,stih-cec";

>> +                       reg = <0x94a087c 0x64>;

>> +                       clocks = <&clk_sysin>;

>> +                       clock-names = "cec-clk";

>> +                       interrupts = <GIC_SPI 140 IRQ_TYPE_NONE>;

>> +                       interrupt-names = "cec-irq";

>> +                       pinctrl-names = "default";

>> +                       pinctrl-0 = <&pinctrl_cec0_default>;

>> +                       resets = <&softreset STIH407_LPM_SOFTRESET>;

>> +                       hdmi-phandle = <&sti_hdmi>;

>> +               };

>>         };

>>  };

>> --

>> 2.11.0

>>

>

>

>
Hans Verkuil (hansverk) March 30, 2017, 9:32 a.m. UTC | #3 
On 03/30/2017 11:30 AM, Patrice CHOTARD wrote:
> Hi Benjamin
> 
> On 03/30/2017 09:41 AM, Benjamin Gaignard wrote:
>> + Patrice for sti DT
> 
> In order to be coherent with all previous STi DT patches,
> 
> can you update the commit message with "ARM: dts: STiH410: update 
> sti-cec for CEC notifier support"

Done.

Regards,

	Hans

> 
> Thanks
> 
> Patrice
> 
>>
>> 2017-03-29 16:15 GMT+02:00 Hans Verkuil <hverkuil@xs4all.nl>:
>>> From: Benjamin Gaignard <benjamin.gaignard@linaro.org>
>>>
>>> To use CEC notifier sti CEC driver needs to get phandle
>>> of the hdmi device.
>>>
>>> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@linaro.org>
>>> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
>>> CC: devicetree@vger.kernel.org
>>> ---
>>>  arch/arm/boot/dts/stih407-family.dtsi | 12 ------------
>>>  arch/arm/boot/dts/stih410.dtsi        | 13 +++++++++++++
>>>  2 files changed, 13 insertions(+), 12 deletions(-)
>>>
>>> diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi
>>> index d753ac36788f..044184580326 100644
>>> --- a/arch/arm/boot/dts/stih407-family.dtsi
>>> +++ b/arch/arm/boot/dts/stih407-family.dtsi
>>> @@ -742,18 +742,6 @@
>>>                                  <&clk_s_c0_flexgen CLK_ETH_PHY>;
>>>                 };
>>>
>>> -               cec: sti-cec@094a087c {
>>> -                       compatible = "st,stih-cec";
>>> -                       reg = <0x94a087c 0x64>;
>>> -                       clocks = <&clk_sysin>;
>>> -                       clock-names = "cec-clk";
>>> -                       interrupts = <GIC_SPI 140 IRQ_TYPE_NONE>;
>>> -                       interrupt-names = "cec-irq";
>>> -                       pinctrl-names = "default";
>>> -                       pinctrl-0 = <&pinctrl_cec0_default>;
>>> -                       resets = <&softreset STIH407_LPM_SOFTRESET>;
>>> -               };
>>> -
>>>                 rng10: rng@08a89000 {
>>>                         compatible      = "st,rng";
>>>                         reg             = <0x08a89000 0x1000>;
>>> diff --git a/arch/arm/boot/dts/stih410.dtsi b/arch/arm/boot/dts/stih410.dtsi
>>> index 3c9672c5b09f..21fe72b183d8 100644
>>> --- a/arch/arm/boot/dts/stih410.dtsi
>>> +++ b/arch/arm/boot/dts/stih410.dtsi
>>> @@ -281,5 +281,18 @@
>>>                                  <&clk_s_c0_flexgen CLK_ST231_DMU>,
>>>                                  <&clk_s_c0_flexgen CLK_FLASH_PROMIP>;
>>>                 };
>>> +
>>> +               sti-cec@094a087c {
>>> +                       compatible = "st,stih-cec";
>>> +                       reg = <0x94a087c 0x64>;
>>> +                       clocks = <&clk_sysin>;
>>> +                       clock-names = "cec-clk";
>>> +                       interrupts = <GIC_SPI 140 IRQ_TYPE_NONE>;
>>> +                       interrupt-names = "cec-irq";
>>> +                       pinctrl-names = "default";
>>> +                       pinctrl-0 = <&pinctrl_cec0_default>;
>>> +                       resets = <&softreset STIH407_LPM_SOFTRESET>;
>>> +                       hdmi-phandle = <&sti_hdmi>;
>>> +               };
>>>         };
>>>  };
>>> --
>>> 2.11.0
>>>
>>
>>
>>
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi
index d753ac36788f..044184580326 100644
--- a/arch/arm/boot/dts/stih407-family.dtsi
+++ b/arch/arm/boot/dts/stih407-family.dtsi
@@ -742,18 +742,6 @@ 
 				 <&clk_s_c0_flexgen CLK_ETH_PHY>;
 		};
 
-		cec: sti-cec@094a087c {
-			compatible = "st,stih-cec";
-			reg = <0x94a087c 0x64>;
-			clocks = <&clk_sysin>;
-			clock-names = "cec-clk";
-			interrupts = <GIC_SPI 140 IRQ_TYPE_NONE>;
-			interrupt-names = "cec-irq";
-			pinctrl-names = "default";
-			pinctrl-0 = <&pinctrl_cec0_default>;
-			resets = <&softreset STIH407_LPM_SOFTRESET>;
-		};
-
 		rng10: rng@08a89000 {
 			compatible      = "st,rng";
 			reg		= <0x08a89000 0x1000>;
diff --git a/arch/arm/boot/dts/stih410.dtsi b/arch/arm/boot/dts/stih410.dtsi
index 3c9672c5b09f..21fe72b183d8 100644
--- a/arch/arm/boot/dts/stih410.dtsi
+++ b/arch/arm/boot/dts/stih410.dtsi
@@ -281,5 +281,18 @@ 
 				 <&clk_s_c0_flexgen CLK_ST231_DMU>,
 				 <&clk_s_c0_flexgen CLK_FLASH_PROMIP>;
 		};
+
+		sti-cec@094a087c {
+			compatible = "st,stih-cec";
+			reg = <0x94a087c 0x64>;
+			clocks = <&clk_sysin>;
+			clock-names = "cec-clk";
+			interrupts = <GIC_SPI 140 IRQ_TYPE_NONE>;
+			interrupt-names = "cec-irq";
+			pinctrl-names = "default";
+			pinctrl-0 = <&pinctrl_cec0_default>;
+			resets = <&softreset STIH407_LPM_SOFTRESET>;
+			hdmi-phandle = <&sti_hdmi>;
+		};
 	};
 };