On Tue, Dec 24, 2013 at 3:48 AM, Simon Horman <horms@xxxxxxxxxxxx> wrote: > On Wed, Dec 18, 2013 at 10:31:58PM +0100, Wolfram Sang wrote: >> From: Wolfram Sang <wsa@xxxxxxxxxxxxxxxxxxxx> >> >> Tested with RIIC2 on a genmai board. Others untested but hopefully >> trivial enough to be added. >> >> Signed-off-by: Wolfram Sang <wsa@xxxxxxxxxxxxxxxxxxxx> >> Acked-by: Magnus Damm <damm@xxxxxxxxxxxxx> >> --- >> >> V2: adapt to established sorting > > Thanks. This seems safe enough and I will queue it up. Note that after dropping these lines: CLKDEV_DEV_ID("fcfee000.i2c", &mstp_clks[MSTP97]), CLKDEV_DEV_ID("fcfee400.i2c", &mstp_clks[MSTP96]), CLKDEV_DEV_ID("fcfee800.i2c", &mstp_clks[MSTP95]), CLKDEV_DEV_ID("fcfeec00.i2c", &mstp_clks[MSTP94]), i2c stopped working. Before: of_irq_parse_one: dev=/i2c@fcfee800/eeprom@50, index=0 at24 2-0050: 16384 byte 24c128 EEPROM, writable, 64 bytes/write i2c-riic fcfee800.i2c: registered with 400000Hz bus speed After: i2c-riic fcfee800.i2c: missing controller clock i2c-riic: probe of fcfee800.i2c failed with error -2 Am I missing some other patch? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- 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