Re: [PATCH v3] [MFD] i2c-htcpld: Add the HTCPLD driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 19 Jan 2010 13:09:48 +0100, Samuel Ortiz wrote:
> Hi Jean,
> 
> On Tue, Jan 19, 2010 at 12:48:12PM +0100, Jean Delvare wrote:
> > Hi Samuel,
> > 
> > On Tue, 19 Jan 2010 12:09:17 +0100, Samuel Ortiz wrote:
> > > 2) Kconfig: we need to depend on I2C=y, to avoid the problematic case of
> > > having I2C_CORE=m and HTC_I2CPLD=y
> > 
> > ??
> > 
> > There is no config option named I2C_CORE. The main option for I2C is,
> > well, I2C,
> Sorry, I meant I2C, not I2C_CORE.
> 
> 
> > and if you depend on I2C, then you can't be built into the
> > kernel if I2C is built as a module. 
> I have:
> 
> drivers/mfd/Kconfig:
> config HTC_I2CPLD
> 	bool "HTC I2C PLD chip support"
> 	depends on I2C
> 
> Then make menuconfig allows me to build a .config with
> 
> CONFIG_I2C=m and CONFIG_HTC_I2CPLD=y
> 
> and that obviously fails to build. If I'm a bool depending on I2C, it has to
> be a built-in I2C.

Ah, my bad, I totally missed that HTC_I2CPLD was a bool. Please just
ignore me, sorry for the noise. You are correct, it should depend on
I2C=y.

-- 
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux