| Message ID | 20170329141543.32935-12-hverkuil@xs4all.nl (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
+ 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 >
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 >> > > >
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 >>> >> >> >> >
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>; + }; }; };