Enable the second I2C bus on Marvell Kirkwood
Subsystems (2)
PATCH 1/1 Enable the second I2C bus on Marvell Kirkwood
mv64xxx_of_config requires that the tclk frequency be found through the clk stuff rather than through device tree, so add another override for the 2nd controller.
--- arch/arm/mach-kirkwood/board-dt.c | 2 ++ arch/arm/mach-kirkwood/common.c | 1 + 2 files changed, 3 insertions(+), 0 deletions(-)
of_serial specifies the tclk frequency (via clock-frequency) in the DT, should the i2c have the same option?
arch/arm/mach-kirkwood/board-dt.c +2 -0
@@ -30,6 +30,8 @@ struct of_dev_auxdata kirkwood_auxdata_lookup[] __initdata = {OF_DEV_AUXDATA ("marvell,orion-spi" ,0xf1010600 ,"orion_spi.0" ,NULL ),OF_DEV_AUXDATA ("marvell,mv64xxx-i2c" ,0xf1011000 ,"mv64xxx_i2c.0" ,NULL ), +OF_DEV_AUXDATA ("marvell,mv64xxx-i2c" ,0xf1011100 ,"mv64xxx_i2c.1" , +NULL ),OF_DEV_AUXDATA ("marvell,orion-wdt" ,0xf1020300 ,"orion_wdt" ,NULL ),OF_DEV_AUXDATA ("marvell,orion-sata" ,0xf1080000 ,"sata_mv.0" ,NULL ), OF_DEV_AUXDATA("marvell,orion-nand", 0xf4000000, "orion_nand", NULL),
arch/arm/mach-kirkwood/common.c +1 -0
@@ -278,6 +278,7 @@ void __init kirkwood_clk_init(void)orion_clkdev_add ("1" ,"pcie" , pex1);orion_clkdev_add (NULL ,"kirkwood-i2s" , audio);orion_clkdev_add (NULL ,MV64XXX_I2C_CTLR_NAME ".0" , runit); +orion_clkdev_add (NULL ,MV64XXX_I2C_CTLR_NAME ".1" , runit);/* Marvell says runit is used by SPI, UART, NAND, TWSI, ..., * so should never be gated.
Signature · 1 line
1.7.5.4
PATCH 1/1 Enable the second I2C bus on Marvell Kirkwood
Quoted 5 lines · Jason Gunthorpe
mv64xxx_of_configrequires that the tclk frequency be found through the clk stuff rather than through device tree, so add another override for the 2nd controller.Signed-off-by Jason Gunthorpe <[email protected]>
Thanks for the patch, Jason. Looks good. One small note, please don't forget to CC: the maintainers in future submissions. In this case, myself and Andrew Lunn.
thx,
Jason.
Quoted 41 lines
--- arch/arm/mach-kirkwood/board-dt.c | 2 ++ arch/arm/mach-kirkwood/common.c | 1 + 2 files changed, 3 insertions(+), 0 deletions(-)
of_serialspecifies the tclk frequency (via clock-frequency) in the DT, should the i2c have the same option?arch/arm/mach-kirkwood/board-dt.c +2
-0@@ -30,6 +30,8 @@ struct of_dev_auxdata kirkwood_auxdata_lookup[] __initdata = {OF_DEV_AUXDATA ("marvell,orion-spi" ,0xf1010600 ,"orion_spi.0" ,NULL ),OF_DEV_AUXDATA ("marvell,mv64xxx-i2c" ,0xf1011000 ,"mv64xxx_i2c.0" ,NULL ), +OF_DEV_AUXDATA ("marvell,mv64xxx-i2c" ,0xf1011100 ,"mv64xxx_i2c.1" , +NULL ),OF_DEV_AUXDATA ("marvell,orion-wdt" ,0xf1020300 ,"orion_wdt" ,NULL ),OF_DEV_AUXDATA ("marvell,orion-sata" ,0xf1080000 ,"sata_mv.0" ,NULL ), OF_DEV_AUXDATA("marvell,orion-nand", 0xf4000000, "orion_nand", NULL),arch/arm/mach-kirkwood/common.c +1
-0@@ -278,6 +278,7 @@ void __init kirkwood_clk_init(void)orion_clkdev_add ("1" ,"pcie" , pex1);orion_clkdev_add (NULL ,"kirkwood-i2s" , audio);orion_clkdev_add (NULL ,MV64XXX_I2C_CTLR_NAME ".0" , runit); +orion_clkdev_add (NULL ,MV64XXX_I2C_CTLR_NAME ".1" , runit);/* Marvell says runit is used by SPI, UART, NAND, TWSI, ..., * so should never be gated. Signature · 7 lines
1.7.5.4 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel at lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
PATCH 1/1 Enable the second I2C bus on Marvell Kirkwood
In reply to Jason GunthorpeQuoted 5 lines · Jason Gunthorpe
mv64xxx_of_configrequires that the tclk frequency be found through the clk stuff rather than through device tree, so add another override for the 2nd controller.Signed-off-by Jason Gunthorpe <[email protected]>
I made one small change, changed subject to:
ARM: Kirkwood: Enable the second I2C bus
Applied to:
git://git.infradead.org/users/jcooper/linux.git kirkwood/boards
thx,
Jason.