[U-Boot] [PATCH v2 06/11] net: tftpput: move common code into separate functions
Wolfgang Denk
wd at denx.de
Wed Oct 26 22:34:01 CEST 2011
Dear Simon Glass,
In message <1319515208-7810-7-git-send-email-sjg at chromium.org> you wrote:
> We want to show block markers on completion of get and put, so
> move this common code into separate functions.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
> net/tftp.c | 68 +++++++++++++++++++++++++++++++++--------------------------
> 1 files changed, 38 insertions(+), 30 deletions(-)
ARRRRRRGHHH!!!!
Your commits are not bisectable.
When applying your commits up to and including this one, all boards
with ethernet support will break with errors like this:
net.c: In function 'NetLoop':
net.c:412: error: too many arguments to function 'TftpStart'
make[1]: *** [/work/wd/tmp-ppc/net/net.o] Error 1
Please always make sure that patches are really orthogonal so we can
use git bisect!
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
War is never imperative.
-- McCoy, "Balance of Terror", stardate 1709.2
More information about the U-Boot
mailing list