On 09/08/2013 11:05, Gregory CLEMENT wrote: > Hello, > > This new version just brings the handle of the "marvell,mv78230-i2" > string as a single compatible string. As there were no other remarks > since about 2 months I hope that this version will be able to be > merged in 3.12. > > This patch set adds support for the I2C Transaction Generator which > offloads CPU from managing I2C transfer step by step. This feature is > currently only available on the I2C controller IP embedded in the > Armada XP SoC. > > This series also contains a real fix for the I2C controller of the > Armada XP SoC. > > The first two patches modify the driver itself and should go through > i2c subsystem. > > The last patch updates the device tree to be able to use this new > feature. I kept this patch in the series for having a coherent series, > but it have already been pulled in the mvebu subsystem. > Wolfram, Jason Cooper drooped the third patch as it conflicted with a patch from Maxime Ripard which adds the AllWinner support. Olof also asked a formal acked-by from a device tree maintainer even if we already answer to Mark Rutland request. Olof also requested that you take the binding update, so I am going to send a new version of this patch set with the last patch split in two parts. As explained earlier today, unless you really want I use be32_to_cpu in the mv64xxx_i2c_offload_msg() I won't change anything else. Regards, > Thanks, > > Changelog: > > v4-v5: > > - handle of the "marvell,mv78230-i2" string as a single compatible > string. Even it is strongly discouraged to use "marvell, > mv78230-i2" alone, it is better to be able to handle it. > > > v3->v4: > > - reverse the order of the compatible strings, with the most > specific first > > - rebased on 3.11-rc1 > > v2->v3: > > - Introduces a new compatible string mv78230-i2c which will be used > for the fix and for the offload feature which are only present on > the Armada XP SoCs > > - Removes the unneeded spin_lock_irqsave pointed by Russell King > > - The offload mechanism is now port of the fsm and handle the > multiple messages. > > - The flag bridge-enabled is renamed to offload_enabled, but the > register name stills contains the BRIDGE word to match the > datasheet. > > - Uses writel_relaxed on the place pointed by Russell King > > - Uses the bool type for the flag (pointed by Thomas Petazzoni) > > - Removes useless code (pointed by Thomas Petazzoni) > > - Updates the bindings documentation > > v1->v2: > - Move the flag for the timing issue from global scope to per device > scope > - Assignment is no more done in if condition > > Gregory CLEMENT (3): > i2c-mv64xxx: Add I2C Transaction Generator support > i2c-mv64xxx: Fix timing issue on Armada XP (errata FE-8471889) > ARM: dts: mvebu: Introduce a new compatible string for mv64xxx-i2c > > .../devicetree/bindings/i2c/i2c-mv64xxx.txt | 13 +- > arch/arm/boot/dts/armada-370-xp.dtsi | 2 - > arch/arm/boot/dts/armada-370.dtsi | 8 + > arch/arm/boot/dts/armada-xp.dtsi | 10 + > drivers/i2c/busses/i2c-mv64xxx.c | 217 ++++++++++++++++++++- > 5 files changed, 237 insertions(+), 13 deletions(-) > -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- 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
