linux-kernel - Re: visibility changes (was: Re: [GIT] kbuild and kconfig fixes for 2.6.37-rc5)

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]

Date:	Tue, 07 Dec 2010 20:36:27 -0200
From:	Mauro Carvalho Chehab <mchehab@...hat.com>
To:	Geert Uytterhoeven <geert@...ux-m68k.org>
CC:	Arnaud Lacombe <lacombar@...il.com>, Michal Marek <mmarek@...e.cz>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	brueckner@...ux.vnet.ibm.com, linux-kbuild@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: visibility changes (was: Re: [GIT] kbuild and kconfig fixes for
 2.6.37-rc5)

Em 07-12-2010 18:45, Geert Uytterhoeven escreveu:
> On Tue, Dec 7, 2010 at 21:40, Arnaud Lacombe <lacombar@...il.com> wrote:
>> On Tue, Dec 7, 2010 at 3:31 PM, Geert Uytterhoeven <geert@...ux-m68k.org> wrote:
>>> On Tue, Dec 7, 2010 at 20:51, Arnaud Lacombe <lacombar@...il.com> wrote:
>>>> On Tue, Dec 7, 2010 at 2:07 PM, Geert Uytterhoeven <geert@...ux-m68k.org> wrote:
>>>>> On Sat, Dec 4, 2010 at 14:57, Michal Marek <mmarek@...e.cz> wrote:
>>>>>> please pull these kbuild/kconfig fixes for the next -rc. The largest
>>>>>> part are fixes for the annoying kconfig warnings, contributed by Arnaud
>>>>>> Lacombe and Mauro Carvalho Chehab. The solution adds a new 'visible if'
>>>>>> keyword to the Kconfig language, so it's not exactly something that one
>>>>>> would expect in the rc phase, but the kconfig change is not huge (the
>>>>>> large diff is due to generated files) and it is deployed only in those
>>>>>> five Kconfig files that had the warnings and were hard to fix by other
>>>>>> means.  The second part is a forgotten patch by Hendrik Brueckner to
>>>>>> usr/initramfs, that fixes build for Blackfin and h8300.
>>>>>
>>>>>> Arnaud Lacombe (5):
>>>>>>      kconfig: add an option to determine a menu's visibility
>>>>>
>>>>> Hmm, this changeset also went in through the v4l tree?
>>>>>
>>>> Mauro merely imported the set in his tree to give them visibility and
>>>> testing, but they went in trough the kbuild tree after being in -next
>>>> for a week.
>>>
>>> Hmm, git show 86e187ff9bce9fbed7bfed92ae34f491cf1af50f doesn't
>>> show a SoB from Michal...
>>>
>> Why would you expect it to ? Most of the commit in Linus' tree are not
>> personally SoB by him.
> 
> Sorry, you're right.
> 
>>>>> BTW, I got confused:
>>>>>  - make allmodconfig causes e.g.:
>>>>>        CONFIG_VIDEO_HELPER_CHIPS_AUTO=y
>>>>>        CONFIG_VIDEO_TVAUDIO=m
>>>>>    => good
>>>>>  - make oldconfig for my config with almost everything selected,
>>>>> except for stuff I know that breaks:
>>>>>        CONFIG_VIDEO_HELPER_CHIPS_AUTO=y
>>>>>        # CONFIG_VIDEO_TVAUDIO is not set
>>>>>    and no way to manually set CONFIG_VIDEO_TVAUDIO=y?
>>>>>    => bad!
>>>> that's difficult to judge without your old configuration.
>>>> VIDEO_TVAUDIO is only selected by VIDEO_BT848 which you don't give
>>>> information about.
>>>
>>> I don't have PCI, so I cannot enable VIDEO_BT848.
>>>
>> so why do you expect VIDEO_TVAUDIO to be enabled ?
> 
> I don't expect it to be enabled, I expected to be able to enable it.
> Allmodconfig
> does enable it.

Why do you want to enable a device that doesn't work? There's no other driver
but bttv that has upport for the devices implemented by tvaudio module.

/me is confused...

Cheers,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/