On Wed, Sep 27, 2017 at 11:42:50AM +1000, Jonathan Gray wrote: > On Tue, Sep 26, 2017 at 01:08:54PM -0700, Stephen Graf wrote: > > I took this dtb from the amd64 dtb package yesterday (dtb-4.13.tgz). > > > > > > > > The address register for i2c01 and i2c02 are the same. I2c02 should be: > > > > reg = <0x1c2b400 0x400>; > > Talk to upstream on the devicetree list. > > Changes first went in with > > commit d8a507e6d9316b1023ba3cc9a7e696067200b495 > Author: Jorik Jonker <jo...@kippendief.biz> > Date: Mon Sep 12 20:12:47 2016 +0200 > > dts: sun8i-h3: add I2C0-2 peripherals to H3 SOC > > These peripherals can only be muxed to these pins, so they are > associated in the DTSI instead of the board files. This makes it very > easy to enable them using overlays or u-boot commands: > > => fdt set /soc/i2c@01c2ac00 status okay > > Signed-off-by: Jorik Jonker <jo...@kippendief.biz> > Signed-off-by: Maxime Ripard <maxime.rip...@free-electrons.com> > > in sun8i-h3.dtsi but later got moved to sunxi-h3-h5.dtsi.
Fixed in the following commit which is in master and should be in 4.15-rc1. commit 4d2ee8d56bdb17bf296800829642bbd34ef08a09 Author: Corentin LABBE <clabbe.montj...@gmail.com> Date: Tue Sep 26 09:22:22 2017 +0200 ARM: dts: sunxi: h3/h5: Fix i2c2 register address The unit address and register address does not match. This patch fix the register address with the good one. Acked-by: Maxime Ripard <maxime.rip...@free-electrons.com> Signed-off-by: Corentin Labbe <clabbe.montj...@gmail.com> Signed-off-by: Chen-Yu Tsai <w...@csie.org> diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi b/arch/arm/boot/dts/sunxi-h3-h5.dtsi index 6c71d5aa6352..24a7a0c84449 100644 --- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi +++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi @@ -575,7 +575,7 @@ i2c2: i2c@1c2b400 { compatible = "allwinner,sun6i-a31-i2c"; - reg = <0x01c2b000 0x400>; + reg = <0x01c2b400 0x400>; interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_BUS_I2C2>; resets = <&ccu RST_BUS_I2C2>;