Skip to contentSkip to conversation

ARM: uniphier: select SPL_DM rather than default in defconfig

PATCH New Masahiro Yamada 3 participants 13 messages Aug 28, 2015 – Aug 30, 2015

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

PATCH 1/3 ARM: uniphier: select SPL_DM rather than default in defconfig

Now UniPhier SoCs highly depend on Driver Model for SPL, too.

---

 arch/arm/Kconfig           | 1 +
 configs/ph1_ld4_defconfig  | 1 -
 configs/ph1_pro4_defconfig | 1 -
 configs/ph1_sld3_defconfig | 1 -
 configs/ph1_sld8_defconfig | 1 -
 5 files changed, 1 insertion(+), 4 deletions(-)
arch/arm/Kconfig +1 -0
index a99ae28..5f3af77 100644
@@ -817,6 +817,7 @@ config ARCH_UNIPHIER
 	select SPL
 	select OF_CONTROL
 	select DM
+	select SPL_DM
 	select DM_SERIAL
 	select DM_I2C
 	help
configs/ph1_ld4_defconfig +0 -1
index f84dcdf..56c215f 100644
@@ -17,7 +17,6 @@ CONFIG_CMD_PING=y
 CONFIG_CMD_TIME=y
 # CONFIG_CMD_MISC is not set
 CONFIG_NET_RANDOM_ETHADDR=y
-CONFIG_SPL_DM=y
 CONFIG_NAND_DENALI=y
 CONFIG_SYS_NAND_DENALI_64BIT=y
 CONFIG_NAND_DENALI_SPARE_AREA_SKIP_BYTES=8
configs/ph1_pro4_defconfig +0 -1
index 2fc844d..7624c54 100644
@@ -16,7 +16,6 @@ CONFIG_CMD_PING=y
 CONFIG_CMD_TIME=y
 # CONFIG_CMD_MISC is not set
 CONFIG_NET_RANDOM_ETHADDR=y
-CONFIG_SPL_DM=y
 CONFIG_NAND_DENALI=y
 CONFIG_SYS_NAND_DENALI_64BIT=y
 CONFIG_NAND_DENALI_SPARE_AREA_SKIP_BYTES=8
configs/ph1_sld3_defconfig +0 -1
index 386a0d1..d495132 100644
@@ -17,7 +17,6 @@ CONFIG_CMD_PING=y
 CONFIG_CMD_TIME=y
 # CONFIG_CMD_MISC is not set
 CONFIG_NET_RANDOM_ETHADDR=y
-CONFIG_SPL_DM=y
 CONFIG_NAND_DENALI=y
 CONFIG_SYS_NAND_DENALI_64BIT=y
 CONFIG_NAND_DENALI_SPARE_AREA_SKIP_BYTES=8
configs/ph1_sld8_defconfig +0 -1
index 2af45d2..1a35a77 100644
@@ -17,7 +17,6 @@ CONFIG_CMD_PING=y
 CONFIG_CMD_TIME=y
 # CONFIG_CMD_MISC is not set
 CONFIG_NET_RANDOM_ETHADDR=y
-CONFIG_SPL_DM=y
 CONFIG_NAND_DENALI=y
 CONFIG_SYS_NAND_DENALI_64BIT=y
 CONFIG_NAND_DENALI_SPARE_AREA_SKIP_BYTES=8
Signature · 1 line
1.9.1
#1 lore raw

PATCH 1/3 ARM: uniphier: select SPL_DM rather than default in defconfig

2015-08-28 20:13 GMT+09:00 Masahiro Yamada <[email protected]>:

Quoted 3 lines

Now UniPhier SoCs highly depend on Driver Model for SPL, too.

Applied to u-boot-uniphier/master.

Signature · 2 lines
Best Regards
Masahiro Yamada
#2 lore raw

PATCH 2/3 ARM: uniphier: enable SPL_OF_CONTROL

In reply to Masahiro Yamada

Device Tree really improves code maintainability and is now available for SPL too.

This is the state-of-the-art implementation in U-boot.

The board files (platform data) are no longer needed.

---

 arch/arm/Kconfig                             |  1 +
 arch/arm/dts/uniphier-ph1-ld4-ref.dts        |  9 ++++++---
 arch/arm/dts/uniphier-ph1-pro4-ref.dts       |  9 ++++++---
 arch/arm/dts/uniphier-ph1-sld3-ref.dts       |  9 ++++++---
 arch/arm/dts/uniphier-ph1-sld8-ref.dts       |  9 ++++++---
 arch/arm/mach-uniphier/ph1-ld4/Makefile      |  1 -
 arch/arm/mach-uniphier/ph1-ld4/platdevice.c  | 15 ---------------
 arch/arm/mach-uniphier/ph1-pro4/Makefile     |  1 -
 arch/arm/mach-uniphier/ph1-pro4/platdevice.c | 15 ---------------
 arch/arm/mach-uniphier/ph1-sld3/Makefile     |  1 -
 arch/arm/mach-uniphier/ph1-sld3/platdevice.c |  1 -
 arch/arm/mach-uniphier/ph1-sld8/platdevice.c | 15 ---------------
 doc/README.uniphier                          | 10 +++++-----
 include/configs/uniphier.h                   |  2 +-
 14 files changed, 31 insertions(+), 67 deletions(-)
 delete mode 100644 arch/arm/mach-uniphier/ph1-ld4/platdevice.c
 delete mode 100644 arch/arm/mach-uniphier/ph1-pro4/platdevice.c
 delete mode 100644 arch/arm/mach-uniphier/ph1-sld3/platdevice.c
 delete mode 100644 arch/arm/mach-uniphier/ph1-sld8/platdevice.c
arch/arm/Kconfig +1 -0
index 5f3af77..cd88df4 100644
@@ -816,6 +816,7 @@ config ARCH_UNIPHIER
 	select SUPPORT_SPL
 	select SPL
 	select OF_CONTROL
+	select SPL_OF_CONTROL
 	select DM
 	select SPL_DM
 	select DM_SERIAL
arch/arm/dts/uniphier-ph1-ld4-ref.dts +6 -3
index 25e487a..7c36581 100644
@@ -1,9 +1,7 @@
 /*
  * Device Tree Source for UniPhier PH1-LD4 Reference Board
  *
- * Copyright (C) 2014-2015 Panasonic Corporation
- * Copyright (C) 2015      Socionext Inc.
- *   Author: Masahiro Yamada <[email protected]>
+ * Copyright (C) 2014-2015 Masahiro Yamada <[email protected]>
  *
  * SPDX-License-Identifier:	GPL-2.0+	X11
  */
@@ -57,3 +55,8 @@
 &usb1 {
 	status = "okay";
 };
+
+/* for U-boot only */
+&uart0 {
+       u-boot,dm-pre-reloc;
+};
arch/arm/dts/uniphier-ph1-pro4-ref.dts +6 -3
index b4b7f61..fd3e8c6 100644
@@ -1,9 +1,7 @@
 /*
  * Device Tree Source for UniPhier PH1-Pro4 Reference Board
  *
- * Copyright (C) 2014-2015 Panasonic Corporation
- * Copyright (C) 2015      Socionext Inc.
- *   Author: Masahiro Yamada <[email protected]>
+ * Copyright (C) 2014-2015 Masahiro Yamada <[email protected]>
  *
  * SPDX-License-Identifier:	GPL-2.0+	X11
  */
@@ -56,3 +54,8 @@
 &usb0 {
 	status = "okay";
 };
+
+/* for U-boot only */
+&uart0 {
+       u-boot,dm-pre-reloc;
+};
arch/arm/dts/uniphier-ph1-sld3-ref.dts +6 -3
index 9dc9296..2406bfc 100644
@@ -1,9 +1,7 @@
 /*
  * Device Tree Source for UniPhier PH1-sLD3 Reference Board
  *
- * Copyright (C) 2014-2015 Panasonic Corporation
- * Copyright (C) 2015      Socionext Inc.
- *   Author: Masahiro Yamada <[email protected]>
+ * Copyright (C) 2014-2015 Masahiro Yamada <[email protected]>
  *
  * SPDX-License-Identifier:	GPL-2.0+	X11
  */
@@ -56,3 +54,8 @@
 &usb1 {
 	status = "okay";
 };
+
+/* for U-boot only */
+&uart0 {
+       u-boot,dm-pre-reloc;
+};
arch/arm/dts/uniphier-ph1-sld8-ref.dts +6 -3
index 2d1359c..b7ed275 100644
@@ -1,9 +1,7 @@
 /*
  * Device Tree Source for UniPhier PH1-sLD8 Reference Board
  *
- * Copyright (C) 2014-2015 Panasonic Corporation
- * Copyright (C) 2015      Socionext Inc.
- *   Author: Masahiro Yamada <[email protected]>
+ * Copyright (C) 2014-2015 Masahiro Yamada <[email protected]>
  *
  * SPDX-License-Identifier:	GPL-2.0+	X11
  */
@@ -57,3 +55,8 @@
 &usb1 {
 	status = "okay";
 };
+
+/* for U-boot only */
+&uart0 {
+       u-boot,dm-pre-reloc;
+};
arch/arm/mach-uniphier/ph1-ld4/Makefile +0 -1
index af815c3..1410b12 100644
@@ -8,7 +8,6 @@ obj-y += bcu_init.o sg_init.o pll_init.o early_clkrst_init.o \
 	early_pinctrl.o pll_spectrum.o umc_init.o ddrphy_init.o
 obj-$(CONFIG_PFC_MICRO_SUPPORT_CARD) += sbc_init.o
 obj-$(CONFIG_DCC_MICRO_SUPPORT_CARD) += sbc_init_3cs.o
-obj-$(CONFIG_SPL_DM) += platdevice.o
 else
 obj-$(CONFIG_BOARD_EARLY_INIT_F) += pinctrl.o clkrst_init.o
 endif
arch/arm/mach-uniphier/ph1-ld4/platdevice.c deleted +0 -0
deleted file mode 100644
index c0e6294..0000000
arch/arm/mach-uniphier/ph1-pro4/Makefile +0 -1
index f6a584e..229f443 100644
@@ -8,7 +8,6 @@ obj-y += sg_init.o pll_init.o early_clkrst_init.o \
 	early_pinctrl.o pll_spectrum.o umc_init.o ddrphy_init.o
 obj-$(CONFIG_PFC_MICRO_SUPPORT_CARD) += sbc_init.o
 obj-$(CONFIG_DCC_MICRO_SUPPORT_CARD) += sbc_init_3cs.o
-obj-$(CONFIG_SPL_DM) += platdevice.o
 else
 obj-$(CONFIG_BOARD_EARLY_INIT_F) += pinctrl.o clkrst_init.o
 endif
arch/arm/mach-uniphier/ph1-pro4/platdevice.c deleted +0 -0
deleted file mode 100644
index 7440ced..0000000
arch/arm/mach-uniphier/ph1-sld3/Makefile +0 -1
index f3f7ad4..aff5d64 100644
@@ -8,7 +8,6 @@ obj-y += bcu_init.o memconf.o sg_init.o pll_init.o early_clkrst_init.o \
 	early_pinctrl.o pll_spectrum.o umc_init.o
 obj-$(CONFIG_PFC_MICRO_SUPPORT_CARD) += sbc_init.o
 obj-$(CONFIG_DCC_MICRO_SUPPORT_CARD) += sbc_init_3cs.o
-obj-$(CONFIG_SPL_DM) += platdevice.o
 else
 obj-$(CONFIG_BOARD_EARLY_INIT_F) += pinctrl.o clkrst_init.o
 endif
arch/arm/mach-uniphier/ph1-sld3/platdevice.c deleted +0 -0
deleted file mode 100644
index 6521067..0000000
arch/arm/mach-uniphier/ph1-sld8/platdevice.c deleted +0 -0
deleted file mode 100644
index aa334a1..0000000
doc/README.uniphier +5 -5
index 52d681b..e936f40 100644
@@ -52,12 +52,12 @@ Burn U-Boot images to NAND
 --------------------------
 
 Write two files to the NAND device as follows:
- - spl/u-boot-spl.bin at the offset address 0x00000000
- - u-boot-dtb.img     at the offset address 0x00010000
+ - spl/u-boot-spl-dtb.bin at the offset address 0x00000000
+ - u-boot-dtb.img         at the offset address 0x00010000
 
 If a TFTP server is available, the images can be easily updated.
-Just copy the u-boot-spl.bin and u-boot-dtb.img to the TFTP public directory,
-and then run the following command at the U-Boot command line:
+Just copy the u-boot-spl-dtb.bin and u-boot-dtb.img to the TFTP public
+directory, and then run the following command at the U-Boot command line:
 
   => run nandupdate
 
@@ -129,4 +129,4 @@ newer SoCs.  Even if it is, EA[25] is not connected on most of the boards.
 
 --
 Masahiro Yamada <[email protected]>
-Jul. 2015
+Aug. 2015
include/configs/uniphier.h +1 -1
index b0a7daa..d59564b 100644
@@ -288,7 +288,7 @@
 	"netdev=eth0\0"						\
 	"verify=n\0"						\
 	"nandupdate=nand erase 0 0x00100000 &&"			\
-		"tftpboot u-boot-spl.bin &&"			\
+		"tftpboot u-boot-spl-dtb.bin &&"		\
 		"nand write $loadaddr 0 0x00010000 &&"		\
 		"tftpboot u-boot-dtb.img &&"			\
 		"nand write $loadaddr 0x00010000 0x000f0000\0"	\
Signature · 1 line
1.9.1
#4 lore raw

PATCH 2/3 ARM: uniphier: enable SPL_OF_CONTROL

2015-08-28 20:28 GMT+09:00 Marek Vasut <[email protected]>:

Quoted 12 lines · Masahiro Yamada
Quoted 8 lines

Device Tree really improves code maintainability and is now available for SPL too.

This is the state-of-the-art implementation in U-boot.

The board files (platform data) are no longer needed.

Do you mean, Reviewed-by ?

You do not have maintainership for any files this commit is touching.

Signature · 2 lines
Best Regards
Masahiro Yamada
#5 lore raw

PATCH 2/3 ARM: uniphier: enable SPL_OF_CONTROL

Quoted 9 lines · Masahiro Yamada

2015-08-28 20:28 GMT+09:00 Marek Vasut <[email protected]>:

Quoted 8 lines · Masahiro Yamada
Quoted 7 lines

Device Tree really improves code maintainability and is now available for SPL too.

This is the state-of-the-art implementation in U-boot.

The board files (platform data) are no longer needed.

Quoted 7 lines
Quoted 3 lines
Quoted 1 line

Do you mean, Reviewed-by ?

You do not have maintainership for any files this commit is touching.

I meant acked-by, but I suspect the meaning of both acked-by and reviewed-by is a bit ambiguous. Is there some document which explains what acked-by and reviewed-by precisely mean ?

Anyway, to make it easier for you, add my:

HTH ;-)

Best regards,
Marek Vasut

#6 lore raw

PATCH 2/3 ARM: uniphier: enable SPL_OF_CONTROL

Hi Marek,

2015-08-29 6:41 GMT+09:00 Marek Vasut <[email protected]>:

Quoted 14 lines · Masahiro Yamada
Quoted 13 lines

2015-08-28 20:28 GMT+09:00 Marek Vasut <[email protected]>:

Quoted 11 lines · Masahiro Yamada
Quoted 8 lines

Device Tree really improves code maintainability and is now available for SPL too.

This is the state-of-the-art implementation in U-boot.

The board files (platform data) are no longer needed.

Quoted 8 lines
Quoted 3 lines

Do you mean, Reviewed-by ?

You do not have maintainership for any files this commit is touching.

I meant acked-by, but I suspect the meaning of both acked-by and reviewed-by is a bit ambiguous. Is there some document which explains what acked-by and reviewed-by precisely mean ?

Personally, I was very convinced with the last answer (Nov 14, 2012; 7:34am) in this thread:
http://linux-kernel.2935.n7.nabble.com/acked-by-meaning-td551744.html

If you want to refer to a more official one,
https://www.kernel.org/doc/Documentation/SubmittingPatches

------------->8------------
Acked-by: is often used by the maintainer of the affected code when that maintainer neither contributed to nor forwarded the patch.

[snip]

Acked-by: does not necessarily indicate acknowledgement of the entire patch. For example, if a patch affects multiple subsystems and has an Acked-by: from one subsystem maintainer then this usually indicates acknowledgement of just the part which affects that maintainer's code. --------------8<---------------

I think this is the typical usage of Acked-by:
http://patchwork.ozlabs.org/patch/508716/

In the kernel development, patch committers give Signed-off-by, but it is not the custom in U-boot.
So, I guess it makes sense that custodians issue Acked-by instead of Signed-off-by.

Signature · 2 lines
Best Regards
Masahiro Yamada
#7 lore raw

PATCH 2/3 ARM: uniphier: enable SPL_OF_CONTROL

Quoted 1 line · Masahiro Yamada

Hi Marek,

Hi Masahiro,

Quoted 23 lines

2015-08-29 6:41 GMT+09:00 Marek Vasut <[email protected]>:

Quoted 14 lines · Masahiro Yamada
Quoted 13 lines

2015-08-28 20:28 GMT+09:00 Marek Vasut <[email protected]>:

Quoted 11 lines · Masahiro Yamada
Quoted 8 lines

Device Tree really improves code maintainability and is now available for SPL too.

This is the state-of-the-art implementation in U-boot.

The board files (platform data) are no longer needed.

Quoted 7 lines
Quoted 3 lines

Do you mean, Reviewed-by ?

You do not have maintainership for any files this commit is touching.

I meant acked-by, but I suspect the meaning of both acked-by and reviewed-by is a bit ambiguous. Is there some document which explains what acked-by and reviewed-by precisely mean ?

Quoted 8 lines

Personally, I was very convinced with the last answer (Nov 14, 2012; 7:34am) in this thread:
http://linux-kernel.2935.n7.nabble.com/acked-by-meaning-td551744.html

If you want to refer to a more official one,
https://www.kernel.org/doc/Documentation/SubmittingPatches

Oh, they did formalize it here. I didn't know about that, thanks!

Quoted 13 lines

------------->8------------
Acked-by: is often used by the maintainer of the affected code when that maintainer neither contributed to nor forwarded the patch.

[snip]

Acked-by: does not necessarily indicate acknowledgement of the entire patch. For example, if a patch affects multiple subsystems and has an
Acked-by: from one subsystem maintainer then this usually indicates acknowledgement of just the part which affects that maintainer's code. --------------8<---------------

Quoted 7 lines

I think this is the typical usage of Acked-by:
http://patchwork.ozlabs.org/patch/508716/

In the kernel development, patch committers give Signed-off-by, but it is not the custom in U-boot.

We should certainly talk about the usage of SoB lines in U-Boot eventually.

Quoted 2 lines

So, I guess it makes sense that custodians issue Acked-by instead of Signed-off-by.

That I am not so sure about. But, feel free to add my R-B to this patch if you feel it's appropriate ;-)

Best regards,
Marek Vasut

#8 lore raw

PATCH 2/3 ARM: uniphier: enable SPL_OF_CONTROL

In reply to Masahiro Yamada
Quoted 21 lines · Masahiro Yamada

Device Tree really improves code maintainability and is now available for SPL too.

This is the state-of-the-art implementation in U-boot.

The board files (platform data) are no longer needed.

---

 arch/arm/Kconfig                             |  1 +
 arch/arm/dts/uniphier-ph1-ld4-ref.dts        |  9 ++++++---
 arch/arm/dts/uniphier-ph1-pro4-ref.dts       |  9 ++++++---
 arch/arm/dts/uniphier-ph1-sld3-ref.dts       |  9 ++++++---
 arch/arm/dts/uniphier-ph1-sld8-ref.dts       |  9 ++++++---
 arch/arm/mach-uniphier/ph1-ld4/Makefile      |  1 -
 arch/arm/mach-uniphier/ph1-ld4/platdevice.c  | 15 ---------------
 arch/arm/mach-uniphier/ph1-pro4/Makefile     |  1 -
 arch/arm/mach-uniphier/ph1-pro4/platdevice.c | 15 ---------------
 arch/arm/mach-uniphier/ph1-sld3/Makefile     |  1 -
 arch/arm/mach-uniphier/ph1-sld3/platdevice.c |  1 -
Quoted 8 lines
 arch/arm/mach-uniphier/ph1-sld8/platdevice.c | 15 ---------------
 doc/README.uniphier                          | 10 +++++-----
 include/configs/uniphier.h                   |  2 +-
 14 files changed, 31 insertions(+), 67 deletions(-)
 delete mode 100644 arch/arm/mach-uniphier/ph1-ld4/platdevice.c
 delete mode 100644 arch/arm/mach-uniphier/ph1-pro4/platdevice.c
 delete mode 100644 arch/arm/mach-uniphier/ph1-sld3/platdevice.c
 delete mode 100644 arch/arm/mach-uniphier/ph1-sld8/platdevice.c

Very nice!

#9 lore raw

PATCH 2/3 ARM: uniphier: enable SPL_OF_CONTROL

In reply to Masahiro Yamada

2015-08-28 20:13 GMT+09:00 Masahiro Yamada <[email protected]>:

Quoted 8 lines

Device Tree really improves code maintainability and is now available for SPL too.

This is the state-of-the-art implementation in U-boot.

The board files (platform data) are no longer needed.

Applied to u-boot-uniphier/master.

Signature · 2 lines
Best Regards
Masahiro Yamada
#10 lore raw

PATCH 3/3 serial: uniphier: drop platform data support

In reply to Masahiro Yamada

This driver is enabled only for UniPhier SoCs and ARCH_UNIPHIER now selects OF_CONTROL and SPL_OF_CONTROL.

This driver no longer needs to support platform data configuration.

---

 drivers/serial/serial_uniphier.c           | 41 +++++++++++-------------------
 include/dm/platform_data/serial-uniphier.h | 18 -------------
 2 files changed, 15 insertions(+), 44 deletions(-)
 delete mode 100644 include/dm/platform_data/serial-uniphier.h
drivers/serial/serial_uniphier.c +15 -26
index abf362a..edb9203 100644
@@ -8,7 +8,6 @@
 #include <linux/serial_reg.h>
 #include <asm/errno.h>
 #include <dm/device.h>
-#include <dm/platform_data/serial-uniphier.h>
 #include <mapmem.h>
 #include <serial.h>
 #include <fdtdec.h>
@@ -34,6 +33,7 @@ struct uniphier_serial {
 
 struct uniphier_serial_private_data {
 	struct uniphier_serial __iomem *membase;
+	unsigned int uartclk;
 };
 
 #define uniphier_serial_port(dev)	\
@@ -41,12 +41,12 @@ struct uniphier_serial_private_data {
 
 static int uniphier_serial_setbrg(struct udevice *dev, int baudrate)
 {
-	struct uniphier_serial_platform_data *plat = dev_get_platdata(dev);
+	struct uniphier_serial_private_data *priv = dev_get_priv(dev);
 	struct uniphier_serial __iomem *port = uniphier_serial_port(dev);
 	const unsigned int mode_x_div = 16;
 	unsigned int divisor;
 
-	divisor = DIV_ROUND_CLOSEST(plat->uartclk, mode_x_div * baudrate);
+	divisor = DIV_ROUND_CLOSEST(priv->uartclk, mode_x_div * baudrate);
 
 	writel(divisor, &port->dlr);
 
@@ -87,17 +87,24 @@ static int uniphier_serial_pending(struct udevice *dev, bool input)
 
 static int uniphier_serial_probe(struct udevice *dev)
 {
-	u32 tmp;
+	DECLARE_GLOBAL_DATA_PTR;
 	struct uniphier_serial_private_data *priv = dev_get_priv(dev);
-	struct uniphier_serial_platform_data *plat = dev_get_platdata(dev);
 	struct uniphier_serial __iomem *port;
+	fdt_addr_t base;
+	fdt_size_t size;
+	u32 tmp;
 
-	port = map_sysmem(plat->base, sizeof(struct uniphier_serial));
+	base = fdtdec_get_addr_size(gd->fdt_blob, dev->of_offset, "reg", &size);
+
+	port = map_sysmem(base, size);
 	if (!port)
 		return -ENOMEM;
 
 	priv->membase = port;
 
+	priv->uartclk = fdtdec_get_int(gd->fdt_blob, dev->of_offset,
+				       "clock-frequency", 0);
+
 	tmp = readl(&port->lcr_mcr);
 	tmp &= ~LCR_MASK;
 	tmp |= UART_LCR_WLEN8 << LCR_SHIFT;
@@ -113,25 +120,11 @@ static int uniphier_serial_remove(struct udevice *dev)
 	return 0;
 }
 
-#if CONFIG_IS_ENABLED(OF_CONTROL)
 static const struct udevice_id uniphier_uart_of_match[] = {
 	{ .compatible = "socionext,uniphier-uart" },
 	{ /* sentinel */ }
 };
 
-static int uniphier_serial_ofdata_to_platdata(struct udevice *dev)
-{
-	struct uniphier_serial_platform_data *plat = dev_get_platdata(dev);
-	DECLARE_GLOBAL_DATA_PTR;
-
-	plat->base = fdtdec_get_addr(gd->fdt_blob, dev->of_offset, "reg");
-	plat->uartclk = fdtdec_get_int(gd->fdt_blob, dev->of_offset,
-				       "clock-frequency", 0);
-
-	return 0;
-}
-#endif
-
 static const struct dm_serial_ops uniphier_serial_ops = {
 	.setbrg = uniphier_serial_setbrg,
 	.getc = uniphier_serial_getc,
@@ -140,15 +133,11 @@ static const struct dm_serial_ops uniphier_serial_ops = {
 };
 
 U_BOOT_DRIVER(uniphier_serial) = {
-	.name = DRIVER_NAME,
+	.name = "uniphier-uart",
 	.id = UCLASS_SERIAL,
-	.of_match = of_match_ptr(uniphier_uart_of_match),
-	.ofdata_to_platdata = of_match_ptr(uniphier_serial_ofdata_to_platdata),
+	.of_match = uniphier_uart_of_match,
 	.probe = uniphier_serial_probe,
 	.remove = uniphier_serial_remove,
 	.priv_auto_alloc_size = sizeof(struct uniphier_serial_private_data),
-	.platdata_auto_alloc_size =
-				sizeof(struct uniphier_serial_platform_data),
 	.ops = &uniphier_serial_ops,
-	.flags = DM_FLAG_PRE_RELOC,
 };
include/dm/platform_data/serial-uniphier.h deleted +0 -0
deleted file mode 100644
index 52343e3..0000000
Signature · 1 line
1.9.1
#11 lore raw

PATCH 3/3 serial: uniphier: drop platform data support

Quoted 5 lines · Masahiro Yamada

This driver is enabled only for UniPhier SoCs and ARCH_UNIPHIER now selects OF_CONTROL and SPL_OF_CONTROL.

This driver no longer needs to support platform data configuration.

Quoted 7 lines

---

 drivers/serial/serial_uniphier.c           | 41 +++++++++++-------------------
 include/dm/platform_data/serial-uniphier.h | 18 -------------
 2 files changed, 15 insertions(+), 44 deletions(-)
 delete mode 100644 include/dm/platform_data/serial-uniphier.h
#12 lore raw

PATCH 3/3 serial: uniphier: drop platform data support

In reply to Masahiro Yamada

2015-08-28 20:13 GMT+09:00 Masahiro Yamada <[email protected]>:

Quoted 6 lines

This driver is enabled only for UniPhier SoCs and ARCH_UNIPHIER now selects OF_CONTROL and SPL_OF_CONTROL.

This driver no longer needs to support platform data configuration.

Applied to u-boot-uniphier/master.

Signature · 2 lines
Best Regards
Masahiro Yamada