public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/8] Add tftpput command for uploading files over network
Date: Sat, 22 Oct 2011 10:21:18 +0200	[thread overview]
Message-ID: <4EA27CFE.2010303@aribaud.net> (raw)
In-Reply-To: <1319259100-11376-1-git-send-email-sjg@chromium.org>

Le 22/10/2011 06:51, Simon Glass a ?crit :
> The tftpboot command permits reading of files over a network interface
> using the Trivial FTP protocol. This patch series adds the ability to
> transfer files the other way.
>
> Why is this useful?
>
> - Uploading boot time data to a server
> - Uploading profiling information
> - Uploading large mounts of data for comparison / checking on a host
>      (e.g. use tftpput and ghex2 instead of the 'md' command)

Especially I find it interesting for backing up things like MTD and 
small disk files (not partitions, though). Most of my work currently is 
trying to bring mainline U-Boot support to existing boards with bad 
U-Boot implementations, and being able to backup things from U-Boot (as 
opposed to having to set up NFS root and Linux boot) would definitely be 
a plus.

> Mostly the existing code can be re-used and I have tried to avoid too
> much refactoring or cleaning up.

:)

> The feature is activated by the CONFIG_CMD_TFTPPUT option.
>
> This has been very lightly tested on a Seaboard with a USB network
> adaptor. I don't think it handles block number overflow.

What size does this limit transfers to?

> Simon Glass (8):
>    Move simple_itoa to vsprintf
>    Add setenv_uint() and setenv_addr()
>    tftpput: Rename TFTP to TFTPGET
>    tftpput: move common code into separate functions
>    tftpput: support selecting get/put for tftp
>    tftpput: add save_addr and save_size global variables
>    tftpput: implement tftp logic
>    tftpput: add tftpput command

Many U-Boot environments use 'tftp' as a shorthand to tftpboot. Did you 
verify that this is not broken by the introduction of 'tftpput'?

Also, I'd be happy to test this if a branch exists that already holds 
these commits.

Amicalement,
-- 
Albert.

  parent reply	other threads:[~2011-10-22  8:21 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-22  4:51 [U-Boot] [PATCH 0/8] Add tftpput command for uploading files over network Simon Glass
2011-10-22  4:51 ` [U-Boot] [PATCH 1/8] Move simple_itoa to vsprintf Simon Glass
2011-10-22  4:51 ` [U-Boot] [PATCH 2/8] Add setenv_uint() and setenv_addr() Simon Glass
2011-10-23  5:29   ` Mike Frysinger
2011-10-25 21:35     ` Simon Glass
2011-10-25 21:39       ` Mike Frysinger
2011-10-25 21:58         ` Simon Glass
2011-10-25 22:01           ` Wolfgang Denk
2011-10-25 22:08             ` Mike Frysinger
2011-10-25 22:41               ` Simon Glass
2011-10-25 22:49                 ` Mike Frysinger
2011-10-25 23:08                   ` Simon Glass
2011-10-25 23:27                     ` Mike Frysinger
2011-10-22  4:51 ` [U-Boot] [PATCH 3/8] tftpput: Rename TFTP to TFTPGET Simon Glass
2011-10-22  4:51 ` [U-Boot] [PATCH 4/8] tftpput: move common code into separate functions Simon Glass
2011-10-22  4:51 ` [U-Boot] [PATCH 5/8] tftpput: support selecting get/put for tftp Simon Glass
2011-10-22  4:51 ` [U-Boot] [PATCH 6/8] tftpput: add save_addr and save_size global variables Simon Glass
2011-10-22  4:51 ` [U-Boot] [PATCH 7/8] tftpput: implement tftp logic Simon Glass
2011-10-22  4:51 ` [U-Boot] [PATCH 8/8] tftpput: add tftpput command Simon Glass
2011-10-22  8:21 ` Albert ARIBAUD [this message]
2011-10-22 16:15   ` [U-Boot] [PATCH 0/8] Add tftpput command for uploading files over network Simon Glass
2011-10-24  4:28     ` Simon Glass
2011-10-25  4:13       ` Simon Glass
2011-10-25  6:44         ` Albert ARIBAUD
2011-10-30 20:13   ` Mike Frysinger

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=4EA27CFE.2010303@aribaud.net \
    --to=albert.u.boot@aribaud.net \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox