| Message ID | 1392571560.28866.16.camel@x220 (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Hi Paul, On 02/16/14 19:26, Paul Bolle wrote: > The Kconfig symbol MACH_CM_T3730 was added in v3.1. It has never been > used. Setting it has no effect. There are no calls for > machine_is_cm_t3730(). This symbol can safely be removed. Indeed... Is it such a burden to keep it just until we switch OMAP3 to DT? Because, it makes a bit harder to hack on the kernel. Well, not too much as it can be reverted, but still one needs to remember to do this... I'd like to keep it just until we remove the board files, please. > > Signed-off-by: Paul Bolle <pebolle@tiscali.nl> > --- > Tested only with "git grep". > > arch/arm/mach-omap2/Kconfig | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig > index 66da3f5..7d4934e 100644 > --- a/arch/arm/mach-omap2/Kconfig > +++ b/arch/arm/mach-omap2/Kconfig > @@ -292,7 +292,6 @@ config MACH_CM_T35 > bool "CompuLab CM-T35/CM-T3730 modules" > depends on ARCH_OMAP3 > default y > - select MACH_CM_T3730 > select OMAP_PACKAGE_CUS > > config MACH_CM_T3517 > @@ -301,9 +300,6 @@ config MACH_CM_T3517 > default y > select OMAP_PACKAGE_CBB > > -config MACH_CM_T3730 > - bool > - > config MACH_SBC3530 > bool "OMAP3 SBC STALKER board" > depends on ARCH_OMAP3 >
Hi Igor, On Mon, 2014-02-17 at 14:33 +0200, Igor Grinberg wrote: > On 02/16/14 19:26, Paul Bolle wrote: > > The Kconfig symbol MACH_CM_T3730 was added in v3.1. It has never been > > used. Setting it has no effect. There are no calls for > > machine_is_cm_t3730(). This symbol can safely be removed. > > Indeed... Please note that I now doubt my reasoning was correct here. I'm going to revisit my recent batch of patches regarding, in short, arm Kconfig symbols used in mach-types.h. (lkml.org is sort of down, so I can't provide a link.) > Is it such a burden to keep it just until we switch OMAP3 to DT? > Because, it makes a bit harder to hack on the kernel. > Well, not too much as it can be reverted, but still one needs > to remember to do this... No, not really. The worst that could happen, from your perspective, is that if it turns out this was the right thing to do, I might send a reminder in a few months. > I'd like to keep it just until we remove the board files, please. Thanks, Paul Bolle
diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig index 66da3f5..7d4934e 100644 --- a/arch/arm/mach-omap2/Kconfig +++ b/arch/arm/mach-omap2/Kconfig @@ -292,7 +292,6 @@ config MACH_CM_T35 bool "CompuLab CM-T35/CM-T3730 modules" depends on ARCH_OMAP3 default y - select MACH_CM_T3730 select OMAP_PACKAGE_CUS config MACH_CM_T3517 @@ -301,9 +300,6 @@ config MACH_CM_T3517 default y select OMAP_PACKAGE_CBB -config MACH_CM_T3730 - bool - config MACH_SBC3530 bool "OMAP3 SBC STALKER board" depends on ARCH_OMAP3
The Kconfig symbol MACH_CM_T3730 was added in v3.1. It has never been used. Setting it has no effect. There are no calls for machine_is_cm_t3730(). This symbol can safely be removed. Signed-off-by: Paul Bolle <pebolle@tiscali.nl> --- Tested only with "git grep". arch/arm/mach-omap2/Kconfig | 4 ---- 1 file changed, 4 deletions(-)