On Thu, Dec 26, 2013 at 10:25 PM, Wolfram Sang <wsa@xxxxxxxxxxxxx> wrote:
> On Thu, Dec 26, 2013 at 10:20:23PM +0100, Geert Uytterhoeven wrote:
>> 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.
>
> ??? Why do you want to drop these lines?
Your v2 dropped them, presumably in response to Sergei's ICKy comments.
And that's what Simon committed...
> If the i2c clocks are not registered, you won't get i2c support? Am I
> missing something?
Ah OK. So you do need them. Good, /me getting less confused ;-)
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
