[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