[PATCH 14/17] i2c: nomadik: Fixup deployment of runtime PM

On 10 February 2014 11:14, Ulf Hansson [off-list ref] wrote:

On 5 February 2014 15:34, Linus Walleij [off-list ref] wrote:

quoted

On Tue, Feb 4, 2014 at 4:58 PM, Ulf Hansson [off-list ref] wrote:

quoted

Since the device is active while a successful probe has been completed,
the reference counting for the clock will be screwed up and never reach
zero.

The issue is resolved by implementing runtime PM callbacks and let them
handle the resources accordingly, including the clock.

Cc: Alessandro Rubini <redacted>
Cc: Linus Walleij <redacted>
Cc: Wolfram Sang <redacted>
Signed-off-by: Ulf Hansson <redacted>

Hm do I read it right as patch 13 breaks runtime PM by leaving
the device active after probe() and this patch
14 fixes it again? Maybe these two patches should be squashed
then.

In v2 I have now squashed patch 13 into this patch 14.

That means patch13 shall be dropped from this patchset.

Kind regards
Uffe

You are right; but the driver will still be working, you just don't
get the benefit from inactivating the device at request inactivity -
as you pointed out.

The reason for why I wanted to do this as separate steps was to make
it easier for reviewing, otherwise the patch(es) would have been quite
big and messy. I am for sure open to  adopt to your proposal, but just
wanted to give you some more background, before I go ahead and send a
v2.

Kind regards
Ulf Hansson

quoted

Yours,
Linus Walleij