Re: [PATCH 1/3] i2c: append hardware lock with bus lock

2011-04-28 Thread Eric Miao
On Thu, Apr 28, 2011 at 4:22 PM, Jean Delvare kh...@linux-fr.org wrote: Hi Haojian, On Thu, 28 Apr 2011 12:02:36 +0800, Haojian Zhuang wrote: Both AP and CP are contained in Marvell PXA910 silicon. These two ARM cores are sharing one pair of I2C pins. In order to keep I2C transaction

Re: [PATCH 1/3] i2c: append hardware lock with bus lock

2011-04-28 Thread Jean Delvare
Hi Eric, On Thu, 28 Apr 2011 16:36:02 +0800, Eric Miao wrote: On Thu, Apr 28, 2011 at 4:22 PM, Jean Delvare kh...@linux-fr.org wrote: Hi Haojian, On Thu, 28 Apr 2011 12:02:36 +0800, Haojian Zhuang wrote: Both AP and CP are contained in Marvell PXA910 silicon. These two ARM cores are

Re: [PATCH 1/3] i2c: append hardware lock with bus lock

2011-04-28 Thread Haojian Zhuang
On Thu, Apr 28, 2011 at 10:37 PM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Thu, Apr 28, 2011 at 04:16:25PM +0200, Jean Delvare wrote: Are you suggesting that the hardware lock wouldn't mind being taken twice by the AP side? If it is the case, then indeed the software mutex is

[PATCH 1/3] i2c: append hardware lock with bus lock

2011-04-28 Thread Haojian Zhuang
Both AP and CP are contained in Marvell PXA910 silicon. These two ARM cores are sharing one pair of I2C pins. In order to keep I2C transaction operated with atomic, hardware lock (RIPC) is required. Because of this, bus lock in AP side can't afford this requirement. Now hardware lock is appended.

[PATCH 2/3] i2c: pxa: support hardware lock

2011-04-28 Thread Haojian Zhuang
Append hardware lock support since it's required by Marvell PXA910. Signed-off-by: Haojian Zhuang haojian.zhu...@marvell.com Cc: Ben Dooks ben-li...@fluff.org Cc: Jean Delvare kh...@linux-fr.org --- drivers/i2c/busses/i2c-pxa.c |3 +++ include/linux/i2c/pxa-i2c.h |3 +++ 2 files

[PATCH 3/3] ARM: mmp: add hardware lock support in PXA910

2011-04-28 Thread Haojian Zhuang
RIPC register is used to implement hardware lock between CP and AP in Marvell PXA910. Signed-off-by: Haojian Zhuang haojian.zhu...@gmail.com Cc: Ben Dooks ben-li...@fluff.org Cc: Jean Delvare kh...@linux-fr.org Cc: Eric Miao eric.y.m...@gmail.com Cc: Russell King li...@arm.linux.org.uk ---

[PATCH] i2c-parport: Fix adapter list handling

2011-04-28 Thread Jean Delvare
Use a standard list with proper locking to handle the list of adapters. Thankfully it only matters on systems with more than one parallel port, which are very rare. Thanks to Lukasz Kapiec for reporting the problem to me. Signed-off-by: Jean Delvare kh...@linux-fr.org Cc: sta...@kernel.org ---

Re: [PATCH] i2c: i2c-sh_mobile bus speed platform data V2

2011-04-28 Thread Simon Horman
On Thu, Apr 28, 2011 at 01:25:36PM +0900, Magnus Damm wrote: From: Magnus Damm d...@opensource.se Add support to the i2c-sh_mobile driver for setting the I2C bus speed using platform data. Signed-off-by: Magnus Damm d...@opensource.se Acked-by: Simon Horman ho...@verge.net.au -- To