From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Date: Tue, 05 May 2015 11:45:43 +0200 Subject: [U-Boot] [PATCH 5/5] nand: sunxi: And a20_nandread command utilizing spl nand read driver In-Reply-To: <87zj5jtmca.fsf@hellsgate.pl> References: <1430319781-15375-1-git-send-email-dkochmanski@turtle-solutions.eu> <1430319781-15375-6-git-send-email-dkochmanski@turtle-solutions.eu> <1430576790.15640.140.camel@hellion.org.uk> <55478040.2090902@redhat.com> <87zj5jtmca.fsf@hellsgate.pl> Message-ID: <55489147.8060603@redhat.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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
>>> >>> 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