[U-Boot] [PATCH v2 06/11] net: tftpput: move common code into separate functions

Simon Glass sjg at chromium.org
Wed Oct 26 23:56:41 CEST 2011


Hi Tom,

On Wed, Oct 26, 2011 at 1:38 PM, Tom Rini <tom.rini at gmail.com> wrote:
> On Wed, Oct 26, 2011 at 1:34 PM, Wolfgang Denk <wd at denx.de> wrote:
>> 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!
>
> https://github.com/dustin/bindir/blob/master/git-test-sequence is very
> helpful in
> the "make sure my whole pull request is bisectable" area.

Yes that is useful - thanks very much. Even more useful if my build
procedure is actually building the code in the commits :-)

Regards,
Simon

>
> --
> Tom
>


More information about the U-Boot mailing list