[PATCH v1 0/3] Enable CEC on stm32mp157c-ev1 board

[PATCH v1 0/3] Enable CEC on stm32mp157c-ev1 board

[yannick fertre]

This patch-set enables CEC (Consumer Electronics Control) on stm32mp157c
eval board.

yannick fertre (3):
  ARM: dts: stm32: add cec support on stm32mp157c
  ARM: dts: stm32: add cec pins to stm32mp157c
  ARM: dts: stm32: add cec support on stm32mp157c-ev1

 arch/arm/boot/dts/stm32mp157-pinctrl.dtsi | 9 +++++++++
 arch/arm/boot/dts/stm32mp157c-ev1.dts     | 6 ++++++
 arch/arm/boot/dts/stm32mp157c.dtsi        | 9 +++++++++
 3 files changed, 24 insertions(+)

-- 
1.9.1

[PATCH v1 1/3] ARM: dts: stm32: add cec support on stm32mp157c

[yannick fertre]

Add cec support on stm32mp157c

Signed-off-by: yannick fertre <yannick.fertre@st.com>
---
 arch/arm/boot/dts/stm32mp157c.dtsi | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/arch/arm/boot/dts/stm32mp157c.dtsi b/arch/arm/boot/dts/stm32mp157c.dtsi
index bc3eddc..b80eb6b 100644
--- a/arch/arm/boot/dts/stm32mp157c.dtsi
+++ b/arch/arm/boot/dts/stm32mp157c.dtsi
@@ -136,6 +136,15 @@
 			status = "disabled";
 		};
 
+		cec: cec@40016000 {
+			compatible = "st,stm32-cec";
+			reg = <0x40016000 0x400>;
+			interrupts = <GIC_SPI 94 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&rcc CEC_K>, <&clk_lse>;
+			clock-names = "cec", "hdmi-cec";
+			status = "disabled";
+		};
+
 		uart7: serial@40018000 {
 			compatible = "st,stm32h7-uart";
 			reg = <0x40018000 0x400>;
-- 
1.9.1

[PATCH v1 2/3] ARM: dts: stm32: add cec pins to stm32mp157c

[yannick fertre]

This patch adds cec support on stm32mp157c eval board.

Signed-off-by: Yannick Fertre <yannick.fertre@st.com>
---
 arch/arm/boot/dts/stm32mp157-pinctrl.dtsi | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi b/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
index 6f044100..67b0349 100644
--- a/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
+++ b/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
@@ -14,6 +14,15 @@
 			ranges = <0 0x50002000 0xa400>;
 			pins-are-numbered;
 
+			cec_pins_a: cec@0 {
+				pins {
+					pinmux = <STM32_PINMUX('A', 15, AF4)>;
+					bias-disable;
+					drive-open-drain;
+					slew-rate = <0>;
+				};
+			};
+
 			gpioa: gpio@50002000 {
 				gpio-controller;
 				#gpio-cells = <2>;
-- 
1.9.1

[PATCH v1 3/3] ARM: dts: stm32: add cec support on stm32mp157c-ev1

[yannick fertre]

This patch enables cec node on stm32mp157c-ev1 board

Signed-off-by: Yannick Fertre <yannick.fertre@st.com>
---
 arch/arm/boot/dts/stm32mp157c-ev1.dts | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/arch/arm/boot/dts/stm32mp157c-ev1.dts b/arch/arm/boot/dts/stm32mp157c-ev1.dts
index 57e6dbc..e2a583b 100644
--- a/arch/arm/boot/dts/stm32mp157c-ev1.dts
+++ b/arch/arm/boot/dts/stm32mp157c-ev1.dts
@@ -19,3 +19,9 @@
 		serial0 = &uart4;
 	};
 };
+
+&cec {
+	pinctrl-names = "default";
+	pinctrl-0 = <&cec_pins_a>;
+	status = "okay";
+};
-- 
1.9.1

Re: [PATCH v1 1/3] ARM: dts: stm32: add cec support on stm32mp157c

[kbuild test robot]

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

Hi yannick,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on robh/for-next]
[also build test ERROR on v4.17-rc2 next-20180424]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/yannick-fertre/Enable-CEC-on-stm32mp157c-ev1-board/20180424-195256
base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: arm-cns3420vb_defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (Debian 7.2.0-11) 7.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=arm 

All errors (new ones prefixed by >>):

   Error: arch/arm/boot/dts/stm32mp157c.dtsi:166.19-20 syntax error
>> FATAL ERROR: Unable to parse input tree

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 9408 bytes --]

Re: [PATCH v1 2/3] ARM: dts: stm32: add cec pins to stm32mp157c

[Alexandre Torgue]

Hi Yannick

On 04/24/2018 09:54 AM, yannick fertre wrote:
> This patch adds cec support on stm32mp157c eval board.
> 
> Signed-off-by: Yannick Fertre <yannick.fertre@st.com>
> ---
>   arch/arm/boot/dts/stm32mp157-pinctrl.dtsi | 9 +++++++++
>   1 file changed, 9 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi b/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
> index 6f044100..67b0349 100644
> --- a/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
> +++ b/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
> @@ -14,6 +14,15 @@
>   			ranges = <0 0x50002000 0xa400>;
>   			pins-are-numbered;
>   
> +			cec_pins_a: cec@0 {
> +				pins {
> +					pinmux = <STM32_PINMUX('A', 15, AF4)>;
> +					bias-disable;
> +					drive-open-drain;
> +					slew-rate = <0>;
> +				};
> +			};
> +
>   			gpioa: gpio@50002000 {
>   				gpio-controller;
>   				#gpio-cells = <2>;
> 
Next time, please put pins group after gpio controller and don't use 
cec@0 but cec-0 to declare pins group. I'll change it this time during 
merge.

Thanks
Alex

Re: [PATCH v1 0/3] Enable CEC on stm32mp157c-ev1 board

[Alexandre Torgue]

hi,

On 04/24/2018 09:54 AM, yannick fertre wrote:
> This patch-set enables CEC (Consumer Electronics Control) on stm32mp157c
> eval board.
> 
> yannick fertre (3):
>    ARM: dts: stm32: add cec support on stm32mp157c
>    ARM: dts: stm32: add cec pins to stm32mp157c
>    ARM: dts: stm32: add cec support on stm32mp157c-ev1
> 
>   arch/arm/boot/dts/stm32mp157-pinctrl.dtsi | 9 +++++++++
>   arch/arm/boot/dts/stm32mp157c-ev1.dts     | 6 ++++++
>   arch/arm/boot/dts/stm32mp157c.dtsi        | 9 +++++++++
>   3 files changed, 24 insertions(+)
> 

Series applied on stm32-next.

Regards
Alex