All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH v2 0/2] Add Arndale board support
@ 2013-02-26  5:13 Inderpal Singh
  2013-02-26  5:13 ` [U-Boot] [PATCH v2 1/2] smdk5250: move board specific options to board specific config file Inderpal Singh
  2013-02-26  5:13 ` [U-Boot] [PATCH v2 2/2] Exynos5: Arndale: Add support for exynos5250 based arndale board Inderpal Singh
  0 siblings, 2 replies; 4+ messages in thread
From: Inderpal Singh @ 2013-02-26  5:13 UTC (permalink / raw)
  To: u-boot

The Arndale board is based on samsung's exynos5250 SOC.
First patch moves board specific options to board specific config file and the
second patch adds the initial config and dts file for the arndale board. 

Changes in v2:
	- Added entry is MAINTAINERS
	- Added Chander's Acked-by

Inderpal Singh (2):
  smdk5250: move board specific options to board specific config file
  Exynos5: Arndale: Add support for exynos5250 based arndale board.

 MAINTAINERS                              |    4 ++
 board/samsung/dts/exynos5250-arndale.dts |   36 ++++++++++++++++++
 boards.cfg                               |    1 +
 include/configs/arndale.h                |   39 +++++++++++++++++++
 include/configs/exynos5250-dt.h          |   60 ------------------------------
 include/configs/smdk5250.h               |   35 +++++++++++++++++
 6 files changed, 115 insertions(+), 60 deletions(-)
 create mode 100644 board/samsung/dts/exynos5250-arndale.dts
 create mode 100644 include/configs/arndale.h

-- 
1.7.9.5

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

* [U-Boot] [PATCH v2 1/2] smdk5250: move board specific options to board specific config file
  2013-02-26  5:13 [U-Boot] [PATCH v2 0/2] Add Arndale board support Inderpal Singh
@ 2013-02-26  5:13 ` Inderpal Singh
  2013-02-27 21:51   ` Simon Glass
  2013-02-26  5:13 ` [U-Boot] [PATCH v2 2/2] Exynos5: Arndale: Add support for exynos5250 based arndale board Inderpal Singh
  1 sibling, 1 reply; 4+ messages in thread
From: Inderpal Singh @ 2013-02-26  5:13 UTC (permalink / raw)
  To: u-boot

Signed-off-by: Inderpal Singh <inderpal.singh@linaro.org>
Acked-by: Chander Kashyap <chander.kashyap@linaro.org>
---
 include/configs/exynos5250-dt.h |   60 ---------------------------------------
 include/configs/smdk5250.h      |   35 +++++++++++++++++++++++
 2 files changed, 35 insertions(+), 60 deletions(-)

diff --git a/include/configs/exynos5250-dt.h b/include/configs/exynos5250-dt.h
index 3fa86b2..d4a589c 100644
--- a/include/configs/exynos5250-dt.h
+++ b/include/configs/exynos5250-dt.h
@@ -73,7 +73,6 @@
 #define CONFIG_SYS_MALLOC_LEN		(CONFIG_ENV_SIZE + (4 << 20))
 
 /* select serial console configuration */
-#define CONFIG_SERIAL3			/* use SERIAL 3 */
 #define CONFIG_BAUDRATE			115200
 #define EXYNOS5_DEFAULT_UART_OFFSET	0x010000
 
@@ -139,7 +138,6 @@
 /* Miscellaneous configurable options */
 #define CONFIG_SYS_LONGHELP		/* undef to save memory */
 #define CONFIG_SYS_HUSH_PARSER		/* use "hush" command parser	*/
-#define CONFIG_SYS_PROMPT		"SMDK5250 # "
 #define CONFIG_SYS_CBSIZE		256	/* Console I/O Buffer Size */
 #define CONFIG_SYS_PBSIZE		384	/* Print Buffer Size */
 #define CONFIG_SYS_MAXARGS		16	/* max number of command args */
@@ -179,7 +177,6 @@
 /* FLASH and environment organization */
 #define CONFIG_SYS_NO_FLASH
 #undef CONFIG_CMD_IMLS
-#define CONFIG_IDENT_STRING		" for SMDK5250"
 
 #define CONFIG_SYS_MMC_ENV_DEV		0
 
@@ -232,57 +229,10 @@
 #define CONFIG_I2C_EDID
 
 /* PMIC */
-#define CONFIG_PMIC
-#define CONFIG_PMIC_I2C
-#define CONFIG_PMIC_MAX77686
-
-/* SPI */
-#define CONFIG_ENV_IS_IN_SPI_FLASH
-#define CONFIG_SPI_FLASH
-
-#ifdef CONFIG_SPI_FLASH
-#define CONFIG_EXYNOS_SPI
-#define CONFIG_CMD_SF
-#define CONFIG_CMD_SPI
-#define CONFIG_SPI_FLASH_WINBOND
-#define CONFIG_SF_DEFAULT_MODE		SPI_MODE_0
-#define CONFIG_SF_DEFAULT_SPEED		50000000
-#define EXYNOS5_SPI_NUM_CONTROLLERS	5
-#endif
-
-#ifdef CONFIG_ENV_IS_IN_SPI_FLASH
-#define CONFIG_ENV_SPI_MODE	SPI_MODE_0
-#define CONFIG_ENV_SECT_SIZE	CONFIG_ENV_SIZE
-#define CONFIG_ENV_SPI_BUS	1
-#define CONFIG_ENV_SPI_MAX_HZ	50000000
-#endif
-
-/* PMIC */
 #define CONFIG_POWER
 #define CONFIG_POWER_I2C
 #define CONFIG_POWER_MAX77686
 
-/* SPI */
-#define CONFIG_ENV_IS_IN_SPI_FLASH
-#define CONFIG_SPI_FLASH
-
-#ifdef CONFIG_SPI_FLASH
-#define CONFIG_EXYNOS_SPI
-#define CONFIG_CMD_SF
-#define CONFIG_CMD_SPI
-#define CONFIG_SPI_FLASH_WINBOND
-#define CONFIG_SF_DEFAULT_MODE		SPI_MODE_0
-#define CONFIG_SF_DEFAULT_SPEED		50000000
-#define EXYNOS5_SPI_NUM_CONTROLLERS	5
-#endif
-
-#ifdef CONFIG_ENV_IS_IN_SPI_FLASH
-#define CONFIG_ENV_SPI_MODE	SPI_MODE_0
-#define CONFIG_ENV_SECT_SIZE	CONFIG_ENV_SIZE
-#define CONFIG_ENV_SPI_BUS	1
-#define CONFIG_ENV_SPI_MAX_HZ	50000000
-#endif
-
 /* Ethernet Controllor Driver */
 #ifdef CONFIG_CMD_NET
 #define CONFIG_SMC911X
@@ -314,14 +264,4 @@
 #define CONFIG_SHA1
 #define CONFIG_SHA256
 
-/* Display */
-#define CONFIG_LCD
-#ifdef CONFIG_LCD
-#define CONFIG_EXYNOS_FB
-#define CONFIG_EXYNOS_DP
-#define LCD_XRES			2560
-#define LCD_YRES			1600
-#define LCD_BPP			LCD_COLOR16
-#endif
-
 #endif	/* __CONFIG_H */
diff --git a/include/configs/smdk5250.h b/include/configs/smdk5250.h
index 81f83a8..51c4215 100644
--- a/include/configs/smdk5250.h
+++ b/include/configs/smdk5250.h
@@ -30,4 +30,39 @@
 #undef CONFIG_DEFAULT_DEVICE_TREE
 #define CONFIG_DEFAULT_DEVICE_TREE	exynos5250-smdk5250
 
+#define CONFIG_SYS_PROMPT		"SMDK5250 # "
+#define CONFIG_SERIAL3			/* use SERIAL 3 */
+#define CONFIG_IDENT_STRING		" for SMDK5250"
+
+/* SPI */
+#define CONFIG_ENV_IS_IN_SPI_FLASH
+#define CONFIG_SPI_FLASH
+
+#ifdef CONFIG_SPI_FLASH
+#define CONFIG_EXYNOS_SPI
+#define CONFIG_CMD_SF
+#define CONFIG_CMD_SPI
+#define CONFIG_SPI_FLASH_WINBOND
+#define CONFIG_SF_DEFAULT_MODE		SPI_MODE_0
+#define CONFIG_SF_DEFAULT_SPEED		50000000
+#define EXYNOS5_SPI_NUM_CONTROLLERS	5
+#endif
+
+#ifdef CONFIG_ENV_IS_IN_SPI_FLASH
+#define CONFIG_ENV_SPI_MODE	SPI_MODE_0
+#define CONFIG_ENV_SECT_SIZE	CONFIG_ENV_SIZE
+#define CONFIG_ENV_SPI_BUS	1
+#define CONFIG_ENV_SPI_MAX_HZ	50000000
+#endif
+
+/* Display */
+#define CONFIG_LCD
+#ifdef CONFIG_LCD
+#define CONFIG_EXYNOS_FB
+#define CONFIG_EXYNOS_DP
+#define LCD_XRES			2560
+#define LCD_YRES			1600
+#define LCD_BPP			LCD_COLOR16
+#endif
+
 #endif	/* __CONFIG_SMDK_H */
-- 
1.7.9.5

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

* [U-Boot] [PATCH v2 2/2] Exynos5: Arndale: Add support for exynos5250 based arndale board.
  2013-02-26  5:13 [U-Boot] [PATCH v2 0/2] Add Arndale board support Inderpal Singh
  2013-02-26  5:13 ` [U-Boot] [PATCH v2 1/2] smdk5250: move board specific options to board specific config file Inderpal Singh
@ 2013-02-26  5:13 ` Inderpal Singh
  1 sibling, 0 replies; 4+ messages in thread
From: Inderpal Singh @ 2013-02-26  5:13 UTC (permalink / raw)
  To: u-boot

The Arndale board is based on samsung's exynos5250 SOC.
This patch adds a initial config and dts file.

Signed-off-by: Inderpal Singh <inderpal.singh@linaro.org>
Acked-by: Chander Kashyap <chander.kashyap@linaro.org>
---
This depends on the first 6 patches from Amar at [1]

[1] http://comments.gmane.org/gmane.comp.boot-loaders.u-boot/151523

 MAINTAINERS                              |    4 +++
 board/samsung/dts/exynos5250-arndale.dts |   36 +++++++++++++++++++++++++++
 boards.cfg                               |    1 +
 include/configs/arndale.h                |   39 ++++++++++++++++++++++++++++++
 4 files changed, 80 insertions(+)
 create mode 100644 board/samsung/dts/exynos5250-arndale.dts
 create mode 100644 include/configs/arndale.h

diff --git a/MAINTAINERS b/MAINTAINERS
index 45e2dd4..da0dd38 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -736,6 +736,10 @@ Chander Kashyap <k.chander@samsung.com>
 	SMDKV310		ARM ARMV7 (EXYNOS4210 SoC)
 	SMDK5250		ARM ARMV7 (EXYNOS5250 SoC)
 
+Inderpal Singh <inderpal.singh@linaro.org>
+
+	arndale			ARM ARMV7 (EXYNOS5250 SoC)
+
 Lukasz Majewski <l.majewski@samsung.com>
 
 	trats			ARM ARMV7 (EXYNOS4210 SoC)
diff --git a/board/samsung/dts/exynos5250-arndale.dts b/board/samsung/dts/exynos5250-arndale.dts
new file mode 100644
index 0000000..9b4d6b2
--- /dev/null
+++ b/board/samsung/dts/exynos5250-arndale.dts
@@ -0,0 +1,36 @@
+/*
+ * SAMSUNG Arndale board device tree source
+ *
+ * Copyright (c) 2013 Samsung Electronics Co., Ltd.
+ *		http://www.samsung.com
+ *
+ * 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/ ARCH_CPU_DTS
+
+/ {
+	model = "SAMSUNG Arndale board based on EXYNOS5250";
+	compatible = "samsung,arndale", "samsung,exynos5250";
+
+	mmc at 12200000 {
+		samsung,bus-width = <8>;
+		samsung,timing = <1 3 3>;
+	};
+
+	mmc at 12210000 {
+		status = "disabled";
+	};
+
+	mmc at 12220000 {
+		samsung,bus-width = <4>;
+		samsung,timing = <1 2 3>;
+	};
+
+	mmc at 12230000 {
+		status = "disabled";
+	};
+};
diff --git a/boards.cfg b/boards.cfg
index b1319aa..2017ce0 100644
--- a/boards.cfg
+++ b/boards.cfg
@@ -287,6 +287,7 @@ smdkc100                     arm         armv7       smdkc100            samsung
 origen			     arm	 armv7	     origen		 samsung	exynos
 s5pc210_universal            arm         armv7       universal_c210      samsung        exynos
 smdk5250		     arm	 armv7	     smdk5250		 samsung	exynos
+arndale			     arm	 armv7	     smdk5250		 samsung	exynos
 smdkv310		     arm	 armv7	     smdkv310		 samsung	exynos
 trats                        arm         armv7       trats               samsung        exynos
 harmony                      arm         armv7:arm720t harmony           nvidia         tegra20
diff --git a/include/configs/arndale.h b/include/configs/arndale.h
new file mode 100644
index 0000000..c71370e
--- /dev/null
+++ b/include/configs/arndale.h
@@ -0,0 +1,39 @@
+/*
+ * Copyright (C) 2013 Samsung Electronics
+ *
+ * Configuration settings for the SAMSUNG Arndale board.
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#ifndef __CONFIG_ARNDALE_H
+#define __CONFIG_ARNDALE_H
+
+#include <configs/exynos5250-dt.h>
+
+#undef CONFIG_DEFAULT_DEVICE_TREE
+#define CONFIG_DEFAULT_DEVICE_TREE	exynos5250-arndale
+
+#define CONFIG_SERIAL2
+#define CONFIG_SYS_PROMPT		"Arndale # "
+#define CONFIG_IDENT_STRING		" for Arndale"
+
+#define CONFIG_ENV_IS_IN_MMC
+
+#endif	/* __CONFIG_ARNDALE_H */
-- 
1.7.9.5

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

* [U-Boot] [PATCH v2 1/2] smdk5250: move board specific options to board specific config file
  2013-02-26  5:13 ` [U-Boot] [PATCH v2 1/2] smdk5250: move board specific options to board specific config file Inderpal Singh
@ 2013-02-27 21:51   ` Simon Glass
  0 siblings, 0 replies; 4+ messages in thread
From: Simon Glass @ 2013-02-27 21:51 UTC (permalink / raw)
  To: u-boot

Hi,

On Mon, Feb 25, 2013 at 9:13 PM, Inderpal Singh
<inderpal.singh@linaro.org> wrote:
> Signed-off-by: Inderpal Singh <inderpal.singh@linaro.org>
> Acked-by: Chander Kashyap <chander.kashyap@linaro.org>
> ---
>  include/configs/exynos5250-dt.h |   60 ---------------------------------------
>  include/configs/smdk5250.h      |   35 +++++++++++++++++++++++
>  2 files changed, 35 insertions(+), 60 deletions(-)
>
> diff --git a/include/configs/exynos5250-dt.h b/include/configs/exynos5250-dt.h

Sorry I have only just seen this - I don't think this patch is wholly correct.

The intent with the -dt.h file is to enable things that might be used
by exynos5250 boards. For example it is OK to enable multiple pmic
drivers, SPI, MMC, etc. even if not all boards use them. Device tree
nodes have a 'status = "disabled" feature to deal with turning
individual peripherals on/ff. I agree that the smdk5250 string is
wrong, but you should perhaps change this to exynos5250 instead. By
all means undef and define things in your board, but we should not
defeat the purpose of the -dt.h file. The device tree should be used
to configure/enable drivers where possible.

Now I realise that what I am saying is not entirely possible right now
(in that for example some drivers are not device tree-enabled), but we
should try to avoid just treating this file as an exynos 'common'
file.

Regards,
Simon


> index 3fa86b2..d4a589c 100644
> --- a/include/configs/exynos5250-dt.h
> +++ b/include/configs/exynos5250-dt.h
> @@ -73,7 +73,6 @@
>  #define CONFIG_SYS_MALLOC_LEN          (CONFIG_ENV_SIZE + (4 << 20))
>
>  /* select serial console configuration */
> -#define CONFIG_SERIAL3                 /* use SERIAL 3 */
>  #define CONFIG_BAUDRATE                        115200
>  #define EXYNOS5_DEFAULT_UART_OFFSET    0x010000
>
> @@ -139,7 +138,6 @@
>  /* Miscellaneous configurable options */
>  #define CONFIG_SYS_LONGHELP            /* undef to save memory */
>  #define CONFIG_SYS_HUSH_PARSER         /* use "hush" command parser    */
> -#define CONFIG_SYS_PROMPT              "SMDK5250 # "
>  #define CONFIG_SYS_CBSIZE              256     /* Console I/O Buffer Size */
>  #define CONFIG_SYS_PBSIZE              384     /* Print Buffer Size */
>  #define CONFIG_SYS_MAXARGS             16      /* max number of command args */
> @@ -179,7 +177,6 @@
>  /* FLASH and environment organization */
>  #define CONFIG_SYS_NO_FLASH
>  #undef CONFIG_CMD_IMLS
> -#define CONFIG_IDENT_STRING            " for SMDK5250"
>
>  #define CONFIG_SYS_MMC_ENV_DEV         0
>
> @@ -232,57 +229,10 @@
>  #define CONFIG_I2C_EDID
>
>  /* PMIC */
> -#define CONFIG_PMIC
> -#define CONFIG_PMIC_I2C
> -#define CONFIG_PMIC_MAX77686
> -
> -/* SPI */
> -#define CONFIG_ENV_IS_IN_SPI_FLASH
> -#define CONFIG_SPI_FLASH
> -
> -#ifdef CONFIG_SPI_FLASH
> -#define CONFIG_EXYNOS_SPI
> -#define CONFIG_CMD_SF
> -#define CONFIG_CMD_SPI
> -#define CONFIG_SPI_FLASH_WINBOND
> -#define CONFIG_SF_DEFAULT_MODE         SPI_MODE_0
> -#define CONFIG_SF_DEFAULT_SPEED                50000000
> -#define EXYNOS5_SPI_NUM_CONTROLLERS    5
> -#endif
> -
> -#ifdef CONFIG_ENV_IS_IN_SPI_FLASH
> -#define CONFIG_ENV_SPI_MODE    SPI_MODE_0
> -#define CONFIG_ENV_SECT_SIZE   CONFIG_ENV_SIZE
> -#define CONFIG_ENV_SPI_BUS     1
> -#define CONFIG_ENV_SPI_MAX_HZ  50000000
> -#endif
> -
> -/* PMIC */
>  #define CONFIG_POWER
>  #define CONFIG_POWER_I2C
>  #define CONFIG_POWER_MAX77686
>
> -/* SPI */
> -#define CONFIG_ENV_IS_IN_SPI_FLASH
> -#define CONFIG_SPI_FLASH
> -
> -#ifdef CONFIG_SPI_FLASH
> -#define CONFIG_EXYNOS_SPI
> -#define CONFIG_CMD_SF
> -#define CONFIG_CMD_SPI
> -#define CONFIG_SPI_FLASH_WINBOND
> -#define CONFIG_SF_DEFAULT_MODE         SPI_MODE_0
> -#define CONFIG_SF_DEFAULT_SPEED                50000000
> -#define EXYNOS5_SPI_NUM_CONTROLLERS    5
> -#endif
> -
> -#ifdef CONFIG_ENV_IS_IN_SPI_FLASH
> -#define CONFIG_ENV_SPI_MODE    SPI_MODE_0
> -#define CONFIG_ENV_SECT_SIZE   CONFIG_ENV_SIZE
> -#define CONFIG_ENV_SPI_BUS     1
> -#define CONFIG_ENV_SPI_MAX_HZ  50000000
> -#endif
> -
>  /* Ethernet Controllor Driver */
>  #ifdef CONFIG_CMD_NET
>  #define CONFIG_SMC911X
> @@ -314,14 +264,4 @@
>  #define CONFIG_SHA1
>  #define CONFIG_SHA256
>
> -/* Display */
> -#define CONFIG_LCD
> -#ifdef CONFIG_LCD
> -#define CONFIG_EXYNOS_FB
> -#define CONFIG_EXYNOS_DP
> -#define LCD_XRES                       2560
> -#define LCD_YRES                       1600
> -#define LCD_BPP                        LCD_COLOR16
> -#endif
> -
>  #endif /* __CONFIG_H */
> diff --git a/include/configs/smdk5250.h b/include/configs/smdk5250.h
> index 81f83a8..51c4215 100644
> --- a/include/configs/smdk5250.h
> +++ b/include/configs/smdk5250.h
> @@ -30,4 +30,39 @@
>  #undef CONFIG_DEFAULT_DEVICE_TREE
>  #define CONFIG_DEFAULT_DEVICE_TREE     exynos5250-smdk5250
>
> +#define CONFIG_SYS_PROMPT              "SMDK5250 # "
> +#define CONFIG_SERIAL3                 /* use SERIAL 3 */
> +#define CONFIG_IDENT_STRING            " for SMDK5250"
> +
> +/* SPI */
> +#define CONFIG_ENV_IS_IN_SPI_FLASH
> +#define CONFIG_SPI_FLASH
> +
> +#ifdef CONFIG_SPI_FLASH
> +#define CONFIG_EXYNOS_SPI
> +#define CONFIG_CMD_SF
> +#define CONFIG_CMD_SPI
> +#define CONFIG_SPI_FLASH_WINBOND
> +#define CONFIG_SF_DEFAULT_MODE         SPI_MODE_0
> +#define CONFIG_SF_DEFAULT_SPEED                50000000
> +#define EXYNOS5_SPI_NUM_CONTROLLERS    5
> +#endif
> +
> +#ifdef CONFIG_ENV_IS_IN_SPI_FLASH
> +#define CONFIG_ENV_SPI_MODE    SPI_MODE_0
> +#define CONFIG_ENV_SECT_SIZE   CONFIG_ENV_SIZE
> +#define CONFIG_ENV_SPI_BUS     1
> +#define CONFIG_ENV_SPI_MAX_HZ  50000000
> +#endif
> +
> +/* Display */
> +#define CONFIG_LCD
> +#ifdef CONFIG_LCD
> +#define CONFIG_EXYNOS_FB
> +#define CONFIG_EXYNOS_DP
> +#define LCD_XRES                       2560
> +#define LCD_YRES                       1600
> +#define LCD_BPP                        LCD_COLOR16
> +#endif
> +
>  #endif /* __CONFIG_SMDK_H */
> --
> 1.7.9.5
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

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

end of thread, other threads:[~2013-02-27 21:51 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-02-26  5:13 [U-Boot] [PATCH v2 0/2] Add Arndale board support Inderpal Singh
2013-02-26  5:13 ` [U-Boot] [PATCH v2 1/2] smdk5250: move board specific options to board specific config file Inderpal Singh
2013-02-27 21:51   ` Simon Glass
2013-02-26  5:13 ` [U-Boot] [PATCH v2 2/2] Exynos5: Arndale: Add support for exynos5250 based arndale board Inderpal Singh

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.