diff options
author | Gregory CLEMENT <gregory.clement@free-electrons.com> | 2013-12-31 10:59:33 -0500 |
---|---|---|
committer | Jason Cooper <jason@lakedaemon.net> | 2014-01-13 21:01:09 -0500 |
commit | 6cf70ae928bae17077efc0d528dec49bc380438b (patch) | |
tree | d795960abeeda498b8ae075163c0b7355d32c2a5 /drivers/i2c/busses/i2c-mv64xxx.c | |
parent | 85e618a1be2b2092318178d1d66bdad49cbbeeeb (diff) |
i2c: mv64xxx: Fix bus hang on A0 version of the Armada XP SoCs
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 <gregory.clement@free-electrons.com>
Reported-by: Andrew Lunn <andrew@lunn.ch>
Acked-by: Wolfram Sang <wsa@the-dreams.de>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Cc: stable@vger.kernel.org # v3.12+: af8d1c63afcb: ARM: mvebu: Add support to get the ID and the revision of a SoC
Cc: stable@vger.kernel.org # v3.12+: 85e618a1be2b: ARM: mvebu: Add quirk for i2c for the OpenBlocks AX3-4 board
Cc: stable@vger.kernel.org # v3.12+
Fixes: 930ab3d403ae (i2c: mv64xxx: Add I2C Transaction Generator support)
Signed-off-by: Jason Cooper <jason@lakedaemon.net>
Diffstat (limited to 'drivers/i2c/busses/i2c-mv64xxx.c')
-rw-r--r-- | drivers/i2c/busses/i2c-mv64xxx.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/i2c/busses/i2c-mv64xxx.c b/drivers/i2c/busses/i2c-mv64xxx.c index 8be7e42aa4de..b8c5187b9ee0 100644 --- a/drivers/i2c/busses/i2c-mv64xxx.c +++ b/drivers/i2c/busses/i2c-mv64xxx.c | |||
@@ -692,6 +692,7 @@ static const struct of_device_id mv64xxx_i2c_of_match_table[] = { | |||
692 | { .compatible = "allwinner,sun4i-i2c", .data = &mv64xxx_i2c_regs_sun4i}, | 692 | { .compatible = "allwinner,sun4i-i2c", .data = &mv64xxx_i2c_regs_sun4i}, |
693 | { .compatible = "marvell,mv64xxx-i2c", .data = &mv64xxx_i2c_regs_mv64xxx}, | 693 | { .compatible = "marvell,mv64xxx-i2c", .data = &mv64xxx_i2c_regs_mv64xxx}, |
694 | { .compatible = "marvell,mv78230-i2c", .data = &mv64xxx_i2c_regs_mv64xxx}, | 694 | { .compatible = "marvell,mv78230-i2c", .data = &mv64xxx_i2c_regs_mv64xxx}, |
695 | { .compatible = "marvell,mv78230-a0-i2c", .data = &mv64xxx_i2c_regs_mv64xxx}, | ||
695 | {} | 696 | {} |
696 | }; | 697 | }; |
697 | MODULE_DEVICE_TABLE(of, mv64xxx_i2c_of_match_table); | 698 | MODULE_DEVICE_TABLE(of, mv64xxx_i2c_of_match_table); |
@@ -783,6 +784,10 @@ mv64xxx_of_config(struct mv64xxx_i2c_data *drv_data, | |||
783 | drv_data->errata_delay = true; | 784 | drv_data->errata_delay = true; |
784 | } | 785 | } |
785 | 786 | ||
787 | if (of_device_is_compatible(np, "marvell,mv78230-a0-i2c")) { | ||
788 | drv_data->offload_enabled = false; | ||
789 | drv_data->errata_delay = true; | ||
790 | } | ||
786 | out: | 791 | out: |
787 | return rc; | 792 | return rc; |
788 | #endif | 793 | #endif |