From mboxrd@z Thu Jan 1 00:00:00 1970 From: marek.vasut@gmail.com (Marek Vasut) Date: Mon, 18 Jun 2012 01:06:20 +0200 Subject: [PATCH] ARM: ux500: bump i2c bus speed to 400kHz In-Reply-To: <1339955837-26691-1-git-send-email-linus.walleij@stericsson.com> References: <1339955837-26691-1-git-send-email-linus.walleij@stericsson.com> Message-ID: <201206180106.21108.marek.vasut@gmail.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Dear Linus Walleij, > From: Linus Walleij > > Some recent discussion revealed inconsistency in the board file > for the MOP500 family. The board can very well do 400kHz on > the I2C busses, so make it do that. Shouldn't this be controlled by DT? Some of the i2c devices can't do 400kHz. > Cc: Srinidhi Kasagar > Cc: Lee Jones > Signed-off-by: Linus Walleij > --- > arch/arm/mach-ux500/board-mop500.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/arch/arm/mach-ux500/board-mop500.c > b/arch/arm/mach-ux500/board-mop500.c index 9c74ac5..e441bd7 100644 > --- a/arch/arm/mach-ux500/board-mop500.c > +++ b/arch/arm/mach-ux500/board-mop500.c > @@ -354,13 +354,13 @@ static struct nmk_i2c_controller u8500_i2c##id##_data > = { \ /* > * The board uses 4 i2c controllers, initialize all of > * them with slave data setup time of 250 ns, > - * Tx & Rx FIFO threshold values as 8 and standard > + * Tx & Rx FIFO threshold values as 1 resp. 8 and fast > * mode of operation > */ > -U8500_I2C_CONTROLLER(0, 0xe, 1, 8, 100000, 200, I2C_FREQ_MODE_FAST); > -U8500_I2C_CONTROLLER(1, 0xe, 1, 8, 100000, 200, I2C_FREQ_MODE_FAST); > -U8500_I2C_CONTROLLER(2, 0xe, 1, 8, 100000, 200, I2C_FREQ_MODE_FAST); > -U8500_I2C_CONTROLLER(3, 0xe, 1, 8, 100000, 200, I2C_FREQ_MODE_FAST); > +U8500_I2C_CONTROLLER(0, 0xe, 1, 8, 400000, 200, I2C_FREQ_MODE_FAST); > +U8500_I2C_CONTROLLER(1, 0xe, 1, 8, 400000, 200, I2C_FREQ_MODE_FAST); > +U8500_I2C_CONTROLLER(2, 0xe, 1, 8, 400000, 200, I2C_FREQ_MODE_FAST); > +U8500_I2C_CONTROLLER(3, 0xe, 1, 8, 400000, 200, I2C_FREQ_MODE_FAST); > > static void __init mop500_i2c_init(struct device *parent) > { Best regards, Marek Vasut