From mboxrd@z Thu Jan 1 00:00:00 1970 From: Grant Likely Subject: Re: [PATCH] of: Have of_device_add call platform_device_add rather than device_add Date: Mon, 26 Nov 2012 15:28:48 +0000 Message-ID: <20121126152848.57CB73E1A30@localhost> References: <20121121072448.GG19837@obsidianresearch.com> <20121121155104.726B83E0AE2@localhost> <20121121174453.GD6406@obsidianresearch.com> <20121121181430.GE6406@obsidianresearch.com> <20121122153621.452CA3E129E@localhost> <20121122173020.GA8473@obsidianresearch.com> Return-path: In-Reply-To: <20121122173020.GA8473@obsidianresearch.com> Sender: linux-kernel-owner@vger.kernel.org To: Jason Gunthorpe Cc: Linux Kernel Mailing List , Rob Herring , Greg Kroah-Hartman , devicetree-discuss List-Id: devicetree@vger.kernel.org On Thu, 22 Nov 2012 10:30:20 -0700, Jason Gunthorpe wrote: > On Thu, Nov 22, 2012 at 03:36:21PM +0000, Grant Likely wrote: > > > Hmm... I've not tried it with assigned-address. I tried with two sibling > > platform devices using just the 'reg' property. That the kernel will > > complain about. For powerpc-only, the patch I posted allows the device > > to get registered anyway even though the range incorrectly overlaps. > > My second example was done with the reg property.. > > gpio0: gpio@10100 { > compatible = "marvell,orion-gpio"; > #gpio-cells = <2>; > gpio-controller; > reg = <0x10100 0x40>; > } > chip_cfg@0 { > compatible = "orc,chip_config"; > // Doubles up on gpio0 > reg = <0x10100 0x4>; > }; > > > f1010100-f101013f : /internal@f1000000/gpio@10100 > f1010100-f1010103 : /internal@f1000000/chip_cfg@0 > > What did you try? Maybe order matters? (Sorry for not replying to my own mail; I'm doing this offline and my sent mail doesn't show) Looks like it is by design. With my dummy devices I see this: 10200c00-102023ff : /amba/overlap@10200c00 10201000-10201fff : /amba/dummy@10201000 10201400-10201bff : /amba/overlap@10201400 All three of those devices are siblings in the device tree. g.