Messages in this thread |  | | | Date | Wed, 30 May 2012 15:37:18 +0800 | | From | Lee Jones <> | | Subject | Re: [PATCH 18/22] mfd: Remove redundant Kconfig entry |
| |
On 30/05/12 15:15, Linus Walleij wrote: > On Wed, May 30, 2012 at 12:47 PM, Lee Jones<lee.jones@linaro.org> wrote: > >> During ab8500-core clean-up the Kconfig entry for AB8500_I2C_CORE >> was left remnant. This patch simply removes it. >> >> Cc: Samuel Ortiz<sameo@linux.intel.com> >> Cc: linux-kernel@vger.kernel.org >> Signed-off-by: Lee Jones<lee.jones@linaro.org> >> --- >> drivers/mfd/Kconfig | 10 ---------- >> 1 file changed, 10 deletions(-) >> >> diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig >> index 11a1b65..b697136 100644 >> --- a/drivers/mfd/Kconfig >> +++ b/drivers/mfd/Kconfig >> @@ -660,16 +660,6 @@ config AB8500_CORE >> the irq_chip parts for handling the Mixed Signal chip events. >> This chip embeds various other multimedia funtionalities as well. >> >> -config AB8500_I2C_CORE >> - bool "AB8500 register access via PRCMU I2C" >> - depends on AB8500_CORE&& MFD_DB8500_PRCMU > > You need to move these dependencies under the AB8500_CORE config > entry instead.
Already done and upstreamed: 21f082a66177852365df0c955ecaef50fba9a691
>> - default y > > I wonder if in arch/arm/mach-ux500/Kconfig MACH_MOP500 should > select this instead, all MOP500 children have this an *won't* work > without it.
As described, I have already moved the dependency in the Kconfig file.
I guess what you've just mentioned is a separate patch (for another time). I'll stick it on my ever growing TODO.
-- Lee Jones Linaro ST-Ericsson Landing Team Lead M: +44 77 88 633 515 Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |