From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Packham Date: Wed, 26 Feb 2020 06:52:01 +0000 Subject: [PATCH] arm: mvebu: update RTC values for PCIe memory wrappers In-Reply-To: <41312d93e1bdf4db55d81e9cd1c17a7073ea8a81.camel@alliedtelesis.co.nz> References: <20200226000249.20480-1-judge.packham@gmail.com> <87k14aneaq.fsf@tarshish> <41312d93e1bdf4db55d81e9cd1c17a7073ea8a81.camel@alliedtelesis.co.nz> Message-ID: <04b26a4c6b68a902266bd2160cbc3371eede2e9f.camel@alliedtelesis.co.nz> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, 2020-02-26 at 19:46 +1300, Chris Packham wrote: > Hi Baruch, > > On Wed, 2020-02-26 at 06:49 +0200, Baruch Siach wrote: > > Hi Chris, > > > > On Wed, Feb 26 2020, Chris Packham wrote: > > > From: Chris Packham > > > > > > Update the RTC (Read Timing Control) values for PCIe memory > > > wrappers > > > following an ERRATA (ERRATA# TDB). This means the PCIe accesses > > > will > > > used slower memory Read Timing, to allow more efficient energy > > > consumption, in order to lower the minimum VDD of the > > > memory. Will > > > lead > > > to more robust memory when voltage drop occurs (VDDSEG) > > > > Have you seen memory access reliability problems because of this > > issue? > > > > We're seeing a few different symptoms but they all consist of a PCIe > transfer failing in some way (as far as we can tell in the TX > direction > from the CPU). > > > > The code is based on changes from Marvell's U-Boot, specifically: > > > > > > > > https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/commit/20cd2704072512de176e048970f2883db901674b > > > > > https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/commit/eb608a7c8dd0d42b87601a61b9c0cc5615ab94b2 > > > > > https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/commit/c4af19ae2bf08cf6e450e741ce4f04d402a5cb6b > > > > > https://github.com/MarvellEmbeddedProcessors/u-boot-marvell/commit/c4af19ae2bf08cf6e450e741ce4f04d402a5cb6b > > > > The last link looks duplicated. > > Hmm I was pretty sure there were 4 commits. I'll double check and > either fix or remove the link. Ah. There are 4 patches, but the last one isn't in the github repo for some reason. I'll drop the link. > > > > > baruch > >