On 4/22/12, Wolfram Sang <[email protected]> wrote: > On Tue, Apr 17, 2012 at 05:04:31PM +0530, Viresh Kumar wrote: >> clk_{un}prepare is mandatory for platforms using common clock framework. >> Since >> this driver is used by SPEAr platform, which supports common clock >> framework, >> add clk_{un}prepare() support for designware i2c. >> >> Signed-off-by: Viresh Kumar <[email protected]> > > ? Can't apply, which version is this against? Looks good otherwise.
Because following is waiting in you todo list and must be applied before this one: :) Author: Deepak Sikri <[email protected]> Date: Mon Sep 26 18:42:10 2011 +0530 i2c/busses: Add PM support This patch adds in support for standby/S2R/hybernate for i2c-designware driver. Signed-off-by: Deepak Sikri <[email protected]> Signed-off-by: Rajeev Kumar <[email protected]> --- drivers/i2c/busses/i2c-designware-platdrv.c | 27 +++++++++++++++++++++++++++ 1 files changed, 27 insertions(+), 0 deletions(-) Actually suspend/resume of above are also fixed by this patch. What are your plans for following patchset: i2c/adapter: Add bus recovery infrastructure Would be good if you can apply that or ask for another version if you see some shortcomings. Otherwise this kind of issues will happen again. Please allocate some time for this, it is really important for other bus drivers. All controllers/slaves have that bug. They will be fixed only once this is applied. :) -- viresh -- 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
