All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH 3/3 v2] arm: mx6: tqma6: Extract baseboard configs into separate config file
@ 2015-03-12 15:34 Stefan Roese
  2015-03-12 15:52 ` Markus Niebel
  2015-04-22 12:50 ` Stefano Babic
  0 siblings, 2 replies; 3+ messages in thread
From: Stefan Roese @ 2015-03-12 15:34 UTC (permalink / raw)
  To: u-boot

This patch extracts all baseboard specific defines into a separate config file.
This makes it easier to add other baseboards that use the TQMa6 SoM.

This patch will be used by the upcoming WRU-IV board support which also
uses the TQMa6 SoM.

Signed-off-by: Stefan Roese <sr@denx.de>
Cc: Markus Niebel <Markus.Niebel@tq-group.com>
Cc: Stefano Babic <sbabic@denx.de>
---
v2:
- Move default DTT sensor definion to end of config header
  for baseboards not implementing any additional sensors
- Remove comment

 include/configs/tqma6.h      | 57 +++++++++++++-------------------------------
 include/configs/tqma6_mba6.h | 30 +++++++++++++++++++++++
 2 files changed, 46 insertions(+), 41 deletions(-)
 create mode 100644 include/configs/tqma6_mba6.h

diff --git a/include/configs/tqma6.h b/include/configs/tqma6.h
index 9022550..d064c81 100644
--- a/include/configs/tqma6.h
+++ b/include/configs/tqma6.h
@@ -35,16 +35,6 @@
 #define PHYS_SDRAM_SIZE			(1024u * SZ_1M)
 #endif
 
-#if defined(CONFIG_MBA6)
-
-#if defined(CONFIG_MX6DL) || defined(CONFIG_MX6S)
-#define CONFIG_DEFAULT_FDT_FILE		"imx6dl-mba6x.dtb"
-#elif defined(CONFIG_MX6Q) || defined(CONFIG_MX6Q)
-#define CONFIG_DEFAULT_FDT_FILE		"imx6q-mba6x.dtb"
-#endif
-
-#endif
-
 #define CONFIG_DISPLAY_CPUINFO
 #define CONFIG_DISPLAY_BOARDINFO
 #define CONFIG_SYS_GENERIC_BOARD
@@ -85,11 +75,6 @@
 
 /* I2C SYSMON (LM75) */
 #define CONFIG_DTT_LM75
-#if defined(CONFIG_MBA6)
-#define CONFIG_DTT_SENSORS		{ 0, 1 }
-#else
-#define CONFIG_DTT_SENSORS		{ 0 }
-#endif
 #define CONFIG_DTT_MAX_TEMP		70
 #define CONFIG_DTT_MIN_TEMP		-30
 #define CONFIG_DTT_HYSTERESIS	3
@@ -150,38 +135,12 @@
 #define CONFIG_PHYLIB
 #define CONFIG_MII
 
-#if defined(CONFIG_MBA6)
-
-#define CONFIG_FEC_XCV_TYPE		RGMII
-#define CONFIG_ETHPRIME			"FEC"
-
-#define CONFIG_FEC_MXC_PHYADDR		0x03
-#define CONFIG_PHY_MICREL
-#define CONFIG_PHY_KSZ9031
-
-#else
-
-#error "define PHY to use for your baseboard"
-
-#endif
-
 #define CONFIG_ARP_TIMEOUT		200UL
 /* Network config - Allow larger/faster download for TFTP/NFS */
 #define CONFIG_IP_DEFRAG
 #define CONFIG_TFTP_BLOCKSIZE	4096
 #define CONFIG_NFS_READ_SIZE	4096
 
-#if defined(CONFIG_MBA6)
-
-#define CONFIG_MXC_UART_BASE		UART2_BASE
-#define CONFIG_CONSOLE_DEV		"ttymxc1"
-
-#else
-
-#error "define baseboard specific things (uart, number of SD-card slots)"
-
-#endif
-
 /* allow to overwrite serial and ethaddr */
 #define CONFIG_ENV_OVERWRITE
 #define CONFIG_CONS_INDEX		1
@@ -495,4 +454,20 @@
 #define CONFIG_CMD_CACHE
 #endif
 
+/*
+ * All the defines above are for the TQMa6 SoM
+ *
+ * Now include the baseboard specific configuration
+ */
+#ifdef CONFIG_MBA6
+#include "tqma6_mba6.h"
+#else
+#error "No baseboard for the TQMa6 defined!"
+#endif
+
+/* Support at least the sensor on TQMa6 SOM */
+#if !defined(CONFIG_DTT_SENSORS)
+#define CONFIG_DTT_SENSORS		{ 0 }
+#endif
+
 #endif /* __CONFIG_H */
diff --git a/include/configs/tqma6_mba6.h b/include/configs/tqma6_mba6.h
new file mode 100644
index 0000000..88c0067
--- /dev/null
+++ b/include/configs/tqma6_mba6.h
@@ -0,0 +1,30 @@
+/*
+ * Copyright (C) 2013, 2014 Markus Niebel <Markus.Niebel@tq-group.com>
+ *
+ * Configuration settings for the TQ Systems TQMa6<Q,S> module.
+ *
+ * SPDX-License-Identifier:	GPL-2.0+
+ */
+
+#ifndef __CONFIG_TQMA6_MBA6_H
+#define __CONFIG_TQMA6_MBA6_H
+
+#if defined(CONFIG_MX6DL) || defined(CONFIG_MX6S)
+#define CONFIG_DEFAULT_FDT_FILE		"imx6dl-mba6x.dtb"
+#elif defined(CONFIG_MX6Q) || defined(CONFIG_MX6Q)
+#define CONFIG_DEFAULT_FDT_FILE		"imx6q-mba6x.dtb"
+#endif
+
+#define CONFIG_DTT_SENSORS		{ 0, 1 }
+
+#define CONFIG_FEC_XCV_TYPE		RGMII
+#define CONFIG_ETHPRIME			"FEC"
+
+#define CONFIG_FEC_MXC_PHYADDR		0x03
+#define CONFIG_PHY_MICREL
+#define CONFIG_PHY_KSZ9031
+
+#define CONFIG_MXC_UART_BASE		UART2_BASE
+#define CONFIG_CONSOLE_DEV		"ttymxc1"
+
+#endif /* __CONFIG_TQMA6_MBA6_H */
-- 
2.3.2

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

* [U-Boot] [PATCH 3/3 v2] arm: mx6: tqma6: Extract baseboard configs into separate config file
  2015-03-12 15:34 [U-Boot] [PATCH 3/3 v2] arm: mx6: tqma6: Extract baseboard configs into separate config file Stefan Roese
@ 2015-03-12 15:52 ` Markus Niebel
  2015-04-22 12:50 ` Stefano Babic
  1 sibling, 0 replies; 3+ messages in thread
From: Markus Niebel @ 2015-03-12 15:52 UTC (permalink / raw)
  To: u-boot

Am 12.03.2015 um 16:34 schrieb Stefan Roese:
> This patch extracts all baseboard specific defines into a separate config file.
> This makes it easier to add other baseboards that use the TQMa6 SoM.
> 
> This patch will be used by the upcoming WRU-IV board support which also
> uses the TQMa6 SoM.
> 
> Signed-off-by: Stefan Roese <sr@denx.de>
> Cc: Markus Niebel <Markus.Niebel@tq-group.com>
> Cc: Stefano Babic <sbabic@denx.de>
> ---
> v2:
> - Move default DTT sensor definion to end of config header
>   for baseboards not implementing any additional sensors
> - Remove comment
> 
>  include/configs/tqma6.h      | 57 +++++++++++++-------------------------------
>  include/configs/tqma6_mba6.h | 30 +++++++++++++++++++++++
>  2 files changed, 46 insertions(+), 41 deletions(-)
>  create mode 100644 include/configs/tqma6_mba6.h
> 
> diff --git a/include/configs/tqma6.h b/include/configs/tqma6.h
> index 9022550..d064c81 100644
> --- a/include/configs/tqma6.h
> +++ b/include/configs/tqma6.h
> @@ -35,16 +35,6 @@
>  #define PHYS_SDRAM_SIZE			(1024u * SZ_1M)
>  #endif
>  
> -#if defined(CONFIG_MBA6)
> -
> -#if defined(CONFIG_MX6DL) || defined(CONFIG_MX6S)
> -#define CONFIG_DEFAULT_FDT_FILE		"imx6dl-mba6x.dtb"
> -#elif defined(CONFIG_MX6Q) || defined(CONFIG_MX6Q)
> -#define CONFIG_DEFAULT_FDT_FILE		"imx6q-mba6x.dtb"
> -#endif
> -
> -#endif
> -
>  #define CONFIG_DISPLAY_CPUINFO
>  #define CONFIG_DISPLAY_BOARDINFO
>  #define CONFIG_SYS_GENERIC_BOARD
> @@ -85,11 +75,6 @@
>  
>  /* I2C SYSMON (LM75) */
>  #define CONFIG_DTT_LM75
> -#if defined(CONFIG_MBA6)
> -#define CONFIG_DTT_SENSORS		{ 0, 1 }
> -#else
> -#define CONFIG_DTT_SENSORS		{ 0 }
> -#endif
>  #define CONFIG_DTT_MAX_TEMP		70
>  #define CONFIG_DTT_MIN_TEMP		-30
>  #define CONFIG_DTT_HYSTERESIS	3
> @@ -150,38 +135,12 @@
>  #define CONFIG_PHYLIB
>  #define CONFIG_MII
>  
> -#if defined(CONFIG_MBA6)
> -
> -#define CONFIG_FEC_XCV_TYPE		RGMII
> -#define CONFIG_ETHPRIME			"FEC"
> -
> -#define CONFIG_FEC_MXC_PHYADDR		0x03
> -#define CONFIG_PHY_MICREL
> -#define CONFIG_PHY_KSZ9031
> -
> -#else
> -
> -#error "define PHY to use for your baseboard"
> -
> -#endif
> -
>  #define CONFIG_ARP_TIMEOUT		200UL
>  /* Network config - Allow larger/faster download for TFTP/NFS */
>  #define CONFIG_IP_DEFRAG
>  #define CONFIG_TFTP_BLOCKSIZE	4096
>  #define CONFIG_NFS_READ_SIZE	4096
>  
> -#if defined(CONFIG_MBA6)
> -
> -#define CONFIG_MXC_UART_BASE		UART2_BASE
> -#define CONFIG_CONSOLE_DEV		"ttymxc1"
> -
> -#else
> -
> -#error "define baseboard specific things (uart, number of SD-card slots)"
> -
> -#endif
> -
>  /* allow to overwrite serial and ethaddr */
>  #define CONFIG_ENV_OVERWRITE
>  #define CONFIG_CONS_INDEX		1
> @@ -495,4 +454,20 @@
>  #define CONFIG_CMD_CACHE
>  #endif
>  
> +/*
> + * All the defines above are for the TQMa6 SoM
> + *
> + * Now include the baseboard specific configuration
> + */
> +#ifdef CONFIG_MBA6
> +#include "tqma6_mba6.h"
> +#else
> +#error "No baseboard for the TQMa6 defined!"
> +#endif
> +
> +/* Support at least the sensor on TQMa6 SOM */
> +#if !defined(CONFIG_DTT_SENSORS)
> +#define CONFIG_DTT_SENSORS		{ 0 }
> +#endif
> +
>  #endif /* __CONFIG_H */
> diff --git a/include/configs/tqma6_mba6.h b/include/configs/tqma6_mba6.h
> new file mode 100644
> index 0000000..88c0067
> --- /dev/null
> +++ b/include/configs/tqma6_mba6.h
> @@ -0,0 +1,30 @@
> +/*
> + * Copyright (C) 2013, 2014 Markus Niebel <Markus.Niebel@tq-group.com>
> + *
> + * Configuration settings for the TQ Systems TQMa6<Q,S> module.
> + *
> + * SPDX-License-Identifier:	GPL-2.0+
> + */
> +
> +#ifndef __CONFIG_TQMA6_MBA6_H
> +#define __CONFIG_TQMA6_MBA6_H
> +
> +#if defined(CONFIG_MX6DL) || defined(CONFIG_MX6S)
> +#define CONFIG_DEFAULT_FDT_FILE		"imx6dl-mba6x.dtb"
> +#elif defined(CONFIG_MX6Q) || defined(CONFIG_MX6Q)
> +#define CONFIG_DEFAULT_FDT_FILE		"imx6q-mba6x.dtb"
> +#endif
> +
> +#define CONFIG_DTT_SENSORS		{ 0, 1 }
> +
> +#define CONFIG_FEC_XCV_TYPE		RGMII
> +#define CONFIG_ETHPRIME			"FEC"
> +
> +#define CONFIG_FEC_MXC_PHYADDR		0x03
> +#define CONFIG_PHY_MICREL
> +#define CONFIG_PHY_KSZ9031
> +
> +#define CONFIG_MXC_UART_BASE		UART2_BASE
> +#define CONFIG_CONSOLE_DEV		"ttymxc1"
> +
> +#endif /* __CONFIG_TQMA6_MBA6_H */
> 
Acked-By: Markus Niebel <Markus.Niebel@tq-group.com>

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

* [U-Boot] [PATCH 3/3 v2] arm: mx6: tqma6: Extract baseboard configs into separate config file
  2015-03-12 15:34 [U-Boot] [PATCH 3/3 v2] arm: mx6: tqma6: Extract baseboard configs into separate config file Stefan Roese
  2015-03-12 15:52 ` Markus Niebel
@ 2015-04-22 12:50 ` Stefano Babic
  1 sibling, 0 replies; 3+ messages in thread
From: Stefano Babic @ 2015-04-22 12:50 UTC (permalink / raw)
  To: u-boot

On 12/03/2015 16:34, Stefan Roese wrote:
> This patch extracts all baseboard specific defines into a separate config file.
> This makes it easier to add other baseboards that use the TQMa6 SoM.
> 
> This patch will be used by the upcoming WRU-IV board support which also
> uses the TQMa6 SoM.
> 
> Signed-off-by: Stefan Roese <sr@denx.de>
> Cc: Markus Niebel <Markus.Niebel@tq-group.com>
> Cc: Stefano Babic <sbabic@denx.de>
> ---

Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic



-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

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

end of thread, other threads:[~2015-04-22 12:50 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-03-12 15:34 [U-Boot] [PATCH 3/3 v2] arm: mx6: tqma6: Extract baseboard configs into separate config file Stefan Roese
2015-03-12 15:52 ` Markus Niebel
2015-04-22 12:50 ` Stefano Babic

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.