Re: [U-Boot] [PATCH] Fix block device accesses beyond 2TiB

2013-06-24 Thread Sascha Silbe
Frédéric Leroy writes: > Le 22/06/2013 17:31, Albert ARIBAUD a écrit : >> > However, the patch issues no warning at compile time and fixes the ide >> > block layer. >> >> OK -- anyone has any idea why Sasha's patch fixes reading from "far" >> blocks but does not fix ex2ls? Frankly, I'd prefer it

Re: [U-Boot] [PATCH] Fix block device accesses beyond 2TiB

2013-06-24 Thread Frédéric Leroy
Le 22/06/2013 17:31, Albert ARIBAUD a écrit : > Hi Frédéric, > > > However, the patch issues no warning at compile time and fixes the ide > > block layer. > > OK -- anyone has any idea why Sasha's patch fixes reading from "far" > blocks but does not fix ex2ls? Frankly, I'd prefer it if the patch >

Re: [U-Boot] [PATCH] Fix block device accesses beyond 2TiB

2013-06-22 Thread Albert ARIBAUD
Hi Frédéric, On Sat, 22 Jun 2013 14:29:26 +0200, Frédéric Leroy wrote: > Hello Albert, Sascha, Marek > > Le 22/06/2013 12:07, Albert ARIBAUD a écrit : > > > Quick review looks OK. > > > > > > Best regards, > > > Marek Vasut > > > > Anyone could test Sascha's patch? Especially Frédédic, can you

Re: [U-Boot] [PATCH] Fix block device accesses beyond 2TiB

2013-06-22 Thread Albert ARIBAUD
On Mon, 17 Jun 2013 22:26:00 +0200, Marek Vasut wrote: > Hello Sascha, > > > With CONFIG_SYS_64BIT_LBA, lbaint_t gets defined as a 64-bit type, > > which is required to represent block numbers for storage devices that > > exceed 2TiB (the block size usually is 512B), e.g. recent hard drives. > >

Re: [U-Boot] [PATCH] Fix block device accesses beyond 2TiB

2013-06-17 Thread Marek Vasut
Hello Sascha, > With CONFIG_SYS_64BIT_LBA, lbaint_t gets defined as a 64-bit type, > which is required to represent block numbers for storage devices that > exceed 2TiB (the block size usually is 512B), e.g. recent hard drives. > > For some obscure reason, the current U-Boot code uses lbaint_t fo

[U-Boot] [PATCH] Fix block device accesses beyond 2TiB

2013-06-14 Thread Sascha Silbe
With CONFIG_SYS_64BIT_LBA, lbaint_t gets defined as a 64-bit type, which is required to represent block numbers for storage devices that exceed 2TiB (the block size usually is 512B), e.g. recent hard drives. For some obscure reason, the current U-Boot code uses lbaint_t for the number of blocks to