linux-kernel - Re: [PATCH 2/9 v2] ARM: S3C64XX: add nand support for mach-real6410

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <AANLkTinwBj2QtLdFMGDmqipJD9g1O9QzRUUZpCn=0q-z@mail.gmail.com>
Date:	Mon, 18 Oct 2010 16:59:34 +0300
From:	Darius Augulis <augulis.darius@...il.com>
To:	Kukjin Kim <kgene.kim@...sung.com>
Cc:	linux-arm-kernel@...ts.infradead.org, paulius.zaleckas@...il.com,
	linux-kernel@...r.kernel.org, ben-linux@...ff.org
Subject: Re: [PATCH 2/9 v2] ARM: S3C64XX: add nand support for mach-real6410

On Mon, Oct 18, 2010 at 4:23 PM, Kukjin Kim <kgene.kim@...sung.com> wrote:
> Darius Augulis wrote:
>>
>> Add nand device support and mtd partition table
>> for mach-real6410
>>
>> Signed-off-by: Darius Augulis <augulis.darius@...il.com>
>> ---
>>
>> Changelog since v1:
>>  - Changed dependency on other patches
>>
>>  arch/arm/mach-s3c64xx/Kconfig         |    1 +
>>  arch/arm/mach-s3c64xx/mach-real6410.c |   40
>> +++++++++++++++++++++++++++++++++
>>  2 files changed, 41 insertions(+), 0 deletions(-)
>>
>> diff --git a/arch/arm/mach-s3c64xx/Kconfig b/arch/arm/mach-s3c64xx/Kconfig
>> index 3bbd418..e4aede1 100644
>> --- a/arch/arm/mach-s3c64xx/Kconfig
>> +++ b/arch/arm/mach-s3c64xx/Kconfig
>> @@ -106,6 +106,7 @@ config MACH_REAL6410
>>       select S3C64XX_SETUP_SDHCI
>>       select S3C_DEV_FB
>>       select S3C64XX_SETUP_FB_24BPP
>> +     select S3C_DEV_NAND
>>       help
>>         Machine support for the CoreWind REAL6410
>>
>> diff --git a/arch/arm/mach-s3c64xx/mach-real6410.c b/arch/arm/mach-
>> s3c64xx/mach-real6410.c
>> index bed75d2..9fd943d 100644
>> --- a/arch/arm/mach-s3c64xx/mach-real6410.c
>> +++ b/arch/arm/mach-s3c64xx/mach-real6410.c
>> @@ -19,6 +19,8 @@
>>  #include <linux/kernel.h>
>>  #include <linux/list.h>
>>  #include <linux/dm9000.h>
>> +#include <linux/mtd/mtd.h>
>> +#include <linux/mtd/partitions.h>
>>  #include <linux/platform_device.h>
>>  #include <linux/serial_core.h>
>>  #include <linux/types.h>
>> @@ -37,6 +39,7 @@
>>  #include <plat/cpu.h>
>>  #include <plat/devs.h>
>>  #include <plat/fb.h>
>> +#include <plat/nand.h>
>>  #include <plat/regs-serial.h>
>>
>>  #include <video/platform_lcd.h>
>> @@ -147,11 +150,47 @@ static struct s3c_fb_platdata real6410_lcd_pdata
>> __initdata = {
>>       .vidcon1        = VIDCON1_INV_HSYNC | VIDCON1_INV_VSYNC,
>>  };
>>
>> +static struct mtd_partition real6410_nand_part[] = {
>> +     [0] = {
>> +             .name   = "uboot",
>> +             .size   = SZ_1M,
>> +             .offset = 0,
>> +     },
>> +     [1] = {
>> +             .name   = "kernel",
>> +             .size   = SZ_2M,
>> +             .offset = SZ_1M,
>> +     },
>> +     [2] = {
>> +             .name   = "rootfs",
>> +             .size   = MTDPART_SIZ_FULL,
>> +             .offset = SZ_1M + SZ_2M,
>> +     },
>> +};
>> +
>> +static struct s3c2410_nand_set real6410_nand_sets[] = {
>> +     [0] = {
>> +             .name           = "nand",
>> +             .nr_chips       = 1,
>> +             .nr_partitions  = ARRAY_SIZE(real6410_nand_part),
>> +             .partitions     = real6410_nand_part,
>> +     },
>> +};
>> +
>> +static struct s3c2410_platform_nand real6410_nand_info = {
>> +     .tacls          = 25,
>> +     .twrph0         = 55,
>> +     .twrph1         = 40,
>> +     .nr_sets        = ARRAY_SIZE(real6410_nand_sets),
>> +     .sets           = real6410_nand_sets,
>> +};
>> +
>>  static struct platform_device *real6410_devices[] __initdata = {
>>       &real6410_device_eth,
>>       &s3c_device_hsmmc0,
>>       &s3c_device_hsmmc1,
>>       &s3c_device_fb,
>> +     &s3c_device_nand,
>>  };
>>
>>  static void __init real6410_map_io(void)
>> @@ -249,6 +288,7 @@ static void __init real6410_machine_init(void)
>>               real6410_lcd_pdata.win[0]->win_mode.yres);
>>
>>       s3c_fb_set_platdata(&real6410_lcd_pdata);
>> +     s3c_nand_set_platdata(&real6410_nand_info);
>>
>>       /* configure nCS1 width to 16 bits */
>>
> Where is 'END' of patch mark?
> maybe broken your patch file...?

Confused. What 'END' mark are you missing?
All patches are sent by stg mail, all apply fine, no conflict or other
messages from stgit.
Please explain what you mean.

p.s. checkpatch.pl also doesn't report any error.

>
> Others, looks ok to me.
>
> Thanks.
>
> Best regards,
> Kgene.
> --
> Kukjin Kim <kgene.kim@...sung.com>, Senior Engineer,
> SW Solution Development Team, Samsung Electronics Co., Ltd.
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/