Re: [PATCH 4/5] ARM: dts: exynos5420: add hdmi hpd gpio pinctrl node

[Tomasz Figa <tomasz.figa@gmail.com>]

Hi Rahul,

On Thursday 20 of June 2013 11:11:50 Rahul Sharma wrote:
> Add pinctrl node for hdmi-hpd gpio pin to exynos5420
> device tree files.
> 
> Signed-off-by: Rahul Sharma <rahul.sharma@samsung.com>
> ---
>  arch/arm/boot/dts/exynos5420-pinctrl.dtsi |    7 +++++++
>  arch/arm/boot/dts/exynos5420-smdk5420.dts |    2 ++
>  2 files changed, 9 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/exynos5420-pinctrl.dtsi
> b/arch/arm/boot/dts/exynos5420-pinctrl.dtsi index 5848c42..5706ad9
> 100644
> --- a/arch/arm/boot/dts/exynos5420-pinctrl.dtsi
> +++ b/arch/arm/boot/dts/exynos5420-pinctrl.dtsi
> @@ -59,6 +59,13 @@
>  			interrupt-controller;
>  			#interrupt-cells = <2>;
>  		};
> +
> +		hdmi_hpd_irq: hdmi-hpd-irq {
> +		        samsung,pins = "gpx3-7";
> +			samsung,pin-function = <0>;
> +			samsung,pin-pud = <1>;
> +			samsung,pin-drv = <0>;
> +		};

This looks like a board-specific pinconf entry. It should be specified in 
the dts of appropriate board, in this case exynos5420-smdk5420.dts .

Best regards,
Tomasz

>  	};
> 
>  	pinctrl@13410000 {
> diff --git a/arch/arm/boot/dts/exynos5420-smdk5420.dts
> b/arch/arm/boot/dts/exynos5420-smdk5420.dts index b3721bd..2d6c74f
> 100644
> --- a/arch/arm/boot/dts/exynos5420-smdk5420.dts
> +++ b/arch/arm/boot/dts/exynos5420-smdk5420.dts
> @@ -33,6 +33,8 @@
> 
>  	hdmi {
>  		hpd-gpio = <&gpx3 7 0>;
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&hdmi_hpd_irq>;
>  	};
> 
>  	i2c_2: i2c@12C80000 {