linux-kernel - Re: [PATCH] ARM: dts: Add initial pin configuration for exynos3250-rinato

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-id: <56E65423.1070001@samsung.com>
Date:	Mon, 14 Mar 2016 15:03:15 +0900
From:	Krzysztof Kozlowski <k.kozlowski@...sung.com>
To:	Chanwoo Choi <cw00.choi@...sung.com>, kgene@...nel.org
Cc:	linux-samsung-soc@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
	devicetree@...r.kernel.org, Kukjin Kim <kgene.kim@...sung.com>
Subject: Re: [PATCH] ARM: dts: Add initial pin configuration for
 exynos3250-rinato

On 14.03.2016 10:12, Chanwoo Choi wrote:
> This patch adds initial pin configuration using pinctrl subsystem
> to reduce leakage power-consumption of gpio pins in normal state.
> All pins included in this patch are NC (not connected) pin.
> 
> Cc: Kukjin Kim <kgene.kim@...sung.com>
> Cc: Krzysztof Kozlowski <k.kozlowski@...sung.com>
> Signed-off-by: Chanwoo Choi <cw00.choi@...sung.com>
> ---
>  arch/arm/boot/dts/exynos3250-pinctrl.dtsi | 38 +++++++++++++++++
>  arch/arm/boot/dts/exynos3250-rinato.dts   | 70 ++++++++++++++++++++++++++++++-
>  2 files changed, 106 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/exynos3250-pinctrl.dtsi b/arch/arm/boot/dts/exynos3250-pinctrl.dtsi
> index 5ab81c39e2c9..ecf79386e891 100644
> --- a/arch/arm/boot/dts/exynos3250-pinctrl.dtsi
> +++ b/arch/arm/boot/dts/exynos3250-pinctrl.dtsi
> @@ -16,11 +16,49 @@
>  #define PIN_PULL_DOWN		1
>  #define PIN_PULL_UP		3
>  
> +#define PIN_DRV_LV1		0
> +#define PIN_DRV_LV2		2
> +#define PIN_DRV_LV3		1
> +#define PIN_DRV_LV4		3
> +
>  #define PIN_PDN_OUT0		0
>  #define PIN_PDN_OUT1		1
>  #define PIN_PDN_INPUT		2
>  #define PIN_PDN_PREV		3
>  
> +#define PIN_IN(_pin, _pull, _drv)			\
> +	_pin {						\
> +		samsung,pins = #_pin;			\
> +		samsung,pin-function = <0>;		\
> +		samsung,pin-pud = <PIN_PULL_ ##_pull>;	\
> +		samsung,pin-drv = <PIN_DRV_ ##_drv>;	\
> +	}
> +
> +#define PIN_OUT(_pin, _drv)				\
> +	_pin {						\
> +		samsung,pins = #_pin;			\
> +		samsung,pin-function = <1>;		\
> +		samsung,pin-pud = <0>;			\
> +		samsung,pin-drv = <PIN_DRV_ ##_drv>;	\
> +	}
> +
> +#define PIN_OUT_SET(_pin, _val, _drv)			\
> +	_pin {						\
> +		samsung,pins = #_pin;			\
> +		samsung,pin-function = <1>;		\
> +		samsung,pin-pud = <0>;			\
> +		samsung,pin-drv = <PIN_DRV_ ##_drv>;	\
> +		samsung,pin-val = <_val>;		\
> +	}
> +
> +#define PIN_CFG(_pin, _sel, _pull, _drv)		\
> +	_pin {						\
> +		samsung,pins = #_pin;			\
> +		samsung,pin-function = <_sel>;		\
> +		samsung,pin-pud = <PIN_PULL_ ##_pull>;	\
> +		samsung,pin-drv = <PIN_DRV_ ##_drv>;	\
> +	}
> +
>  #define PIN_SLP(_pin, _mode, _pull)				\
>  	_pin {							\
>  		samsung,pins = #_pin;				\
> diff --git a/arch/arm/boot/dts/exynos3250-rinato.dts b/arch/arm/boot/dts/exynos3250-rinato.dts
> index 3e64d5dcdd60..1c78a2153776 100644
> --- a/arch/arm/boot/dts/exynos3250-rinato.dts
> +++ b/arch/arm/boot/dts/exynos3250-rinato.dts
> @@ -681,7 +681,21 @@
>  
>  &pinctrl_0 {
>  	pinctrl-names = "default";
> -	pinctrl-0 = <&sleep0>;
> +	pinctrl-0 = <&initial0 &sleep0>;
> +
> +	initial0: initial-state {
> +		PIN_IN(gpa1-4, DOWN, LV1);
> +		PIN_IN(gpa1-5, DOWN, LV1);
> +
> +		PIN_IN(gpc0-0, DOWN, LV1);
> +		PIN_IN(gpc0-1, DOWN, LV1);
> +		PIN_IN(gpc0-2, DOWN, LV1);
> +		PIN_IN(gpc0-3, DOWN, LV1);
> +		PIN_IN(gpc0-4, DOWN, LV1);
> +
> +		PIN_IN(gpd0-0, DOWN, LV1);
> +		PIN_IN(gpd0-1, DOWN, LV1);
> +	};
>  
>  	sleep0: sleep-state {
>  		PIN_SLP(gpa0-0, INPUT, DOWN);
> @@ -735,7 +749,59 @@
>  
>  &pinctrl_1 {
>  	pinctrl-names = "default";
> -	pinctrl-0 = <&sleep1>;
> +	pinctrl-0 = <&initial1 &sleep1>;
> +
> +	initial1: initial-state {
> +		PIN_IN(gpe0-6, DOWN, LV1);
> +		PIN_IN(gpe0-7, DOWN, LV1);
> +
> +		PIN_IN(gpe1-0, DOWN, LV1);
> +		PIN_IN(gpe1-3, DOWN, LV1);
> +		PIN_IN(gpe1-4, DOWN, LV1);
> +		PIN_IN(gpe1-5, DOWN, LV1);
> +		PIN_IN(gpe1-6, DOWN, LV1);
> +
> +		PIN_IN(gpk2-0, DOWN, LV1);
> +		PIN_IN(gpk2-1, DOWN, LV1);
> +		PIN_IN(gpk2-2, DOWN, LV1);
> +		PIN_IN(gpk2-3, DOWN, LV1);
> +		PIN_IN(gpk2-4, DOWN, LV1);
> +		PIN_IN(gpk2-5, DOWN, LV1);
> +		PIN_IN(gpk2-6, DOWN, LV1);
> +
> +		PIN_IN(gpm0-0, DOWN, LV1);
> +		PIN_IN(gpm0-1, DOWN, LV1);
> +		PIN_IN(gpm0-2, DOWN, LV1);
> +		PIN_IN(gpm0-3, DOWN, LV1);
> +		PIN_IN(gpm0-4, DOWN, LV1);
> +		PIN_IN(gpm0-5, DOWN, LV1);
> +		PIN_IN(gpm0-6, DOWN, LV1);
> +		PIN_IN(gpm0-7, DOWN, LV1);
> +
> +		PIN_IN(gpm1-0, DOWN, LV1);
> +		PIN_IN(gpm1-1, DOWN, LV1);
> +		PIN_IN(gpm1-2, DOWN, LV1);
> +		PIN_IN(gpm1-3, DOWN, LV1);
> +		PIN_IN(gpm1-4, DOWN, LV1);
> +		PIN_IN(gpm1-5, DOWN, LV1);
> +		PIN_IN(gpm1-6, DOWN, LV1);

A blank line here. Beside that:

Reviewed-by: Krzysztof Kozlowski <k.kozlowski@...sung.com>



Best regards,

Krzysztof