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 :) Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
signature.asc
Description: Digital signature
