linux-i2c.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: [BUG] mv64xxx i2c bus lock w/ commit eda6bee6c7
       [not found]       ` <87fvw8ehne.fsf-LkuqDEemtHBg9hUCZPvPmw@public.gmane.org>
@ 2013-06-24 18:30         ` Wolfram Sang
  2013-06-24 21:31           ` Arnaud Ebalard
  0 siblings, 1 reply; 3+ messages in thread
From: Wolfram Sang @ 2013-06-24 18:30 UTC (permalink / raw)
  To: Arnaud Ebalard
  Cc: Rodolfo Giometti, Mark A. Greer, Mauro Barella, Ben Dooks,
	linux-i2c-u79uwXL29TY76Z2rM5mHXA, Guenter Roeck, Simon Guinot,
	Andrew Lunn, Jason Cooper, Linux ARM Kernel Mailing List

[-- Attachment #1: Type: text/plain, Size: 584 bytes --]


> does nothing fancy either. While debugging I ended up on Debian bug
> #622325 (http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=622325) which
> made the driver a little less suspect when a revert of commit eda6bee6c7
> fixed the issue for me.

Either wait for 3.11 or pick 4243fa0bad551b8c8d4ff7104e8fd557ae848845
"(I2C: mv64xxx: fix race between FSM/interrupt and process context")
from my for-next branch. (Note the other patches from Russell, probably
all worth to pick). Would be nice if you could pass the info to the
Debian bug as well.

Thanks,

   Wolfram


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread
* Re: [BUG] mv64xxx i2c bus lock w/ commit eda6bee6c7
  2013-06-24 18:30         ` [BUG] mv64xxx i2c bus lock w/ commit eda6bee6c7 Wolfram Sang
@ 2013-06-24 21:31           ` Arnaud Ebalard
  2013-06-25 21:09             ` Arnaud Ebalard
  0 siblings, 1 reply; 3+ messages in thread
From: Arnaud Ebalard @ 2013-06-24 21:31 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: Linux ARM Kernel Mailing List, Andrew Lunn, Simon Guinot,
	Mark A. Greer, linux-i2c, Ben Dooks, Rodolfo Giometti,
	Mauro Barella, Guenter Roeck, Jason Cooper

Hi,

Wolfram Sang <wsa@the-dreams.de> writes:

>> does nothing fancy either. While debugging I ended up on Debian bug
>> #622325 (http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=622325) which
>> made the driver a little less suspect when a revert of commit eda6bee6c7
>> fixed the issue for me.
>
> Either wait for 3.11 or pick 4243fa0bad551b8c8d4ff7104e8fd557ae848845
> "(I2C: mv64xxx: fix race between FSM/interrupt and process context")
> from my for-next branch. (Note the other patches from Russell, probably
> all worth to pick).

Thanks for the tip. Will try it tomorrow.

> Would be nice if you could pass the info to the Debian bug as well.

I will.

Cheers,

a+

^ permalink raw reply	[flat|nested] 3+ messages in thread
* Re: [BUG] mv64xxx i2c bus lock w/ commit eda6bee6c7
  2013-06-24 21:31           ` Arnaud Ebalard
@ 2013-06-25 21:09             ` Arnaud Ebalard
  0 siblings, 0 replies; 3+ messages in thread
From: Arnaud Ebalard @ 2013-06-25 21:09 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: Linux ARM Kernel Mailing List, Andrew Lunn, Simon Guinot,
	Mark A. Greer, linux-i2c-u79uwXL29TY76Z2rM5mHXA, Ben Dooks,
	Rodolfo Giometti, Mauro Barella, Guenter Roeck, Jason Cooper

Hi Wolfram,

arno-LkuqDEemtHBg9hUCZPvPmw@public.gmane.org (Arnaud Ebalard) writes:

> Wolfram Sang <wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org> writes:
>
>>> does nothing fancy either. While debugging I ended up on Debian bug
>>> #622325 (http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=622325) which
>>> made the driver a little less suspect when a revert of commit eda6bee6c7
>>> fixed the issue for me.
>>
>> Either wait for 3.11 or pick 4243fa0bad551b8c8d4ff7104e8fd557ae848845
>> "(I2C: mv64xxx: fix race between FSM/interrupt and process context")
>> from my for-next branch. (Note the other patches from Russell, probably
>> all worth to pick).
>
> Thanks for the tip. Will try it tomorrow.

Done. It works as expected, i.e. no i2c lock at boot on my ReadynNAS
102 once applied. Note that I had to also apply other patches from
Russel on which this specific commit somewhat depends.

>> Would be nice if you could pass the info to the Debian bug as well.

Done as well. I put you in the Cc: list.

Thanks for your work,

Cheers,

a+

^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-06-25 21:09 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <87a9mkttxr.fsf@natisbad.org>
     [not found] ` <20130623153931.GD7626@kw.sim.vm.gnt>
     [not found]   ` <20130623160659.GB17536@roeck-us.net>
     [not found]     ` <87fvw8ehne.fsf@natisbad.org>
     [not found]       ` <87fvw8ehne.fsf-LkuqDEemtHBg9hUCZPvPmw@public.gmane.org>
2013-06-24 18:30         ` [BUG] mv64xxx i2c bus lock w/ commit eda6bee6c7 Wolfram Sang
2013-06-24 21:31           ` Arnaud Ebalard
2013-06-25 21:09             ` Arnaud Ebalard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).