From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kurt Miller Date: Thu, 26 Sep 2019 08:53:58 -0400 Subject: [U-Boot] =?utf-8?q?rockchip=3A_rk3399=3A_TPL=3A_rockpro64=3A_Wron?= =?utf-8?b?ZyBtZW1vcnkgc2l6ZSBkZXRlY3RlZOOAkOivt+azqOaEj++8jOmCruS7tg==?= =?utf-8?b?55SxdS1ib290LWJvdW5jZXNAbGlzdHMuZGVueC5kZeS7o+WPkeOAkSBkZXRl?= =?utf-8?q?cted?= In-Reply-To: References: <1567028727.8573.69.camel@intricatesoftware.com> <8fe13318-28cc-fd10-98b7-9780531c14f8@rock-chips.com> Message-ID: <1569502438.20259.207.camel@intricatesoftware.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 Thu, 2019-09-19 at 11:03 +0530, Jagan Teki wrote: > Hi Kever, > > On Wed, Sep 18, 2019 at 10:31 AM Jagan Teki wrote: > > > > > > On Wed, Sep 18, 2019 at 9:09 AM Kever Yang wrote: > > > > > > > > > Hi Jagan, > > > > > >      Seems like your and Kurt's board have different DRAM type: > > > > > > - 16bit row + 1 CS > > > > > > - 15bit row + 2 CS > > > > > > Capacity detect function is missing for the driver now, and it's not able > > > > > > to detect the correct size like Kurt is using. > > Got it. let me send the quick check patch, thanks. > Are you looking to fix this for the release? if yes, then we can > create another sdram*.dtsi for this specific dram board otherwise we > can fix this MW. Adding cap detection would alter the existing > behaviour of sdram driver, which indeed a risky to do at this point of > time since we are close to release. Hi Kever, Jagen, Is this being considered for fixing prior to release? My offer to test patches is still available.  Thank you, -Kurt