[PATCH 5/9] ARM: dts: imx50: remove the use of pingrp macros

Shawn Guo shawn.guo at linaro.org
Sat Jan 25 11:43:07 EST 2014


We created the pingrp macros in imx50-pingrp.h for purpose of less LOC
when same pin group is used by multiple boards.  However, DT maintainers
take it as an abuse of DTC macro support.  So let's get rid of it to
make the pins used by given device more intuitive.

Signed-off-by: Shawn Guo <shawn.guo at linaro.org>
---
 arch/arm/boot/dts/imx50-evk.dts  |   28 +++++++-
 arch/arm/boot/dts/imx50-pingrp.h |  146 --------------------------------------
 arch/arm/boot/dts/imx50.dtsi     |    2 +-
 3 files changed, 26 insertions(+), 150 deletions(-)
 delete mode 100644 arch/arm/boot/dts/imx50-pingrp.h

diff --git a/arch/arm/boot/dts/imx50-evk.dts b/arch/arm/boot/dts/imx50-evk.dts
index a859264..1b22512 100644
--- a/arch/arm/boot/dts/imx50-evk.dts
+++ b/arch/arm/boot/dts/imx50-evk.dts
@@ -61,15 +61,37 @@
 &iomuxc {
 	imx50-evk {
 		pinctrl_cspi: cspigrp {
-			fsl,pins = <MX50_CSPI_PINGRP1>;
+			fsl,pins = <
+				MX50_PAD_CSPI_SCLK__CSPI_SCLK		0x00
+				MX50_PAD_CSPI_MISO__CSPI_MISO		0x00
+				MX50_PAD_CSPI_MOSI__CSPI_MOSI		0x00
+				MX50_PAD_CSPI_SS0__GPIO4_11		0xc4
+				MX50_PAD_ECSPI1_MOSI__CSPI_SS1		0xf4
+			>;
 		};
 
 		pinctrl_fec: fecgrp {
-			fsl,pins = <MX50_FEC_PINGRP1>;
+			fsl,pins = <
+				MX50_PAD_SSI_RXFS__FEC_MDC		0x80
+				MX50_PAD_SSI_RXC__FEC_MDIO		0x80
+				MX50_PAD_DISP_D0__FEC_TX_CLK		0x80
+				MX50_PAD_DISP_D1__FEC_RX_ERR		0x80
+				MX50_PAD_DISP_D2__FEC_RX_DV		0x80
+				MX50_PAD_DISP_D3__FEC_RDATA_1		0x80
+				MX50_PAD_DISP_D4__FEC_RDATA_0		0x80
+				MX50_PAD_DISP_D5__FEC_TX_EN		0x80
+				MX50_PAD_DISP_D6__FEC_TDATA_1		0x80
+				MX50_PAD_DISP_D7__FEC_TDATA_0		0x80
+			>;
 		};
 
 		pinctrl_uart1: uart1grp {
-			fsl,pins = <MX50_UART1_PINGRP1>;
+			fsl,pins = <
+				MX50_PAD_UART1_TXD__UART1_TXD_MUX	0x1e4
+				MX50_PAD_UART1_RXD__UART1_RXD_MUX	0x1e4
+				MX50_PAD_UART1_RTS__UART1_RTS		0x1e4
+				MX50_PAD_UART1_CTS__UART1_CTS		0x1e4
+			>;
 		};
 	};
 };
diff --git a/arch/arm/boot/dts/imx50-pingrp.h b/arch/arm/boot/dts/imx50-pingrp.h
deleted file mode 100644
index 0f4ce8c..0000000
--- a/arch/arm/boot/dts/imx50-pingrp.h
+++ /dev/null
@@ -1,146 +0,0 @@
-/*
- * Copyright (C) 2013 Freescale Semiconductor, Inc.
- *
- * 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.
- *
- */
-
-#ifndef __DTS_IMX50_PINGRP_H
-#define __DTS_IMX50_PINGRP_H
-
-#include "imx50-pinfunc.h"
-
-#define MX50_CSPI_PINGRP1 \
-	MX50_PAD_CSPI_SCLK__CSPI_SCLK			0x00 \
-	MX50_PAD_CSPI_MISO__CSPI_MISO			0x00 \
-	MX50_PAD_CSPI_MOSI__CSPI_MOSI			0x00 \
-	MX50_PAD_CSPI_SS0__GPIO4_11			0xc4 \
-	MX50_PAD_ECSPI1_MOSI__CSPI_SS1			0xf4
-
-#define MX50_ECSPI1_PINGRP1 \
-	MX50_PAD_ECSPI1_SCLK__ECSPI1_SCLK		0x00 \
-	MX50_PAD_ECSPI1_SS0__ECSPI1_SS0			0x00 \
-	MX50_PAD_ECSPI1_MISO__ECSPI1_MISO		0x00 \
-	MX50_PAD_ECSPI1_MOSI__ECSPI1_MOSI		0x00
-
-#define MX50_ESDHC1_PINGRP1 \
-	MX50_PAD_SD1_D0__ESDHC1_DAT0			0x1d4 \
-	MX50_PAD_SD1_D1__ESDHC1_DAT1			0x1d4 \
-	MX50_PAD_SD1_D2__ESDHC1_DAT2			0x1d4 \
-	MX50_PAD_SD1_D3__ESDHC1_DAT3			0x1d4 \
-	MX50_PAD_SD1_CMD__ESDHC1_CMD			0x1e4 \
-	MX50_PAD_SD1_CLK__ESDHC1_CLK			0xd4
-
-#define MX50_ESDHC1_PINGRP2 \
-	MX50_PAD_SD1_D0__ESDHC1_DAT0			0x1d4 \
-	MX50_PAD_SD1_D1__ESDHC1_DAT1			0x1d4 \
-	MX50_PAD_SD1_D2__ESDHC1_DAT2			0x1d4 \
-	MX50_PAD_SD1_D3__ESDHC1_DAT3			0x1d4 \
-	MX50_PAD_UART3_TXD__ESDHC1_DAT4			0x1d4 \
-	MX50_PAD_UART3_RXD__ESDHC1_DAT5			0x1d4 \
-	MX50_PAD_UART4_TXD__ESDHC1_DAT6			0x1d4 \
-	MX50_PAD_UART4_RXD__ESDHC1_DAT7			0x1d4 \
-	MX50_PAD_SD1_CMD__ESDHC1_CMD			0x14 \
-	MX50_PAD_SD1_CLK__ESDHC1_CLK			0xd4
-
-#define MX50_ESDHC2_PINGRP1 \
-	MX50_PAD_SD2_CMD__ESDHC2_CMD			0x1e4 \
-	MX50_PAD_SD2_CLK__ESDHC2_CLK			0xd4 \
-	MX50_PAD_SD2_D0__ESDHC2_DAT0			0x1d4 \
-	MX50_PAD_SD2_D1__ESDHC2_DAT1			0x1d4 \
-	MX50_PAD_SD2_D2__ESDHC2_DAT2			0x1d4 \
-	MX50_PAD_SD2_D3__ESDHC2_DAT3			0x1d4 \
-	MX50_PAD_SD2_D4__ESDHC2_DAT4			0x1d4 \
-	MX50_PAD_SD2_D5__ESDHC2_DAT5			0x1d4 \
-	MX50_PAD_SD2_D6__ESDHC2_DAT6			0x1d4 \
-	MX50_PAD_SD2_D7__ESDHC2_DAT7			0x1d4
-
-#define MX50_ESDHC3_PINGRP1 \
-	MX50_PAD_SD3_D0__ESDHC3_DAT0			0x1d4 \
-	MX50_PAD_SD3_D1__ESDHC3_DAT1			0x1d4 \
-	MX50_PAD_SD3_D2__ESDHC3_DAT2			0x1d4 \
-	MX50_PAD_SD3_D3__ESDHC3_DAT3			0x1d4 \
-	MX50_PAD_SD3_D4__ESDHC3_DAT4			0x1d4 \
-	MX50_PAD_SD3_D5__ESDHC3_DAT5			0x1d4 \
-	MX50_PAD_SD3_D6__ESDHC3_DAT6			0x1d4 \
-	MX50_PAD_SD3_D7__ESDHC3_DAT7			0x1d4 \
-	MX50_PAD_SD3_CMD__ESDHC3_CMD			0x1e4 \
-	MX50_PAD_SD3_CLK__ESDHC3_CLK			0xd4
-
-#define MX50_FEC_PINGRP1 \
-	MX50_PAD_SSI_RXFS__FEC_MDC			0x80 \
-	MX50_PAD_SSI_RXC__FEC_MDIO			0x80 \
-	MX50_PAD_DISP_D0__FEC_TX_CLK			0x80 \
-	MX50_PAD_DISP_D1__FEC_RX_ERR			0x80 \
-	MX50_PAD_DISP_D2__FEC_RX_DV			0x80 \
-	MX50_PAD_DISP_D3__FEC_RDATA_1			0x80 \
-	MX50_PAD_DISP_D4__FEC_RDATA_0			0x80 \
-	MX50_PAD_DISP_D5__FEC_TX_EN			0x80 \
-	MX50_PAD_DISP_D6__FEC_TDATA_1			0x80 \
-	MX50_PAD_DISP_D7__FEC_TDATA_0			0x80
-
-#define MX50_FEC_PINGRP2 \
-	MX50_PAD_I2C3_SCL__FEC_MDC			0x80 \
-	MX50_PAD_I2C3_SDA__FEC_MDIO			0x80 \
-	MX50_PAD_DISP_D0__FEC_TX_CLK			0x80 \
-	MX50_PAD_DISP_D10__FEC_RX_DV			0x80 \
-	MX50_PAD_DISP_D11__FEC_RDATA_1			0x80 \
-	MX50_PAD_DISP_D12__FEC_RDATA_0			0x80 \
-	MX50_PAD_DISP_D13__FEC_TX_EN			0x80 \
-	MX50_PAD_DISP_D14__FEC_TDATA_1			0x80 \
-	MX50_PAD_DISP_D15__FEC_TDATA_0			0x80
-
-#define MX50_I2C1_PINGRP1 \
-	MX50_PAD_I2C1_SDA__I2C1_SDA			0x12c \
-	MX50_PAD_I2C1_SCL__I2C1_SCL			0x12c
-
-#define MX50_I2C2_PINGRP1 \
-	MX50_PAD_I2C2_SDA__I2C2_SDA			0x12c \
-	MX50_PAD_I2C2_SCL__I2C2_SCL			0x12c
-
-#define MX50_I2C3_PINGRP1 \
-	MX50_PAD_I2C3_SDA__I2C3_SDA			0x12c \
-	MX50_PAD_I2C3_SCL__I2C3_SCL			0x12c
-
-#define MX50_OWIRE_PINGRP1 \
-	MX50_PAD_OWIRE__OWIRE_LINE			0x84
-
-#define MX50_UART1_PINGRP1 \
-	MX50_PAD_UART1_TXD__UART1_TXD_MUX		0x1e4 \
-	MX50_PAD_UART1_RXD__UART1_RXD_MUX		0x1e4 \
-	MX50_PAD_UART1_RTS__UART1_RTS			0x1e4 \
-	MX50_PAD_UART1_CTS__UART1_CTS			0x1e4
-
-#define MX50_UART2_PINGRP1 \
-	MX50_PAD_UART2_TXD__UART2_TXD_MUX		0x1e4 \
-	MX50_PAD_UART2_RXD__UART2_RXD_MUX		0x1e4 \
-	MX50_PAD_UART2_RTS__UART2_RTS			0x1e4 \
-	MX50_PAD_UART2_CTS__UART2_CTS			0x1e4
-
-#define MX50_UART2_PINGRP2 \
-	MX50_PAD_I2C1_SCL__UART2_TXD_MUX		0x1e4 \
-	MX50_PAD_I2C1_SDA__UART2_RXD_MUX		0x1e4 \
-	MX50_PAD_I2C2_SDA__UART2_RTS			0x1e4 \
-	MX50_PAD_I2C2_SCL__UART2_CTS			0x1e4
-
-#define MX50_UART3_PINGRP1 \
-	MX50_PAD_UART3_TXD__UART3_TXD_MUX		0x1e4 \
-	MX50_PAD_UART3_RXD__UART3_RXD_MUX		0x1e4 \
-	MX50_PAD_ECSPI1_SCLK__UART3_RTS			0x1e4 \
-	MX50_PAD_ECSPI1_MOSI__UART3_CTS			0x1e4
-
-#define MX50_UART4_PINGRP1 \
-	MX50_PAD_UART4_TXD__UART4_TXD_MUX		0x1e4 \
-	MX50_PAD_UART4_RXD__UART4_RXD_MUX		0x1e4 \
-	MX50_PAD_ECSPI1_MISO__UART4_RTS			0x1e4 \
-	MX50_PAD_ECSPI1_SS0__UART4_CTS			0x1e4
-
-#define MX50_UART5_PINGRP1 \
-	MX50_PAD_ECSPI2_MISO__UART5_TXD_MUX		0x1e4 \
-	MX50_PAD_ECSPI2_SS0__UART5_RXD_MUX		0x1e4 \
-	MX50_PAD_ECSPI2_SCLK__UART5_RTS			0x1e4 \
-	MX50_PAD_ECSPI2_MOSI__UART5_CTS			0x1e4
-
-#endif /* __DTS_IMX50_PINGRP_H */
diff --git a/arch/arm/boot/dts/imx50.dtsi b/arch/arm/boot/dts/imx50.dtsi
index 01c0499..7152472 100644
--- a/arch/arm/boot/dts/imx50.dtsi
+++ b/arch/arm/boot/dts/imx50.dtsi
@@ -12,7 +12,7 @@
  */
 
 #include "skeleton.dtsi"
-#include "imx50-pingrp.h"
+#include "imx50-pinfunc.h"
 #include <dt-bindings/clock/imx5-clock.h>
 
 / {
-- 
1.7.9.5





More information about the linux-arm-kernel mailing list