[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <AANLkTik+m5YQZxsPGSsJuQYFThpqs4fa3i8RHbnHWCzR@mail.gmail.com>
Date: Sun, 5 Dec 2010 16:43:28 +0000
From: trapDoor <trapdoor6@...il.com>
To: Arnaud Lacombe <lacombar@...il.com>
Cc: Michal Marek <mmarek@...e.cz>,
Linus Torvalds <torvalds@...ux-foundation.org>,
brueckner@...ux.vnet.ibm.com, mchehab@...hat.com,
linux-kbuild@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [GIT] kbuild and kconfig fixes for 2.6.37-rc5
On Sun, Dec 5, 2010 at 4:13 PM, Arnaud Lacombe <lacombar@...il.com> wrote:
> Hi,
>
> On Sun, Dec 5, 2010 at 10:42 AM, trapDoor <trapdoor6@...il.com> wrote:
>> On Sat, Dec 4, 2010 at 1:57 PM, Michal Marek <mmarek@...e.cz> wrote:
>>> Hi Linus,
>>>
>>> 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.
>>>
>>> Michal
>>>
>>> The following changes since commit e8a7e48bb248a1196484d3f8afa53bded2b24e71:
>>>
>>> Linux 2.6.37-rc4 (2010-11-29 20:42:04 -0800)
>>>
>>> are available in the git repository at:
>>> git://git.kernel.org/pub/scm/linux/kernel/git/mmarek/kbuild-2.6.git rc-fixes
>>>
>>> Arnaud Lacombe (5):
>>> kconfig: add an option to determine a menu's visibility
>>> kconfig: regen parser
>>> Revert "i2c: Fix Kconfig dependencies"
>>> media/video: convert Kconfig to use the menu's `visible' keyword
>>> i2c/algos: convert Kconfig to use the menu's `visible' keyword
>>>
>>> Hendrik Brueckner (1):
>>> initramfs: Really fix build break on symbol-prefixed archs
>>>
>>> Mauro Carvalho Chehab (1):
>>> [media] Fix Kconfig errors due to two visible menus
>>>
>>> Michal Marek (1):
>>> Merge branch 'kconfig_fixes' of git://git.kernel.org/.../mchehab/linux-next into kbuild/rc-fixes
>>>
>>> drivers/i2c/Kconfig | 3 +-
>>> drivers/i2c/algos/Kconfig | 14 +-
>>> drivers/media/common/tuners/Kconfig | 8 +-
>>> drivers/media/dvb/frontends/Kconfig | 5 +-
>>> drivers/media/video/Kconfig | 2 +-
>>> scripts/kconfig/expr.h | 1 +
>>> scripts/kconfig/lkc.h | 1 +
>>> scripts/kconfig/menu.c | 11 +
>>> scripts/kconfig/zconf.gperf | 1 +
>>> scripts/kconfig/zconf.hash.c_shipped | 122 ++++----
>>> scripts/kconfig/zconf.tab.c_shipped | 570 +++++++++++++++++----------------
>>> scripts/kconfig/zconf.y | 21 +-
>>> usr/initramfs_data.S | 5 +-
>>> 13 files changed, 401 insertions(+), 363 deletions(-)
>>> --
>>> 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/
>>>
>>
>> Hello,
>> I couldn't wait and I merged the changes into my copy of mainline
>> tree. The merge was successful, no conflicts and the merge-diffstat
>> was exactly the same as above. However, when running oldconfig on the
>> patched kernel I still get lots of warnings:
>>
>> warning: (GPIO_RDC321X && PCI && GPIOLIB) selects MFD_RDC321X which
>> has unmet direct dependencies (MFD_SUPPORT && PCI)
>> warning: (I2C_ISCH && I2C && PCI || GPIO_SCH && GPIOLIB && PCI ||
>> GPIO_VX855 && GPIOLIB || GPIO_RDC321X && PCI && GPIOLIB ||
>> MFD_88PM860X && MFD_SUPPORT && I2C=y && GENERIC_HARDIRQS || MFD_ASIC3
>> && MFD_SUPPORT && GENERIC_HARDIRQS && GPIOLIB && ARM ||
>> MFD_SH_MOBILE_SDHI && MFD_SUPPORT && (SUPERH || ARCH_SHMOBILE) ||
>> MFD_DAVINCI_VOICECODEC && MFD_SUPPORT || HTC_PASIC3 && MFD_SUPPORT ||
>> TPS6507X && MFD_SUPPORT && I2C || TWL4030_CODEC && MFD_SUPPORT &&
>> TWL4030_CORE || MFD_STMPE && MFD_SUPPORT && I2C=y && GENERIC_HARDIRQS
>> || MFD_TC35892 && MFD_SUPPORT && I2C=y && GENERIC_HARDIRQS ||
>> MFD_T7L66XB && MFD_SUPPORT && ARM && HAVE_CLK || MFD_TC6387XB &&
>> MFD_SUPPORT && ARM && HAVE_CLK || MFD_TC6393XB && MFD_SUPPORT &&
>> GPIOLIB && ARM || MFD_MAX8925 && MFD_SUPPORT && I2C=y &&
>> GENERIC_HARDIRQS || MFD_MAX8998 && MFD_SUPPORT && I2C=y &&
>> GENERIC_HARDIRQS || MFD_WM8400 && MFD_SUPPORT && I2C || MFD_WM831X_I2C
>> && MFD_SUPPORT && I2C=y && GENERIC_HARDIRQS || MFD_WM831X_SPI &&
>> MFD_SUPPORT && SPI_MASTER && GENERIC_HARDIRQS || MFD_WM8994 &&
>> MFD_SUPPORT && I2C=y && GENERIC_HARDIRQS || MFD_MC13XXX && MFD_SUPPORT
>> && SPI_MASTER || AB3100_CORE && MFD_SUPPORT && I2C=y && ABX500_CORE ||
>> AB8500_CORE && MFD_SUPPORT && GENERIC_HARDIRQS && ABX500_CORE &&
>> SPI_MASTER && ARCH_U8500 || AB3550_CORE && MFD_SUPPORT && I2C=y &&
>> GENERIC_HARDIRQS && ABX500_CORE || MFD_TIMBERDALE && MFD_SUPPORT &&
>> PCI && GPIOLIB || LPC_SCH && MFD_SUPPORT && PCI || MFD_RDC321X &&
>> MFD_SUPPORT && PCI || MFD_JANZ_CMODIO && MFD_SUPPORT && PCI ||
>> MFD_JZ4740_ADC && MFD_SUPPORT && MACH_JZ4740 || MFD_TPS6586X &&
>> MFD_SUPPORT && I2C=y && GPIOLIB && GENERIC_HARDIRQS || MFD_VX855 &&
>> MFD_SUPPORT && PCI) selects MFD_CORE which has unmet direct
>> dependencies (MFD_SUPPORT)
>>
> Technically, this is two warning, though really verbose :)
>
True.
>>
>> Were these warnings supposed to be fixed by this patch set as well? Or
>> is it something in my config (attached)?
>>
> No. The kconfig language update is meant to be a solution for the +150
> warnings triggered by the media and i2c tree, without making it
> impossible to maintain. The notion of "dependency" was too strong for
> the expected usage.
>
> - Arnaud
>
OK. Thanks for clarifying.
--
Tomasz
--
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/
Powered by blists - more mailing lists