* [PATCH 1/2] arm: mach-omap2: Kconfig: fix incorrect option
@ 2010-12-15 12:49 ` Aaro Koskinen
0 siblings, 0 replies; 10+ messages in thread
From: Aaro Koskinen @ 2010-12-15 12:49 UTC (permalink / raw)
To: tony, linux-omap, linux-arm-kernel; +Cc: Aaro Koskinen
There is no MFD config option, MFD_SUPPORT should be selected instead.
This will prevent build errors when trying out different configurations.
Signed-off-by: Aaro Koskinen <aaro.koskinen@nokia.com>
---
arch/arm/mach-omap2/Kconfig | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index ab784bf..db20351 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -15,7 +15,7 @@ config ARCH_OMAP2PLUS_TYPICAL
select SERIAL_OMAP_CONSOLE
select I2C
select I2C_OMAP
- select MFD
+ select MFD_SUPPORT
select MENELAUS if ARCH_OMAP2
select TWL4030_CORE if ARCH_OMAP3 || ARCH_OMAP4
select TWL4030_POWER if ARCH_OMAP3 || ARCH_OMAP4
--
1.5.6.5
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 1/2] arm: mach-omap2: Kconfig: fix incorrect option
@ 2010-12-15 12:49 ` Aaro Koskinen
0 siblings, 0 replies; 10+ messages in thread
From: Aaro Koskinen @ 2010-12-15 12:49 UTC (permalink / raw)
To: linux-arm-kernel
There is no MFD config option, MFD_SUPPORT should be selected instead.
This will prevent build errors when trying out different configurations.
Signed-off-by: Aaro Koskinen <aaro.koskinen@nokia.com>
---
arch/arm/mach-omap2/Kconfig | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index ab784bf..db20351 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -15,7 +15,7 @@ config ARCH_OMAP2PLUS_TYPICAL
select SERIAL_OMAP_CONSOLE
select I2C
select I2C_OMAP
- select MFD
+ select MFD_SUPPORT
select MENELAUS if ARCH_OMAP2
select TWL4030_CORE if ARCH_OMAP3 || ARCH_OMAP4
select TWL4030_POWER if ARCH_OMAP3 || ARCH_OMAP4
--
1.5.6.5
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 2/2] arm: mach-omap2: Kconfig: devkit8000 should select needed options
2010-12-15 12:49 ` Aaro Koskinen
@ 2010-12-15 12:49 ` Aaro Koskinen
-1 siblings, 0 replies; 10+ messages in thread
From: Aaro Koskinen @ 2010-12-15 12:49 UTC (permalink / raw)
To: tony, linux-omap, linux-arm-kernel; +Cc: Aaro Koskinen
It's not possible to compile a kernel for this board without I2C,
MFD_SUPPORT and TWL4030_CORE, so those should be selected. This will
prevent build errors when trying out different configurations.
Signed-off-by: Aaro Koskinen <aaro.koskinen@nokia.com>
---
arch/arm/mach-omap2/Kconfig | 4 ++++
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index db20351..bb32f35 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -129,6 +129,10 @@ config MACH_DEVKIT8000
default y
select OMAP_PACKAGE_CUS
select OMAP_MUX
+ select I2C
+ select I2C_OMAP
+ select MFD_SUPPORT
+ select TWL4030_CORE
config MACH_OMAP_LDP
bool "OMAP3 LDP board"
--
1.5.6.5
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 2/2] arm: mach-omap2: Kconfig: devkit8000 should select needed options
@ 2010-12-15 12:49 ` Aaro Koskinen
0 siblings, 0 replies; 10+ messages in thread
From: Aaro Koskinen @ 2010-12-15 12:49 UTC (permalink / raw)
To: linux-arm-kernel
It's not possible to compile a kernel for this board without I2C,
MFD_SUPPORT and TWL4030_CORE, so those should be selected. This will
prevent build errors when trying out different configurations.
Signed-off-by: Aaro Koskinen <aaro.koskinen@nokia.com>
---
arch/arm/mach-omap2/Kconfig | 4 ++++
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
index db20351..bb32f35 100644
--- a/arch/arm/mach-omap2/Kconfig
+++ b/arch/arm/mach-omap2/Kconfig
@@ -129,6 +129,10 @@ config MACH_DEVKIT8000
default y
select OMAP_PACKAGE_CUS
select OMAP_MUX
+ select I2C
+ select I2C_OMAP
+ select MFD_SUPPORT
+ select TWL4030_CORE
config MACH_OMAP_LDP
bool "OMAP3 LDP board"
--
1.5.6.5
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] arm: mach-omap2: Kconfig: devkit8000 should select needed options
2010-12-15 12:49 ` Aaro Koskinen
@ 2010-12-15 17:37 ` Tony Lindgren
-1 siblings, 0 replies; 10+ messages in thread
From: Tony Lindgren @ 2010-12-15 17:37 UTC (permalink / raw)
To: Aaro Koskinen; +Cc: linux-omap, linux-arm-kernel
Hi Aaro,
* Aaro Koskinen <aaro.koskinen@nokia.com> [101215 04:48]:
> It's not possible to compile a kernel for this board without I2C,
> MFD_SUPPORT and TWL4030_CORE, so those should be selected. This will
> prevent build errors when trying out different configurations.
This one I'm not so convinced about. We should be able to compile
support for each board and enable and disable these kind of options
just fine if CONFIG_ARCH_OMAP2PLUS_TYPICAL is disabled.
In the long run we really want to have just a minimal kernel
and have everything else as modules for the default configs
and boot using initramfs.
Probably the best way to deal with issues like this is to
have omap generic platform init code for the common devices
that gets built if those options are selected. Otherwise
we'll end up with ifdefs all over the board-*.c files.
Regards,
Tony
> Signed-off-by: Aaro Koskinen <aaro.koskinen@nokia.com>
> ---
> arch/arm/mach-omap2/Kconfig | 4 ++++
> 1 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
> index db20351..bb32f35 100644
> --- a/arch/arm/mach-omap2/Kconfig
> +++ b/arch/arm/mach-omap2/Kconfig
> @@ -129,6 +129,10 @@ config MACH_DEVKIT8000
> default y
> select OMAP_PACKAGE_CUS
> select OMAP_MUX
> + select I2C
> + select I2C_OMAP
> + select MFD_SUPPORT
> + select TWL4030_CORE
>
> config MACH_OMAP_LDP
> bool "OMAP3 LDP board"
> --
> 1.5.6.5
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 2/2] arm: mach-omap2: Kconfig: devkit8000 should select needed options
@ 2010-12-15 17:37 ` Tony Lindgren
0 siblings, 0 replies; 10+ messages in thread
From: Tony Lindgren @ 2010-12-15 17:37 UTC (permalink / raw)
To: linux-arm-kernel
Hi Aaro,
* Aaro Koskinen <aaro.koskinen@nokia.com> [101215 04:48]:
> It's not possible to compile a kernel for this board without I2C,
> MFD_SUPPORT and TWL4030_CORE, so those should be selected. This will
> prevent build errors when trying out different configurations.
This one I'm not so convinced about. We should be able to compile
support for each board and enable and disable these kind of options
just fine if CONFIG_ARCH_OMAP2PLUS_TYPICAL is disabled.
In the long run we really want to have just a minimal kernel
and have everything else as modules for the default configs
and boot using initramfs.
Probably the best way to deal with issues like this is to
have omap generic platform init code for the common devices
that gets built if those options are selected. Otherwise
we'll end up with ifdefs all over the board-*.c files.
Regards,
Tony
> Signed-off-by: Aaro Koskinen <aaro.koskinen@nokia.com>
> ---
> arch/arm/mach-omap2/Kconfig | 4 ++++
> 1 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
> index db20351..bb32f35 100644
> --- a/arch/arm/mach-omap2/Kconfig
> +++ b/arch/arm/mach-omap2/Kconfig
> @@ -129,6 +129,10 @@ config MACH_DEVKIT8000
> default y
> select OMAP_PACKAGE_CUS
> select OMAP_MUX
> + select I2C
> + select I2C_OMAP
> + select MFD_SUPPORT
> + select TWL4030_CORE
>
> config MACH_OMAP_LDP
> bool "OMAP3 LDP board"
> --
> 1.5.6.5
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] arm: mach-omap2: Kconfig: devkit8000 should select needed options
2010-12-15 17:37 ` Tony Lindgren
@ 2010-12-17 12:41 ` Aaro Koskinen
-1 siblings, 0 replies; 10+ messages in thread
From: Aaro Koskinen @ 2010-12-17 12:41 UTC (permalink / raw)
To: Tony Lindgren; +Cc: linux-omap, linux-arm-kernel
Hi,
On Wed, 15 Dec 2010, Tony Lindgren wrote:
> * Aaro Koskinen <aaro.koskinen@nokia.com> [101215 04:48]:
>> It's not possible to compile a kernel for this board without I2C,
>> MFD_SUPPORT and TWL4030_CORE, so those should be selected. This will
>> prevent build errors when trying out different configurations.
>
> This one I'm not so convinced about. We should be able to compile
> support for each board and enable and disable these kind of options
> just fine if CONFIG_ARCH_OMAP2PLUS_TYPICAL is disabled.
So in this case we should move the stuff with special dependencies from
the board file into e.g. board-devkit8000-peripherals, which would be
a kernel module?
A.
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 2/2] arm: mach-omap2: Kconfig: devkit8000 should select needed options
@ 2010-12-17 12:41 ` Aaro Koskinen
0 siblings, 0 replies; 10+ messages in thread
From: Aaro Koskinen @ 2010-12-17 12:41 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On Wed, 15 Dec 2010, Tony Lindgren wrote:
> * Aaro Koskinen <aaro.koskinen@nokia.com> [101215 04:48]:
>> It's not possible to compile a kernel for this board without I2C,
>> MFD_SUPPORT and TWL4030_CORE, so those should be selected. This will
>> prevent build errors when trying out different configurations.
>
> This one I'm not so convinced about. We should be able to compile
> support for each board and enable and disable these kind of options
> just fine if CONFIG_ARCH_OMAP2PLUS_TYPICAL is disabled.
So in this case we should move the stuff with special dependencies from
the board file into e.g. board-devkit8000-peripherals, which would be
a kernel module?
A.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/2] arm: mach-omap2: Kconfig: devkit8000 should select needed options
2010-12-17 12:41 ` Aaro Koskinen
@ 2010-12-18 0:21 ` Tony Lindgren
-1 siblings, 0 replies; 10+ messages in thread
From: Tony Lindgren @ 2010-12-18 0:21 UTC (permalink / raw)
To: Aaro Koskinen; +Cc: linux-omap, linux-arm-kernel
* Aaro Koskinen <aaro.koskinen@nokia.com> [101217 04:41]:
> Hi,
>
> On Wed, 15 Dec 2010, Tony Lindgren wrote:
> >* Aaro Koskinen <aaro.koskinen@nokia.com> [101215 04:48]:
> >>It's not possible to compile a kernel for this board without I2C,
> >>MFD_SUPPORT and TWL4030_CORE, so those should be selected. This will
> >>prevent build errors when trying out different configurations.
> >
> >This one I'm not so convinced about. We should be able to compile
> >support for each board and enable and disable these kind of options
> >just fine if CONFIG_ARCH_OMAP2PLUS_TYPICAL is disabled.
>
> So in this case we should move the stuff with special dependencies from
> the board file into e.g. board-devkit8000-peripherals, which would be
> a kernel module?
Or how about just have them in ifdef CONFIG_I2C_OMAP and TWL4030_CORE
in the board-devkit8000.c file?
Regards,
Tony
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 2/2] arm: mach-omap2: Kconfig: devkit8000 should select needed options
@ 2010-12-18 0:21 ` Tony Lindgren
0 siblings, 0 replies; 10+ messages in thread
From: Tony Lindgren @ 2010-12-18 0:21 UTC (permalink / raw)
To: linux-arm-kernel
* Aaro Koskinen <aaro.koskinen@nokia.com> [101217 04:41]:
> Hi,
>
> On Wed, 15 Dec 2010, Tony Lindgren wrote:
> >* Aaro Koskinen <aaro.koskinen@nokia.com> [101215 04:48]:
> >>It's not possible to compile a kernel for this board without I2C,
> >>MFD_SUPPORT and TWL4030_CORE, so those should be selected. This will
> >>prevent build errors when trying out different configurations.
> >
> >This one I'm not so convinced about. We should be able to compile
> >support for each board and enable and disable these kind of options
> >just fine if CONFIG_ARCH_OMAP2PLUS_TYPICAL is disabled.
>
> So in this case we should move the stuff with special dependencies from
> the board file into e.g. board-devkit8000-peripherals, which would be
> a kernel module?
Or how about just have them in ifdef CONFIG_I2C_OMAP and TWL4030_CORE
in the board-devkit8000.c file?
Regards,
Tony
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2010-12-18 0:21 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-12-15 12:49 [PATCH 1/2] arm: mach-omap2: Kconfig: fix incorrect option Aaro Koskinen
2010-12-15 12:49 ` Aaro Koskinen
2010-12-15 12:49 ` [PATCH 2/2] arm: mach-omap2: Kconfig: devkit8000 should select needed options Aaro Koskinen
2010-12-15 12:49 ` Aaro Koskinen
2010-12-15 17:37 ` Tony Lindgren
2010-12-15 17:37 ` Tony Lindgren
2010-12-17 12:41 ` Aaro Koskinen
2010-12-17 12:41 ` Aaro Koskinen
2010-12-18 0:21 ` Tony Lindgren
2010-12-18 0:21 ` Tony Lindgren
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.