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 <baruch@xxxxxxxxxx>
> >
> > 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------------{=
- baruch@xxxxxxxxxx - 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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
