On 10/10/2013 02:33 PM, Stephen GALLIMORE wrote: >> -----Original Message----- >> From: Maxime COQUELIN [mailto:maxime.coquelin@xxxxxx] >> Sent: 08 October 2013 17:43 >> ..... >> + >> + i2c@fed40000 { >> + compatible = "st,comms-ssc-i2c"; >> + reg = <0xfed40000 0x110>; >> + interrupts = <GIC_SPI 187 IRQ_TYPE_EDGE_RISING>; > This should specify level (high) triggered, not edge triggered, for our > SoC integrations. > > Note that level triggered is the default GIC setup and was therefore > what was being used when you specified "0" for the flags previously. Thanks Stephen. As discussed this morning, I agree this should be high level triggered. This will be fixed in next revision. > >> + clocks = <&CLKS_ICN_REG_0>; >> + clock-names = "ssc"; >> + clock-frequency = <400000>; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&pinctrl_i2c0_default>; >> + >> + status = "disabled"; >> + }; -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html