[U-Boot] [PATCH] powerpc/eeprom: cleanup mac command
Wolfgang Denk
wd at denx.de
Thu Jul 28 20:39:57 CEST 2011
Dear York Sun,
In message <1311869371.29459.11.camel at oslab-l1> you wrote:
>
> > Why are you turning a generic, board and architecture independent
> > command into a FSL specific one?
>
> I believe this "generic" command is only for FSL boards. Moving it to
> right place.
Even if it supports only FSL boards at the moemnt, the change should
be in the opposite direction: move all the generic code out of the FSL
directories and make it publicly available so others can use this tool
as well.
> > In case you ever move / rename a file: please supply the options so
> > git will detect the move / copy / rename.
>
> The meaningful code was moved into an existing file, so it wasn't
> renaming a file.
That's what you think. Did you check what git says about it?
So: still NAK. Please do it the other way round.
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
Nearly everyone is in favor of going to heaven but too many are
hoping they'll live long enough to see an easing of the entrance
requirements. Never appeal to a man's "better nature." he might not
have one.
More information about the U-Boot
mailing list