Skip to contentSkip to conversation

Drop two more omap3 legacy board files for v4.3 merge window

PATCH New Tony Lindgren 3 participants 10 messages Jul 16, 2015 – Jul 17, 2015

Subsystems (3)
lore
Share & tools
lore

Tony Lindgren

author maintainer
#0 lore raw

cover Drop two more omap3 legacy board files for v4.3 merge window

Hi all,

I think we can drop these now. This just leaves n900 and ldp with n900 pending patches for legacy proc support.

Regards,

Tony

Tony Lindgren (2):
ARM: OMAP2+: Remove legacy booting support for LogicPD Torpedo
ARM: OMAP2+: Remove legacy booting support for Pandora

 arch/arm/mach-omap2/Kconfig              |  20 -
 arch/arm/mach-omap2/Makefile             |   3 -
 arch/arm/mach-omap2/board-omap3logic.c   | 249 ------------
 arch/arm/mach-omap2/board-omap3pandora.c | 633 -------------------------------
 4 files changed, 905 deletions(-)
 delete mode 100644 arch/arm/mach-omap2/board-omap3logic.c
 delete mode 100644 arch/arm/mach-omap2/board-omap3pandora.c
Signature · 1 line
2.1.4
author maintainer
#1 lore raw

PATCH 1/2 ARM: OMAP2+: Remove legacy booting support for LogicPD Torpedo

We've been moving all omap2+ based systems to boot in device tree only mode for a few years now. Only omap3 has legacy booting support remaining. Most omap3 boards already have related arch/arm/boot/*.dts* files for booting with device tree.

This board has support for device tree based booting, and we've been printing warnings about the legacy booting being deprecated for a few merge cycles now. Let's attempt to remove the legacy booting for it.

The reason for removing the legacy booting support now rather than later is we can simply revert this patch if necessary if we run into some unexpected issues that are not trivial to fix for the device tree based booting.

---
 arch/arm/mach-omap2/Kconfig            |  20 ---
 arch/arm/mach-omap2/Makefile           |   2 -
 arch/arm/mach-omap2/board-omap3logic.c | 249 ---------------------------------
 3 files changed, 271 deletions(-)
 delete mode 100644 arch/arm/mach-omap2/board-omap3logic.c
arch/arm/mach-omap2/Kconfig +0 -20
index ecc04ff..e68ad9f 100644
@@ -177,26 +177,6 @@ config MACH_OMAP_LDP
 	default y
 	select OMAP_PACKAGE_CBB
 
-config MACH_OMAP3530_LV_SOM
-	bool "OMAP3 Logic 3530 LV SOM board"
-	depends on ARCH_OMAP3
-	default y
-	select OMAP_PACKAGE_CBB
-	help
-	 Support for the LogicPD OMAP3530 SOM Development kit
-	 for full description please see the products webpage at
-	 http://www.logicpd.com/products/development-kits/texas-instruments-zoom%E2%84%A2-omap35x-development-kit
-
-config MACH_OMAP3_TORPEDO
-	bool "OMAP3 Logic 35x Torpedo board"
-	depends on ARCH_OMAP3
-	default y
-	select OMAP_PACKAGE_CBB
-	help
-	 Support for the LogicPD OMAP35x Torpedo Development kit
-	 for full description please see the products webpage at
-	 http://www.logicpd.com/products/development-kits/zoom-omap35x-torpedo-development-kit
-
 config MACH_OMAP3517EVM
 	bool "OMAP3517/ AM3517 EVM board"
 	depends on ARCH_OMAP3
arch/arm/mach-omap2/Makefile +0 -2
index 903c85b..9e4b17a 100644
@@ -243,8 +243,6 @@ obj-$(CONFIG_SOC_OMAP2420)		+= msdi.o
 # Specific board support
 obj-$(CONFIG_MACH_OMAP_GENERIC)		+= board-generic.o pdata-quirks.o
 obj-$(CONFIG_MACH_OMAP_LDP)		+= board-ldp.o
-obj-$(CONFIG_MACH_OMAP3530_LV_SOM)      += board-omap3logic.o
-obj-$(CONFIG_MACH_OMAP3_TORPEDO)        += board-omap3logic.o
 obj-$(CONFIG_MACH_OMAP3_PANDORA)	+= board-omap3pandora.o
 obj-$(CONFIG_MACH_NOKIA_N8X0)		+= board-n8x0.o
 obj-$(CONFIG_MACH_NOKIA_RX51)		+= board-rx51.o sdram-nokia.o
arch/arm/mach-omap2/board-omap3logic.c deleted +0 -249

249 lines not on this page · raw message

Signature · 1 line
2.1.4
author maintainer
#2 lore raw

PATCH 2/2 ARM: OMAP2+: Remove legacy booting support for Pandora

In reply to Tony Lindgren

We've been moving all omap2+ based systems to boot in device tree only mode for a few years now. Only omap3 has legacy booting support remaining. Most omap3 boards already have related arch/arm/boot/*.dts* files for booting with device tree.

This board has support for device tree based booting, and we've been printing warnings about the legacy booting being deprecated for a few merge cycles now. Let's attempt to remove the legacy booting for it.

The reason for removing the legacy booting support now rather than later is we can simply revert this patch if necessary if we run into some unexpected issues that are not trivial to fix for the device tree based booting.

Cc Signed-off-by: Grazvydas Ignotas <[email protected]>
---
 arch/arm/mach-omap2/Makefile             |   1 -
 arch/arm/mach-omap2/board-omap3pandora.c | 633 -------------------------------
 2 files changed, 634 deletions(-)
 delete mode 100644 arch/arm/mach-omap2/board-omap3pandora.c
arch/arm/mach-omap2/Makefile +0 -1
index 9e4b17a..9bb78a8 100644
@@ -243,7 +243,6 @@ obj-$(CONFIG_SOC_OMAP2420)		+= msdi.o
 # Specific board support
 obj-$(CONFIG_MACH_OMAP_GENERIC)		+= board-generic.o pdata-quirks.o
 obj-$(CONFIG_MACH_OMAP_LDP)		+= board-ldp.o
-obj-$(CONFIG_MACH_OMAP3_PANDORA)	+= board-omap3pandora.o
 obj-$(CONFIG_MACH_NOKIA_N8X0)		+= board-n8x0.o
 obj-$(CONFIG_MACH_NOKIA_RX51)		+= board-rx51.o sdram-nokia.o
 obj-$(CONFIG_MACH_NOKIA_RX51)		+= board-rx51-peripherals.o
arch/arm/mach-omap2/board-omap3pandora.c deleted +0 -633

633 lines not on this page · raw message

Signature · 1 line
2.1.4
#3 lore raw

PATCH 2/2 ARM: OMAP2+: Remove legacy booting support for Pandora

Hi,

Quoted 10 lines · Tony Lindgren

We've been moving all omap2+ based systems to boot in device tree only mode for a few years now. Only omap3 has legacy booting support remaining. Most omap3 boards already have related arch/arm/boot/*.dts* files for booting with device tree.

This board has support for device tree based booting, and we've been printing warnings about the legacy booting being deprecated for a few merge cycles now. Let's attempt to remove the legacy booting for it.

Quoted 4 lines

The reason for removing the legacy booting support now rather than later is we can simply revert this patch if necessary if we run into some unexpected issues that are not trivial to fix for the device tree based booting.

It seems we lose wifi, backlight, audio and usb host mainline support with this as pandora's .dts currently lacks all that stuff. That said I'm not aware of any mainline users (everyone seems to be on our "vendor" kernel), so maybe we can add those later.

Quoted 2 lines
Cc Signed-off-by: Grazvydas Ignotas <[email protected]>

I have not signed this off...

Quoted 7 lines
---
 arch/arm/mach-omap2/Makefile             |   1 -
 arch/arm/mach-omap2/board-omap3pandora.c | 633 -------------------------------
 2 files changed, 634 deletions(-)
 delete mode 100644 arch/arm/mach-omap2/board-omap3pandora.c

Gra?vydas

author maintainer
#4 lore raw

PATCH 2/2 ARM: OMAP2+: Remove legacy booting support for Pandora

Quoted 18 lines · Grazvydas Ignotas

Hi,

Quoted 10 lines · Tony Lindgren

We've been moving all omap2+ based systems to boot in device tree only mode for a few years now. Only omap3 has legacy booting support remaining. Most omap3 boards already have related arch/arm/boot/*.dts* files for booting with device tree.

This board has support for device tree based booting, and we've been printing warnings about the legacy booting being deprecated for a few merge cycles now. Let's attempt to remove the legacy booting for it.

Quoted 4 lines

The reason for removing the legacy booting support now rather than later is we can simply revert this patch if necessary if we run into some unexpected issues that are not trivial to fix for the device tree based booting.

Quoted 4 lines

It seems we lose wifi, backlight, audio and usb host mainline support with this as pandora's .dts currently lacks all that stuff. That said I'm not aware of any mainline users (everyone seems to be on our "vendor" kernel), so maybe we can add those later.

Hmm wifi should be easy, isn't that just libertas_sdio?

Quoted 3 lines
Quoted 1 line
Cc Signed-off-by: Grazvydas Ignotas <[email protected]>

I have not signed this off...

Oops sorry I copied your email address from git log.

author maintainer
#5 lore raw

PATCH 2/2 ARM: OMAP2+: Remove legacy booting support for Pandora

Quoted 14 lines · Tony Lindgren
Quoted 13 lines · Grazvydas Ignotas

Hi,

Quoted 10 lines · Tony Lindgren

We've been moving all omap2+ based systems to boot in device tree only mode for a few years now. Only omap3 has legacy booting support remaining. Most omap3 boards already have related arch/arm/boot/*.dts* files for booting with device tree.

This board has support for device tree based booting, and we've been printing warnings about the legacy booting being deprecated for a few merge cycles now. Let's attempt to remove the legacy booting for it.

Quoted 7 lines
Quoted 7 lines
Quoted 4 lines

The reason for removing the legacy booting support now rather than later is we can simply revert this patch if necessary if we run into some unexpected issues that are not trivial to fix for the device tree based booting.

It seems we lose wifi, backlight, audio and usb host mainline support with this as pandora's .dts currently lacks all that stuff.

More on that later on, but a question on the vendor kernel first..

Quoted 2 lines
Quoted 2 lines

That said I'm not aware of any mainline users (everyone seems to be on our "vendor" kernel), so maybe we can add those later.

What all is keeping people from using mainline kernel on pandora? Is it the sgx or are there other reasons too remaining?

Quoted 1 line

Hmm wifi should be easy, isn't that just libertas_sdio?

Nope wl1251, we can initialize with pdata-quirks.c for now if it does not yet support device tree except for the SPI version. The same goes for the other devices too if needed, I'd assume the the EHCI is similar to beagle though.

And of course we can still wait on removing the pandora board file if you want to. Sounds like that may not be needed though because of people using "vendor" kernel in this case even with the legacy booting?

Regards,

Tony

#6 lore raw

PATCH 2/2 ARM: OMAP2+: Remove legacy booting support for Pandora

Quoted 23 lines · Tony Lindgren
Quoted 14 lines · Tony Lindgren
Quoted 13 lines · Grazvydas Ignotas

Hi,

Quoted 10 lines · Tony Lindgren

We've been moving all omap2+ based systems to boot in device tree only mode for a few years now. Only omap3 has legacy booting support remaining. Most omap3 boards already have related arch/arm/boot/*.dts* files for booting with device tree.

This board has support for device tree based booting, and we've been printing warnings about the legacy booting being deprecated for a few merge cycles now. Let's attempt to remove the legacy booting for it.

Quoted 7 lines
Quoted 7 lines
Quoted 4 lines

The reason for removing the legacy booting support now rather than later is we can simply revert this patch if necessary if we run into some unexpected issues that are not trivial to fix for the device tree based booting.

It seems we lose wifi, backlight, audio and usb host mainline support with this as pandora's .dts currently lacks all that stuff.

Quoted 7 lines

More on that later on, but a question on the vendor kernel first..

Quoted 2 lines
Quoted 2 lines

That said I'm not aware of any mainline users (everyone seems to be on our "vendor" kernel), so maybe we can add those later.

What all is keeping people from using mainline kernel on pandora? Is it the sgx or are there other reasons too remaining?

There are multiple things, besides SGX:
- lack of 1GHz support due to lack of ABB/AVS. Meanwhile we have been running all pandoras at 1GHz for years without ABB/AVS on our kernel and no problems have been reported by users.
- overclocking support. It seems most DM3730s can do 1.2GHz and all can do 1.1GHz, running the device at that clock noticeably speeds up things, so we provide that functionality out of the box. No instances of SoC damage were reported over the years.
- using aufs for software packages, aufs wasn't merged and probably will never be
- keypad Fn key handling in the driver to simulate hardware Fn (not allowed in mainline)
- lack of driver for analog nubs

That said there probably are some people who can live without the above and prefer new features of new mainline kernels.

Quoted 6 lines
Quoted 1 line

Hmm wifi should be easy, isn't that just libertas_sdio?

Nope wl1251, we can initialize with pdata-quirks.c for now if it does not yet support device tree except for the SPI version. The same goes for the other devices too if needed, I'd assume the the EHCI is similar to beagle though.

Yeah maybe I should try to come up with a patch...

Quoted 4 lines

And of course we can still wait on removing the pandora board file if you want to. Sounds like that may not be needed though because of people using "vendor" kernel in this case even with the legacy booting?

Yes if it interferes with other mainline work just go ahead and remove it.

Quoted 4 lines

Regards,

Tony

Gra?vydas

author maintainer
#7 lore raw

PATCH 2/2 ARM: OMAP2+: Remove legacy booting support for Pandora

Quoted 32 lines · Grazvydas Ignotas
Quoted 23 lines · Tony Lindgren
Quoted 14 lines · Tony Lindgren
Quoted 13 lines · Grazvydas Ignotas

Hi,

Quoted 10 lines · Tony Lindgren

We've been moving all omap2+ based systems to boot in device tree only mode for a few years now. Only omap3 has legacy booting support remaining. Most omap3 boards already have related arch/arm/boot/*.dts* files for booting with device tree.

This board has support for device tree based booting, and we've been printing warnings about the legacy booting being deprecated for a few merge cycles now. Let's attempt to remove the legacy booting for it.

Quoted 7 lines
Quoted 7 lines
Quoted 4 lines

The reason for removing the legacy booting support now rather than later is we can simply revert this patch if necessary if we run into some unexpected issues that are not trivial to fix for the device tree based booting.

It seems we lose wifi, backlight, audio and usb host mainline support with this as pandora's .dts currently lacks all that stuff.

Quoted 7 lines

More on that later on, but a question on the vendor kernel first..

Quoted 2 lines
Quoted 2 lines

That said I'm not aware of any mainline users (everyone seems to be on our "vendor" kernel), so maybe we can add those later.

What all is keeping people from using mainline kernel on pandora? Is it the sgx or are there other reasons too remaining?

Quoted 13 lines

There are multiple things, besides SGX:
- lack of 1GHz support due to lack of ABB/AVS. Meanwhile we have been running all pandoras at 1GHz for years without ABB/AVS on our kernel and no problems have been reported by users.
- overclocking support. It seems most DM3730s can do 1.2GHz and all can do 1.1GHz, running the device at that clock noticeably speeds up things, so we provide that functionality out of the box. No instances of SoC damage were reported over the years.
- using aufs for software packages, aufs wasn't merged and probably will never be
- keypad Fn key handling in the driver to simulate hardware Fn (not allowed in mainline)
- lack of driver for analog nubs

OK yeah the 1GHz mode would be nice to have for sure.

Quoted 2 lines

That said there probably are some people who can live without the above and prefer new features of new mainline kernels.

Yes seems that can be done quite easilly :)

Quoted 8 lines
Quoted 6 lines
Quoted 1 line

Hmm wifi should be easy, isn't that just libertas_sdio?

Nope wl1251, we can initialize with pdata-quirks.c for now if it does not yet support device tree except for the SPI version. The same goes for the other devices too if needed, I'd assume the the EHCI is similar to beagle though.

Yeah maybe I should try to come up with a patch...

OK

Quoted 6 lines
Quoted 4 lines

And of course we can still wait on removing the pandora board file if you want to. Sounds like that may not be needed though because of people using "vendor" kernel in this case even with the legacy booting?

Yes if it interferes with other mainline work just go ahead and remove it.

OK will do then.

Thanks,

Tony

#8 lore raw

cover Drop two more omap3 legacy board files for v4.3 merge window

In reply to Tony Lindgren

Hi,

Quoted 2 lines · Tony Lindgren

I think we can drop these now. This just leaves n900 and ldp with n900 pending patches for legacy proc support.

So I guess n900 board file removal depends on the atag procfs support in DT. What's the problem with ldp? Its DT file seem to cover everything in the board file. I guess both of them will finally be removed in 4.4?

-- 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/20150717/afc5f085/attachment.sig>

author maintainer
#9 lore raw

cover Drop two more omap3 legacy board files for v4.3 merge window

Quoted 10 lines · Sebastian Reichel

Hi,

Quoted 2 lines · Tony Lindgren

I think we can drop these now. This just leaves n900 and ldp with n900 pending patches for legacy proc support.

So I guess n900 board file removal depends on the atag procfs support in DT. What's the problem with ldp? Its DT file seem to cover everything in the board file. I guess both of them will finally be removed in 4.4?

Yes the atags procfs and setting system_rev as being discussed with the patches posted by Pali.

LDP actually works better with device tree, we just want to keep the legacy booting around until the n900 issues are sorted out as it's hooked to Russell's build and boot test rack.

Regards,

Tony