[U-Boot] [PATCH v2 3/6] TFTP: rename "server" to "remote"
Detlev Zundel
dzu at denx.de
Wed May 4 12:37:47 CEST 2011
Hi Luca,
> Detlev Zundel wrote:
>>
>> I'll start a new thread to discuss this. Hopefully we then come up with
>> a policy to stick into that wiki page.
>>
>
> Now that the checkpatch policy is much more clear, I started to clean up
> the networking stuff, on top of which the TFTP server sits.
Thanks a lot for starting this cleanup. Your work is very much
appreciated.
> net/net.c alone counts 76 errors and 197 warnings. In terms of modified
> lines, it's going to be a big job, so I am doing it in separate patches, one
> per each category of errors/warnings (whitespace issues, lines >80 chars,
> parentheses issues etc).
> Is this choice ok?
This makes sense to me yes.
> Also, it's going to be a bigger job than the TFTP server itself, so
> I'll send a
> separate patchset for the cleanup work, and hold the TFTP server until the
> cleanup is worked out.
If you really start the clenaup, then this order makes absolute sense.
Thanks in advance!
Detlev
--
Every time history repeats itself the price goes up.
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de
More information about the U-Boot
mailing list