Hi Simon, On Wed, Jul 6, 2011 at 7:32 AM, Simon Glass <s...@chromium.org> wrote: > Hi Chander, > > On Mon, Jun 27, 2011 at 1:37 AM, Chander Kashyap > <chander.kash...@linaro.org> wrote: >> >> Origen board is based upon S5PV310 SoC which is similiar to >> S5PC210 SoC. >> >> Signed-off-by: Chander Kashyap <chander.kash...@linaro.org> >> --- >> MAINTAINERS | 1 + >> board/samsung/origen/Makefile | 46 ++++ >> board/samsung/origen/lowlevel_init.S | 468 >> ++++++++++++++++++++++++++++++++++ >> board/samsung/origen/mem_setup.S | 392 ++++++++++++++++++++++++++++ >> board/samsung/origen/origen.c | 103 ++++++++ >> boards.cfg | 1 + >> include/configs/origen.h | 167 ++++++++++++ >> 7 files changed, 1178 insertions(+), 0 deletions(-) >> create mode 100644 board/samsung/origen/Makefile >> create mode 100644 board/samsung/origen/lowlevel_init.S >> create mode 100644 board/samsung/origen/mem_setup.S >> create mode 100644 board/samsung/origen/origen.c >> create mode 100644 include/configs/origen.h >> > There is a lot of assembler in this patch. I wonder if it would make sense > to use C for more of this?
The size of the SPL is restricted so that was the reason that assembly was used. > Regards, > Simon > Cheers Angus -- Angus Ainslie <angus.ains...@linaro.org> Team Lead, Samsung Landing Team _______________________________________________ linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev
