linux-kernel - Re: [PATCH] OMAPDSS: enable omapdss for ARCH

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <51139E63.2030302@ti.com>
Date:	Thu, 7 Feb 2013 14:30:27 +0200
From:	Tomi Valkeinen <tomi.valkeinen@...com>
To:	Arnd Bergmann <arnd@...db.de>
CC:	Rob Clark <robdclark@...il.com>,
	<linux-arm-kernel@...ts.infradead.org>, <patches@...aro.org>,
	<linux-kernel@...r.kernel.org>, <arm@...nel.org>,
	Tony Lindgren <tony@...mide.com>, Greg KH <greg@...ah.com>
Subject: Re: [PATCH] OMAPDSS: enable omapdss for ARCH_MULTIPLATFORM

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 enabled by select. So select is only a problem if
omapdss is enabled when omapdss's dependencies are missing.

> I suggested doing only the minimum that is needed to unbreak the
> allyesconfig build, which is to enable just omapdss but not omapfb
> and the displays, in case they don't actually build on anything else.

omapfb and the displays build also fine on all platforms. But it's true
that they are not needed to fix allyesconfig.

> 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 harder to justify.

Well, as I see, the bug is omapdrm using "select", not "depends on". So
if I'd have to pick one patch, I'd send 0003. That would fix allyesconfig.

Applying only 0003 means that omapdrm (and omapdss) won't be built on
ARCH_MULTIPLATFORM, but building omapdrm on ARCH_MULTIPLATFORM is a
feature added in this merge window, so leaving it out is not a regression.

But I'm not sure if I'm being overly pedantic here. Patch 0001 would fix
allyesconfig, and allow building omapdrm with ARCH_MULTIPLATFORM.
However, the fixing there happens as a side effect, and so the 0001
patch isn't even called a "fix" in its subject. That's the reason I'm a
bit reluctant with the 0001 patch as a fix.

 Tomi

Download attachment "signature.asc" of type "application/pgp-signature" (900 bytes)