All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: use 'status' property for PCIe nodes
@ 2013-10-29  6:12 Jingoo Han
  2013-10-29 15:50 ` Kukjin Kim
  0 siblings, 1 reply; 3+ messages in thread
From: Jingoo Han @ 2013-10-29  6:12 UTC (permalink / raw)
  To: 'Kukjin Kim'
  Cc: linux-samsung-soc, 'Olof Johansson', 'Jingoo Han'

Set the default status for PCIe to disabled in the exynos5440.dtsi
file and let the board dts files such as exynos5440-ssdk5440.dts
enable the PCIe. However, keep the PCIe for SD5v1 board disabled,
because there is no PCIe slot on SD5v1 board.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
Tested on SSDK5440 board.

 arch/arm/boot/dts/exynos5440-ssdk5440.dts |    2 ++
 arch/arm/boot/dts/exynos5440.dtsi         |    2 ++
 2 files changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/exynos5440-ssdk5440.dts b/arch/arm/boot/dts/exynos5440-ssdk5440.dts
index a7cb848..d58cb78 100644
--- a/arch/arm/boot/dts/exynos5440-ssdk5440.dts
+++ b/arch/arm/boot/dts/exynos5440-ssdk5440.dts
@@ -68,9 +68,11 @@
 
 	pcie@290000 {
 		reset-gpio = <&pin_ctrl 5 0>;
+		status = "okay";
 	};
 
 	pcie@2a0000 {
 		reset-gpio = <&pin_ctrl 22 0>;
+		status = "okay";
 	};
 };
diff --git a/arch/arm/boot/dts/exynos5440.dtsi b/arch/arm/boot/dts/exynos5440.dtsi
index 5d6cf49..8da1070 100644
--- a/arch/arm/boot/dts/exynos5440.dtsi
+++ b/arch/arm/boot/dts/exynos5440.dtsi
@@ -276,6 +276,7 @@
 		interrupt-map-mask = <0 0 0 0>;
 		interrupt-map = <0x0 0 &gic 53>;
 		num-lanes = <4>;
+		status = "disabled";
 	};
 
 	pcie@2a0000 {
@@ -296,5 +297,6 @@
 		interrupt-map-mask = <0 0 0 0>;
 		interrupt-map = <0x0 0 &gic 56>;
 		num-lanes = <4>;
+		status = "disabled";
 	};
 };
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: dts: use 'status' property for PCIe nodes
  2013-10-29  6:12 [PATCH] ARM: dts: use 'status' property for PCIe nodes Jingoo Han
@ 2013-10-29 15:50 ` Kukjin Kim
  2013-10-29 19:46   ` Olof Johansson
  0 siblings, 1 reply; 3+ messages in thread
From: Kukjin Kim @ 2013-10-29 15:50 UTC (permalink / raw)
  To: Jingoo Han
  Cc: 'Kukjin Kim', linux-samsung-soc, 'Olof Johansson'

On 10/28/13 23:12, Jingoo Han wrote:
> Set the default status for PCIe to disabled in the exynos5440.dtsi
> file and let the board dts files such as exynos5440-ssdk5440.dts
> enable the PCIe. However, keep the PCIe for SD5v1 board disabled,
> because there is no PCIe slot on SD5v1 board.
>
> Signed-off-by: Jingoo Han<jg1.han@samsung.com>

Acked-by: Kukjin Kim <kgene.kim@samsung.com>

Olof, please pick this into your tree directly :-)

If any problems, please kindly let me know.

Thanks,
Kukjin

> ---
> Tested on SSDK5440 board.
>
>   arch/arm/boot/dts/exynos5440-ssdk5440.dts |    2 ++
>   arch/arm/boot/dts/exynos5440.dtsi         |    2 ++
>   2 files changed, 4 insertions(+)
>
> diff --git a/arch/arm/boot/dts/exynos5440-ssdk5440.dts b/arch/arm/boot/dts/exynos5440-ssdk5440.dts
> index a7cb848..d58cb78 100644
> --- a/arch/arm/boot/dts/exynos5440-ssdk5440.dts
> +++ b/arch/arm/boot/dts/exynos5440-ssdk5440.dts
> @@ -68,9 +68,11 @@
>
>   	pcie@290000 {
>   		reset-gpio =<&pin_ctrl 5 0>;
> +		status = "okay";
>   	};
>
>   	pcie@2a0000 {
>   		reset-gpio =<&pin_ctrl 22 0>;
> +		status = "okay";
>   	};
>   };
> diff --git a/arch/arm/boot/dts/exynos5440.dtsi b/arch/arm/boot/dts/exynos5440.dtsi
> index 5d6cf49..8da1070 100644
> --- a/arch/arm/boot/dts/exynos5440.dtsi
> +++ b/arch/arm/boot/dts/exynos5440.dtsi
> @@ -276,6 +276,7 @@
>   		interrupt-map-mask =<0 0 0 0>;
>   		interrupt-map =<0x0 0&gic 53>;
>   		num-lanes =<4>;
> +		status = "disabled";
>   	};
>
>   	pcie@2a0000 {
> @@ -296,5 +297,6 @@
>   		interrupt-map-mask =<0 0 0 0>;
>   		interrupt-map =<0x0 0&gic 56>;
>   		num-lanes =<4>;
> +		status = "disabled";
>   	};
>   };

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: dts: use 'status' property for PCIe nodes
  2013-10-29 15:50 ` Kukjin Kim
@ 2013-10-29 19:46   ` Olof Johansson
  0 siblings, 0 replies; 3+ messages in thread
From: Olof Johansson @ 2013-10-29 19:46 UTC (permalink / raw)
  To: Kukjin Kim; +Cc: Jingoo Han, linux-samsung-soc

On Tue, Oct 29, 2013 at 08:50:14AM -0700, Kukjin Kim wrote:
> On 10/28/13 23:12, Jingoo Han wrote:
> >Set the default status for PCIe to disabled in the exynos5440.dtsi
> >file and let the board dts files such as exynos5440-ssdk5440.dts
> >enable the PCIe. However, keep the PCIe for SD5v1 board disabled,
> >because there is no PCIe slot on SD5v1 board.
> >
> >Signed-off-by: Jingoo Han<jg1.han@samsung.com>
> 
> Acked-by: Kukjin Kim <kgene.kim@samsung.com>
> 
> Olof, please pick this into your tree directly :-)
> 
> If any problems, please kindly let me know.

Done, thanks.

-Olof

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-10-29 19:46 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-29  6:12 [PATCH] ARM: dts: use 'status' property for PCIe nodes Jingoo Han
2013-10-29 15:50 ` Kukjin Kim
2013-10-29 19:46   ` Olof Johansson

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.