From mboxrd@z Thu Jan 1 00:00:00 1970 From: ezequiel.garcia@free-electrons.com (Ezequiel Garcia) Date: Tue, 18 Jun 2013 18:10:03 -0300 Subject: [PATCH v3 03/12] bus: mvebu-mbus: Add static window allocation to the DT binding In-Reply-To: <20130618205522.GA13691@obsidianresearch.com> References: <20130618174622.GD2204@obsidianresearch.com> <51C0A5F8.8030300@gmail.com> <201306182039.50736.arnd@arndb.de> <51C0AA8E.9080807@gmail.com> <20130618184753.GA6090@obsidianresearch.com> <51C0ADF7.5050609@gmail.com> <20130618191018.GB6578@obsidianresearch.com> <51C0B4A0.90204@gmail.com> <20130618204911.GB2516@localhost> <20130618205522.GA13691@obsidianresearch.com> Message-ID: <20130618211002.GC2516@localhost> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Jun 18, 2013 at 02:55:22PM -0600, Jason Gunthorpe wrote: > On Tue, Jun 18, 2013 at 05:49:11PM -0300, Ezequiel Garcia wrote: > > Hi Sebastian, > > > > You loose +1 internet points for dropping me from Cc ;-) > > > > On Tue, Jun 18, 2013 at 09:27:28PM +0200, Sebastian Hesselbarth wrote: > > > On 06/18/2013 09:10 PM, Jason Gunthorpe wrote: > > > > > > > > The forms could be: > > > > > > > > 0IAA0000 > > > > FK000000 > > > > - K=0 -> internal regs > > > > - K=1 -> PCI-E thingy > > > > etc > > > > 1IIAA000 (future expansion example) > > > > > > Ok, got it. Any encoding is fine that allows to distinguish real > > > remap windows and fake ones. I assumed that maybe someday there > > > will be more than 4b target id so 0x80 as special case indicator > > > leaves 7b of normal target id in the _current_ mapping. > > > > > > > I'm also wondering about why we not care about target IDs being more > > than 4 bits. > > > > Jason: (I'm checking now but perhaps you know better than me): > > Is there any MBus-architectural reason for you assuring the > > target ID will always be within 4-bits? > > The manuals I have for the register set say 4 bits is allocated for > mbus targets.. Are yours different? > Nope. 4 bits here as well. > If they change it then the window register layout changes and then the > mbus driver probably need to change as well. > Okey, then. -- 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 v3 03/12] bus: mvebu-mbus: Add static window allocation to the DT binding Date: Tue, 18 Jun 2013 18:10:03 -0300 Message-ID: <20130618211002.GC2516@localhost> References: <20130618174622.GD2204@obsidianresearch.com> <51C0A5F8.8030300@gmail.com> <201306182039.50736.arnd@arndb.de> <51C0AA8E.9080807@gmail.com> <20130618184753.GA6090@obsidianresearch.com> <51C0ADF7.5050609@gmail.com> <20130618191018.GB6578@obsidianresearch.com> <51C0B4A0.90204@gmail.com> <20130618204911.GB2516@localhost> <20130618205522.GA13691@obsidianresearch.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <20130618205522.GA13691-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@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: Jason Gunthorpe Cc: Lior Amsalem , Andrew Lunn , Jason Cooper , devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org, Maen Suleiman , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Sebastian Hesselbarth List-Id: devicetree@vger.kernel.org T24gVHVlLCBKdW4gMTgsIDIwMTMgYXQgMDI6NTU6MjJQTSAtMDYwMCwgSmFzb24gR3VudGhvcnBl IHdyb3RlOgo+IE9uIFR1ZSwgSnVuIDE4LCAyMDEzIGF0IDA1OjQ5OjExUE0gLTAzMDAsIEV6ZXF1 aWVsIEdhcmNpYSB3cm90ZToKPiA+IEhpIFNlYmFzdGlhbiwKPiA+IAo+ID4gWW91IGxvb3NlICsx IGludGVybmV0IHBvaW50cyBmb3IgZHJvcHBpbmcgbWUgZnJvbSBDYyA7LSkKPiA+IAo+ID4gT24g VHVlLCBKdW4gMTgsIDIwMTMgYXQgMDk6Mjc6MjhQTSArMDIwMCwgU2ViYXN0aWFuIEhlc3NlbGJh cnRoIHdyb3RlOgo+ID4gPiBPbiAwNi8xOC8yMDEzIDA5OjEwIFBNLCBKYXNvbiBHdW50aG9ycGUg d3JvdGU6Cj4gPiA+ID4KPiA+ID4gPiBUaGUgZm9ybXMgY291bGQgYmU6Cj4gPiA+ID4KPiA+ID4g PiAgIDBJQUEwMDAwCj4gPiA+ID4gICBGSzAwMDAwMAo+ID4gPiA+ICAgICAtIEs9MCAtPiAgaW50 ZXJuYWwgcmVncwo+ID4gPiA+ICAgICAtIEs9MSAtPiAgUENJLUUgdGhpbmd5Cj4gPiA+ID4gICAg ICBldGMKPiA+ID4gPiAgIDFJSUFBMDAwIChmdXR1cmUgZXhwYW5zaW9uIGV4YW1wbGUpCj4gPiA+ IAo+ID4gPiBPaywgZ290IGl0LiBBbnkgZW5jb2RpbmcgaXMgZmluZSB0aGF0IGFsbG93cyB0byBk aXN0aW5ndWlzaCByZWFsCj4gPiA+IHJlbWFwIHdpbmRvd3MgYW5kIGZha2Ugb25lcy4gSSBhc3N1 bWVkIHRoYXQgbWF5YmUgc29tZWRheSB0aGVyZQo+ID4gPiB3aWxsIGJlIG1vcmUgdGhhbiA0YiB0 YXJnZXQgaWQgc28gMHg4MCBhcyBzcGVjaWFsIGNhc2UgaW5kaWNhdG9yCj4gPiA+IGxlYXZlcyA3 YiBvZiBub3JtYWwgdGFyZ2V0IGlkIGluIHRoZSBfY3VycmVudF8gbWFwcGluZy4KPiA+ID4gCj4g PiAKPiA+IEknbSBhbHNvIHdvbmRlcmluZyBhYm91dCB3aHkgd2Ugbm90IGNhcmUgYWJvdXQgdGFy Z2V0IElEcyBiZWluZyBtb3JlCj4gPiB0aGFuIDQgYml0cy4KPiA+IAo+ID4gSmFzb246IChJJ20g Y2hlY2tpbmcgbm93IGJ1dCBwZXJoYXBzIHlvdSBrbm93IGJldHRlciB0aGFuIG1lKToKPiA+IElz IHRoZXJlIGFueSBNQnVzLWFyY2hpdGVjdHVyYWwgcmVhc29uIGZvciB5b3UgYXNzdXJpbmcgdGhl Cj4gPiB0YXJnZXQgSUQgd2lsbCBhbHdheXMgYmUgd2l0aGluIDQtYml0cz8KPiAKPiBUaGUgbWFu dWFscyBJIGhhdmUgZm9yIHRoZSByZWdpc3RlciBzZXQgc2F5IDQgYml0cyBpcyBhbGxvY2F0ZWQg Zm9yCj4gbWJ1cyB0YXJnZXRzLi4gQXJlIHlvdXJzIGRpZmZlcmVudD8KPiAKCk5vcGUuIDQgYml0 cyBoZXJlIGFzIHdlbGwuCgo+IElmIHRoZXkgY2hhbmdlIGl0IHRoZW4gdGhlIHdpbmRvdyByZWdp c3RlciBsYXlvdXQgY2hhbmdlcyBhbmQgdGhlbiB0aGUKPiBtYnVzIGRyaXZlciBwcm9iYWJseSBu ZWVkIHRvIGNoYW5nZSBhcyB3ZWxsLgo+IAoKT2tleSwgdGhlbi4KCi0tIApFemVxdWllbCBHYXJj w61hLCBGcmVlIEVsZWN0cm9ucwpFbWJlZGRlZCBMaW51eCwgS2VybmVsIGFuZCBBbmRyb2lkIEVu Z2luZWVyaW5nCmh0dHA6Ly9mcmVlLWVsZWN0cm9ucy5jb20KX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX18KZGV2aWNldHJlZS1kaXNjdXNzIG1haWxpbmcgbGlz dApkZXZpY2V0cmVlLWRpc2N1c3NAbGlzdHMub3psYWJzLm9yZwpodHRwczovL2xpc3RzLm96bGFi cy5vcmcvbGlzdGluZm8vZGV2aWNldHJlZS1kaXNjdXNzCg==