23 Sep
2009
23 Sep
'09
10:52 p.m.
Dear Kumar Gala,
In message 19E9B551-F442-4BA2-98A3-C941620E4E64@kernel.crashing.org you wrote:
Dope, posted the wrong version. I agree w/the long lines, not sure what to do about it. Move to something like:
typedef struct ccsr_local { /* 0x0 - Control Configuration Status Registers Base Address Register High */ u32 ccsrbarh; /* 0x4 - Control Configuration Status Registers Base Address Register Low */ u32 ccsrbarl;
...
even this exceeds the 80 column limit.
Why do you need to carry the offset in the comment?
What about:
Control Config Status Regs Base Addr Reg ?
or
BAR = Base Address Register (once initially)
Control Config Status Regs BAR ?
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
Time is an illusion perpetrated by the manufacturers of space.