Dear Kumar Gala,
In message <19e9b551-f442-4ba2-98a3-c941620e4...@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: w...@denx.de
Time is an illusion perpetrated by the manufacturers of space.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
