[U-Boot] [PATCH-ARM 1/3] Add support for the s3c2440 cpu excluding nand driver

Minkyu Kang promsoft at gmail.com
Mon Nov 2 05:55:37 CET 2009


Dear kevin.morfitt and Tom

2009/11/2 Tom <Tom.Rix at windriver.com>:
> kevin.morfitt at fearnside-systems.co.uk wrote:
>> This patch adds support for the s3c2440 cpu, excluding the nand driver.
>>
>> Tested on an Embest SBC2440-II Board with local u-boot patches as I don't have
>> any s3c2400 or s3c2410 boards but need this patch applying before I can submit
>> patches for the SBC2440-II Board. Also, ran MAKEALL for all ARM9 targets and no
>> new warnings or errors were found.
>>
>> Note that checkpatch.pl shows one error:
>
> Thank you for using checkpatch.
>
>>
>> ERROR: Invalid UTF-8, patch and commit message should be encoded in UTF-8
>> #656: FILE: include/s3c2440.h:3:
>> + * David M�ller ELSOFT AG Switzerland. d.mueller at elsoft.ch
>>            ^
>> As David's name correctly contains a non-UTF-8 character I've ignored this error.
>>
>> Signed-off-by: Kevin Morfitt <kevin.morfitt at fearnside-systems.co.uk>
>> ---
>>  common/serial.c                              |    4 +-
>>  cpu/arm920t/s3c24x0/Makefile                 |    6 +-
>>  cpu/arm920t/s3c24x0/arch_pre_lowlevel_init.S |   81 +++++++++++++
>
> Why not just lowlevel_init.S ?
> It looks like this is a common lowlevel_init but this looks like
> a mistake since the other s3c34x0 boards have not used it up to to this
> point.  Since it looks like this option is being enabled in the
> other boards, this change must be broken out at its own patch.
>
>
>>  cpu/arm920t/s3c24x0/speed.c                  |   41 +++++--
>>  cpu/arm920t/s3c24x0/timer.c                  |   19 +---
>>  cpu/arm920t/s3c24x0/usb.c                    |   17 +--
>>  cpu/arm920t/s3c24x0/usb_ohci.c               |   11 +--
>>  cpu/arm920t/start.S                          |   39 +------
>>  drivers/i2c/s3c24x0_i2c.c                    |   18 ++--
>>  drivers/mtd/nand/s3c2410_nand.c              |    2 +-
>>  drivers/rtc/s3c24x0_rtc.c                    |    7 +-
>>  drivers/serial/serial_s3c24x0.c              |    6 +-
>>  drivers/usb/host/ohci-hcd.c                  |    3 +-
>>  include/common.h                             |    5 +-
>
>>  include/configs/VCMA9.h                      |    4 +-
>>  include/configs/sbc2410x.h                   |    4 +-
>>  include/configs/smdk2400.h                   |    4 +-
>>  include/configs/smdk2410.h                   |    4 +-
>>  include/configs/trab.h                       |    4 +-
>
> This is typical of what you are doing with the config files.
>> +#define      CONFIG_S3C24X0          1       /* in a SAMSUNG S3C24x0-type SoC     */
>> +#define      CONFIG_S3C2410          1       /* specifically a SAMSUNG S3C2410 SoC */
> It is good that you are trying to generalize the boards, but
> this separate change and must be split.  This new patch should come first.
>
>>  include/s3c2410.h                            |   25 ++++
>>  include/s3c2440.h                            |  163 ++++++++++++++++++++++++++
>>  include/s3c24x0.h                            |   94 ++++++++++++++-
>>  include/s3c24x0_cpu.h                        |   29 +++++
>
> These 4 files belong in include/asm-arch/arch-s3c24x0 or
> where Minkyu thinks is appropriate.

yes right,
I think these file must be move to include/asm-arch/arch-s3c24x0
kevin, could you please make the patch about this also?
It seems to be another patch.
If you busy, I'll do it :)

>
> On your include file s3c2440.h
>
> For your #defines, the whitespace between the identifier and the value
> must be tabs.  You have spaces.
>
> The static inline functions need space beween one function definition
> and the next.  They also need to use tabs
>
>>  23 files changed, 471 insertions(+), 119 deletions(-)
>>  create mode 100644 cpu/arm920t/s3c24x0/arch_pre_lowlevel_init.S
>>  create mode 100644 include/s3c2440.h
>>  create mode 100644 include/s3c24x0_cpu.h
>>
>
> Tom
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>

Thanks
Minkyu Kang
-- 
from. prom.
www.promsoft.net


More information about the U-Boot mailing list