[U-Boot] [PATCH] Flex-OneNAND driver

[Amit Kumar Sharma]

Hi

We can provide two device registration for SLC and MLC are 
but I don't know how useful it is because FlexOneNand 
provides boundary settings  and user can configure SLC and 
MLC area and other point is still device registration is 
separate how user will get any gain as it will be on same 
die.

Regards
Amit


----- Original Message ----- 
From: "Scott Wood" <scottwood at freescale.com>
To: "apgmoorthy" <moorthy.apg at samsung.com>
Cc: <u-boot at lists.denx.de>
Sent: Thursday, October 23, 2008 4:39 AM
Subject: Re: [U-Boot] [PATCH] Flex-OneNAND driver


> On Mon, Sep 22, 2008 at 11:58:51AM +0530, apgmoorthy 
> wrote:
>> Hi All,
>> This patch adds support for Samsung Flex-OneNAND devices.
>>
>> Flex-OneNAND combines SLC and MLC technologies into a 
>> single
>> device. SLC area provides increased reliability and 
>> speed, suitable
>> for storing code and data, such as bootloader, kernel
>> and root file system. MLC area provides high density and 
>> is best used
>> for storing user data. Users can configure the size of 
>> SLC and MLC
>> regions through 'onenand setboundary' command.
>>
>> Signed-off-by: Rohit Hagargundgi <h.rohit at samsung.com>
>
> Sorry for the late reply...
>
>>  extern struct mtd_info onenand_mtd;
>>  extern struct onenand_chip onenand_chip;
>> +loff_t flexonenand_get_addr(int block)
>
> Space before function declarations.
>
>> + for (block = start; block <= end; block++) {
>> + if (FLEXONENAND(this))
>> + instr.addr = flexonenand_get_addr(block);
>> + else
>> + instr.addr = block << onenand_chip.erase_shift;
>> +
>> + if (FLEXONENAND(this) && (mtd->numeraseregions > 1)) {
>> + for (i = 0; i < mtd->numeraseregions &&
>> + mtd->eraseregions[i].offset <= instr.addr;
>> i++)
>
> Patch is line-wrapped.
>
> Can some of this be abstracted through the driver 
> interface, rather than
> putting a bunch of stuff into what should be a relatively 
> straightforward
> command-line wrapper?
>
> Perhaps the two regions should be exposed as separate 
> devices.
>
> -Scott
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot