From: Hans de Goede <hdegoede@redhat.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 5/5] nand: sunxi: And a20_nandread command utilizing spl nand read driver
Date: Tue, 05 May 2015 11:45:43 +0200 [thread overview]
Message-ID: <55489147.8060603@redhat.com> (raw)
In-Reply-To: <87zj5jtmca.fsf@hellsgate.pl>
Hi,
On 05-05-15 11:17, Daniel Kochma?ski wrote:
>
> Hans de Goede writes:
>
>> Hi,
>>
>> On 05/02/2015 04:26 PM, Ian Campbell wrote:
>>> On Wed, 2015-04-29 at 17:03 +0200, Daniel Kochma?ski wrote:
>>>> This patch adds a20_nandread command. It is simple function utilizing
>>>> function from SPL nand driver `nand_spl_load_image`.
>>>>
>>>> Usage: a20_nandread <address> <offset> <bytes>
>>>
>>> This stuff should really be integrated with cmd_nand.c rather than
>>> adding an adhoc SoC specific interface.
>>
>> I want to go even further and I see that this really needs to be integrated
>> with the blkdev code so that one can just do:
>>
>> ls nand 0:1
>>
>> And things will just work, these way we can also just have the standard
>> boot cmds from config_distro_bootcmd.h work rather then needing special
>> nand boot commands.
>>
>> Regards,
>>
>> Hans
>>
>
> It was more a convenient function then proper integration.
I understand but for upstream nand support we really want a proper solution,
note that I'm find with having the nand support being read only, but it
should really be integrated enough that things like:
ls nand 0:1
Will work.
> I'll look
> into that, but it may take time - I'm quite unfamiliar with U-Boot nand
> interface.
Thanks!
Regards,
Hans
next prev parent reply other threads:[~2015-05-05 9:45 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-29 15:02 [U-Boot] [PATCH 0/5] nand: sunxi: Add SPL support for booting from NAND Daniel Kochmański
2015-04-29 15:02 ` [U-Boot] [PATCH 1/5] nand: sunxi: change BLOCK_SIZE in mksunxiboot to match NAND block size Daniel Kochmański
2015-05-02 14:08 ` Ian Campbell
2015-05-05 9:02 ` Daniel Kochmański
2015-05-09 13:51 ` Ian Campbell
2015-04-29 15:02 ` [U-Boot] [PATCH 2/5] nand: sunxi: Add support for booting from internal NAND memory Daniel Kochmański
2015-05-02 14:21 ` Ian Campbell
2015-05-05 9:14 ` Daniel Kochmański
2015-05-05 9:19 ` Daniel Kochmański
2015-05-09 13:53 ` Ian Campbell
2015-05-09 14:33 ` Hans de Goede
2015-05-18 23:47 ` Scott Wood
2015-04-29 15:02 ` [U-Boot] [PATCH 3/5] nand: sunxi: Add secondary U-Boot offset on second syndrome partition Daniel Kochmański
2015-05-02 14:24 ` Ian Campbell
2015-05-05 14:21 ` Tim Harvey
2015-05-05 14:34 ` Daniel Kochmański
2015-05-18 23:10 ` Scott Wood
2015-04-29 15:03 ` [U-Boot] [PATCH 4/5] nand: sunxi: Add multiimage preload option Daniel Kochmański
2015-05-02 14:24 ` Ian Campbell
2015-05-18 23:43 ` Scott Wood
2015-04-29 15:03 ` [U-Boot] [PATCH 5/5] nand: sunxi: And a20_nandread command utilizing spl nand read driver Daniel Kochmański
2015-05-02 14:26 ` Ian Campbell
2015-05-04 14:20 ` Hans de Goede
2015-05-05 9:17 ` Daniel Kochmański
2015-05-05 9:45 ` Hans de Goede [this message]
2015-05-18 23:52 ` Scott Wood
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=55489147.8060603@redhat.com \
--to=hdegoede@redhat.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.