On Tuesday 24 February 2015 18:29:02 Chen-Yu Tsai wrote:
>
> + rsb@01f03400 {
> + compatible = "allwinner,sun8i-a23-rsb";
> + reg = <0x01f03400 0x400>;
> + interrupts = <0 39 4>;
> + clocks = <&apb0_gates 3>;
> + clock-frequency = <3000000>;
> + resets = <&apb0_rst 3>;
> +
> + axp223: pmic@2d {
> + compatible = "x-powers,axp223", "x-powers,axp221";
> + reg = <0x2d>;
> + allwinner,rsb-hw-addr = <0x3e3>;
> +
> + /* ... */
> + };
> + };
The child node cannot have a 'reg' property if the parent does not
have #address-cells/#size-cells. You should add these as mandatory
properties in the list.
I don't really understand the need for having two addresses (runtime
and hardware). Could the runtime address be configured at runtime?
Alternatively, could you use #address-cells=<2> and put both into
'reg'?
Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
