[U-Boot] [PATCH v3 4/7] tegra: fdt: Add NAND controller binding and definitions
Simon Glass
sjg at chromium.org
Tue Apr 17 22:18:31 CEST 2012
+Jim, who wrote the driver originally
Hi Scott,
On Tue, Apr 17, 2012 at 12:06 PM, Scott Wood <scottwood at freescale.com> wrote:
> On 04/17/2012 01:50 PM, Simon Glass wrote:
>> diff --git a/doc/device-tree-bindings/nand/nvidia,tegra20-nand.txt b/doc/device-tree-bindings/nand/nvidia,tegra20-nand.txt
>> new file mode 100644
>> index 0000000..2484556
>> --- /dev/null
>> +++ b/doc/device-tree-bindings/nand/nvidia,tegra20-nand.txt
>> @@ -0,0 +1,68 @@
>> +NAND Flash
>> +----------
>> +
>> +(there isn't yet a generic binding in Linux, so this describes what is in
>> +U-Boot. There should not be Linux-specific or U-Boot specific binding, just
>> +a binding that describes this hardware. But agreeing a binding in Linux in
>> +the absence of a driver may be beyond my powers.)
>> +
>> +The device node for a NAND flash device is as follows:
>> +
>> +Required properties :
>> + - compatible : Should be "manufacturer,device", "nand-flash"
>
> Again, "nand-flash" is not an appropriate compatible. There is no
> generic nand-flash binding.
OK, so I should just drop this.
>
>> + - nvidia,page-data-bytes : Number of bytes in the data area
>> + - nvidia,page-spare-bytes : Number of bytes in spare area
>> + spare area = skipped-spare-bytes + data-ecc-bytes + tag-bytes
>> + + tag-ecc-bytes
>
> Do you really need this stuff to be in the device tree? You should be
> able to determine this information from the ID table.
I suspect so - the driver originally had a lot of CONFIGs for this.
Maybe someone who wants to take it further could do this as part of
supporting ONFI?
I will see if Jim Lin can take another look.
>
>> + - nvidia,skipped-spare-bytes : Number of bytes to skip at start of spare area
>> + (these are typically used for bad block maintenance)
>
> So this binding can't deal with the bad block marker being somewhere
> other than the beginning of the spare area (e.g. 8-bit small page NAND)?
I suppose it depends on what you want to put afterwards.
>
>> + - nvidia,data-ecc-bytes : Number of ECC bytes for data area
>
> Number of ECC bytes per page? Number of ECC bytes per ECC block?
> Number of data bytes per ECC block?
>
>> + - nvidia,tag-bytes :Number of tag bytes in spare area
>
> What are tag bytes?
I know that term from YAFFS, but other than that I am not sure.
>
>> +Nvidia NAND Controller
>> +----------------------
>> +
>> +The device node for a NAND flash controller is as follows:
>> +
>> +Optional properties:
>> +
>> +nvidia,wp-gpios : GPIO of write-protect line, three cells in the format:
>> + phandle, parameter, flags
>
> Doesn't the number of cells depend on the GPIO controller binding?
Yes, but this is the binding Tegra uses.
>
>> +nvidia,nand-width : bus width of the NAND device in bits
>> +
>> + - nvidia,nand-timing : Timing parameters for the NAND. Each is in ns.
>> + Order is: MAX_TRP_TREA, TWB, Max(tCS, tCH, tALS, tALH),
>> + TWHR, Max(tCS, tCH, tALS, tALH), TWH, TWP, TRH, TADL
>
> Might want to point out that there's one cell per timing parameter.
OK.
I'm going to wait and see if Jim can pick up this driver and make the
device-tree change you requested.
>
> -Scott
>
Regards,
Simon
More information about the U-Boot
mailing list