
20 Jan
2011
20 Jan
'11
4:43 p.m.
Dear seedshope,
In message 4D3841B2.8050102@gmail.com you wrote: ...
+int dram_init(void) +{
- gd->ram_size = PHYS_SDRAM_1_SIZE + PHYS_SDRAM_2_SIZE;
It is shoule to use get_ram_size() function
This, like the others, are valid comments - but:
Can you please explain why you have to quote all the 1000+ unrelated lines of the patch?
It is even difficult to spot your 4 lines of comments in 1000+ lines of quoted patch. Plase don't do that.
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
He had quite a powerful intellect, but it was as powerful like a
locomotive, and ran on rails and was therefore almost impossible to
steer. - Terry Pratchett, _Lords and Ladies_