Hi Wolfram, Ping?
On Sun, Jan 05, 2014 at 07:04:10AM +0200, Baruch Siach wrote: > On Sat, Jan 04, 2014 at 11:48:46PM +0100, Wolfram Sang wrote: > > On Mon, Dec 30, 2013 at 10:31:19AM +0200, Baruch Siach wrote: > > > Since 93abe8e4 (clk: add non CONFIG_HAVE_CLK routines) code using clk.h > > > need > > > not depend on HAVE_CLK. Also, remove a redundant clk.h include from core > > > driver > > > file. > > > > > > Signed-off-by: Baruch Siach <[email protected]> > > > > Is the patch description correct? The core driver simply does not need > > clk.h and thus the dependency. No redundancy here, or? > > The platform code (i2c-designware-platform.c) uses the clk API, but the > Kconfig dependency on HAVE_CLK is not needed anymore. The core code > (i2c-designware-core.c) doesn't use clk. Originally the driver was in a > single > file, including the clk handling. This code has been moved to the platform > file, so the core file doesn't need the clk.h include. > > Logically, these are two separate issues. But I'm not sure it is worth > splitting into two patches. What do you think? baruch -- http://baruch.siach.name/blog/ ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - [email protected] - tel: +972.2.679.5364, http://www.tkos.co.il - -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
