[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openbsd-arm
Subject:    Re: error in dtb for orange pi one ?
From:       Jonathan Gray <jsg () jsg ! id ! au>
Date:       2017-09-27 1:42:50
Message-ID: 20170927014250.GA69394 () largo ! jsg ! id ! au
[Download RAW message or body]

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 <jorik@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 <jorik@kippendief.biz>
    Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>

in sun8i-h3.dtsi but later got moved to sunxi-h3-h5.dtsi.

> 
>  
> 
> dtc -O dts -I dtb -o temp.dts
> /usr/local/share/dtb/arm/sun8i-h3-orangepi-one.dtb
> 
> .
> 
>  
> 
>            i2c@01c2ac00 {
> 
>                         compatible = "allwinner,sun6i-a31-i2c";
> 
>                         reg = <0x1c2ac00 0x400>;
> 
>                         interrupts = <0x0 0x6 0x4>;
> 
>                         clocks = <0x2 0x3b>;
> 
>                         resets = <0x2 0x2e>;
> 
>                         pinctrl-names = "default";
> 
>                         pinctrl-0 = <0x13>;
> 
>                         status = "disabled";
> 
>                         #address-cells = <0x1>;
> 
>                         #size-cells = <0x0>;
> 
>                 };
> 
>  
> 
>                 i2c@01c2b000 {
> 
>                         compatible = "allwinner,sun6i-a31-i2c";
> 
>                         reg = <0x1c2b000 0x400>;
> 
>                         interrupts = <0x0 0x7 0x4>;
> 
>                         clocks = <0x2 0x3c>;
> 
>                         resets = <0x2 0x2f>;
> 
>                         pinctrl-names = "default";
> 
>                         pinctrl-0 = <0x14>;
> 
>                         status = "disabled";
> 
>                         #address-cells = <0x1>;
> 
>                         #size-cells = <0x0>;
> 
>                 };
> 
>  
> 
>                 i2c@01c2b400 {
> 
>                         compatible = "allwinner,sun6i-a31-i2c";
> 
>                         reg = <0x1c2b000 0x400>;
> 
>                         interrupts = <0x0 0x8 0x4>;
> 
>                         clocks = <0x2 0x3d>;
> 
>                         resets = <0x2 0x30>;
> 
>                         pinctrl-names = "default";
> 
>                         pinctrl-0 = <0x15>;
> 
>                         status = "disabled";
> 
>                         #address-cells = <0x1>;
> 
>                         #size-cells = <0x0>;
> 
>                 };
> 

[prev in list] [next in list] [prev in thread] [next in thread]