* [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3
@ 2010-10-26 3:29 ` Samreen
0 siblings, 0 replies; 22+ messages in thread
From: Samreen @ 2010-10-26 3:17 UTC (permalink / raw)
To: Tomi Valkeinen, Grazvydas Ignotas, Vaibhav Hiremath,
Roger Quadros, Senthilvadivu
Cc: linux-omap, linux-fbdev, Samreen
The defconfig options for display are taken in the respective Kconfig
to enable display by default on OMAP3 platforms
Signed-off-by: Samreen <samreen@ti.com>
---
Version4:
Remove the enabling of the display panels by default.
Version3:
Eliminate the separate default number of FBs for
different architecture. Keeping default FBs as 3 as before.
Version2:
Enables by default NEC panel used in zoom2/3/3630sdp, instead of
Sharp LQ043T1DG01 panel enabled in previous version of this patch.
drivers/video/omap2/dss/Kconfig | 6 ++++--
drivers/video/omap2/omapfb/Kconfig | 1 +
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/video/omap2/dss/Kconfig b/drivers/video/omap2/dss/Kconfig
index 43b6440..f3244a2 100644
--- a/drivers/video/omap2/dss/Kconfig
+++ b/drivers/video/omap2/dss/Kconfig
@@ -1,6 +1,7 @@
menuconfig OMAP2_DSS
tristate "OMAP2/3 Display Subsystem support (EXPERIMENTAL)"
depends on ARCH_OMAP2 || ARCH_OMAP3
+ default y
help
OMAP2/3 Display Subsystem support.
@@ -9,7 +10,7 @@ if OMAP2_DSS
config OMAP2_VRAM_SIZE
int "VRAM size (MB)"
range 0 32
- default 0
+ default 4
help
The amount of SDRAM to reserve at boot time for video RAM use.
This VRAM will be used by omapfb and other drivers that need
@@ -102,7 +103,8 @@ config OMAP2_DSS_FAKE_VSYNC
config OMAP2_DSS_MIN_FCK_PER_PCK
int "Minimum FCK/PCK ratio (for scaling)"
range 0 32
- default 0
+ default 4 if ARCH_OMAP2 || ARCH_OMAP3
+ default 0 if ARCH_OMAP4
help
This can be used to adjust the minimum FCK/PCK ratio.
diff --git a/drivers/video/omap2/omapfb/Kconfig b/drivers/video/omap2/omapfb/Kconfig
index 65149b2..923bf48 100644
--- a/drivers/video/omap2/omapfb/Kconfig
+++ b/drivers/video/omap2/omapfb/Kconfig
@@ -1,6 +1,7 @@
menuconfig FB_OMAP2
tristate "OMAP2/3 frame buffer support (EXPERIMENTAL)"
depends on FB && OMAP2_DSS
+ default y
select OMAP2_VRAM
select OMAP2_VRFB if ARCH_OMAP2 || ARCH_OMAP3
--
1.5.6.3
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3
@ 2010-10-26 3:29 ` Samreen
0 siblings, 0 replies; 22+ messages in thread
From: Samreen @ 2010-10-26 3:29 UTC (permalink / raw)
To: Tomi Valkeinen, Grazvydas Ignotas, Vaibhav Hiremath,
Roger Quadros, Senthilvadivu
Cc: linux-omap, linux-fbdev, Samreen
The defconfig options for display are taken in the respective Kconfig
to enable display by default on OMAP3 platforms
Signed-off-by: Samreen <samreen@ti.com>
---
Version4:
Remove the enabling of the display panels by default.
Version3:
Eliminate the separate default number of FBs for
different architecture. Keeping default FBs as 3 as before.
Version2:
Enables by default NEC panel used in zoom2/3/3630sdp, instead of
Sharp LQ043T1DG01 panel enabled in previous version of this patch.
drivers/video/omap2/dss/Kconfig | 6 ++++--
drivers/video/omap2/omapfb/Kconfig | 1 +
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/video/omap2/dss/Kconfig b/drivers/video/omap2/dss/Kconfig
index 43b6440..f3244a2 100644
--- a/drivers/video/omap2/dss/Kconfig
+++ b/drivers/video/omap2/dss/Kconfig
@@ -1,6 +1,7 @@
menuconfig OMAP2_DSS
tristate "OMAP2/3 Display Subsystem support (EXPERIMENTAL)"
depends on ARCH_OMAP2 || ARCH_OMAP3
+ default y
help
OMAP2/3 Display Subsystem support.
@@ -9,7 +10,7 @@ if OMAP2_DSS
config OMAP2_VRAM_SIZE
int "VRAM size (MB)"
range 0 32
- default 0
+ default 4
help
The amount of SDRAM to reserve at boot time for video RAM use.
This VRAM will be used by omapfb and other drivers that need
@@ -102,7 +103,8 @@ config OMAP2_DSS_FAKE_VSYNC
config OMAP2_DSS_MIN_FCK_PER_PCK
int "Minimum FCK/PCK ratio (for scaling)"
range 0 32
- default 0
+ default 4 if ARCH_OMAP2 || ARCH_OMAP3
+ default 0 if ARCH_OMAP4
help
This can be used to adjust the minimum FCK/PCK ratio.
diff --git a/drivers/video/omap2/omapfb/Kconfig b/drivers/video/omap2/omapfb/Kconfig
index 65149b2..923bf48 100644
--- a/drivers/video/omap2/omapfb/Kconfig
+++ b/drivers/video/omap2/omapfb/Kconfig
@@ -1,6 +1,7 @@
menuconfig FB_OMAP2
tristate "OMAP2/3 frame buffer support (EXPERIMENTAL)"
depends on FB && OMAP2_DSS
+ default y
select OMAP2_VRAM
select OMAP2_VRFB if ARCH_OMAP2 || ARCH_OMAP3
--
1.5.6.3
^ permalink raw reply related [flat|nested] 22+ messages in thread* RE: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3
2010-10-26 3:29 ` Samreen
@ 2010-11-16 6:21 ` Nilofer, Samreen
-1 siblings, 0 replies; 22+ messages in thread
From: Nilofer, Samreen @ 2010-11-16 6:09 UTC (permalink / raw)
To: Tomi Valkeinen, Grazvydas Ignotas, Hiremath, Vaibhav,
Roger Quadros, Guruswamy, Senthilv
Cc: linux-omap, linux-fbdev
Hi,
Any more comments on this patch?
Warm Regards,
Samreen
Nilofer, Samreen wrote:
> The defconfig options for display are taken in the respective
> Kconfig to enable display by default on OMAP3 platforms
>
> Signed-off-by: Samreen <samreen@ti.com>
> ---
> Version4:
> Remove the enabling of the display panels by default.
>
> Version3:
> Eliminate the separate default number of FBs for
> different architecture. Keeping default FBs as 3 as before.
>
> Version2:
> Enables by default NEC panel used in zoom2/3/3630sdp,
> instead of Sharp LQ043T1DG01 panel enabled in previous
> version of this patch.
>
> drivers/video/omap2/dss/Kconfig | 6 ++++--
> drivers/video/omap2/omapfb/Kconfig | 1 +
> 2 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/video/omap2/dss/Kconfig
> b/drivers/video/omap2/dss/Kconfig index 43b6440..f3244a2 100644
> --- a/drivers/video/omap2/dss/Kconfig
> +++ b/drivers/video/omap2/dss/Kconfig
> @@ -1,6 +1,7 @@
> menuconfig OMAP2_DSS
> tristate "OMAP2/3 Display Subsystem support (EXPERIMENTAL)"
> depends on ARCH_OMAP2 || ARCH_OMAP3
> + default y
> help
> OMAP2/3 Display Subsystem support.
>
> @@ -9,7 +10,7 @@ if OMAP2_DSS
> config OMAP2_VRAM_SIZE
> int "VRAM size (MB)"
> range 0 32
> - default 0
> + default 4
> help
> The amount of SDRAM to reserve at boot time for video RAM use.
> This VRAM will be used by omapfb and other drivers
> that need @@ -102,7 +103,8 @@ config OMAP2_DSS_FAKE_VSYNC
> config OMAP2_DSS_MIN_FCK_PER_PCK
> int "Minimum FCK/PCK ratio (for scaling)"
> range 0 32
> - default 0
> + default 4 if ARCH_OMAP2 || ARCH_OMAP3
> + default 0 if ARCH_OMAP4
> help
> This can be used to adjust the minimum FCK/PCK ratio.
>
> diff --git a/drivers/video/omap2/omapfb/Kconfig
> b/drivers/video/omap2/omapfb/Kconfig
> index 65149b2..923bf48 100644
> --- a/drivers/video/omap2/omapfb/Kconfig
> +++ b/drivers/video/omap2/omapfb/Kconfig
> @@ -1,6 +1,7 @@
> menuconfig FB_OMAP2
> tristate "OMAP2/3 frame buffer support (EXPERIMENTAL)"
> depends on FB && OMAP2_DSS
> + default y
>
> select OMAP2_VRAM
> select OMAP2_VRFB if ARCH_OMAP2 || ARCH_OMAP3
^ permalink raw reply [flat|nested] 22+ messages in thread
* RE: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display
@ 2010-11-16 6:21 ` Nilofer, Samreen
0 siblings, 0 replies; 22+ messages in thread
From: Nilofer, Samreen @ 2010-11-16 6:21 UTC (permalink / raw)
To: Tomi Valkeinen, Grazvydas Ignotas, Hiremath, Vaibhav,
Roger Quadros, Guruswamy, Senthilv
Cc: linux-omap, linux-fbdev
Hi,
Any more comments on this patch?
Warm Regards,
Samreen
Nilofer, Samreen wrote:
> The defconfig options for display are taken in the respective
> Kconfig to enable display by default on OMAP3 platforms
>
> Signed-off-by: Samreen <samreen@ti.com>
> ---
> Version4:
> Remove the enabling of the display panels by default.
>
> Version3:
> Eliminate the separate default number of FBs for
> different architecture. Keeping default FBs as 3 as before.
>
> Version2:
> Enables by default NEC panel used in zoom2/3/3630sdp,
> instead of Sharp LQ043T1DG01 panel enabled in previous
> version of this patch.
>
> drivers/video/omap2/dss/Kconfig | 6 ++++--
> drivers/video/omap2/omapfb/Kconfig | 1 +
> 2 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/video/omap2/dss/Kconfig
> b/drivers/video/omap2/dss/Kconfig index 43b6440..f3244a2 100644
> --- a/drivers/video/omap2/dss/Kconfig
> +++ b/drivers/video/omap2/dss/Kconfig
> @@ -1,6 +1,7 @@
> menuconfig OMAP2_DSS
> tristate "OMAP2/3 Display Subsystem support (EXPERIMENTAL)"
> depends on ARCH_OMAP2 || ARCH_OMAP3
> + default y
> help
> OMAP2/3 Display Subsystem support.
>
> @@ -9,7 +10,7 @@ if OMAP2_DSS
> config OMAP2_VRAM_SIZE
> int "VRAM size (MB)"
> range 0 32
> - default 0
> + default 4
> help
> The amount of SDRAM to reserve at boot time for video RAM use.
> This VRAM will be used by omapfb and other drivers
> that need @@ -102,7 +103,8 @@ config OMAP2_DSS_FAKE_VSYNC
> config OMAP2_DSS_MIN_FCK_PER_PCK
> int "Minimum FCK/PCK ratio (for scaling)"
> range 0 32
> - default 0
> + default 4 if ARCH_OMAP2 || ARCH_OMAP3
> + default 0 if ARCH_OMAP4
> help
> This can be used to adjust the minimum FCK/PCK ratio.
>
> diff --git a/drivers/video/omap2/omapfb/Kconfig
> b/drivers/video/omap2/omapfb/Kconfig
> index 65149b2..923bf48 100644
> --- a/drivers/video/omap2/omapfb/Kconfig
> +++ b/drivers/video/omap2/omapfb/Kconfig
> @@ -1,6 +1,7 @@
> menuconfig FB_OMAP2
> tristate "OMAP2/3 frame buffer support (EXPERIMENTAL)"
> depends on FB && OMAP2_DSS
> + default y
>
> select OMAP2_VRAM
> select OMAP2_VRFB if ARCH_OMAP2 || ARCH_OMAP3
^ permalink raw reply [flat|nested] 22+ messages in thread
* RE: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display
2010-11-16 6:21 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display Nilofer, Samreen
@ 2010-11-16 13:50 ` Tomi Valkeinen
-1 siblings, 0 replies; 22+ messages in thread
From: Tomi Valkeinen @ 2010-11-16 13:50 UTC (permalink / raw)
To: ext Nilofer, Samreen, ext Tony Lindgren
Cc: Grazvydas Ignotas, Hiremath, Vaibhav,
Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
On Tue, 2010-11-16 at 07:09 +0100, ext Nilofer, Samreen wrote:
> Hi,
> Any more comments on this patch?
I'm not sure if enabling kernel options by default is a good thing.
Somehow I remember that enabling things by default is not looked kindly
at. Shouldn't they be enabled in the arch/arm/configs config file, if
they are wanted?
Tony, do you have opinnion on this? Should DSS features be enabled by
default in the Kconfig files, or only in the board config file?
Also, see one comment inline.
> Warm Regards,
> Samreen
> Nilofer, Samreen wrote:
> > The defconfig options for display are taken in the respective
> > Kconfig to enable display by default on OMAP3 platforms
> >
> > Signed-off-by: Samreen <samreen@ti.com>
> > ---
> > Version4:
> > Remove the enabling of the display panels by default.
> >
> > Version3:
> > Eliminate the separate default number of FBs for
> > different architecture. Keeping default FBs as 3 as before.
> >
> > Version2:
> > Enables by default NEC panel used in zoom2/3/3630sdp,
> > instead of Sharp LQ043T1DG01 panel enabled in previous
> > version of this patch.
> >
> > drivers/video/omap2/dss/Kconfig | 6 ++++--
> > drivers/video/omap2/omapfb/Kconfig | 1 +
> > 2 files changed, 5 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/video/omap2/dss/Kconfig
> > b/drivers/video/omap2/dss/Kconfig index 43b6440..f3244a2 100644
> > --- a/drivers/video/omap2/dss/Kconfig
> > +++ b/drivers/video/omap2/dss/Kconfig
> > @@ -1,6 +1,7 @@
> > menuconfig OMAP2_DSS
> > tristate "OMAP2/3 Display Subsystem support (EXPERIMENTAL)"
> > depends on ARCH_OMAP2 || ARCH_OMAP3
> > + default y
> > help
> > OMAP2/3 Display Subsystem support.
> >
> > @@ -9,7 +10,7 @@ if OMAP2_DSS
> > config OMAP2_VRAM_SIZE
> > int "VRAM size (MB)"
> > range 0 32
> > - default 0
> > + default 4
> > help
> > The amount of SDRAM to reserve at boot time for video RAM use.
> > This VRAM will be used by omapfb and other drivers
> > that need @@ -102,7 +103,8 @@ config OMAP2_DSS_FAKE_VSYNC
> > config OMAP2_DSS_MIN_FCK_PER_PCK
> > int "Minimum FCK/PCK ratio (for scaling)"
> > range 0 32
> > - default 0
> > + default 4 if ARCH_OMAP2 || ARCH_OMAP3
> > + default 0 if ARCH_OMAP4
I think having default ratio of 4 would mean that for example high res
modes wouldn't work on beagle. They require high pixel clock, which
requires 1/2 pck/lck ratio.
> > help
> > This can be used to adjust the minimum FCK/PCK ratio.
> >
> > diff --git a/drivers/video/omap2/omapfb/Kconfig
> > b/drivers/video/omap2/omapfb/Kconfig
> > index 65149b2..923bf48 100644
> > --- a/drivers/video/omap2/omapfb/Kconfig
> > +++ b/drivers/video/omap2/omapfb/Kconfig
> > @@ -1,6 +1,7 @@
> > menuconfig FB_OMAP2
> > tristate "OMAP2/3 frame buffer support (EXPERIMENTAL)"
> > depends on FB && OMAP2_DSS
> > + default y
> >
> > select OMAP2_VRAM
> > select OMAP2_VRFB if ARCH_OMAP2 || ARCH_OMAP3
^ permalink raw reply [flat|nested] 22+ messages in thread
* RE: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3
@ 2010-11-16 13:50 ` Tomi Valkeinen
0 siblings, 0 replies; 22+ messages in thread
From: Tomi Valkeinen @ 2010-11-16 13:50 UTC (permalink / raw)
To: ext Nilofer, Samreen, ext Tony Lindgren
Cc: Grazvydas Ignotas, Hiremath, Vaibhav,
Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
On Tue, 2010-11-16 at 07:09 +0100, ext Nilofer, Samreen wrote:
> Hi,
> Any more comments on this patch?
I'm not sure if enabling kernel options by default is a good thing.
Somehow I remember that enabling things by default is not looked kindly
at. Shouldn't they be enabled in the arch/arm/configs config file, if
they are wanted?
Tony, do you have opinnion on this? Should DSS features be enabled by
default in the Kconfig files, or only in the board config file?
Also, see one comment inline.
> Warm Regards,
> Samreen
> Nilofer, Samreen wrote:
> > The defconfig options for display are taken in the respective
> > Kconfig to enable display by default on OMAP3 platforms
> >
> > Signed-off-by: Samreen <samreen@ti.com>
> > ---
> > Version4:
> > Remove the enabling of the display panels by default.
> >
> > Version3:
> > Eliminate the separate default number of FBs for
> > different architecture. Keeping default FBs as 3 as before.
> >
> > Version2:
> > Enables by default NEC panel used in zoom2/3/3630sdp,
> > instead of Sharp LQ043T1DG01 panel enabled in previous
> > version of this patch.
> >
> > drivers/video/omap2/dss/Kconfig | 6 ++++--
> > drivers/video/omap2/omapfb/Kconfig | 1 +
> > 2 files changed, 5 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/video/omap2/dss/Kconfig
> > b/drivers/video/omap2/dss/Kconfig index 43b6440..f3244a2 100644
> > --- a/drivers/video/omap2/dss/Kconfig
> > +++ b/drivers/video/omap2/dss/Kconfig
> > @@ -1,6 +1,7 @@
> > menuconfig OMAP2_DSS
> > tristate "OMAP2/3 Display Subsystem support (EXPERIMENTAL)"
> > depends on ARCH_OMAP2 || ARCH_OMAP3
> > + default y
> > help
> > OMAP2/3 Display Subsystem support.
> >
> > @@ -9,7 +10,7 @@ if OMAP2_DSS
> > config OMAP2_VRAM_SIZE
> > int "VRAM size (MB)"
> > range 0 32
> > - default 0
> > + default 4
> > help
> > The amount of SDRAM to reserve at boot time for video RAM use.
> > This VRAM will be used by omapfb and other drivers
> > that need @@ -102,7 +103,8 @@ config OMAP2_DSS_FAKE_VSYNC
> > config OMAP2_DSS_MIN_FCK_PER_PCK
> > int "Minimum FCK/PCK ratio (for scaling)"
> > range 0 32
> > - default 0
> > + default 4 if ARCH_OMAP2 || ARCH_OMAP3
> > + default 0 if ARCH_OMAP4
I think having default ratio of 4 would mean that for example high res
modes wouldn't work on beagle. They require high pixel clock, which
requires 1/2 pck/lck ratio.
> > help
> > This can be used to adjust the minimum FCK/PCK ratio.
> >
> > diff --git a/drivers/video/omap2/omapfb/Kconfig
> > b/drivers/video/omap2/omapfb/Kconfig
> > index 65149b2..923bf48 100644
> > --- a/drivers/video/omap2/omapfb/Kconfig
> > +++ b/drivers/video/omap2/omapfb/Kconfig
> > @@ -1,6 +1,7 @@
> > menuconfig FB_OMAP2
> > tristate "OMAP2/3 frame buffer support (EXPERIMENTAL)"
> > depends on FB && OMAP2_DSS
> > + default y
> >
> > select OMAP2_VRAM
> > select OMAP2_VRFB if ARCH_OMAP2 || ARCH_OMAP3
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display
2010-11-16 13:50 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3 Tomi Valkeinen
@ 2010-11-16 19:38 ` Tony Lindgren
-1 siblings, 0 replies; 22+ messages in thread
From: Tony Lindgren @ 2010-11-16 19:38 UTC (permalink / raw)
To: Tomi Valkeinen
Cc: ext Nilofer, Samreen, Grazvydas Ignotas, Hiremath, Vaibhav,
Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
* Tomi Valkeinen <tomi.valkeinen@nokia.com> [101116 05:41]:
> On Tue, 2010-11-16 at 07:09 +0100, ext Nilofer, Samreen wrote:
> > Hi,
> > Any more comments on this patch?
>
> I'm not sure if enabling kernel options by default is a good thing.
> Somehow I remember that enabling things by default is not looked kindly
> at. Shouldn't they be enabled in the arch/arm/configs config file, if
> they are wanted?
>
> Tony, do you have opinnion on this? Should DSS features be enabled by
> default in the Kconfig files, or only in the board config file?
The best thing would be to have DSS y if ARCH_OMAP2PLUS or similar,
then have all the features built in and pass the configuration options
in platform_data. That still allows disabling certain features with
Kconfig if needed for memory consumption etc. But the omap2plus_defconfig
should "just work" on all the boards.
Tony
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3
@ 2010-11-16 19:38 ` Tony Lindgren
0 siblings, 0 replies; 22+ messages in thread
From: Tony Lindgren @ 2010-11-16 19:38 UTC (permalink / raw)
To: Tomi Valkeinen
Cc: ext Nilofer, Samreen, Grazvydas Ignotas, Hiremath, Vaibhav,
Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
* Tomi Valkeinen <tomi.valkeinen@nokia.com> [101116 05:41]:
> On Tue, 2010-11-16 at 07:09 +0100, ext Nilofer, Samreen wrote:
> > Hi,
> > Any more comments on this patch?
>
> I'm not sure if enabling kernel options by default is a good thing.
> Somehow I remember that enabling things by default is not looked kindly
> at. Shouldn't they be enabled in the arch/arm/configs config file, if
> they are wanted?
>
> Tony, do you have opinnion on this? Should DSS features be enabled by
> default in the Kconfig files, or only in the board config file?
The best thing would be to have DSS y if ARCH_OMAP2PLUS or similar,
then have all the features built in and pass the configuration options
in platform_data. That still allows disabling certain features with
Kconfig if needed for memory consumption etc. But the omap2plus_defconfig
should "just work" on all the boards.
Tony
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3
2010-11-16 19:38 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3 Tony Lindgren
@ 2010-11-16 19:55 ` Kevin Hilman
-1 siblings, 0 replies; 22+ messages in thread
From: Kevin Hilman @ 2010-11-16 19:55 UTC (permalink / raw)
To: Tony Lindgren
Cc: Tomi Valkeinen, ext Nilofer, Samreen, Grazvydas Ignotas,
Hiremath, Vaibhav, Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
Tony Lindgren <tony@atomide.com> writes:
> * Tomi Valkeinen <tomi.valkeinen@nokia.com> [101116 05:41]:
>> On Tue, 2010-11-16 at 07:09 +0100, ext Nilofer, Samreen wrote:
>> > Hi,
>> > Any more comments on this patch?
>>
>> I'm not sure if enabling kernel options by default is a good thing.
>> Somehow I remember that enabling things by default is not looked kindly
>> at. Shouldn't they be enabled in the arch/arm/configs config file, if
>> they are wanted?
>>
>> Tony, do you have opinnion on this? Should DSS features be enabled by
>> default in the Kconfig files, or only in the board config file?
>
> The best thing would be to have DSS y if ARCH_OMAP2PLUS or similar,
> then have all the features built in and pass the configuration options
> in platform_data. That still allows disabling certain features with
> Kconfig if needed for memory consumption etc. But the omap2plus_defconfig
> should "just work" on all the boards.
Personally, I'd much rather see DSS core and optional features default
to being built as a module.
Other than a splash screen, there's no reason DSS is needed for a
minimal boot.
Kevin
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3
@ 2010-11-16 19:55 ` Kevin Hilman
0 siblings, 0 replies; 22+ messages in thread
From: Kevin Hilman @ 2010-11-16 19:55 UTC (permalink / raw)
To: Tony Lindgren
Cc: Tomi Valkeinen, ext Nilofer, Samreen, Grazvydas Ignotas,
Hiremath, Vaibhav, Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
Tony Lindgren <tony@atomide.com> writes:
> * Tomi Valkeinen <tomi.valkeinen@nokia.com> [101116 05:41]:
>> On Tue, 2010-11-16 at 07:09 +0100, ext Nilofer, Samreen wrote:
>> > Hi,
>> > Any more comments on this patch?
>>
>> I'm not sure if enabling kernel options by default is a good thing.
>> Somehow I remember that enabling things by default is not looked kindly
>> at. Shouldn't they be enabled in the arch/arm/configs config file, if
>> they are wanted?
>>
>> Tony, do you have opinnion on this? Should DSS features be enabled by
>> default in the Kconfig files, or only in the board config file?
>
> The best thing would be to have DSS y if ARCH_OMAP2PLUS or similar,
> then have all the features built in and pass the configuration options
> in platform_data. That still allows disabling certain features with
> Kconfig if needed for memory consumption etc. But the omap2plus_defconfig
> should "just work" on all the boards.
Personally, I'd much rather see DSS core and optional features default
to being built as a module.
Other than a splash screen, there's no reason DSS is needed for a
minimal boot.
Kevin
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display
2010-11-16 19:55 ` Kevin Hilman
@ 2010-11-16 20:10 ` Tony Lindgren
-1 siblings, 0 replies; 22+ messages in thread
From: Tony Lindgren @ 2010-11-16 20:10 UTC (permalink / raw)
To: Kevin Hilman
Cc: Tomi Valkeinen, ext Nilofer, Samreen, Grazvydas Ignotas,
Hiremath, Vaibhav, Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
* Kevin Hilman <khilman@deeprootsystems.com> [101116 11:45]:
> Tony Lindgren <tony@atomide.com> writes:
>
> > * Tomi Valkeinen <tomi.valkeinen@nokia.com> [101116 05:41]:
> >> On Tue, 2010-11-16 at 07:09 +0100, ext Nilofer, Samreen wrote:
> >> > Hi,
> >> > Any more comments on this patch?
> >>
> >> I'm not sure if enabling kernel options by default is a good thing.
> >> Somehow I remember that enabling things by default is not looked kindly
> >> at. Shouldn't they be enabled in the arch/arm/configs config file, if
> >> they are wanted?
> >>
> >> Tony, do you have opinnion on this? Should DSS features be enabled by
> >> default in the Kconfig files, or only in the board config file?
> >
> > The best thing would be to have DSS y if ARCH_OMAP2PLUS or similar,
> > then have all the features built in and pass the configuration options
> > in platform_data. That still allows disabling certain features with
> > Kconfig if needed for memory consumption etc. But the omap2plus_defconfig
> > should "just work" on all the boards.
>
> Personally, I'd much rather see DSS core and optional features default
> to being built as a module.
>
> Other than a splash screen, there's no reason DSS is needed for a
> minimal boot.
Sure a module would be even better. My point is that the selection of
all the features should be enabled by default and the board options come
from platform_data.
Tony
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3
@ 2010-11-16 20:10 ` Tony Lindgren
0 siblings, 0 replies; 22+ messages in thread
From: Tony Lindgren @ 2010-11-16 20:10 UTC (permalink / raw)
To: Kevin Hilman
Cc: Tomi Valkeinen, ext Nilofer, Samreen, Grazvydas Ignotas,
Hiremath, Vaibhav, Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
* Kevin Hilman <khilman@deeprootsystems.com> [101116 11:45]:
> Tony Lindgren <tony@atomide.com> writes:
>
> > * Tomi Valkeinen <tomi.valkeinen@nokia.com> [101116 05:41]:
> >> On Tue, 2010-11-16 at 07:09 +0100, ext Nilofer, Samreen wrote:
> >> > Hi,
> >> > Any more comments on this patch?
> >>
> >> I'm not sure if enabling kernel options by default is a good thing.
> >> Somehow I remember that enabling things by default is not looked kindly
> >> at. Shouldn't they be enabled in the arch/arm/configs config file, if
> >> they are wanted?
> >>
> >> Tony, do you have opinnion on this? Should DSS features be enabled by
> >> default in the Kconfig files, or only in the board config file?
> >
> > The best thing would be to have DSS y if ARCH_OMAP2PLUS or similar,
> > then have all the features built in and pass the configuration options
> > in platform_data. That still allows disabling certain features with
> > Kconfig if needed for memory consumption etc. But the omap2plus_defconfig
> > should "just work" on all the boards.
>
> Personally, I'd much rather see DSS core and optional features default
> to being built as a module.
>
> Other than a splash screen, there's no reason DSS is needed for a
> minimal boot.
Sure a module would be even better. My point is that the selection of
all the features should be enabled by default and the board options come
from platform_data.
Tony
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display
2010-11-16 20:10 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3 Tony Lindgren
@ 2010-11-17 12:28 ` Tomi Valkeinen
-1 siblings, 0 replies; 22+ messages in thread
From: Tomi Valkeinen @ 2010-11-17 12:28 UTC (permalink / raw)
To: ext Tony Lindgren
Cc: Kevin Hilman, ext Nilofer, Samreen, Grazvydas Ignotas, Hiremath,
Vaibhav, Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
On Tue, 2010-11-16 at 21:10 +0100, ext Tony Lindgren wrote:
> * Kevin Hilman <khilman@deeprootsystems.com> [101116 11:45]:
> > Tony Lindgren <tony@atomide.com> writes:
> >
> > > * Tomi Valkeinen <tomi.valkeinen@nokia.com> [101116 05:41]:
> > >> On Tue, 2010-11-16 at 07:09 +0100, ext Nilofer, Samreen wrote:
> > >> > Hi,
> > >> > Any more comments on this patch?
> > >>
> > >> I'm not sure if enabling kernel options by default is a good thing.
> > >> Somehow I remember that enabling things by default is not looked kindly
> > >> at. Shouldn't they be enabled in the arch/arm/configs config file, if
> > >> they are wanted?
> > >>
> > >> Tony, do you have opinnion on this? Should DSS features be enabled by
> > >> default in the Kconfig files, or only in the board config file?
> > >
> > > The best thing would be to have DSS y if ARCH_OMAP2PLUS or similar,
> > > then have all the features built in and pass the configuration options
> > > in platform_data. That still allows disabling certain features with
> > > Kconfig if needed for memory consumption etc. But the omap2plus_defconfig
> > > should "just work" on all the boards.
> >
> > Personally, I'd much rather see DSS core and optional features default
> > to being built as a module.
> >
> > Other than a splash screen, there's no reason DSS is needed for a
> > minimal boot.
>
> Sure a module would be even better. My point is that the selection of
> all the features should be enabled by default and the board options come
> from platform_data.
Ok, let's build DSS & all panel drivers as modules by default.
Somehow I've gotten the impression from linux ml that enabling features
by default is bad. But perhaps it's more about intervening features than
normal drivers.
Samreen, will you change the patch to set DSS & panel drivers to be
built as modules by default? Optimally all display interfaces should be
enabled, but I believe at least SDI and DSI are currently slightly
broken, because they prevent DSS from initializing if SDI/DSI regulator
is not defined in the board file. So perhaps we should start with only
DPI and VENC enabled (plus all the DPI panels).
Also, this patch could be based on top of Bryan Wu's new generic panel
patch set, which I'll perhaps apply today to my tree if I find no more
to comment about.
Tomi
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3
@ 2010-11-17 12:28 ` Tomi Valkeinen
0 siblings, 0 replies; 22+ messages in thread
From: Tomi Valkeinen @ 2010-11-17 12:28 UTC (permalink / raw)
To: ext Tony Lindgren
Cc: Kevin Hilman, ext Nilofer, Samreen, Grazvydas Ignotas, Hiremath,
Vaibhav, Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
On Tue, 2010-11-16 at 21:10 +0100, ext Tony Lindgren wrote:
> * Kevin Hilman <khilman@deeprootsystems.com> [101116 11:45]:
> > Tony Lindgren <tony@atomide.com> writes:
> >
> > > * Tomi Valkeinen <tomi.valkeinen@nokia.com> [101116 05:41]:
> > >> On Tue, 2010-11-16 at 07:09 +0100, ext Nilofer, Samreen wrote:
> > >> > Hi,
> > >> > Any more comments on this patch?
> > >>
> > >> I'm not sure if enabling kernel options by default is a good thing.
> > >> Somehow I remember that enabling things by default is not looked kindly
> > >> at. Shouldn't they be enabled in the arch/arm/configs config file, if
> > >> they are wanted?
> > >>
> > >> Tony, do you have opinnion on this? Should DSS features be enabled by
> > >> default in the Kconfig files, or only in the board config file?
> > >
> > > The best thing would be to have DSS y if ARCH_OMAP2PLUS or similar,
> > > then have all the features built in and pass the configuration options
> > > in platform_data. That still allows disabling certain features with
> > > Kconfig if needed for memory consumption etc. But the omap2plus_defconfig
> > > should "just work" on all the boards.
> >
> > Personally, I'd much rather see DSS core and optional features default
> > to being built as a module.
> >
> > Other than a splash screen, there's no reason DSS is needed for a
> > minimal boot.
>
> Sure a module would be even better. My point is that the selection of
> all the features should be enabled by default and the board options come
> from platform_data.
Ok, let's build DSS & all panel drivers as modules by default.
Somehow I've gotten the impression from linux ml that enabling features
by default is bad. But perhaps it's more about intervening features than
normal drivers.
Samreen, will you change the patch to set DSS & panel drivers to be
built as modules by default? Optimally all display interfaces should be
enabled, but I believe at least SDI and DSI are currently slightly
broken, because they prevent DSS from initializing if SDI/DSI regulator
is not defined in the board file. So perhaps we should start with only
DPI and VENC enabled (plus all the DPI panels).
Also, this patch could be based on top of Bryan Wu's new generic panel
patch set, which I'll perhaps apply today to my tree if I find no more
to comment about.
Tomi
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3
2010-11-17 12:28 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3 Tomi Valkeinen
@ 2010-11-18 6:18 ` Paul Mundt
-1 siblings, 0 replies; 22+ messages in thread
From: Paul Mundt @ 2010-11-18 6:18 UTC (permalink / raw)
To: Tomi Valkeinen
Cc: ext Tony Lindgren, Kevin Hilman, ext Nilofer, Samreen,
Grazvydas Ignotas, Hiremath, Vaibhav,
Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
On Wed, Nov 17, 2010 at 02:28:11PM +0200, Tomi Valkeinen wrote:
> On Tue, 2010-11-16 at 21:10 +0100, ext Tony Lindgren wrote:
> > Sure a module would be even better. My point is that the selection of
> > all the features should be enabled by default and the board options come
> > from platform_data.
>
> Ok, let's build DSS & all panel drivers as modules by default.
>
> Somehow I've gotten the impression from linux ml that enabling features
> by default is bad. But perhaps it's more about intervening features than
> normal drivers.
>
The general rule is to avoid default enabling unless you really need it,
but it still remains optional (which is why it's not being selected,
instead). Some, like gpiolib, have come up with WANT/NEED options for the
platform code to select in order to work out the desired behaviour, and
you may benefit from a similar approach for your subsystem if it's really
that integral for some parts.
The flip side of course is that if you expect your users to primarily be
using the defconfigs provided, you can simply leave it default disabled
in the Kconfig and set the options you want in the defconfigs.
Unless you can say with certainty that all OMAP3 boards are going to want
DSS enabled or modular by default, it's almost always better to just
leave it up to the defconfigs.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3
@ 2010-11-18 6:18 ` Paul Mundt
0 siblings, 0 replies; 22+ messages in thread
From: Paul Mundt @ 2010-11-18 6:18 UTC (permalink / raw)
To: Tomi Valkeinen
Cc: ext Tony Lindgren, Kevin Hilman, ext Nilofer, Samreen,
Grazvydas Ignotas, Hiremath, Vaibhav,
Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
On Wed, Nov 17, 2010 at 02:28:11PM +0200, Tomi Valkeinen wrote:
> On Tue, 2010-11-16 at 21:10 +0100, ext Tony Lindgren wrote:
> > Sure a module would be even better. My point is that the selection of
> > all the features should be enabled by default and the board options come
> > from platform_data.
>
> Ok, let's build DSS & all panel drivers as modules by default.
>
> Somehow I've gotten the impression from linux ml that enabling features
> by default is bad. But perhaps it's more about intervening features than
> normal drivers.
>
The general rule is to avoid default enabling unless you really need it,
but it still remains optional (which is why it's not being selected,
instead). Some, like gpiolib, have come up with WANT/NEED options for the
platform code to select in order to work out the desired behaviour, and
you may benefit from a similar approach for your subsystem if it's really
that integral for some parts.
The flip side of course is that if you expect your users to primarily be
using the defconfigs provided, you can simply leave it default disabled
in the Kconfig and set the options you want in the defconfigs.
Unless you can say with certainty that all OMAP3 boards are going to want
DSS enabled or modular by default, it's almost always better to just
leave it up to the defconfigs.
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display
2010-11-18 6:18 ` Paul Mundt
@ 2010-11-18 16:44 ` Tony Lindgren
-1 siblings, 0 replies; 22+ messages in thread
From: Tony Lindgren @ 2010-11-18 16:44 UTC (permalink / raw)
To: Paul Mundt
Cc: Tomi Valkeinen, Kevin Hilman, ext Nilofer, Samreen,
Grazvydas Ignotas, Hiremath, Vaibhav,
Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
* Paul Mundt <lethal@linux-sh.org> [101117 22:09]:
> On Wed, Nov 17, 2010 at 02:28:11PM +0200, Tomi Valkeinen wrote:
> > On Tue, 2010-11-16 at 21:10 +0100, ext Tony Lindgren wrote:
> > > Sure a module would be even better. My point is that the selection of
> > > all the features should be enabled by default and the board options come
> > > from platform_data.
> >
> > Ok, let's build DSS & all panel drivers as modules by default.
> >
> > Somehow I've gotten the impression from linux ml that enabling features
> > by default is bad. But perhaps it's more about intervening features than
> > normal drivers.
> >
> The general rule is to avoid default enabling unless you really need it,
> but it still remains optional (which is why it's not being selected,
> instead). Some, like gpiolib, have come up with WANT/NEED options for the
> platform code to select in order to work out the desired behaviour, and
> you may benefit from a similar approach for your subsystem if it's really
> that integral for some parts.
>
> The flip side of course is that if you expect your users to primarily be
> using the defconfigs provided, you can simply leave it default disabled
> in the Kconfig and set the options you want in the defconfigs.
>
> Unless you can say with certainty that all OMAP3 boards are going to want
> DSS enabled or modular by default, it's almost always better to just
> leave it up to the defconfigs.
I wish we could just do "default m if ARCH_OMAP2PLUS_TYPICAL"..
But meanwhile setting it as a module in omap2plus_defconfig does
the trick though like you say.
Regards,
Tony
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3
@ 2010-11-18 16:44 ` Tony Lindgren
0 siblings, 0 replies; 22+ messages in thread
From: Tony Lindgren @ 2010-11-18 16:44 UTC (permalink / raw)
To: Paul Mundt
Cc: Tomi Valkeinen, Kevin Hilman, ext Nilofer, Samreen,
Grazvydas Ignotas, Hiremath, Vaibhav,
Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
* Paul Mundt <lethal@linux-sh.org> [101117 22:09]:
> On Wed, Nov 17, 2010 at 02:28:11PM +0200, Tomi Valkeinen wrote:
> > On Tue, 2010-11-16 at 21:10 +0100, ext Tony Lindgren wrote:
> > > Sure a module would be even better. My point is that the selection of
> > > all the features should be enabled by default and the board options come
> > > from platform_data.
> >
> > Ok, let's build DSS & all panel drivers as modules by default.
> >
> > Somehow I've gotten the impression from linux ml that enabling features
> > by default is bad. But perhaps it's more about intervening features than
> > normal drivers.
> >
> The general rule is to avoid default enabling unless you really need it,
> but it still remains optional (which is why it's not being selected,
> instead). Some, like gpiolib, have come up with WANT/NEED options for the
> platform code to select in order to work out the desired behaviour, and
> you may benefit from a similar approach for your subsystem if it's really
> that integral for some parts.
>
> The flip side of course is that if you expect your users to primarily be
> using the defconfigs provided, you can simply leave it default disabled
> in the Kconfig and set the options you want in the defconfigs.
>
> Unless you can say with certainty that all OMAP3 boards are going to want
> DSS enabled or modular by default, it's almost always better to just
> leave it up to the defconfigs.
I wish we could just do "default m if ARCH_OMAP2PLUS_TYPICAL"..
But meanwhile setting it as a module in omap2plus_defconfig does
the trick though like you say.
Regards,
Tony
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3
2010-11-18 16:44 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3 Tony Lindgren
@ 2010-11-18 18:38 ` Paul Mundt
-1 siblings, 0 replies; 22+ messages in thread
From: Paul Mundt @ 2010-11-18 18:38 UTC (permalink / raw)
To: Tony Lindgren
Cc: Tomi Valkeinen, Kevin Hilman, ext Nilofer, Samreen,
Grazvydas Ignotas, Hiremath, Vaibhav,
Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
On Thu, Nov 18, 2010 at 08:44:14AM -0800, Tony Lindgren wrote:
> * Paul Mundt <lethal@linux-sh.org> [101117 22:09]:
> > Unless you can say with certainty that all OMAP3 boards are going to want
> > DSS enabled or modular by default, it's almost always better to just
> > leave it up to the defconfigs.
>
> I wish we could just do "default m if ARCH_OMAP2PLUS_TYPICAL"..
> But meanwhile setting it as a module in omap2plus_defconfig does
> the trick though like you say.
>
Something like this should be quite doable if the Kconfig fragments work
is ever completed. If this is something that interests you you may wish
to give it a bit of a nudge :-)
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3
@ 2010-11-18 18:38 ` Paul Mundt
0 siblings, 0 replies; 22+ messages in thread
From: Paul Mundt @ 2010-11-18 18:38 UTC (permalink / raw)
To: Tony Lindgren
Cc: Tomi Valkeinen, Kevin Hilman, ext Nilofer, Samreen,
Grazvydas Ignotas, Hiremath, Vaibhav,
Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
On Thu, Nov 18, 2010 at 08:44:14AM -0800, Tony Lindgren wrote:
> * Paul Mundt <lethal@linux-sh.org> [101117 22:09]:
> > Unless you can say with certainty that all OMAP3 boards are going to want
> > DSS enabled or modular by default, it's almost always better to just
> > leave it up to the defconfigs.
>
> I wish we could just do "default m if ARCH_OMAP2PLUS_TYPICAL"..
> But meanwhile setting it as a module in omap2plus_defconfig does
> the trick though like you say.
>
Something like this should be quite doable if the Kconfig fragments work
is ever completed. If this is something that interests you you may wish
to give it a bit of a nudge :-)
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display
2010-11-18 18:38 ` Paul Mundt
@ 2010-11-18 19:10 ` Tony Lindgren
-1 siblings, 0 replies; 22+ messages in thread
From: Tony Lindgren @ 2010-11-18 19:10 UTC (permalink / raw)
To: Paul Mundt
Cc: Tomi Valkeinen, Kevin Hilman, ext Nilofer, Samreen,
Grazvydas Ignotas, Hiremath, Vaibhav,
Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
* Paul Mundt <lethal@linux-sh.org> [101118 10:29]:
> On Thu, Nov 18, 2010 at 08:44:14AM -0800, Tony Lindgren wrote:
> > * Paul Mundt <lethal@linux-sh.org> [101117 22:09]:
> > > Unless you can say with certainty that all OMAP3 boards are going to want
> > > DSS enabled or modular by default, it's almost always better to just
> > > leave it up to the defconfigs.
> >
> > I wish we could just do "default m if ARCH_OMAP2PLUS_TYPICAL"..
> > But meanwhile setting it as a module in omap2plus_defconfig does
> > the trick though like you say.
> >
> Something like this should be quite doable if the Kconfig fragments work
> is ever completed. If this is something that interests you you may wish
> to give it a bit of a nudge :-)
Yeah maybe eventually at some point.. Got several merge cycles of
other things to fix.. And the defconfig can be used for now :)
Tony
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3
@ 2010-11-18 19:10 ` Tony Lindgren
0 siblings, 0 replies; 22+ messages in thread
From: Tony Lindgren @ 2010-11-18 19:10 UTC (permalink / raw)
To: Paul Mundt
Cc: Tomi Valkeinen, Kevin Hilman, ext Nilofer, Samreen,
Grazvydas Ignotas, Hiremath, Vaibhav,
Quadros Roger (Nokia-MS/Helsinki),
Guruswamy, Senthilvadivu, linux-omap, linux-fbdev
* Paul Mundt <lethal@linux-sh.org> [101118 10:29]:
> On Thu, Nov 18, 2010 at 08:44:14AM -0800, Tony Lindgren wrote:
> > * Paul Mundt <lethal@linux-sh.org> [101117 22:09]:
> > > Unless you can say with certainty that all OMAP3 boards are going to want
> > > DSS enabled or modular by default, it's almost always better to just
> > > leave it up to the defconfigs.
> >
> > I wish we could just do "default m if ARCH_OMAP2PLUS_TYPICAL"..
> > But meanwhile setting it as a module in omap2plus_defconfig does
> > the trick though like you say.
> >
> Something like this should be quite doable if the Kconfig fragments work
> is ever completed. If this is something that interests you you may wish
> to give it a bit of a nudge :-)
Yeah maybe eventually at some point.. Got several merge cycles of
other things to fix.. And the defconfig can be used for now :)
Tony
^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2010-11-18 19:10 UTC | newest]
Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-10-26 3:17 [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3 Samreen
2010-10-26 3:29 ` Samreen
2010-11-16 6:09 ` Nilofer, Samreen
2010-11-16 6:21 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display Nilofer, Samreen
2010-11-16 13:50 ` Tomi Valkeinen
2010-11-16 13:50 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3 Tomi Valkeinen
2010-11-16 19:38 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display Tony Lindgren
2010-11-16 19:38 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3 Tony Lindgren
2010-11-16 19:55 ` Kevin Hilman
2010-11-16 19:55 ` Kevin Hilman
2010-11-16 20:10 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display Tony Lindgren
2010-11-16 20:10 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3 Tony Lindgren
2010-11-17 12:28 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display Tomi Valkeinen
2010-11-17 12:28 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3 Tomi Valkeinen
2010-11-18 6:18 ` Paul Mundt
2010-11-18 6:18 ` Paul Mundt
2010-11-18 16:44 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display Tony Lindgren
2010-11-18 16:44 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3 Tony Lindgren
2010-11-18 18:38 ` Paul Mundt
2010-11-18 18:38 ` Paul Mundt
2010-11-18 19:10 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display Tony Lindgren
2010-11-18 19:10 ` [PATCH v4] OMAP3: DSS: Kconfig changes to enable display options on OMAP3 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.