
4 Oct
2009
4 Oct
'09
1:32 a.m.
Dear Stefan Roese,
In message 200910021450.19551.sr@denx.de you wrote:
But looking at this patch, it seems that larger parts in this (supposedly) board-specific code are actually more or less identical.
Should we not factor out such common code?
Yes, this could (should) be done. But there are differences in these board specific functions. This consolidation could introduce new errors/bugs. And I didn't want to do such a thing, now that the merge window is closed. And
I did not intend to ask for this fior the current release.
frankly, I don't have the time for such a cleanup right now. But I'll put this on my to-do list.
Thanks.
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
"Beware of programmers carrying screwdrivers." - Chip Salzenberg