Re: [PATCH] ARM: dts: add initial dts file for ORIGEN4QUAD

2012-11-27 Thread Tomasz Figa 
Hi Kyuho,

On Tuesday 27 of November 2012 19:28:09 kh.choi wrote:
> Hi Tomasz,
> 
> 2012-11-26 오후 8:57, Tomasz Figa 쓴 글:
> > Hi Kyungmin, Kyuho,
> > 
> > On Monday 26 of November 2012 20:52:12 Kyungmin Park wrote:
> >> Hi,
> >> 
> >> On 11/26/12, chlrb...@gmail.com  wrote:
> >>> From: Kyuho Choi 
> >>> 
> >>> This patch adds initial dts file for ORIGEN4QUAD board.
> >>> ORIGEN4QUAD board based on Samsung EXYNOS4412 SoC.
> >>> More properties will be added later.
> >>> 
> >>> Signed-off-by: Kyuho Choi 
> >>> ---
> >>> 
> >>>   arch/arm/boot/dts/exynos4412-origen4quad.dts |   45
> >>> 
> >>> ++
> >>> 
> >>>   1 files changed, 45 insertions(+), 0 deletions(-)
> >>>   create mode 100644 arch/arm/boot/dts/exynos4412-origen4quad.dts
> >>> 
> >>> diff --git a/arch/arm/boot/dts/exynos4412-origen4quad.dts
> >>> b/arch/arm/boot/dts/exynos4412-origen4quad.dts
> >>> new file mode 100644
> >>> index 000..390a2ab
> >>> --- /dev/null
> >>> +++ b/arch/arm/boot/dts/exynos4412-origen4quad.dts
> >>> @@ -0,0 +1,45 @@
> >>> +/*
> >>> + * Samsung's Exynos4412 based Origen4Quad board device tree source
> >>> + *
> >>> + * Copyright (c) 2012-2013 Isnignal Co., Ltd.
> >>> + *   http://www.insignal.co.kr
> >>> + *
> >>> + * Device tree source file for Insignal's Origen4Quad board which
> >>> is
> >>> based on
> >>> + * Samsung's Exynos4412 SoC.
> >>> + *
> >>> + * This program is free software; you can redistribute it and/or
> >>> modify + * it under the terms of the GNU General Public License
> >>> version 2 as + * published by the Free Software Foundation.
> >>> +*/
> >>> +
> >>> +/dts-v1/;
> >>> +/include/ "exynos4412.dtsi"
> >>> +
> >>> +/ {
> >>> + model = "Insignal Origen4Quad board based on Exynos4412";
> >>> + compatible = "insignal,origen4quad", "samsung,exynos4412";
> >>> +
> >>> + memory {
> >>> + reg = <0x4000 0x4000>;
> >>> + };
> >> 
> >> does it test to boot with this dts file?
> >> does it support 1GiB memory section?
> > 
> > It will most likely fail to boot with u-boot versions that do not
> > support replacing memory configuration in device tree and without
> > CONFIG_ARM_ATAG_DTB_COMPAT enabled.
> > 
> > In other words, the memory setup from dts is used only if both of the
> > 
> > following are true:
> >   - CONFIG_ARM_ATAG_DTB_COMPAT is disabled
> >   - u-boot does not modify memory setup in device tree passed to the
> >   kernel
> I'd working based on `exynos4_defconfig` for ORIGEN4QUAD board.
> `exynos4_defconfig`s default disabled CONFIG_ARM_ATAG_DTB_COMPAT.
> And ORIGEN4QUAD working on u-boot v2012.10.

This means that u-boot version you use modifies memory setup in device 
tree before passing it to the kernel.

You can try to boot with empty memory node or no memory node at all and 
check what memory configuration the kernel gets in /proc/device-tree, for 
example using

xxd -c8 -g4 /proc/device-tree/memory/reg

Still, the maximum section size for mach-exynos is set in kernel to 256MiB 
and so the memory in reg property such be defined in chunks less or equal 
256 MiB.

Best regards,
-- 
Tomasz Figa
Samsung Poland R&D Center
SW Solution Development, Linux Platform

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] ARM: dts: add initial dts file for ORIGEN4QUAD

2012-11-27 Thread kh.choi 

Hi,

2012-11-26 오후 8:52, Kyungmin Park 쓴 글:

Hi,

On 11/26/12, chlrb...@gmail.com  wrote:

From: Kyuho Choi 

This patch adds initial dts file for ORIGEN4QUAD board.
ORIGEN4QUAD board based on Samsung EXYNOS4412 SoC.
More properties will be added later.

Signed-off-by: Kyuho Choi 
---
  arch/arm/boot/dts/exynos4412-origen4quad.dts |   45
++
  1 files changed, 45 insertions(+), 0 deletions(-)
  create mode 100644 arch/arm/boot/dts/exynos4412-origen4quad.dts

diff --git a/arch/arm/boot/dts/exynos4412-origen4quad.dts
b/arch/arm/boot/dts/exynos4412-origen4quad.dts
new file mode 100644
index 000..390a2ab
--- /dev/null
+++ b/arch/arm/boot/dts/exynos4412-origen4quad.dts
@@ -0,0 +1,45 @@
+/*
+ * Samsung's Exynos4412 based Origen4Quad board device tree source
+ *
+ * Copyright (c) 2012-2013 Isnignal Co., Ltd.
+ * http://www.insignal.co.kr
+ *
+ * Device tree source file for Insignal's Origen4Quad board which is based
on
+ * Samsung's Exynos4412 SoC.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+*/
+
+/dts-v1/;
+/include/ "exynos4412.dtsi"
+
+/ {
+   model = "Insignal Origen4Quad board based on Exynos4412";
+   compatible = "insignal,origen4quad", "samsung,exynos4412";
+
+   memory {
+   reg = <0x4000 0x4000>;
+   };

does it test to boot with this dts file?
does it support 1GiB memory section?

Yes, I tested this dts.
ORIGEN4QUAD board support 1GiB memory.
But in current patch state, only support 760MiB on boot.


Thank you,
Kyungmin Park

+
+   chosen {
+   bootargs ="root=/dev/ram0 rw ramdisk=32768 initrd=0x4100,32M
console=ttySAC2,115200 init=/linuxrc";
+   };
+
+   serial@1380 {
+   status = "okay";
+   };
+
+   serial@1381 {
+   status = "okay";
+   };
+
+   serial@1382 {
+   status = "okay";
+   };
+
+   serial@1383 {
+   status = "okay";
+   };
+};
--
1.7.5.4


___
linux-arm-kernel mailing list
linux-arm-ker...@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel


--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html



--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] ARM: dts: add initial dts file for ORIGEN4QUAD

2012-11-27 Thread kh.choi 

Hi Tomasz,

2012-11-26 오후 8:57, Tomasz Figa 쓴 글:

Hi Kyungmin, Kyuho,

On Monday 26 of November 2012 20:52:12 Kyungmin Park wrote:

Hi,

On 11/26/12, chlrb...@gmail.com  wrote:

From: Kyuho Choi 

This patch adds initial dts file for ORIGEN4QUAD board.
ORIGEN4QUAD board based on Samsung EXYNOS4412 SoC.
More properties will be added later.

Signed-off-by: Kyuho Choi 
---

  arch/arm/boot/dts/exynos4412-origen4quad.dts |   45

++

  1 files changed, 45 insertions(+), 0 deletions(-)
  create mode 100644 arch/arm/boot/dts/exynos4412-origen4quad.dts

diff --git a/arch/arm/boot/dts/exynos4412-origen4quad.dts
b/arch/arm/boot/dts/exynos4412-origen4quad.dts
new file mode 100644
index 000..390a2ab
--- /dev/null
+++ b/arch/arm/boot/dts/exynos4412-origen4quad.dts
@@ -0,0 +1,45 @@
+/*
+ * Samsung's Exynos4412 based Origen4Quad board device tree source
+ *
+ * Copyright (c) 2012-2013 Isnignal Co., Ltd.
+ * http://www.insignal.co.kr
+ *
+ * Device tree source file for Insignal's Origen4Quad board which is
based on
+ * Samsung's Exynos4412 SoC.
+ *
+ * This program is free software; you can redistribute it and/or
modify + * it under the terms of the GNU General Public License
version 2 as + * published by the Free Software Foundation.
+*/
+
+/dts-v1/;
+/include/ "exynos4412.dtsi"
+
+/ {
+   model = "Insignal Origen4Quad board based on Exynos4412";
+   compatible = "insignal,origen4quad", "samsung,exynos4412";
+
+   memory {
+   reg = <0x4000 0x4000>;
+   };


does it test to boot with this dts file?
does it support 1GiB memory section?


It will most likely fail to boot with u-boot versions that do not support
replacing memory configuration in device tree and without
CONFIG_ARM_ATAG_DTB_COMPAT enabled.

In other words, the memory setup from dts is used only if both of the
following are true:
  - CONFIG_ARM_ATAG_DTB_COMPAT is disabled
  - u-boot does not modify memory setup in device tree passed to the kernel

I'd working based on `exynos4_defconfig` for ORIGEN4QUAD board.
`exynos4_defconfig`s default disabled CONFIG_ARM_ATAG_DTB_COMPAT.
And ORIGEN4QUAD working on u-boot v2012.10.


Best regards,


--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] ARM: dts: add initial dts file for ORIGEN4QUAD

2012-11-26 Thread Tomasz Figa 
Hi Kyungmin, Kyuho,

On Monday 26 of November 2012 20:52:12 Kyungmin Park wrote:
> Hi,
> 
> On 11/26/12, chlrb...@gmail.com  wrote:
> > From: Kyuho Choi 
> > 
> > This patch adds initial dts file for ORIGEN4QUAD board.
> > ORIGEN4QUAD board based on Samsung EXYNOS4412 SoC.
> > More properties will be added later.
> > 
> > Signed-off-by: Kyuho Choi 
> > ---
> > 
> >  arch/arm/boot/dts/exynos4412-origen4quad.dts |   45
> > 
> > ++
> > 
> >  1 files changed, 45 insertions(+), 0 deletions(-)
> >  create mode 100644 arch/arm/boot/dts/exynos4412-origen4quad.dts
> > 
> > diff --git a/arch/arm/boot/dts/exynos4412-origen4quad.dts
> > b/arch/arm/boot/dts/exynos4412-origen4quad.dts
> > new file mode 100644
> > index 000..390a2ab
> > --- /dev/null
> > +++ b/arch/arm/boot/dts/exynos4412-origen4quad.dts
> > @@ -0,0 +1,45 @@
> > +/*
> > + * Samsung's Exynos4412 based Origen4Quad board device tree source
> > + *
> > + * Copyright (c) 2012-2013 Isnignal Co., Ltd.
> > + * http://www.insignal.co.kr
> > + *
> > + * Device tree source file for Insignal's Origen4Quad board which is
> > based on
> > + * Samsung's Exynos4412 SoC.
> > + *
> > + * This program is free software; you can redistribute it and/or
> > modify + * it under the terms of the GNU General Public License
> > version 2 as + * published by the Free Software Foundation.
> > +*/
> > +
> > +/dts-v1/;
> > +/include/ "exynos4412.dtsi"
> > +
> > +/ {
> > +   model = "Insignal Origen4Quad board based on Exynos4412";
> > +   compatible = "insignal,origen4quad", "samsung,exynos4412";
> > +
> > +   memory {
> > +   reg = <0x4000 0x4000>;
> > +   };
> 
> does it test to boot with this dts file?
> does it support 1GiB memory section?

It will most likely fail to boot with u-boot versions that do not support 
replacing memory configuration in device tree and without 
CONFIG_ARM_ATAG_DTB_COMPAT enabled.

In other words, the memory setup from dts is used only if both of the 
following are true:
 - CONFIG_ARM_ATAG_DTB_COMPAT is disabled
 - u-boot does not modify memory setup in device tree passed to the kernel

Best regards,
-- 
Tomasz Figa
Samsung Poland R&D Center
SW Solution Development, Linux Platform

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] ARM: dts: add initial dts file for ORIGEN4QUAD

2012-11-26 Thread Kyungmin Park 
Hi,

On 11/26/12, chlrb...@gmail.com  wrote:
> From: Kyuho Choi 
>
> This patch adds initial dts file for ORIGEN4QUAD board.
> ORIGEN4QUAD board based on Samsung EXYNOS4412 SoC.
> More properties will be added later.
>
> Signed-off-by: Kyuho Choi 
> ---
>  arch/arm/boot/dts/exynos4412-origen4quad.dts |   45
> ++
>  1 files changed, 45 insertions(+), 0 deletions(-)
>  create mode 100644 arch/arm/boot/dts/exynos4412-origen4quad.dts
>
> diff --git a/arch/arm/boot/dts/exynos4412-origen4quad.dts
> b/arch/arm/boot/dts/exynos4412-origen4quad.dts
> new file mode 100644
> index 000..390a2ab
> --- /dev/null
> +++ b/arch/arm/boot/dts/exynos4412-origen4quad.dts
> @@ -0,0 +1,45 @@
> +/*
> + * Samsung's Exynos4412 based Origen4Quad board device tree source
> + *
> + * Copyright (c) 2012-2013 Isnignal Co., Ltd.
> + *   http://www.insignal.co.kr
> + *
> + * Device tree source file for Insignal's Origen4Quad board which is based
> on
> + * Samsung's Exynos4412 SoC.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> +*/
> +
> +/dts-v1/;
> +/include/ "exynos4412.dtsi"
> +
> +/ {
> + model = "Insignal Origen4Quad board based on Exynos4412";
> + compatible = "insignal,origen4quad", "samsung,exynos4412";
> +
> + memory {
> + reg = <0x4000 0x4000>;
> + };
does it test to boot with this dts file?
does it support 1GiB memory section?

Thank you,
Kyungmin Park
> +
> + chosen {
> + bootargs ="root=/dev/ram0 rw ramdisk=32768 initrd=0x4100,32M
> console=ttySAC2,115200 init=/linuxrc";
> + };
> +
> + serial@1380 {
> + status = "okay";
> + };
> +
> + serial@1381 {
> + status = "okay";
> + };
> +
> + serial@1382 {
> + status = "okay";
> + };
> +
> + serial@1383 {
> + status = "okay";
> + };
> +};
> --
> 1.7.5.4
>
>
> ___
> linux-arm-kernel mailing list
> linux-arm-ker...@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html