[PATCH 2/5] MTD: bcm63xxpart: make sure CFE and NVRAM partitions are at least 64K

Jonas Gorski jonas.gorski at gmail.com
Sat Dec 17 08:27:38 EST 2011


Hi Guillaume,

On 17 December 2011 14:13, Guillaume LECERF <glecerf at gmail.com> wrote:
> Hi Jonas,
>
> 2011/12/17 Jonas Gorski <jonas.gorski at gmail.com>:
>> -       spareaddr = roundup(totallen, master->erasesize) + master->erasesize;
>> -       sparelen = master->size - spareaddr - master->erasesize;
>> +       spareaddr = roundup(totallen, master->erasesize) + cfelen;
>
> spareaddr = roundup(totallen, cfelen) + cfelen ?

The intention is to align the spareaddr to the next eraseblock (cfelen
can be bigger than the erasesize). Maybe writing it as

       spareaddr = roundup(cfelen + totallen, master->erasesize);

would be cleaner.


Jonas



More information about the linux-mtd mailing list