On Mon, Aug 18, 2014 at 10:27 PM, Maxime Ripard <maxime.rip...@free-electrons.com> wrote: > On Sun, Aug 17, 2014 at 11:20:44PM +0800, Chen-Yu Tsai wrote: >> Hi, >> >> On Sun, Aug 17, 2014 at 4:02 PM, Maxime Ripard >> <maxime.rip...@free-electrons.com> wrote: >> > Hi, >> > >> > On Sun, Aug 17, 2014 at 11:52:17AM +0800, Chen-Yu Tsai wrote: >> >> Add nodes for the 3 i2c controllers found on A23 SoCs to the sun8i DTSI. >> >> >> >> Signed-off-by: Chen-Yu Tsai <w...@csie.org> >> >> --- >> >> arch/arm/boot/dts/sun8i-a23.dtsi | 36 >> >> ++++++++++++++++++++++++++++++++++++ >> >> 1 file changed, 36 insertions(+) >> >> >> >> diff --git a/arch/arm/boot/dts/sun8i-a23.dtsi >> >> b/arch/arm/boot/dts/sun8i-a23.dtsi >> >> index 4658998..53ef350 100644 >> >> --- a/arch/arm/boot/dts/sun8i-a23.dtsi >> >> +++ b/arch/arm/boot/dts/sun8i-a23.dtsi >> >> @@ -387,6 +387,42 @@ >> >> status = "disabled"; >> >> }; >> >> >> >> + i2c0: i2c@01c2ac00 { >> >> + compatible = "allwinner,sun6i-a31-i2c"; >> >> + reg = <0x01c2ac00 0x400>; >> >> + interrupts = <0 6 4>; >> >> + clocks = <&apb2_gates 0>; >> >> + clock-frequency = <100000>; >> > >> > It's the default, you don't need to set it. >> >> Ah, just checked the driver, it is indeed the default. >> All the other sunxi dtsi have it though. Do we want to be consistent? > > Yep, it's usually a good thing :)
I'll do additional patches to remove them from the other dtsi then. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/