[U-Boot] [PATCH 3/7 v6] ARM: Add arm1176 core with S3C6400 SoC

Wolfgang Denk wd at denx.de
Sat Aug 9 11:58:59 CEST 2008


In message <20080809091101.GC18040 at game.jcrosoft.org> you wrote:
> 
> > +		/*
> > +		 * for 10 ms clock period @ PCLK with 4 bit divider = 1/2 and
> > +		 * prescaler = 16. Should be 10390 @33.25MHz and 15625 @ 50 MHz
> > +		 */
> > +		timer_load_val = get_PCLK() / PRESCALER * (100 / 4); /* 100s */
> > +		/*printf("Calculated %lu timer_load_val\n", timer_load_val);*/
> please remove if not need

Or even better convert it into a debug() call so it can be enabled
again for debugging purposes.

> > +		timers->TCFG1 = (timers->TCFG1 & ~0xf0000) | 0x20000;
> > +	}
> > +
> please add some empty line to be more readable

No. One empty line os enough here.

> > +#define NFADDR           	(ELFIN_NAND_BASE+NFADDR_OFFSET)
>                  ^^^^^^^^^^^
> please remove whitesapce

Not remove, but use TABs vor vertical alignment.


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
We fight only when there is no other choice. We prefer  the  ways  of
peaceful contact.
	-- Kirk, "Spectre of the Gun", stardate 4385.3



More information about the U-Boot mailing list