On Wed, Sep 23, 2015 at 11:43:25PM +0100, Dinh Nguyen wrote: > On Wed, Sep 23, 2015 at 12:54 AM, Mark Rutland <mark.rutland@xxxxxxx> wrote: > >> +/ { > >> + compatible = "altr,socfpga-stratix10"; > >> + #address-cells = <2>; > >> + #size-cells = <2>; > > > > [...] > > > >> + soc { > >> + #address-cells = <1>; > >> + #size-cells = <1>; > >> + compatible = "simple-bus"; > >> + device_type = "soc"; > >> + interrupt-parent = <&intc>; > > > > As far as I am aware, this requires a (non-empty) ranges property, given the > > parent node has {address,size}-cells = <2>. > > > > Does the kernel not complain currently? > > > > I do not see any complaints without ranges here, but with ranges, I get: I'm surprised (and a little worried) that we don't warn in the absence of ranges when #{address,size}-cells changes. That doesn't feel right. > DTC arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dtb > Warning (ranges_format): /soc has empty "ranges" property but its > #address-cells (1) differs from / (2) > Warning (ranges_format): /soc has empty "ranges" property but its > #size-cells (1) differs from / (2) As I mentioned, the ranges proeprty should be non-empty. Thanks, Mark. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html