Re: [U-Boot] [PATCH 11/14] sunxi: Fill memory before comparing it when doing dram init on sun6i

2014-12-22 Thread Hans de Goede
Hi, On 22-12-14 15:19, Ian Campbell wrote: On Fri, 2014-12-19 at 17:55 +0100, Hans de Goede wrote: Hi, On 18-12-14 20:12, Ian Campbell wrote: On Tue, 2014-12-16 at 21:31 +0100, Hans de Goede wrote: The sun8i boot0 code fills the DRAM with a "random" pattern before comparing it at different o

Re: [U-Boot] [PATCH 11/14] sunxi: Fill memory before comparing it when doing dram init on sun6i

2014-12-22 Thread Ian Campbell
On Mon, 2014-12-22 at 16:32 +0200, Siarhei Siamashka wrote: > On Mon, 22 Dec 2014 14:19:20 + > Ian Campbell wrote: > > > On Fri, 2014-12-19 at 17:55 +0100, Hans de Goede wrote: > > > Hi, > > > > > > On 18-12-14 20:12, Ian Campbell wrote: > > > > On Tue, 2014-12-16 at 21:31 +0100, Hans de Goe

Re: [U-Boot] [PATCH 11/14] sunxi: Fill memory before comparing it when doing dram init on sun6i

2014-12-22 Thread Siarhei Siamashka
On Mon, 22 Dec 2014 14:19:20 + Ian Campbell wrote: > On Fri, 2014-12-19 at 17:55 +0100, Hans de Goede wrote: > > Hi, > > > > On 18-12-14 20:12, Ian Campbell wrote: > > > On Tue, 2014-12-16 at 21:31 +0100, Hans de Goede wrote: > > >> The sun8i boot0 code fills the DRAM with a "random" pattern

Re: [U-Boot] [PATCH 11/14] sunxi: Fill memory before comparing it when doing dram init on sun6i

2014-12-22 Thread Ian Campbell
On Fri, 2014-12-19 at 17:55 +0100, Hans de Goede wrote: > Hi, > > On 18-12-14 20:12, Ian Campbell wrote: > > On Tue, 2014-12-16 at 21:31 +0100, Hans de Goede wrote: > >> The sun8i boot0 code fills the DRAM with a "random" pattern before > >> comparing > >> it at different offsets to do columns, e

Re: [U-Boot] [PATCH 11/14] sunxi: Fill memory before comparing it when doing dram init on sun6i

2014-12-21 Thread Siarhei Siamashka
On Fri, 19 Dec 2014 17:56:40 +0100 Hans de Goede wrote: > Hi, > > On 19-12-14 11:08, Siarhei Siamashka wrote: > > On Thu, 18 Dec 2014 19:12:13 + > > Ian Campbell wrote: > > > >> On Tue, 2014-12-16 at 21:31 +0100, Hans de Goede wrote: > >>> The sun8i boot0 code fills the DRAM with a "random"

Re: [U-Boot] [PATCH 11/14] sunxi: Fill memory before comparing it when doing dram init on sun6i

2014-12-19 Thread Hans de Goede
Hi, On 19-12-14 11:08, Siarhei Siamashka wrote: On Thu, 18 Dec 2014 19:12:13 + Ian Campbell wrote: On Tue, 2014-12-16 at 21:31 +0100, Hans de Goede wrote: The sun8i boot0 code fills the DRAM with a "random" pattern before comparing it at different offsets to do columns, etc. detection. T

Re: [U-Boot] [PATCH 11/14] sunxi: Fill memory before comparing it when doing dram init on sun6i

2014-12-19 Thread Hans de Goede
Hi, On 18-12-14 20:12, Ian Campbell wrote: On Tue, 2014-12-16 at 21:31 +0100, Hans de Goede wrote: The sun8i boot0 code fills the DRAM with a "random" pattern before comparing it at different offsets to do columns, etc. detection. The sun6i boot0 code does not do it, but it seems like a good id

Re: [U-Boot] [PATCH 11/14] sunxi: Fill memory before comparing it when doing dram init on sun6i

2014-12-19 Thread Siarhei Siamashka
On Thu, 18 Dec 2014 19:12:13 + Ian Campbell wrote: > On Tue, 2014-12-16 at 21:31 +0100, Hans de Goede wrote: > > The sun8i boot0 code fills the DRAM with a "random" pattern before comparing > > it at different offsets to do columns, etc. detection. The sun6i boot0 code > > does not do it, but

Re: [U-Boot] [PATCH 11/14] sunxi: Fill memory before comparing it when doing dram init on sun6i

2014-12-18 Thread Ian Campbell
On Tue, 2014-12-16 at 21:31 +0100, Hans de Goede wrote: > The sun8i boot0 code fills the DRAM with a "random" pattern before comparing > it at different offsets to do columns, etc. detection. The sun6i boot0 code > does not do it, but it seems like a good idea to do this regardless. Is this the ri

[U-Boot] [PATCH 11/14] sunxi: Fill memory before comparing it when doing dram init on sun6i

2014-12-16 Thread Hans de Goede
The sun8i boot0 code fills the DRAM with a "random" pattern before comparing it at different offsets to do columns, etc. detection. The sun6i boot0 code does not do it, but it seems like a good idea to do this regardless. The new mctl_mem_fill function this introduces is added as an inline helper