On 4/22/12, Wolfram Sang <w.sang@xxxxxxxxxxxxxx> 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 <viresh.kumar@xxxxxx>
>
> ? 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 <deepak.sikri@xxxxxx>
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 <deepak.sikri@xxxxxx>
Signed-off-by: Rajeev Kumar <rajeev-dlh.kumar@xxxxxx>
---
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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
