From mboxrd@z Thu Jan 1 00:00:00 1970 From: ezequiel.garcia@free-electrons.com (Ezequiel Garcia) Date: Tue, 11 Jun 2013 10:31:45 -0300 Subject: [PATCH 04/14] bus: mvebu-mbus: Add static window allocation to the DT binding In-Reply-To: <201306072101.44694.arnd@arndb.de> References: <1370623671-7748-1-git-send-email-ezequiel.garcia@free-electrons.com> <1370623671-7748-5-git-send-email-ezequiel.garcia@free-electrons.com> <201306072101.44694.arnd@arndb.de> Message-ID: <20130611133143.GB3996@localhost> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Arnd, I have another question! On Fri, Jun 07, 2013 at 09:01:44PM +0200, Arnd Bergmann wrote: [...] > > +Example: > > + > > + soc { > > + compatible = "marvell,armada370-mbus", "simple-bus"; > > + reg = <0xd0020000 0x100>, <0xd0020180 0x20>; > > + }; > > + > > +** How does it work? > > + > > +The MBus driver controls the allocation and release of the addresses > > +decoding windows needed to access devices. > > + > > +Each window, is identified by its target ID and attribute ID. In order > > +to represent this, each of mbus-node first-level child has to declare > > +a suitable 'ranges' translation entry for the MBus to allocate a window > > +for it. This entry will encode the target and attribute in a 'windowid' > > +ad-hoc cell, like this: > > + > > + soc { > > + bootrom { > > + ranges = <0 0x01e00000 0xfff00000 0x100000>; > > + }; > > + }; > > I think I'm a bit lost here. Is the "soc" node in this example the node > that is described as compatible="marvell,armada370-mbus"? Maybe expand > the example a little here to clarify this. > Exactly: that's the mbus-compatible node. I'll clarify a bit the documentation so this is clearer. I kept the 'soc' name in the examples as that's the name in the dts files. Do you think I should rename to 'mbus' in the dts files? I haven't done that since that would imply to change all the dts files. Thanks for the help, -- Ezequiel Garc?a, Free Electrons Embedded Linux, Kernel and Android Engineering http://free-electrons.com From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ezequiel Garcia Subject: Re: [PATCH 04/14] bus: mvebu-mbus: Add static window allocation to the DT binding Date: Tue, 11 Jun 2013 10:31:45 -0300 Message-ID: <20130611133143.GB3996@localhost> References: <1370623671-7748-1-git-send-email-ezequiel.garcia@free-electrons.com> <1370623671-7748-5-git-send-email-ezequiel.garcia@free-electrons.com> <201306072101.44694.arnd@arndb.de> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <201306072101.44694.arnd-r2nGTMty4D4@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: "devicetree-discuss" To: Arnd Bergmann Cc: Lior Amsalem , Jason Cooper , Andrew Lunn , devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org, Jason Gunthorpe , Maen Suleiman , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Sebastian Hesselbarth List-Id: devicetree@vger.kernel.org SGkgQXJuZCwKCkkgaGF2ZSBhbm90aGVyIHF1ZXN0aW9uIQoKT24gRnJpLCBKdW4gMDcsIDIwMTMg YXQgMDk6MDE6NDRQTSArMDIwMCwgQXJuZCBCZXJnbWFubiB3cm90ZToKWy4uLl0KPiA+ICtFeGFt cGxlOgo+ID4gKwo+ID4gKwlzb2Mgewo+ID4gKwkJY29tcGF0aWJsZSA9ICJtYXJ2ZWxsLGFybWFk YTM3MC1tYnVzIiwgInNpbXBsZS1idXMiOwo+ID4gKwkJcmVnID0gPDB4ZDAwMjAwMDAgMHgxMDA+ LCA8MHhkMDAyMDE4MCAweDIwPjsKPiA+ICsJfTsKPiA+ICsKPiA+ICsqKiBIb3cgZG9lcyBpdCB3 b3JrPwo+ID4gKwo+ID4gK1RoZSBNQnVzIGRyaXZlciBjb250cm9scyB0aGUgYWxsb2NhdGlvbiBh bmQgcmVsZWFzZSBvZiB0aGUgYWRkcmVzc2VzCj4gPiArZGVjb2Rpbmcgd2luZG93cyBuZWVkZWQg dG8gYWNjZXNzIGRldmljZXMuCj4gPiArCj4gPiArRWFjaCB3aW5kb3csIGlzIGlkZW50aWZpZWQg YnkgaXRzIHRhcmdldCBJRCBhbmQgYXR0cmlidXRlIElELiBJbiBvcmRlcgo+ID4gK3RvIHJlcHJl c2VudCB0aGlzLCBlYWNoIG9mIG1idXMtbm9kZSBmaXJzdC1sZXZlbCBjaGlsZCBoYXMgdG8gZGVj bGFyZQo+ID4gK2Egc3VpdGFibGUgJ3JhbmdlcycgdHJhbnNsYXRpb24gZW50cnkgZm9yIHRoZSBN QnVzIHRvIGFsbG9jYXRlIGEgd2luZG93Cj4gPiArZm9yIGl0LiBUaGlzIGVudHJ5IHdpbGwgZW5j b2RlIHRoZSB0YXJnZXQgYW5kIGF0dHJpYnV0ZSBpbiBhICd3aW5kb3dpZCcKPiA+ICthZC1ob2Mg Y2VsbCwgbGlrZSB0aGlzOgo+ID4gKwo+ID4gKwlzb2Mgewo+ID4gKwkJYm9vdHJvbSB7Cj4gPiAr CQkJcmFuZ2VzID0gPDAgMHgwMWUwMDAwMCAweGZmZjAwMDAwIDB4MTAwMDAwPjsKPiA+ICsJCX07 Cj4gPiArCX07Cj4gCj4gSSB0aGluayBJJ20gYSBiaXQgbG9zdCBoZXJlLiBJcyB0aGUgInNvYyIg bm9kZSBpbiB0aGlzIGV4YW1wbGUgdGhlIG5vZGUKPiB0aGF0IGlzIGRlc2NyaWJlZCBhcyBjb21w YXRpYmxlPSJtYXJ2ZWxsLGFybWFkYTM3MC1tYnVzIj8gTWF5YmUgZXhwYW5kCj4gdGhlIGV4YW1w bGUgYSBsaXR0bGUgaGVyZSB0byBjbGFyaWZ5IHRoaXMuCj4gCgpFeGFjdGx5OiB0aGF0J3MgdGhl IG1idXMtY29tcGF0aWJsZSBub2RlLiBJJ2xsIGNsYXJpZnkgYSBiaXQgdGhlIGRvY3VtZW50YXRp b24Kc28gdGhpcyBpcyBjbGVhcmVyLgoKSSBrZXB0IHRoZSAnc29jJyBuYW1lIGluIHRoZSBleGFt cGxlcyBhcyB0aGF0J3MgdGhlIG5hbWUgaW4gdGhlIGR0cyBmaWxlcy4gCkRvIHlvdSB0aGluayBJ IHNob3VsZCByZW5hbWUgdG8gJ21idXMnIGluIHRoZSBkdHMgZmlsZXM/CgpJIGhhdmVuJ3QgZG9u ZSB0aGF0IHNpbmNlIHRoYXQgd291bGQgaW1wbHkgdG8gY2hhbmdlIGFsbCB0aGUgZHRzCmZpbGVz LgoKVGhhbmtzIGZvciB0aGUgaGVscCwKLS0gCkV6ZXF1aWVsIEdhcmPDrWEsIEZyZWUgRWxlY3Ry b25zCkVtYmVkZGVkIExpbnV4LCBLZXJuZWwgYW5kIEFuZHJvaWQgRW5naW5lZXJpbmcKaHR0cDov L2ZyZWUtZWxlY3Ryb25zLmNvbQpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fXwpkZXZpY2V0cmVlLWRpc2N1c3MgbWFpbGluZyBsaXN0CmRldmljZXRyZWUtZGlz Y3Vzc0BsaXN0cy5vemxhYnMub3JnCmh0dHBzOi8vbGlzdHMub3psYWJzLm9yZy9saXN0aW5mby9k ZXZpY2V0cmVlLWRpc2N1c3MK