Dear "kevin.morf...@fearnside-systems.co.uk", In message <4a263923.2030...@fearnside-systems.co.uk> you wrote: > > >> +#define BWSCON 0x48000000 > >> + > >> +#define DW8 (0x0) > >> +#define DW16 (0x1) > >> +#define DW32 (0x2) > >> +#define WAIT (0x1 << 2) > >> +#define UBLB (0x1 << 3) > >> > > what is all theses macro? for what use? > > > They make setting the fields of the memory controller registers more > readable. For example BSWCON is > the Bus Width and Wait Control Register, DW32 sets a Data Width value > bit-field in BWSCON to 8-bit > etc. They're used later in the patch to configure the memory controller > registers. The names are the same > as those used in the S3C2440 data sheet.
Please make sure to use C structs to descrive device register layout, and use proper I/O accessor functions for the actual access. > >> + gpio->GPBCON = 0x00055555; > >> + gpio->GPBUP = 0x000007FF; > >> + gpio->GPBDAT = 0x000001C0; /* Switch on LED1. */ > >> + gpio->GPCCON = 0xAAAAAAAA; > >> + gpio->GPCUP = 0x0000FFFF; > >> + gpio->GPDCON = 0xAAAAAAAA; Such code will not be accepted. Please use C structs and I/O accessor functions. 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 "Consistency requires you to be as ignorant today as you were a year ago." - Bernard Berenson _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot