Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-02-07 Thread Arnd Bergmann
On Thursday 07 February 2013, Tomi Valkeinen wrote: > > I would suggest only the first patch, since Linus quite specifically > > asked only for serious bug fixes. I think an allyesconfig build > > breakage is serious enough, but doing multiple patches for one > > bug should not be necessary and is

Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-02-07 Thread Tomi Valkeinen
On 2013-02-07 13:55, Arnd Bergmann wrote: > omapdrm still has the 'select' statement in it if you only send the > first patch, so it should not be possible to disable omapdss when > it is actually needed. Yes, you're right. After adding ARCH_MULTIPLATFORM, the kconfig enables omapdss fully even

Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-02-07 Thread Arnd Bergmann
On Thursday 07 February 2013, Tomi Valkeinen wrote: > On 2013-02-06 16:29, Arnd Bergmann wrote: > > On Wednesday 06 February 2013 16:15:57 Tomi Valkeinen wrote: > > >> I have patches to add the ARCH_MULTIPLATFORM for omapdss, and to fix the > >> omap_vout and omapdrm Kconfig files. Each of them

Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-02-07 Thread Tomi Valkeinen
On 2013-02-06 16:29, Arnd Bergmann wrote: > On Wednesday 06 February 2013 16:15:57 Tomi Valkeinen wrote: >> I have patches to add the ARCH_MULTIPLATFORM for omapdss, and to fix the >> omap_vout and omapdrm Kconfig files. Each of them changes only one line. >> Arnd, are you ok with queuing those

Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-02-07 Thread Tomi Valkeinen
On 2013-02-06 16:29, Arnd Bergmann wrote: On Wednesday 06 February 2013 16:15:57 Tomi Valkeinen wrote: I have patches to add the ARCH_MULTIPLATFORM for omapdss, and to fix the omap_vout and omapdrm Kconfig files. Each of them changes only one line. Arnd, are you ok with queuing those patches

Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-02-07 Thread Arnd Bergmann
On Thursday 07 February 2013, Tomi Valkeinen wrote: On 2013-02-06 16:29, Arnd Bergmann wrote: On Wednesday 06 February 2013 16:15:57 Tomi Valkeinen wrote: I have patches to add the ARCH_MULTIPLATFORM for omapdss, and to fix the omap_vout and omapdrm Kconfig files. Each of them changes

Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-02-07 Thread Tomi Valkeinen
On 2013-02-07 13:55, Arnd Bergmann wrote: omapdrm still has the 'select' statement in it if you only send the first patch, so it should not be possible to disable omapdss when it is actually needed. Yes, you're right. After adding ARCH_MULTIPLATFORM, the kconfig enables omapdss fully even if

Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-02-07 Thread Arnd Bergmann
On Thursday 07 February 2013, Tomi Valkeinen wrote: I would suggest only the first patch, since Linus quite specifically asked only for serious bug fixes. I think an allyesconfig build breakage is serious enough, but doing multiple patches for one bug should not be necessary and is much

Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-02-06 Thread Arnd Bergmann
On Wednesday 06 February 2013 16:15:57 Tomi Valkeinen wrote: > > That would normally be me, but I've been on a long leave, and just came > back. It'll take me some time to get back on track. > > I don't think it makes sense to add ARCH_MULTIPLATFORM only for omapdss, > like this patch does. I

Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-02-06 Thread Tomi Valkeinen
Hi Arnd, Rob, On 2013-02-05 18:31, Arnd Bergmann wrote: > On Tuesday 22 January 2013, Arnd Bergmann wrote: >> On Tuesday 22 January 2013, Rob Clark wrote: >>> At least core omapdss does not have any build dependencies on >>> ARCH_OMAP2PLUS, and adding this dependency in the Kconfig breaks omapdrm

Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-02-06 Thread Tomi Valkeinen
Hi Arnd, Rob, On 2013-02-05 18:31, Arnd Bergmann wrote: On Tuesday 22 January 2013, Arnd Bergmann wrote: On Tuesday 22 January 2013, Rob Clark wrote: At least core omapdss does not have any build dependencies on ARCH_OMAP2PLUS, and adding this dependency in the Kconfig breaks omapdrm for

Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-02-06 Thread Arnd Bergmann
On Wednesday 06 February 2013 16:15:57 Tomi Valkeinen wrote: That would normally be me, but I've been on a long leave, and just came back. It'll take me some time to get back on track. I don't think it makes sense to add ARCH_MULTIPLATFORM only for omapdss, like this patch does. I think we

Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-02-05 Thread Arnd Bergmann
On Tuesday 22 January 2013, Arnd Bergmann wrote: > On Tuesday 22 January 2013, Rob Clark wrote: > > At least core omapdss does not have any build dependencies on > > ARCH_OMAP2PLUS, and adding this dependency in the Kconfig breaks omapdrm > > for ARCH_MULTIPLATFORM builds. > > > > Signed-off-by:

Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-02-05 Thread Arnd Bergmann
On Tuesday 22 January 2013, Arnd Bergmann wrote: On Tuesday 22 January 2013, Rob Clark wrote: At least core omapdss does not have any build dependencies on ARCH_OMAP2PLUS, and adding this dependency in the Kconfig breaks omapdrm for ARCH_MULTIPLATFORM builds. Signed-off-by: Rob Clark

Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-01-22 Thread Arnd Bergmann
On Tuesday 22 January 2013, Rob Clark wrote: > At least core omapdss does not have any build dependencies on > ARCH_OMAP2PLUS, and adding this dependency in the Kconfig breaks omapdrm > for ARCH_MULTIPLATFORM builds. > > Signed-off-by: Rob Clark Acked-by: Arnd Bergmann -- To unsubscribe from

[PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-01-22 Thread Rob Clark
At least core omapdss does not have any build dependencies on ARCH_OMAP2PLUS, and adding this dependency in the Kconfig breaks omapdrm for ARCH_MULTIPLATFORM builds. Signed-off-by: Rob Clark --- drivers/video/omap2/Kconfig | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git

[PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-01-22 Thread Rob Clark
At least core omapdss does not have any build dependencies on ARCH_OMAP2PLUS, and adding this dependency in the Kconfig breaks omapdrm for ARCH_MULTIPLATFORM builds. Signed-off-by: Rob Clark robdcl...@gmail.com --- drivers/video/omap2/Kconfig | 7 ++- 1 file changed, 6 insertions(+), 1

Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

2013-01-22 Thread Arnd Bergmann
On Tuesday 22 January 2013, Rob Clark wrote: At least core omapdss does not have any build dependencies on ARCH_OMAP2PLUS, and adding this dependency in the Kconfig breaks omapdrm for ARCH_MULTIPLATFORM builds. Signed-off-by: Rob Clark robdcl...@gmail.com Acked-by: Arnd Bergmann