[U-Boot] [PATCH 4/4] s5pc1xx: add support SMDKC100 board

Minkyu Kang promsoft at gmail.com
Tue Sep 8 03:46:07 CEST 2009


Dear Jean-Christophe

2009/9/5 Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com>:
>> diff --git a/board/samsung/smdkc100/onenand.c b/board/samsung/smdkc100/onenand.c
> I guess this is not board specific but soc specific
> so please move it to drivers/mtd/onenand/

no, this is related with onenand clock.
It is board specific.

>> new file mode 100644
>> index 0000000..75bb8a9
>> --- /dev/null
>> +++ b/board/samsung/smdkc100/onenand.c
>> @@ -0,0 +1,98 @@
>> +/*
>> + *  Copyright (C) 2008-2009 Samsung Electronics
>> + *  Kyungmin Park <kyungmin.park at samsung.com>
>> + *
>> + * See file CREDITS for list of people who contributed to this
>> + * project.
>> + *
>> + * This program is free software; you can redistribute it and/or
>> + * modify it under the terms of the GNU General Public License as
>> + * published by the Free Software Foundation; either version 2 of
>> + * the License, or (at your option) any later version.
>> + *
>> + * 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
>> + * GNU General Public License for more details.
>> + *
>> + * You should have received a copy of the GNU General Public License
>> + * along with this program; if not, write to the Free Software
>> + * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
>> + * MA 02111-1307 USA
>> + */
>> +
>> +#include <common.h>
>> +#include <linux/mtd/compat.h>
>> +#include <linux/mtd/mtd.h>
>> +#include <linux/mtd/onenand.h>
>> +
>> +#include <onenand_uboot.h>
>> +
>> +#include <samsung_onenand.h>
>> +
>> +#include <asm/io.h>
>> +#include <asm/arch/clock.h>
>> +
>> +extern void s3c_onenand_init(struct mtd_info *);
> please move this to a header

agreed

>> +
>> diff --git a/board/samsung/smdkc100/smdkc100.c b/board/samsung/smdkc100/smdkc100.c
>> new file mode 100644
>> index 0000000..4539ced
>> --- /dev/null
>> diff --git a/board/samsung/smdkc100/u-boot.lds b/board/samsung/smdkc100/u-boot.lds
> no need please remove
>> new file mode 100644
>> index 0000000..27f8201
>> --- /dev/null
>> +/***********************************************************
>
>> +
>> +#define CONFIG_RAMDISK_BOOT  "root=/dev/ram0 rw rootfstype=ext2" \
>> +             " console=ttySAC0,115200n8" \
>> +             " mem=80M"
> why do you restrict the memsize of the kernel?
>> +
>> +#define CONFIG_COMMON_BOOT   "console=ttySAC0,115200n8" \
>> +             " mem=128M " \
>> +             " " MTDPARTS_DEFAULT
>> +
>> +
>> +#define CONFIG_ENV_OVERWRITE
>> +#define CONFIG_EXTRA_ENV_SETTINGS                                    \
>> +     CONFIG_UPDATEB \
>> +     "updatek=onenand erase 0x60000 0x300000;" \
>> +     " onenand write 0x31008000 0x60000 0x300000\0" \
>> +     "updateu=onenand erase block 147-4095;" \
>> +     " onenand write 0x32000000 0x1260000 0x8C0000\0" \
> something like this will be more readable

ok.

>        "updatek="                                              \
>                "onenand erase 0x60000 0x300000; "              \
>                "onenand write 0x31008000 0x60000 0x300000\0"   \
>        "updateu="                                              \
>                "onenand erase block 147-4095; "                \
>                "onenand write 0x32000000 0x1260000 0x8C0000\0" \
>        "bootk="                                                \
>                "onenand read 0x30007FC0 0x60000 0x300000; "    \
>                "bootm 0x30007FC0\0"                            \
>        "flashboot="                                            \
>                "set bootargs "                                 \
>                        "root=/dev/mtdblock${bootblock} "       \
>                        "rootfstype=${rootfstype} "             \
>                        "ubi.mtd=${ubiblock} ${opts} "          \
>                        CONFIG_COMMON_BOOT "; "                 \
>                "run bootk\0"                                   \
>> +     "ubifsboot=set bootargs root=ubi0!rootfs rootfstype=ubifs" \
>> +      " ubi.mtd=${ubiblock} ${opts} " CONFIG_COMMON_BOOT "; run bootk\0" \
>> +     "boottrace=setenv opts initcall_debug; run bootcmd\0" \
>> +     "android=set bootargs root=ubi0!ramdisk ubi.mtd=${ubiblock}" \
>> +      " rootfstype=ubifs init=/init.sh " CONFIG_COMMON_BOOT "; run bootk\0" \
>> +     "nfsboot=set bootargs root=/dev/nfs ubi.mtd=${ubiblock}" \
>> +      " nfsroot=${nfsroot},nolock ip=${ipaddr}:${serverip}:${gatewayip}:" \
>> +      "${netmask}:nowplus:usb0:off " CONFIG_COMMON_BOOT "; run bootk\0" \
>> +     "ramboot=set bootargs " CONFIG_RAMDISK_BOOT \
>> +      " initrd=0x33000000,8M ramdisk=8192\0" \
>> +     "rootfstype=cramfs\0" \
>> +     "mtdparts=" MTDPARTS_DEFAULT "\0" \
>> +     "meminfo=mem=128M\0" \
>> +     "nfsroot=/nfsroot/arm\0" \
>> +     "bootblock=5\0" \
>> +     "ubiblock=4\0" \
>> +     "ubi=enabled"
>> +
>> +/*
> Best Regards,
> J.
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>

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


More information about the U-Boot mailing list