27 Apr
2012
27 Apr
'12
6:17 p.m.
Dear Tom Rini,
In message 4F9AB2EE.3070202@ti.com you wrote:
Argh... But you cannot change this without explicit ACK from the respective board maintainers.
Then we're really SOL here. Even if we limit it to "I dropped 300/600/900/1200/2400/4800 baud rate support, is that ok".
What's wrong with keeping a customizes #define for the boards that use nonstandard settings?
Finally, your patch does not apply (I guess it is an incremental patch over obsolete code?).
That's odd actually, it was on v2012.04'ish and a pull and rebase -q went quietly. It was build-tested on MAKEALL -a powerpc/arm/x86.
But you see the problem in the patch, don't you?
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
"It follows that any commander in chief who undertakes to carry out a
plan which he considers defective is at fault; he must put forth his
reasons, insist of the plan being changed, and finally tender his
resignation rather than be the instrument of his army's downfall."
- Napoleon, "Military Maxims and Thought"