On Tue, Jul 20, 2010 at 4:22 AM, Scott Wood <scottw...@freescale.com> wrote:
> On Mon, Jul 19, 2010 at 09:05:05PM +0800, Lei Wen wrote:
>> Rebased version for this nand chip has one problem in detecting its
>> page size using in nand_base.c.
>> If we set page size 0 in nand_ids.c, we would get the calculation
>> result as page size 2048, while the true page size is 4096.
>
> Is the ID data bad, or is there a bug in nand_get_flash_type(), or is it
> some new ID format that needs support?

Samsung seem modify the ID name rule, which make the calculation
method nand_get_flash_type doesn't work.
In mainline linux code, it give a hardcode way to identify this type
of nand with new calculation method.

>
>> I think it is reasonable to set the 32Gigabit nand as 4k page by
>> default, as all nand been product in this size has
>> more than 4k page.
>
> "more than"?  Are there larger page size NANDs out there?

8k page nand exists, but they are for even larger nand, like 64Gb, or 128Gb.

>
> And if so, wouldn't hard-coding the page size at 4096 bytes be a problem?
>
> Is there zero possibility that a 2k page NAND in this size could be made in
> the future (e.g. for compatibility with controllers that don't support 4k
> pages)?

Maybe we could post another patch for that kind of nand?...
>
> -Scott
>
>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to