[U-Boot] [U-Boot, v6, 4/9] nand: Extend nand_(read|write)_skip_bad with *actual and limit parameters

Igor Grinberg grinberg at compulab.co.il
Thu Apr 11 08:37:48 CEST 2013


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 04/10/13 16:08, Tom Rini wrote:
> On Thu, Mar 14, 2013 at 05:32:50AM -0000, Tom Rini wrote:
> 
>> We make these two functions take a size_t pointer to how much space
>> was used on NAND to read or write the buffer (when reads/writes happen)
>> so that bad blocks can be accounted for.  We also make them take an
>> loff_t limit on how much data can be read or written.  This means that
>> we can now catch the case of when writing to a partition would exceed
>> the partition size due to bad blocks.  To do this we also need to make
>> check_skip_len count not just complete blocks used but partial ones as
>> well.  All callers of nand_(read|write)_skip_bad are adjusted to call
>> these with the most sensible limits available.
>>
>> The changes were started by Pantelis and finished by Tom.
>>
>> Signed-off-by: Pantelis Antoniou <panto at antoniou-consulting.com>
>> Signed-off-by: Tom Rini <trini at ti.com>
> 
> With current master, the following changes are needed to adapt cm_t35 to
> these call changes:

I guess, you will squash both patches into one, right?

Acked-by: Igor Grinberg <grinberg at compulab.co.il>

> 
> diff --git a/board/cm_t35/cm_t35.c b/board/cm_t35/cm_t35.c
> index 629ce4a..84c36ba 100644
> --- a/board/cm_t35/cm_t35.c
> +++ b/board/cm_t35/cm_t35.c
> @@ -91,6 +91,7 @@ static int splash_load_from_nand(u32 bmp_load_addr)
>  
>  	res = nand_read_skip_bad(&nand_info[nand_curr_device],
>  			splash_screen_nand_offset, &bmp_header_size,
> +			NULL, nand_info[nand_curr_device].size,
>  			(u_char *)bmp_load_addr);
>  	if (res < 0)
>  		return res;
> @@ -103,6 +104,7 @@ static int splash_load_from_nand(u32 bmp_load_addr)
>  
>  	return nand_read_skip_bad(&nand_info[nand_curr_device],
>  			splash_screen_nand_offset, &bmp_size,
> +			NULL, nand_info[nand_curr_device].size,
>  			(u_char *)bmp_load_addr);
>  
>  splash_address_too_high:
> 

- -- 
Regards,
Igor.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.17 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQIcBAEBAgAGBQJRZlo8AAoJEBDE8YO64EfazSoP/1u/f8FQwl7zZDcwlUIWSJPe
nsk66wJxMteHKPG9Ib3DGKNW+ncAEfNYWH7Pzqg9rnsp+oWf/bMpTil/qWV0J8Qp
QSTC3ez5QOYONBn/Zb8MyOyMcHVnbOFrKiHIR+7zjPhaWWRzVAM7VBxiYS/gtGUY
tiK8ccqH9w8r+t4s6fbWZ8WWTX0ZryKcCiGxiNnjFGz5xphsS9+AvTbIL7n2r+ji
mie9SD/Lb6GCeEhgYT9XBgo7gjx6Q0RNEXZL5fP+YWpII73KPFzvzzMTRo2jwWxr
/vggymfjJ3amAFvMuFuIk3OEbH5CSWKnfX5pwyW2TIZ+GZttiFw040ZurmA6k3f9
LdwhFdgrp0BzRkwC92FLR+aMyV8A8WgFYkMysuqDlJkiaSXLVFlcLB43le6kuuyq
uKbRDTMMNiyHncet6znIjgmH3wzYECXlRyAkH7K0mz6DrkmAGraYizUy/XwxK6cr
/Zh0rmR0Owg/XZKZRRdhQnwGf/B9+FufNZehgYJlhOFHVFwm9bbHX02k16RD2PP5
P/AgdV8psCR0HvH9dJ+/93Y0r/paIp3PeAAt4nZxMwiHp07cJwqpjwgQUWDjxPkK
q3R6H0otAh7cDzE84QotNzeHlR1GuxFU6wQLuTE16EDrGE7BHyMaYjuhRXH9zHek
ptNpTuQm5/O7yn6tQHvV
=6ma+
-----END PGP SIGNATURE-----


More information about the U-Boot mailing list