From mboxrd@z Thu Jan 1 00:00:00 1970 From: Philipp Tomsich Subject: Re: [PATCH 00/92] ram: rk3399: Add LPDDR4 support Date: Wed, 12 Jun 2019 17:40:48 +0200 Message-ID: <50A468C5-717E-46F8-B015-1F99A1742AE5@theobroma-systems.com> References: <20190611145135.21399-1-jagan@amarulasolutions.com> <349AB0AF-4DEE-4BC8-8FEF-B3DBD34564E4@theobroma-systems.com> Mime-Version: 1.0 (Mac OS X Mail 11.5 \(3445.9.1\)) Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" To: Jagan Teki Cc: YouMin Chen , U-Boot-Denx , Akash Gajjar , "open list:ARM/Rockchip SoC..." , linux-amarula List-Id: linux-rockchip.vger.kernel.org Cgo+IE9uIDEyLjA2LjIwMTksIGF0IDE3OjMwLCBKYWdhbiBUZWtpIDxqYWdhbkBhbWFydWxhc29s dXRpb25zLmNvbT4gd3JvdGU6Cj4gCj4gT24gVHVlLCBKdW4gMTEsIDIwMTkgYXQgODozNiBQTSBQ aGlsaXBwIFRvbXNpY2gKPiA8cGhpbGlwcC50b21zaWNoQHRoZW9icm9tYS1zeXN0ZW1zLmNvbSA8 bWFpbHRvOnBoaWxpcHAudG9tc2ljaEB0aGVvYnJvbWEtc3lzdGVtcy5jb20+PiB3cm90ZToKPj4g Cj4+IAo+PiAKPj4+IE9uIDExLjA2LjIwMTksIGF0IDE3OjAzLCBKYWdhbiBUZWtpIDxqYWdhbkBh bWFydWxhc29sdXRpb25zLmNvbT4gd3JvdGU6Cj4+PiAKPj4+IE9uIFR1ZSwgSnVuIDExLCAyMDE5 IGF0IDg6MjMgUE0gUGhpbGlwcCBUb21zaWNoCj4+PiA8cGhpbGlwcC50b21zaWNoQHRoZW9icm9t YS1zeXN0ZW1zLmNvbT4gd3JvdGU6Cj4+Pj4gCj4+Pj4gCj4+Pj4gCj4+Pj4+IE9uIDExLjA2LjIw MTksIGF0IDE2OjUwLCBKYWdhbiBUZWtpIDxqYWdhbkBhbWFydWxhc29sdXRpb25zLmNvbT4gd3Jv dGU6Cj4+Pj4+IAo+Pj4+PiBZZXMsIGl0IGNhbiBiZSBwb3NzaWJsZSB0byBicmVhayB0aGlzIHNl cmllcyBpbnRvIG11bHRpcGxlIHN1YiBzZXJpZXMKPj4+Pj4gYnV0IGlkZWEgaGVyZSBpcyB0byBt YXJrIGFsbCB0aGUgcmVxdWlyZWQgY2hhbmdlcyB0byBzdXBwb3J0IExQRERSNAo+Pj4+PiBpbiBy azMzOTkgaW4gb25lIHNldC4gaWYgcmVxdWlyZWQgd2UgY2FuIGJyZWFrIGl0IGZyb20gbmV4dCB2 ZXJzaW9ucy4KPj4+Pj4gCj4+Pj4+IFRoaXMgaXMgdGhlIGluaXRpYWwgc2V0IGZvciBzdXBwb3J0 aW5nIExQRERSNCB3aXRoIGFzc29jaWF0ZWQKPj4+Pj4gZmVhdHVyZXMuCj4+Pj4+IAo+Pj4+PiBU aGFua3MgdG8KPj4+Pj4gLSBZb3VNaW4gQ2hlbgo+Pj4+PiAtIEFrYXNoIEdhamphcgo+Pj4+PiAt IEtldmVyIFlhbmcKPj4+Pj4gZm9yIHN1cHBvcnRpbmcgYWxsIHRoZSBoZWxwIG9uIHRoaXMgd29y ay4KPj4+Pj4gCj4+Pj4+IE9uIHN1bW1hcnkgdGhpcyBzZXJpZXMgc3VwcG9ydAo+Pj4+PiAtIENv ZGUgd2FybmluZyBhbmQgZml4ZXMKPj4+Pj4gLSByYW5rIGRldGVjdGlvbiwgdGhpcyB3b3VsZCBy ZXF1aXJlZCB0byBwcm9iZSBzaW5nbGUgY2hhbm5lbAo+Pj4+PiBzZHJhbSBjb25maWd1cmVkIGlu IE5hbm9QSS1ORU80Cj4+Pj4+IC0gTFBERFI0IHN1cHBvcnQsIHRlc3RlZCBpbiBSb2NrcHJvNjQg YW5kIFJvY2stUEktNAo+Pj4+PiAKPj4+Pj4gcGF0Y2ggMDAwMSAtIDAwMzM6IGZpeCBjb2RlIHdh cm5pbmdzLCBwcmludHMsIG5ldyBtYWNyb3MKPj4+Pj4gCj4+Pj4+IHBhdGNoIDAwMzQgLSAwMDUx OiByYW5rIGRldGVjdGlvbiwgc2RyYW0gZGVidWcgY29kZQo+Pj4+PiAKPj4+Pj4gcGF0Y2ggMDA1 MjogVXNlIEREUjMtMTgwMCBvbiBOYW5vUEktTkVPNAo+Pj4+PiAKPj4+Pj4gcGF0Y2ggMDA1MyAt IDAwODk6IGxwZGRyNCBzdXBwb3J0Cj4+Pj4+IAo+Pj4+PiBwYXRjaCAwMDkwOiBMUEREUjQtMTAw IHRpbWluZ3MKPj4+Pj4gCj4+Pj4+IHBhdGNoIDAwOTE6IFVzZSBMUEREUjQtMTAwIG9uIFJvY2tw cm82NAo+Pj4+PiAKPj4+Pj4gcGF0Y2ggMDA5MjogVXNlIExQRERSNC0xMDAgb24gUm9jay1QSSA0 Cj4+Pj4+IAo+Pj4+PiBOb3RlOiBQdW1hIHJrMzM5OSBoYXMgU1BMIHNpemUgb3ZlcmZsb3csIGJl dHRlciB0byBlbmFibGUgVFBMCj4+Pj4+IGZvciB0aGlzIGJvYXJkLgo+Pj4+IAo+Pj4+IFdlIG5l ZWQgdG8ga2VlcCBQdW1hIG9uIGEgU1BMLW9ubHkgY29uZmlndXJhdGlvbiBmb3IgdGhlIHRpbWUg YmVpbmcuCj4+Pj4gUGxlYXNlIG1ha2Ugc3VyZSB0aGF0IHRoZSBMUEREUjQgY29kZSBpcyBhbiBv cHRpb25hbCBmZWF0dXJlIHRoYXQgZG9lcyBub3QKPj4+PiBpbmNyZWFzZSB0aGUgRFJBTS1kcml2 ZXIgc2l6ZSBmb3IgYm9hcmRzIHRoYXQgZG9u4oCZdCBuZWVkL3dhbnQgaXQuCj4+PiAKPj4+IFdl IGhhdmUgZmV3IGJvYXJkcyBkbyBoYXZlIFRQTC1ydW5uYWJsZSwgd291bGQgYmUgYW55IHRlY2hu aWNhbCBpc3N1ZQo+Pj4gdG8gc3dpdGNoIHB1bWEgdG8gVFBMPyBiZWNhdXNlIHdlIGhhdmUgbHBk ZHI0IGNvZGUgcGFydCBvZiBleGlzdGluZwo+Pj4gZHJpdmVyIGl0c2VsZiBhbmQgaXQgcmVxdWly ZSBleHRyYSBpZmRlZiB0byBjb25zaWRlciB3aGljaCBpbmRlZWQgbG9vawo+Pj4gYXdmdWwgZnJv bSBjb2RlIHBvaW50LW9mLXZpZXcuCj4+IAo+PiBPdXIgc2VjdXJlIGJvb3QgcHJvY2VzcyAoaS5l LiBzaWduaW5nIHRvb2xzKSBjdXJyZW50bHkgZGVwZW5kcyBvbiB0aGlzIGFuZAo+PiB0aGUgY2hh bmdlb3ZlciB3b27igJl0IGJlIHF1aWNr4oCmCj4gCj4gTm90IHNvIHF1aWNrLCB3ZSBoYXZlIHRp bWUgdGlsbCBNVy4gaXNuJ3QgaXQgcG9zc2libGU/IGVuYWJsaW5nIHNlY3VyZQo+IHRvb2xzIGlu IGJvdGggVFBMIGFuZCBTUEwgb3IgVFBMLWFsb25lIHdvdWxkIGJlIG1lYW5pbmdmdWwgdHJhaWwu IHdoYXQKPiBkbyB5b3UgdGhpbms/CgpXZSBhcmVu4oCZdCB0YWxraW5nIGFib3V0IGEgc2luZ2xl IE1XIGhlcmUsIGdpdmVuIHRoYXQgc3VtbWVyIGlzIHN0YXJ0aW5nIHRvIGVhdAp1cCBzb21lIG9m IG15IHJlc291cmNlc+KApgoKVGhhbmtzLApQaGlsLgoKX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX18KVS1Cb290IG1haWxpbmcgbGlzdApVLUJvb3RAbGlzdHMu ZGVueC5kZQpodHRwczovL2xpc3RzLmRlbnguZGUvbGlzdGluZm8vdS1ib290Cg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: Philipp Tomsich Date: Wed, 12 Jun 2019 17:40:48 +0200 Subject: [U-Boot] [PATCH 00/92] ram: rk3399: Add LPDDR4 support In-Reply-To: References: <20190611145135.21399-1-jagan@amarulasolutions.com> <349AB0AF-4DEE-4BC8-8FEF-B3DBD34564E4@theobroma-systems.com> Message-ID: <50A468C5-717E-46F8-B015-1F99A1742AE5@theobroma-systems.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: u-boot@lists.denx.de > On 12.06.2019, at 17:30, Jagan Teki wrote: > > On Tue, Jun 11, 2019 at 8:36 PM Philipp Tomsich > > wrote: >> >> >> >>> On 11.06.2019, at 17:03, Jagan Teki wrote: >>> >>> On Tue, Jun 11, 2019 at 8:23 PM Philipp Tomsich >>> wrote: >>>> >>>> >>>> >>>>> On 11.06.2019, at 16:50, Jagan Teki wrote: >>>>> >>>>> Yes, it can be possible to break this series into multiple sub series >>>>> but idea here is to mark all the required changes to support LPDDR4 >>>>> in rk3399 in one set. if required we can break it from next versions. >>>>> >>>>> This is the initial set for supporting LPDDR4 with associated >>>>> features. >>>>> >>>>> Thanks to >>>>> - YouMin Chen >>>>> - Akash Gajjar >>>>> - Kever Yang >>>>> for supporting all the help on this work. >>>>> >>>>> On summary this series support >>>>> - Code warning and fixes >>>>> - rank detection, this would required to probe single channel >>>>> sdram configured in NanoPI-NEO4 >>>>> - LPDDR4 support, tested in Rockpro64 and Rock-PI-4 >>>>> >>>>> patch 0001 - 0033: fix code warnings, prints, new macros >>>>> >>>>> patch 0034 - 0051: rank detection, sdram debug code >>>>> >>>>> patch 0052: Use DDR3-1800 on NanoPI-NEO4 >>>>> >>>>> patch 0053 - 0089: lpddr4 support >>>>> >>>>> patch 0090: LPDDR4-100 timings >>>>> >>>>> patch 0091: Use LPDDR4-100 on Rockpro64 >>>>> >>>>> patch 0092: Use LPDDR4-100 on Rock-PI 4 >>>>> >>>>> Note: Puma rk3399 has SPL size overflow, better to enable TPL >>>>> for this board. >>>> >>>> We need to keep Puma on a SPL-only configuration for the time being. >>>> Please make sure that the LPDDR4 code is an optional feature that does not >>>> increase the DRAM-driver size for boards that don’t need/want it. >>> >>> We have few boards do have TPL-runnable, would be any technical issue >>> to switch puma to TPL? because we have lpddr4 code part of existing >>> driver itself and it require extra ifdef to consider which indeed look >>> awful from code point-of-view. >> >> Our secure boot process (i.e. signing tools) currently depends on this and >> the changeover won’t be quick… > > Not so quick, we have time till MW. isn't it possible? enabling secure > tools in both TPL and SPL or TPL-alone would be meaningful trail. what > do you think? We aren’t talking about a single MW here, given that summer is starting to eat up some of my resources… Thanks, Phil.