[U-Boot] [PATCH v2 4/7] rtc: add support for 4543 RTC (manufactured by e.g. EPSON)

Wolfgang Denk wd at denx.de
Fri Mar 27 11:51:23 CET 2009


Dear Anatolij,

In message <49CCAB93.6030901 at denx.de> you wrote:
> 
> > + * This program is distributed in the hope that it will be useful,
> > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> ----------------------------------------------------------^
> please, remove space here.
> 
> <snip>
> > +/*
> > + * Note: The acrobatics below is due to the hideously ingenius idea of
> > + * the chip designers.  As the chip does not allow register
> -------------------------^
> please, remove space here.
> 
> > + * addressing, all values need to be read and written in one go.  Sure
> -------------------------------------------------------------------^
> please, remove space here.

Nope. These are actually intentional. Good type setting systems add
wider horizontal space after a full stop than between words. The
double space after a full stop is the ASSCII representation of this.

It may be a matter of taste, and since many editors don't support
this (at least not out of the box), so it is not a required style
element, but in any case there is no reason to remove this.

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 at denx.de
Lispers are among  the  best  grads  of  the  Sweep-It-Under-Someone-
Else's-Carpet  School of Simulated Simplicity. [Was that sufficiently
incendiary? :-)]  - Larry Wall in <1992Jan10.201804.11926 at netlabs.com


More information about the U-Boot mailing list