
28 Oct
2010
28 Oct
'10
6:47 p.m.
Dear Sanjeev Premi,
In message 1288283932-3881-1-git-send-email-premi@ti.com you wrote:
This patch moves the most common config options into one file. In addition to movement, attempt has been made to consolidate related config options together.
Ideally, this patch should contain "deletion" from current omap3_evm.h. But for this RFC, I am maintaining current file and adding (only for example) another file omap3_evm_full.h in next patch.
This is not a good idea, as it makes it very hard to compare what actually was moved from one file to the other. I would much rather prefer a patch that shows this in a single step.
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@denx.de
There are three things I always forget. Names, faces - the third I
can't remember. - Italo Svevo