[U-Boot] [PATCH 3/3] ARM: at91: change nand flash table

[Tom Rini <trini@ti.com>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 02/21/2013 01:58 AM, Bo Shen wrote:
> Hi Tom,
> 
> On 2/21/2013 10:05, Tom Rini wrote:
>> On Wed, Feb 20, 2013 at 06:16:25PM +0800, Bo Shen wrote:
>> 
>>> Change nand flash partition tablke according to
>>> www.at91.com/linux4sam
>>> 
>>> more information: 
>>> http://www.at91.com/linux4sam/bin/view/Linux4SAM/GettingStarted#Linux4SAM_NandFlash_demo_Memory
>>>
>>>
>>>
>>> 
Signed-off-by: Bo Shen <voice.shen@atmel.com>
>> 
>> Question.  Is there a reason to not being doing the MTDIDS /
>> MTDPARTS and adding CONFIG_CMD_MTDPARTS ?
> 
> Yes, MTDIDS/MTDPARTS is missing, however, we can set mtdids and
> mtdpart environment to use mtdparts command. (Maybe this is not a
> good idea, will consider to add MTDIDS / MTDPARTS later, not in
> this patch series.)

Right.  But without CONFIG_CMD_MTDPARTS you don't have the mtdparts
commands / infrastructure in U-Boot :)

>> Doing that should also "just" enable the nand info to be passed
>> in the device tree.
> 
> I am not fully understand this. You means add MTDIDS / MTDPARTS, it
> can automatically pass the nand info to device tree, not through
> the command line, am I right?

I was mis-remembering the code.  If you have
CONFIG_FDT_FIXUP_PARTITIONS set and the board support code calls
fdt_fixup_mtdparts then yes, the mtdparts info will be put into the
device tree and then CONFIG_MTD_OF_PARTS in the kernel should take
that and run with it.  At least I think that's all there is to it,
I've meant to, but not had a chance, to try that all out again.
Perhaps soon tho.

- -- 
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQIcBAEBAgAGBQJRJi4EAAoJENk4IS6UOR1WDGoP/ineyNosxYT2e1yLIWNKDeI6
S+tTcYU6fLyI/0rB9mSfALXCbWVaVq6NVmkpwwKBYZhOSOAt7QiQTPI4wVRVnK85
801qUEIBral3NKeSULkQJ/Nk22SjwRWu+lURksuHISkPuXNKMve0R66pJeZYR1Eo
mLCEW+yI+ugnoNJgUCOZNFBTdsbMQzScSCRzJ5MEorHS2xBuygZxfHpw8CKKjlbs
sEENBtc5bXqmZ33n0Yw1Hy08eAsySowSqo3z4OOFNvILq2Jh0RvIrYXQ04W8Uakr
GAwXV3kPLbbAaZ4r7nIfWnIdAq0QWB/AK0N5EaXReiW+k3BJbvcS+0Aha+o1RlWs
Cxho2n3UGuuPOB792ioobbd74NuoLGZR5arj14nrD5mI2edh7p9jd0yiYAW61Lri
+D+iE2JVxsAfj9qWy+1B6Ha8b7VmnR8qnsLgXiAFtoHG546M5aINJlr1wLAKsiDc
CGNeFi1Xh4StApcOOzq53NfNxoCAPhr7c/0Zw0wkFbyztUrWPjqmp07HocRBINWl
jFqiiSrxIskeQe56qd8nc2u7wFw/Of7kDBbMiVqJ24mQwM8pp0LQLLXdRdi9Xj53
mXvl9GtiFAXPaoOkLA5X5M4cySC3yhftoHAd94/Soj6B1cRajhYXIGEL5+ZX5wRx
Jg15Mk+3nt2gFXfbXnPy
=wEzg
-----END PGP SIGNATURE-----