On Wed, Jan 08, 2014 at 04:06:28PM +0100, Gregory CLEMENT wrote: > The first variants of Armada XP SoCs (A0 stepping) have issues related > to the i2c controller which prevent to use the offload mechanism and > lead to a kernel hang during boot. > > The commit introduces a new the compatible string > marvell,mv78230-a0-i2c for the i2c controller. When this compatible > string is used the driver disables the offload mechanism and the > kernel no more hangs on these SoCs. > > Signed-off-by: Gregory CLEMENT <[email protected]> > Reported-by: Andrew Lunn <[email protected]> > Cc: [email protected] > --- > drivers/i2c/busses/i2c-mv64xxx.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/drivers/i2c/busses/i2c-mv64xxx.c > b/drivers/i2c/busses/i2c-mv64xxx.c > index 8be7e42aa4de..f424c0f89946 100644 > --- a/drivers/i2c/busses/i2c-mv64xxx.c > +++ b/drivers/i2c/busses/i2c-mv64xxx.c > @@ -692,6 +692,10 @@ static const struct of_device_id > mv64xxx_i2c_of_match_table[] = { > { .compatible = "allwinner,sun4i-i2c", .data = &mv64xxx_i2c_regs_sun4i}, > { .compatible = "marvell,mv64xxx-i2c", .data = > &mv64xxx_i2c_regs_mv64xxx}, > { .compatible = "marvell,mv78230-i2c", .data = > &mv64xxx_i2c_regs_mv64xxx}, > + { > + .compatible = "marvell,mv78230-a0-i2c", > + .data = &mv64xxx_i2c_regs_mv64xxx > + },
I think a oneliner entry like the entries above is easier to read, but that is very minor... Acked-by: Wolfram Sang <[email protected]>
signature.asc
Description: Digital signature
