Drop legacy support for omap3517
Subsystems (7)
Tony Lindgren
maintainercover 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
Tony Lindgren
maintainerPATCH 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
@@ -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
@@ -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
Tony Lindgren
maintainerPATCH 2/4 ARM: OMAP3: Remove legacy support for am3517crane
In reply to Tony LindgrenThis 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
@@ -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
@@ -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
Tony Lindgren
maintainerPATCH 3/4 ARM: OMAP3: Remove cm-t3517 legacy support
In reply to Tony LindgrenThis 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 | 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
@@ -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
@@ -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
Tony Lindgren
maintainerPATCH 4/4 ARM: OMAP3: Remove legacy support for am35xx-emac
In reply to Tony LindgrenThis 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
@@ -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
@@ -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
@@ -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
@@ -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
cover Drop legacy support for omap3517
In reply to Tony LindgrenQuoted 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
Tony Lindgren
maintainercover 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
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_revat 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>
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
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>
cover Drop legacy support for omap3517
In reply to Tony LindgrenQuoted 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_revat 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>
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_revat 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
@@ -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
@@ -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>
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_hsmmcdriver does not exportslot_namesysfs entry because it not supported by DT yet. Entryslot_nameis 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
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_hsmmcdriver does not exportslot_namesysfs entry because it not supported by DT yet. Entryslot_nameis 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>
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_hsmmcdriver does not exportslot_namesysfs entry because it not supported by DT yet. Entryslot_nameis 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.chas already support forslot_namesysfs entry, just it is not filled from DT data (only fromplatform_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
cover Drop legacy support for omap3517
In reply to Arnd BergmannQuoted 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_hsmmcdriver does not exportslot_namesysfs entry because it not supported by DT yet. Entryslot_nameis 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
cover Drop legacy support for omap3517
In reply to Pali RohárQuoted 35 lines · Pali Roh?r
Another regression for DT setup (which does not occur for board code):
omap_hsmmcdriver does not exportslot_namesysfs entry because it not supported by DT yet. Entryslot_nameis 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@@ -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
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>
cover Drop legacy support for omap3517
In reply to Pali RohárQuoted 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_revat 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_hsmmcdriver does not exportslot_namesysfs entry because it not supported by DT yet. Entryslot_nameis 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
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_hsmmcdriver does not exportslot_namesysfs entry because it not supported by DT yet. Entryslot_nameis 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>