Skip to contentSkip to conversation

Drop legacy support for omap3517

PATCH New Tony Lindgren 5 participants 19 messages Jan 13, 2015 – Jan 26, 2015

Subsystems (7)
lore
Share & tools
lore

Tony Lindgren

author maintainer
#0 lore raw

cover Drop legacy support for omap3517

Hi all,

It seems we can now drop omap3517 legacy booting support to get a bit closer to making all of omap3 boot in device tree only mode.

All these boards have at least minimal support for booting with device tree, and pretty much anything supported with the legacy board files can be configured also for device tree based booting if not done already.

If anybody has issues booting 3517 with appended device tree, please post the isssues to the mailing lists so we can promptly fix those.

Regards,

Tony

Tony Lindgren (4):
ARM: OMAP3: Remove legacy support for am3517-evm
ARM: OMAP3: Remove legacy support for am3517crane
ARM: OMAP3: Remove cm-t3517 legacy support
ARM: OMAP3: Remove legacy support for am35xx-emac

 arch/arm/mach-omap2/Kconfig                |  17 --
 arch/arm/mach-omap2/Makefile               |   8 -
 arch/arm/mach-omap2/am35xx-emac.c          | 114 ---------
 arch/arm/mach-omap2/am35xx-emac.h          |  15 --
 arch/arm/mach-omap2/am35xx.h               |  46 ----
 arch/arm/mach-omap2/board-am3517crane.c    | 150 ------------
 arch/arm/mach-omap2/board-am3517evm.c      | 373 -----------------------------
 arch/arm/mach-omap2/board-cm-t3517.c       | 335 --------------------------
 arch/arm/mach-omap2/omap_hwmod_3xxx_data.c |  33 +--
 arch/arm/mach-omap2/pdata-quirks.c         |   1 -
 10 files changed, 2 insertions(+), 1090 deletions(-)
 delete mode 100644 arch/arm/mach-omap2/am35xx-emac.c
 delete mode 100644 arch/arm/mach-omap2/am35xx-emac.h
 delete mode 100644 arch/arm/mach-omap2/am35xx.h
 delete mode 100644 arch/arm/mach-omap2/board-am3517crane.c
 delete mode 100644 arch/arm/mach-omap2/board-am3517evm.c
 delete mode 100644 arch/arm/mach-omap2/board-cm-t3517.c
Signature · 1 line
2.1.4
author maintainer
#1 lore raw

PATCH 1/4 ARM: OMAP3: Remove legacy support for am3517-evm

This board is working with device tree based booting so there should not be any need to keep the legacy booting support around. People using this board can boot it with appended DTB with existing bootloader.

By removing the 3517 legacy booting support we can get a bit closer to making all of omap3 boot in device tree only mode.

---
 arch/arm/mach-omap2/Kconfig           |   6 -
 arch/arm/mach-omap2/Makefile          |   2 -
 arch/arm/mach-omap2/board-am3517evm.c | 373 ----------------------------------
 3 files changed, 381 deletions(-)
 delete mode 100644 arch/arm/mach-omap2/board-am3517evm.c
arch/arm/mach-omap2/Kconfig +0 -6
index 6ab656c..dcf126d 100644
@@ -213,12 +213,6 @@ config MACH_OVERO
 	default y
 	select OMAP_PACKAGE_CBB
 
-config MACH_OMAP3517EVM
-	bool "OMAP3517/ AM3517 EVM board"
-	depends on ARCH_OMAP3
-	default y
-	select OMAP_PACKAGE_CBB
-
 config MACH_CRANEBOARD
 	bool "AM3517/05 CRANE board"
 	depends on ARCH_OMAP3
arch/arm/mach-omap2/Makefile +0 -2
index 5d27dfd..b68fb66 100644
@@ -253,8 +253,6 @@ obj-$(CONFIG_MACH_CM_T35)		+= board-cm-t35.o
 obj-$(CONFIG_MACH_CM_T3517)		+= board-cm-t3517.o
 obj-$(CONFIG_MACH_TOUCHBOOK)		+= board-omap3touchbook.o
 
-obj-$(CONFIG_MACH_OMAP3517EVM)		+= board-am3517evm.o
-
 obj-$(CONFIG_MACH_CRANEBOARD)		+= board-am3517crane.o
 
 obj-$(CONFIG_MACH_SBC3530)		+= board-omap3stalker.o
arch/arm/mach-omap2/board-am3517evm.c deleted +0 -373

373 lines not on this page · raw message

Signature · 1 line
2.1.4
author maintainer
#2 lore raw

PATCH 2/4 ARM: OMAP3: Remove legacy support for am3517crane

In reply to Tony Lindgren

This board is working with device tree based booting so there should not be any need to keep the legacy booting support around. People using this board can boot it with appended DTB with existing bootloader.

By removing the 3517 legacy booting support we can get a bit closer to making all of omap3 boot in device tree only mode.

---
 arch/arm/mach-omap2/Kconfig             |   5 --
 arch/arm/mach-omap2/Makefile            |   2 -
 arch/arm/mach-omap2/board-am3517crane.c | 150 --------------------------------
 3 files changed, 157 deletions(-)
 delete mode 100644 arch/arm/mach-omap2/board-am3517crane.c
arch/arm/mach-omap2/Kconfig +0 -5
index dcf126d..ba3a7a1 100644
@@ -213,11 +213,6 @@ config MACH_OVERO
 	default y
 	select OMAP_PACKAGE_CBB
 
-config MACH_CRANEBOARD
-	bool "AM3517/05 CRANE board"
-	depends on ARCH_OMAP3
-	select OMAP_PACKAGE_CBB
-
 config MACH_OMAP3_PANDORA
 	bool "OMAP3 Pandora"
 	depends on ARCH_OMAP3
arch/arm/mach-omap2/Makefile +0 -2
index b68fb66..39b1ee1 100644
@@ -253,8 +253,6 @@ obj-$(CONFIG_MACH_CM_T35)		+= board-cm-t35.o
 obj-$(CONFIG_MACH_CM_T3517)		+= board-cm-t3517.o
 obj-$(CONFIG_MACH_TOUCHBOOK)		+= board-omap3touchbook.o
 
-obj-$(CONFIG_MACH_CRANEBOARD)		+= board-am3517crane.o
-
 obj-$(CONFIG_MACH_SBC3530)		+= board-omap3stalker.o
 
 # Platform specific device init code
arch/arm/mach-omap2/board-am3517crane.c deleted +0 -150

150 lines not on this page · raw message

Signature · 1 line
2.1.4
author maintainer
#3 lore raw

PATCH 3/4 ARM: OMAP3: Remove cm-t3517 legacy support

In reply to Tony Lindgren

This board is working with device tree based booting so there should not be any need to keep the legacy booting support around. People using this board can boot it with appended DTB with existing bootloader.

By removing the 3517 legacy booting support we can get a bit closer to making all of omap3 boot in device tree only mode.

Cc Dmitry Lifshitz <[email protected]>
Cc Igor Grinberg <[email protected]>
---
 arch/arm/mach-omap2/Kconfig          |   6 -
 arch/arm/mach-omap2/Makefile         |   1 -
 arch/arm/mach-omap2/board-cm-t3517.c | 335 -----------------------------------
 3 files changed, 342 deletions(-)
 delete mode 100644 arch/arm/mach-omap2/board-cm-t3517.c
arch/arm/mach-omap2/Kconfig +0 -6
index ba3a7a1..209456e 100644
@@ -252,12 +252,6 @@ config MACH_CM_T35
 	select MACH_CM_T3730
 	select OMAP_PACKAGE_CUS
 
-config MACH_CM_T3517
-	bool "CompuLab CM-T3517 module"
-	depends on ARCH_OMAP3
-	default y
-	select OMAP_PACKAGE_CBB
-
 config MACH_CM_T3730
        bool
 
arch/arm/mach-omap2/Makefile +0 -1
index 39b1ee1..29da3f4 100644
@@ -250,7 +250,6 @@ obj-$(CONFIG_MACH_NOKIA_RX51)		+= board-rx51.o sdram-nokia.o
 obj-$(CONFIG_MACH_NOKIA_RX51)		+= board-rx51-peripherals.o
 obj-$(CONFIG_MACH_NOKIA_RX51)		+= board-rx51-video.o
 obj-$(CONFIG_MACH_CM_T35)		+= board-cm-t35.o
-obj-$(CONFIG_MACH_CM_T3517)		+= board-cm-t3517.o
 obj-$(CONFIG_MACH_TOUCHBOOK)		+= board-omap3touchbook.o
 
 obj-$(CONFIG_MACH_SBC3530)		+= board-omap3stalker.o
arch/arm/mach-omap2/board-cm-t3517.c deleted +0 -335

335 lines not on this page · raw message

Signature · 1 line
2.1.4
author maintainer
#4 lore raw

PATCH 4/4 ARM: OMAP3: Remove legacy support for am35xx-emac

In reply to Tony Lindgren

This is no longer needed now that 3517 is booting in device tree only mode.

---
 arch/arm/mach-omap2/Makefile               |   3 -
 arch/arm/mach-omap2/am35xx-emac.c          | 114 -----------------------------
 arch/arm/mach-omap2/am35xx-emac.h          |  15 ----
 arch/arm/mach-omap2/am35xx.h               |  46 ------------
 arch/arm/mach-omap2/omap_hwmod_3xxx_data.c |  33 +--------
 arch/arm/mach-omap2/pdata-quirks.c         |   1 -
 6 files changed, 2 insertions(+), 210 deletions(-)
 delete mode 100644 arch/arm/mach-omap2/am35xx-emac.c
 delete mode 100644 arch/arm/mach-omap2/am35xx-emac.h
 delete mode 100644 arch/arm/mach-omap2/am35xx.h
arch/arm/mach-omap2/Makefile +0 -3
index 29da3f4..bc1a056 100644
@@ -281,7 +281,4 @@ ifneq ($(CONFIG_HWSPINLOCK_OMAP),)
 obj-y					+= hwspinlock.o
 endif
 
-emac-$(CONFIG_TI_DAVINCI_EMAC)		:= am35xx-emac.o
-obj-y					+= $(emac-m) $(emac-y)
-
 obj-y					+= common-board-devices.o twl-common.o dss-common.o
arch/arm/mach-omap2/am35xx-emac.c deleted +0 -114

114 lines not on this page · raw message

arch/arm/mach-omap2/am35xx-emac.h deleted +0 -15
deleted file mode 100644
index 15c6f9c..0000000
@@ -1,15 +0,0 @@
-/*
- * Copyright (C) 2011 Ilya Yanok, Emcraft Systems
- *
- * 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.
- */
-
-#define AM35XX_DEFAULT_MDIO_FREQUENCY	1000000
-
-#if defined(CONFIG_TI_DAVINCI_EMAC) || defined(CONFIG_TI_DAVINCI_EMAC_MODULE)
-void am35xx_emac_init(unsigned long mdio_bus_freq, u8 rmii_en);
-#else
-static inline void am35xx_emac_init(unsigned long mdio_bus_freq, u8 rmii_en) {}
-#endif
arch/arm/mach-omap2/am35xx.h deleted +0 -46

46 lines not on this page · raw message

arch/arm/mach-omap2/omap_hwmod_3xxx_data.c +2 -31
index 11468ee..4e8e93c 100644
@@ -29,8 +29,6 @@
 #include <linux/platform_data/mailbox-omap.h>
 #include <plat/dmtimer.h>
 
-#include "am35xx.h"
-
 #include "soc.h"
 #include "omap_hwmod.h"
 #include "omap_hwmod_common_data.h"
@@ -50,6 +48,8 @@
  * elsewhere.
  */
 
+#define AM35XX_IPSS_USBOTGSS_BASE      0x5C040000
+
 /*
  * IP blocks
  */
@@ -3459,15 +3459,6 @@ static struct omap_hwmod_ocp_if am35xx_mdio__l3 = {
 	.user		= OCP_USER_MPU,
 };
 
-static struct omap_hwmod_addr_space am35xx_mdio_addrs[] = {
-	{
-		.pa_start	= AM35XX_IPSS_MDIO_BASE,
-		.pa_end		= AM35XX_IPSS_MDIO_BASE + SZ_4K - 1,
-		.flags		= ADDR_TYPE_RT,
-	},
-	{ }
-};
-
 /* l4_core -> davinci mdio  */
 /*
  * XXX Should be connected to an IPSS hwmod, not the L4_CORE directly;
@@ -3478,25 +3469,15 @@ static struct omap_hwmod_ocp_if am35xx_l4_core__mdio = {
 	.master		= &omap3xxx_l4_core_hwmod,
 	.slave		= &am35xx_mdio_hwmod,
 	.clk		= "emac_fck",
-	.addr		= am35xx_mdio_addrs,
 	.user		= OCP_USER_MPU,
 };
 
-static struct omap_hwmod_irq_info am35xx_emac_mpu_irqs[] = {
-	{ .name = "rxthresh",	.irq = 67 + OMAP_INTC_START, },
-	{ .name = "rx_pulse",	.irq = 68 + OMAP_INTC_START, },
-	{ .name = "tx_pulse",	.irq = 69 + OMAP_INTC_START },
-	{ .name = "misc_pulse",	.irq = 70 + OMAP_INTC_START },
-	{ .irq = -1 },
-};
-
 static struct omap_hwmod_class am35xx_emac_class = {
 	.name = "davinci_emac",
 };
 
 static struct omap_hwmod am35xx_emac_hwmod = {
 	.name		= "davinci_emac",
-	.mpu_irqs	= am35xx_emac_mpu_irqs,
 	.class		= &am35xx_emac_class,
 	/*
 	 * According to Mark Greer, the MPU will not return from WFI
@@ -3519,15 +3500,6 @@ static struct omap_hwmod_ocp_if am35xx_emac__l3 = {
 	.user		= OCP_USER_MPU,
 };
 
-static struct omap_hwmod_addr_space am35xx_emac_addrs[] = {
-	{
-		.pa_start	= AM35XX_IPSS_EMAC_BASE,
-		.pa_end		= AM35XX_IPSS_EMAC_BASE + 0x30000 - 1,
-		.flags		= ADDR_TYPE_RT,
-	},
-	{ }
-};
-
 /* l4_core -> davinci emac  */
 /*
  * XXX Should be connected to an IPSS hwmod, not the L4_CORE directly;
@@ -3538,7 +3510,6 @@ static struct omap_hwmod_ocp_if am35xx_l4_core__emac = {
 	.master		= &omap3xxx_l4_core_hwmod,
 	.slave		= &am35xx_emac_hwmod,
 	.clk		= "emac_ick",
-	.addr		= am35xx_emac_addrs,
 	.user		= OCP_USER_MPU,
 };
 
arch/arm/mach-omap2/pdata-quirks.c +0 -1
index 3d7eee1..190fa43 100644
@@ -19,7 +19,6 @@
 #include <linux/platform_data/pinctrl-single.h>
 #include <linux/platform_data/iommu-omap.h>
 
-#include "am35xx.h"
 #include "common.h"
 #include "common-board-devices.h"
 #include "dss-common.h"
Signature · 1 line
2.1.4
#5 lore raw

cover Drop legacy support for omap3517

In reply to Tony Lindgren
Quoted 8 lines · On Tuesday 13 January 2015 09:57:41 Tony Lindgren wrote:

It seems we can now drop omap3517 legacy booting support to get a bit closer to making all of omap3 boot in device tree only mode.

All these boards have at least minimal support for booting with device tree, and pretty much anything supported with the legacy board files can be configured also for device tree based booting if not done already.

Ah, very nice.

Just out of curiosity, what are the remaining showstoppers for the 3430 and 3530 based boards?

Arnd

author maintainer
#6 lore raw

cover Drop legacy support for omap3517

Quoted 10 lines · Arnd Bergmann
Quoted 8 lines · On Tuesday 13 January 2015 09:57:41 Tony Lindgren wrote:

It seems we can now drop omap3517 legacy booting support to get a bit closer to making all of omap3 boot in device tree only mode.

All these boards have at least minimal support for booting with device tree, and pretty much anything supported with the legacy board files can be configured also for device tree based booting if not done already.

Quoted 4 lines

Ah, very nice.

Just out of curiosity, what are the remaining showstoppers for the 3430 and 3530 based boards?

Not much really. We're now printing a warning to get a people to upgrade their systems. So with some boards we need to wait a while.

Then here are the remaining items that I'm aware of:

- A regression at least on some n900 with appended DTB not booting properly any longer reported by Pali

- A regression where the legacy ATAGs don't seem to get properly translated by the uncompress code for atag_rev at least reported by Pali

- A few missing .dts files for devkit8000, omap3logic, omap3stalker, omap3pandora and omap3touchbook

Pali and Sebastian probably know best of any other remaining issues to drop n900 legacy booting.

If people have the boards listed above, please pitch in to do the .dts files or send the boards to me and I can do minimal .dts files for them.

Regards,

Tony

#7 lore raw

cover Drop legacy support for omap3517

Quoted 35 lines · On Tuesday 13 January 2015 21:42:20 Tony Lindgren wrote:
Quoted 10 lines · Arnd Bergmann
Quoted 8 lines · On Tuesday 13 January 2015 09:57:41 Tony Lindgren wrote:

It seems we can now drop omap3517 legacy booting support to get a bit closer to making all of omap3 boot in device tree only mode.

All these boards have at least minimal support for booting with device tree, and pretty much anything supported with the legacy board files can be configured also for device tree based booting if not done already.

Quoted 4 lines

Ah, very nice.

Just out of curiosity, what are the remaining showstoppers for the 3430 and 3530 based boards?

Not much really. We're now printing a warning to get a people to upgrade their systems. So with some boards we need to wait a while.

Then here are the remaining items that I'm aware of:

- A regression at least on some n900 with appended DTB not booting properly any longer reported by Pali

- A regression where the legacy ATAGs don't seem to get properly translated by the uncompress code for atag_rev at least reported by Pali

- A few missing .dts files for devkit8000, omap3logic, omap3stalker, omap3pandora and omap3touchbook

Pali and Sebastian probably know best of any other remaining issues to drop n900 legacy booting.

Quoted 7 lines

If people have the boards listed above, please pitch in to do the .dts files or send the boards to me and I can do minimal .dts files for them.

Regards,

Tony

CCing Pavel Machek

He reported also some (still not fixed) regressions on n900.

Signature · 9 lines
Pali Roh?r
pali.rohar at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150113/3ba7d988/attachment.sig>
#8 lore raw

cover Drop legacy support for omap3517

Quoted 4 lines

CCing Pavel Machek

He reported also some (still not fixed) regressions on n900.

adp1653 flash driver does not have device tree bindings, yet. Patches were submitted, but I guess I'll need to push them some more.
Pavel

Signature · 2 lines
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
#9 lore raw

cover Drop legacy support for omap3517

Quoted 4 lines · Pavel Machek
Quoted 1 line

He reported also some (still not fixed) regressions on n900.

adp1653 flash driver does not have device tree bindings, yet. Patches were submitted, but I guess I'll need to push them some more.

Yes, but that's not a regression, since mainline kernel does not load adp1653 when booted in legacy mode.

-- Sebastian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150118/7af19704/attachment-0001.sig>

#10 lore raw

cover Drop legacy support for omap3517

In reply to Tony Lindgren
Quoted 28 lines · Tony Lindgren
Quoted 10 lines · Arnd Bergmann
Quoted 8 lines · On Tuesday 13 January 2015 09:57:41 Tony Lindgren wrote:

It seems we can now drop omap3517 legacy booting support to get a bit closer to making all of omap3 boot in device tree only mode.

All these boards have at least minimal support for booting with device tree, and pretty much anything supported with the legacy board files can be configured also for device tree based booting if not done already.

Quoted 4 lines

Ah, very nice.

Just out of curiosity, what are the remaining showstoppers for the 3430 and 3530 based boards?

Not much really. We're now printing a warning to get a people to upgrade their systems. So with some boards we need to wait a while.

Then here are the remaining items that I'm aware of:

- A regression at least on some n900 with appended DTB not booting properly any longer reported by Pali

- A regression where the legacy ATAGs don't seem to get properly translated by the uncompress code for atag_rev at least reported by Pali

Quoted 5 lines

- A few missing .dts files for devkit8000, omap3logic, omap3stalker, omap3pandora and omap3touchbook

Pali and Sebastian probably know best of any other remaining issues to drop n900 legacy booting.

Apart from the bugs there is only one feature-wise regression when the system is booted via DT, which is the accelerometer. I will (re)send patches for that in the next days.

-- Sebastian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150115/86546acf/attachment.sig>

#11 lore raw

cover Drop legacy support for omap3517

Quoted 39 lines · On Thursday 15 January 2015 15:25:36 Sebastian Reichel wrote:
Quoted 38 lines · Tony Lindgren
Quoted 10 lines · Arnd Bergmann
Quoted 8 lines · On Tuesday 13 January 2015 09:57:41 Tony Lindgren wrote:

It seems we can now drop omap3517 legacy booting support to get a bit closer to making all of omap3 boot in device tree only mode.

All these boards have at least minimal support for booting with device tree, and pretty much anything supported with the legacy board files can be configured also for device tree based booting if not done already.

Quoted 4 lines

Ah, very nice.

Just out of curiosity, what are the remaining showstoppers for the 3430 and 3530 based boards?

Not much really. We're now printing a warning to get a people to upgrade their systems. So with some boards we need to wait a while.

Then here are the remaining items that I'm aware of:

- A regression at least on some n900 with appended DTB not booting

properly any longer reported by Pali

- A regression where the legacy ATAGs don't seem to get properly

translated by the uncompress code for atag_rev at least reported by Pali

- A few missing .dts files for devkit8000, omap3logic, omap3stalker,

omap3pandora and omap3touchbook

Quoted 8 lines
Quoted 2 lines

Pali and Sebastian probably know best of any other remaining issues to drop n900 legacy booting.

Apart from the bugs there is only one feature-wise regression when the system is booted via DT, which is the accelerometer. I will (re)send patches for that in the next days.

-- Sebastian

Another regression for DT setup (which does not occur for board code):

omap_hsmmc driver does not export slot_name sysfs entry because it not supported by DT yet. Entry slot_name is used by userspace application to determinate if mmc block device is internal eMMC memory or external uSD card. So support for this property also in DT is needed.

Here is simple patch which fix this problem:

arch/arm/boot/dts/omap3-n900.dts +2 -0
index 8571027..31ca609 100644
@@ -665,6 +665,7 @@
 };
 
 &mmc1 {
+	slot-name = "external";
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc1_pins>;
 	vmmc-supply = <&vmmc1>;
@@ -674,6 +675,7 @@
 
 /* most boards use vaux3, only some old versions use vmmc2 instead */
 &mmc2 {
+	slot-name = "internal";
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc2_pins>;
 	vmmc-supply = <&vaux3>;
drivers/mmc/host/omap_hsmmc.c +2 -0
index 7c71dcd..cd189eb 100644
@@ -2021,6 +2021,8 @@ static struct omap_hsmmc_platform_data *of_get_hsmmc_pdata(struct device *dev)
 	if (of_find_property(np, "enable-sdio-wakeup", NULL))
 		pdata->pm_caps |= MMC_PM_WAKE_SDIO_IRQ;
 
+	of_property_read_string(np, "slot-name", &pdata->name);
+
 	return pdata;
 }
 #else

With this patch identification works fine:

/ # cat /sys/block/mmcblk0/device/../slot_name
external
/ # cat /sys/block/mmcblk1/device/../slot_name
internal

Without patch omap_hsmmc driver does not create slot_name.

Signature · 9 lines
Pali Roh?r
pali.rohar at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150124/e7a5459a/attachment.sig>
#12 lore raw

cover Drop legacy support for omap3517

Quoted 11 lines · On Saturday 24 January 2015 13:00:00 Pali Roh?r wrote:

Another regression for DT setup (which does not occur for board code):

omap_hsmmc driver does not export slot_name sysfs entry because it not supported by DT yet. Entry slot_name is used by userspace application to determinate if mmc block device is internal eMMC memory or external uSD card. So support for this property also in DT is needed.

Here is simple patch which fix this problem:

The sysfs file only exists for the two OMAP drivers, and the naming is used only by board-n8x0, board-rx51 and board-omap3logic.

It may be better to add a board-specific hack for these to keep the existing user space working but not spread the use of this file anywhere else.

Arnd

#13 lore raw

cover Drop legacy support for omap3517

Quoted 15 lines · On Saturday 24 January 2015 23:08:00 Arnd Bergmann wrote:
Quoted 10 lines · On Saturday 24 January 2015 13:00:00 Pali Roh?r wrote:

Another regression for DT setup (which does not occur for board code):

omap_hsmmc driver does not export slot_name sysfs entry because it not supported by DT yet. Entry slot_name is used by userspace application to determinate if mmc block device is internal eMMC memory or external uSD card. So support for this property also in DT is needed.

Here is simple patch which fix this problem:

The sysfs file only exists for the two OMAP drivers, and the naming is used only by board-n8x0, board-rx51 and board-omap3logic.

Quoted 5 lines

It may be better to add a board-specific hack for these to keep the existing user space working but not spread the use of this file anywhere else.

Arnd

Why? omap_hsmmc.c has already support for slot_name sysfs entry, just it is not filled from DT data (only from platform_data)...

Signature · 9 lines
Pali Roh?r
pali.rohar at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150124/9901f95a/attachment.sig>
#14 lore raw

cover Drop legacy support for omap3517

Quoted 16 lines · On Saturday 24 January 2015 23:14:58 Pali Roh?r wrote:
Quoted 15 lines · On Saturday 24 January 2015 23:08:00 Arnd Bergmann wrote:
Quoted 10 lines · On Saturday 24 January 2015 13:00:00 Pali Roh?r wrote:

Another regression for DT setup (which does not occur for board code):

omap_hsmmc driver does not export slot_name sysfs entry because it not supported by DT yet. Entry slot_name is used by userspace application to determinate if mmc block device is internal eMMC memory or external uSD card. So support for this property also in DT is needed.

Here is simple patch which fix this problem:

The sysfs file only exists for the two OMAP drivers, and the naming is used only by board-n8x0, board-rx51 and board-omap3logic.

Quoted 6 lines
Quoted 3 lines

It may be better to add a board-specific hack for these to keep the existing user space working but not spread the use of this file anywhere else.

Why? omap_hsmmc.c has already support for slot_name sysfs entry, just it is not filled from DT data (only from platform_data)...

It's a highly nonstandard interface, none of the other controllers support it, and even changing the the other omap machines to use the same strings might break existing user space that might rely on whatever gets printed in those properties today.

We should limit the use of those strings to the boards that are known to run software that requires them to avoid regressions. We could also define a standard way of finding out whether an mmc device is removable. We already have the MMC_CAP_NONREMOVABLE flag in the kernel and could have a read-only boolean attribute in sysfs for it.

Putting a "name" property into DT because some random user space requires a particular name to show up in sysfs however does not seem a legitimate hardware description.

Arnd

#15 lore raw

cover Drop legacy support for omap3517

In reply to Arnd Bergmann
Quoted 13 lines · Arnd Bergmann
Quoted 11 lines · On Saturday 24 January 2015 13:00:00 Pali Roh?r wrote:

Another regression for DT setup (which does not occur for board code):

omap_hsmmc driver does not export slot_name sysfs entry because it not supported by DT yet. Entry slot_name is used by userspace application to determinate if mmc block device is internal eMMC memory or external uSD card. So support for this property also in DT is needed.

Here is simple patch which fix this problem:

Quoted 6 lines

The sysfs file only exists for the two OMAP drivers, and the naming is used only by board-n8x0, board-rx51 and board-omap3logic.

It may be better to add a board-specific hack for these to keep the existing user space working but not spread the use of this file anywhere else.

I'd guess this is going to be useful on more than just n900, and thus is worth supporting generically...

Pavel

Signature · 2 lines
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
#16 lore raw

cover Drop legacy support for omap3517

In reply to Pali Rohár
Quoted 35 lines · Pali Roh?r

Another regression for DT setup (which does not occur for board code):

omap_hsmmc driver does not export slot_name sysfs entry because it not supported by DT yet. Entry slot_name is used by userspace application to determinate if mmc block device is internal eMMC memory or external uSD card. So support for this property also in DT is needed.

Here is simple patch which fix this problem:

arch/arm/boot/dts/omap3-n900.dts +2 -0
index 8571027..31ca609 100644
@@ -665,6 +665,7 @@
 };
 
 &mmc1 {
+	slot-name = "external";
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc1_pins>;
 	vmmc-supply = <&vmmc1>;
@@ -674,6 +675,7 @@
 
 /* most boards use vaux3, only some old versions use vmmc2 instead */
 &mmc2 {
+	slot-name = "internal";
 	pinctrl-names = "default";
 	pinctrl-0 = <&mmc2_pins>;
 	vmmc-supply = <&vaux3>;
drivers/mmc/host/omap_hsmmc.c +0 -0
index 7c71dcd..cd189eb 100644

drivers/mmc/host/omap_hsmmc.c static struct omap_hsmmc_platform_data *of_get_hsmmc_pdata(struct device *dev) · line 2028

Quoted 8 lines
drivers/mmc/host/omap_hsmmc.c +2 -0
@@ -2021,6 +2021,8 @@ static struct omap_hsmmc_platform_data *of_get_hsmmc_pdata(struct device *dev)
 	if (of_find_property(np, "enable-sdio-wakeup", NULL))
 		pdata->pm_caps |= MMC_PM_WAKE_SDIO_IRQ;
 
+	of_property_read_string(np, "slot-name", &pdata->name);
+
 	return pdata;
 }
 #else

I suggest to use "label" as DT property name, which is also used for labeling GPIOs, LEDs, partitions, ... in DT.

-- Sebastian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150125/e97be3c8/attachment.sig>

#17 lore raw

cover Drop legacy support for omap3517

In reply to Pali Rohár
Quoted 51 lines · Pali Roh?r
Quoted 39 lines · On Thursday 15 January 2015 15:25:36 Sebastian Reichel wrote:
Quoted 38 lines · Tony Lindgren
Quoted 10 lines · Arnd Bergmann
Quoted 8 lines · On Tuesday 13 January 2015 09:57:41 Tony Lindgren wrote:

It seems we can now drop omap3517 legacy booting support to get a bit closer to making all of omap3 boot in device tree only mode.

All these boards have at least minimal support for booting with device tree, and pretty much anything supported with the legacy board files can be configured also for device tree based booting if not done already.

Quoted 4 lines

Ah, very nice.

Just out of curiosity, what are the remaining showstoppers for the 3430 and 3530 based boards?

Not much really. We're now printing a warning to get a people to upgrade their systems. So with some boards we need to wait a while.

Then here are the remaining items that I'm aware of:

- A regression at least on some n900 with appended DTB not booting

properly any longer reported by Pali

- A regression where the legacy ATAGs don't seem to get properly

translated by the uncompress code for atag_rev at least reported by Pali

- A few missing .dts files for devkit8000, omap3logic, omap3stalker,

omap3pandora and omap3touchbook

Quoted 8 lines
Quoted 2 lines

Pali and Sebastian probably know best of any other remaining issues to drop n900 legacy booting.

Apart from the bugs there is only one feature-wise regression when the system is booted via DT, which is the accelerometer. I will (re)send patches for that in the next days.

-- Sebastian

Another regression for DT setup (which does not occur for board code):

Quoted 7 lines

omap_hsmmc driver does not export slot_name sysfs entry because it not supported by DT yet. Entry slot_name is used by userspace application to determinate if mmc block device is internal eMMC memory or external uSD card. So support for this property also in DT is needed.

Here is simple patch which fix this problem:

Thanks!

Does it need line into Documentation/devicetree/ somewhere?

With that,

Pavel

Signature · 2 lines
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
#18 lore raw

cover Drop legacy support for omap3517

Quoted 8 lines · Pavel Machek
Quoted 8 lines

Another regression for DT setup (which does not occur for board code):

omap_hsmmc driver does not export slot_name sysfs entry because it not supported by DT yet. Entry slot_name is used by userspace application to determinate if mmc block device is internal eMMC memory or external uSD card. So support for this property also in DT is needed.

Quoted 5 lines
Quoted 1 line

Here is simple patch which fix this problem:

Thanks!

Does it need line into Documentation/devicetree/ somewhere?

Documentation/devicetree/bindings/mmc/mmc.txt (generic) Documentation/devicetree/bindings/mmc/ti-omap-hsmmc.txt (omap specific)

-- Sebastian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150125/2ee8f6e0/attachment-0001.sig>