* [U-Boot] [PATCH] Fix help text of ext2load and fatload.
@ 2014-07-09 20:40 Pavel Machek
2014-07-09 21:05 ` Marek Vasut
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Pavel Machek @ 2014-07-09 20:40 UTC (permalink / raw)
To: u-boot
Fix help text of ext2load and fatload to match code in fs/fs.c
Signed-off-by: Pavel Machek <pavel@denx.de>
diff --git a/common/cmd_ext2.c b/common/cmd_ext2.c
index 5a4bcc1..2675ae8 100644
--- a/common/cmd_ext2.c
+++ b/common/cmd_ext2.c
@@ -45,7 +45,7 @@ U_BOOT_CMD(
U_BOOT_CMD(
ext2load, 6, 0, do_ext2load,
"load binary file from a Ext2 filesystem",
- "<interface> <dev[:part]> [addr] [filename] [bytes]\n"
+ "<interface> [<dev[:part]> [addr [filename [bytes [pos]]]]]\n"
" - load binary file 'filename' from 'dev' on 'interface'\n"
" to address 'addr' from ext2 filesystem."
);
diff --git a/common/cmd_fat.c b/common/cmd_fat.c
index a12d8fa..b7e743a 100644
--- a/common/cmd_fat.c
+++ b/common/cmd_fat.c
@@ -26,7 +26,7 @@ int do_fat_fsload (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
U_BOOT_CMD(
fatload, 7, 0, do_fat_fsload,
"load binary file from a dos filesystem",
- "<interface> [<dev[:part]>] <addr> <filename> [bytes [pos]]\n"
+ "<interface> [<dev[:part]> [<addr> [<filename> [bytes [pos]]]]]\n"
" - Load binary file 'filename' from 'dev' on 'interface'\n"
" to address 'addr' from dos filesystem.\n"
" 'pos' gives the file position to start loading from.\n"
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
^ permalink raw reply related [flat|nested] 6+ messages in thread* [U-Boot] [PATCH] Fix help text of ext2load and fatload.
2014-07-09 20:40 [U-Boot] [PATCH] Fix help text of ext2load and fatload Pavel Machek
@ 2014-07-09 21:05 ` Marek Vasut
2014-07-09 22:57 ` Pavel Machek
2014-07-14 15:26 ` Tom Rini
2014-07-22 19:22 ` [U-Boot] " Tom Rini
2 siblings, 1 reply; 6+ messages in thread
From: Marek Vasut @ 2014-07-09 21:05 UTC (permalink / raw)
To: u-boot
On Wednesday, July 09, 2014 at 10:40:07 PM, Pavel Machek wrote:
> Fix help text of ext2load and fatload to match code in fs/fs.c
Which code in fs/fs.c do you mean? Are you referring to do_load() ?
Reviewed-by: Marek Vasut <marex@denx.de>
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] Fix help text of ext2load and fatload.
2014-07-09 20:40 [U-Boot] [PATCH] Fix help text of ext2load and fatload Pavel Machek
2014-07-09 21:05 ` Marek Vasut
@ 2014-07-14 15:26 ` Tom Rini
2014-07-15 21:37 ` Pavel Machek
2014-07-22 19:22 ` [U-Boot] " Tom Rini
2 siblings, 1 reply; 6+ messages in thread
From: Tom Rini @ 2014-07-14 15:26 UTC (permalink / raw)
To: u-boot
On Wed, Jul 09, 2014 at 10:40:07PM +0200, Pavel Machek wrote:
>
> Fix help text of ext2load and fatload to match code in fs/fs.c
>
> Signed-off-by: Pavel Machek <pavel@denx.de>
>
> diff --git a/common/cmd_ext2.c b/common/cmd_ext2.c
> index 5a4bcc1..2675ae8 100644
> --- a/common/cmd_ext2.c
> +++ b/common/cmd_ext2.c
> @@ -45,7 +45,7 @@ U_BOOT_CMD(
> U_BOOT_CMD(
> ext2load, 6, 0, do_ext2load,
> "load binary file from a Ext2 filesystem",
> - "<interface> <dev[:part]> [addr] [filename] [bytes]\n"
> + "<interface> [<dev[:part]> [addr [filename [bytes [pos]]]]]\n"
Wait, what? [] is optional, <> is mandatory. With the rest of the
related patches I see you fix a problem about assuming addr but we still
have to pass along a dev at least, yes?
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140714/55183982/attachment.pgp>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] Fix help text of ext2load and fatload.
2014-07-14 15:26 ` Tom Rini
@ 2014-07-15 21:37 ` Pavel Machek
0 siblings, 0 replies; 6+ messages in thread
From: Pavel Machek @ 2014-07-15 21:37 UTC (permalink / raw)
To: u-boot
On Mon 2014-07-14 11:26:32, Tom Rini wrote:
> On Wed, Jul 09, 2014 at 10:40:07PM +0200, Pavel Machek wrote:
> >
> > Fix help text of ext2load and fatload to match code in fs/fs.c
> >
> > Signed-off-by: Pavel Machek <pavel@denx.de>
> >
> > diff --git a/common/cmd_ext2.c b/common/cmd_ext2.c
> > index 5a4bcc1..2675ae8 100644
> > --- a/common/cmd_ext2.c
> > +++ b/common/cmd_ext2.c
> > @@ -45,7 +45,7 @@ U_BOOT_CMD(
> > U_BOOT_CMD(
> > ext2load, 6, 0, do_ext2load,
> > "load binary file from a Ext2 filesystem",
> > - "<interface> <dev[:part]> [addr] [filename] [bytes]\n"
> > + "<interface> [<dev[:part]> [addr [filename [bytes [pos]]]]]\n"
>
> Wait, what? [] is optional, <> is mandatory. With the rest of the
> related patches I see you fix a problem about assuming addr but we still
> have to pass along a dev at least, yes?
Do we?
fs/fs.c:
if (fs_set_blk_dev(argv[1], (argc >= 3) ? argv[2] : NULL, fstype))
return 1;
This tries to handle case where interface is specified but dev is
not. Aha, and it seems to use bootdevice environment variable if
device is not specified (and also "hostfs" does not use bootdevice).
SOCFPGA_CYCLONE5 # setenv bootdevice 0:1
SOCFPGA_CYCLONE5 # fatload mmc
** Bad device mmc 0 **
(In older u-boots, bootdevice is indeed mandatory. And clearly I
should get recent u-boot with mmc support; working on that.).
IOW I think the patch is correct as-is.
Best regards,
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] Fix help text of ext2load and fatload.
2014-07-09 20:40 [U-Boot] [PATCH] Fix help text of ext2load and fatload Pavel Machek
2014-07-09 21:05 ` Marek Vasut
2014-07-14 15:26 ` Tom Rini
@ 2014-07-22 19:22 ` Tom Rini
2 siblings, 0 replies; 6+ messages in thread
From: Tom Rini @ 2014-07-22 19:22 UTC (permalink / raw)
To: u-boot
On Wed, Jul 09, 2014 at 10:40:07PM +0200, Pavel Machek wrote:
> Fix help text of ext2load and fatload to match code in fs/fs.c
>
> Signed-off-by: Pavel Machek <pavel@denx.de>
> Reviewed-by: Marek Vasut <marex@denx.de>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140722/96cca4d7/attachment.pgp>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-07-22 19:22 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-09 20:40 [U-Boot] [PATCH] Fix help text of ext2load and fatload Pavel Machek
2014-07-09 21:05 ` Marek Vasut
2014-07-09 22:57 ` Pavel Machek
2014-07-14 15:26 ` Tom Rini
2014-07-15 21:37 ` Pavel Machek
2014-07-22 19:22 ` [U-Boot] " Tom Rini
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox