linux-kernel - Re: [PATCH] s3c24xx nand: add command line partition table parsing support

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <5767b9100809171308j58d05b36i453badc96e1e668c@mail.gmail.com>
Date:	Thu, 18 Sep 2008 04:08:47 +0800
From:	"Conke Hu" <conke.hu@...il.com>
To:	"Russell King" <rmk+lkml@....linux.org.uk>
Cc:	"Linux Kernel Mailing List" <linux-kernel@...r.kernel.org>,
	linux-arm@...r.kernel.org, ben@...tec.co.uk
Subject: Re: [PATCH] s3c24xx nand: add command line partition table parsing support

Thank you, Russell
then I'd sent it to linux-arm-kernel@...ts.arm.linux.org.uk &
ben-linux@...ff.org, right?
I am subscribing linux-arm-kernel@...ts.arm.linux.org.uk :)

regards,
Conke

On Thu, Sep 18, 2008 at 1:58 AM, Russell King <rmk+lkml@....linux.org.uk> wrote:
> By sending this to the linux-arm list at vger, you're missing out the
> majority of the ARM community.  Please use the proper (listed) mailing
> list in MAINTAINERS.
>
> On Thu, Sep 18, 2008 at 01:32:16AM +0800, Conke Hu wrote:
>> Add mtd id and  command line partition table parsing support for
>> s3c24xx nand driver.
>>
>> Signed-off-by: Conke Hu <conke.hu@...il.com>
>> Signed-off-by: Linke Wang <linke.wang@...il.com>
>> ------------------
>> diff --git a/drivers/mtd/nand/s3c2410.c b/drivers/mtd/nand/s3c2410.c
>> index 556139e..bab7216 100644
>> --- a/drivers/mtd/nand/s3c2410.c
>> +++ b/drivers/mtd/nand/s3c2410.c
>> @@ -48,6 +48,8 @@
>>  #include <asm/plat-s3c/regs-nand.h>
>>  #include <asm/plat-s3c/nand.h>
>>
>> +#define S3C_NAND_NAME_LEN 16
>> +
>>  #ifdef CONFIG_MTD_NAND_S3C2410_HWECC
>>  static int hardware_ecc = 1;
>>  #else
>> @@ -60,6 +62,9 @@ static int clock_stop = 1;
>>  static const int clock_stop = 0;
>>  #endif
>>
>> +#ifdef CONFIG_MTD_PARTITIONS
>> +static const char *s3c_part_probes[] = {"cmdlinepart", NULL};
>> +#endif
>>
>>  /* new oob placement block for use with hardware ecc generation
>>   */
>> @@ -649,6 +654,15 @@ static int s3c2410_nand_add_partition(struct
>> s3c2410_nand_info *info,
>>                                     struct s3c2410_nand_mtd *mtd,
>>                                     struct s3c2410_nand_set *set)
>>  {
>> +#ifdef CONFIG_MTD_CMDLINE_PARTS
>> +     int nr_parts;
>> +     struct mtd_partition *partitions;
>> +
>> +     nr_parts = parse_mtd_partitions(&mtd->mtd, s3c_part_probes, &partitions, 0);
>> +     if (nr_parts > 0 && partitions != NULL)
>> +             return add_mtd_partitions(&mtd->mtd, partitions, nr_parts);
>> +#endif
>> +
>>       if (set == NULL)
>>               return add_mtd_device(&mtd->mtd);
>>
>> @@ -808,6 +822,7 @@ static int s3c24xx_nand_probe(struct platform_device *pdev,
>>       int size;
>>       int nr_sets;
>>       int setno;
>> +     char *name;
>>
>>       pr_debug("s3c2410_nand_probe(%p)\n", pdev);
>>
>> @@ -875,7 +890,7 @@ static int s3c24xx_nand_probe(struct platform_device *pdev,
>>
>>       /* allocate our information */
>>
>> -     size = nr_sets * sizeof(*info->mtds);
>> +     size = nr_sets * (sizeof(*info->mtds) + S3C_NAND_NAME_LEN);
>>       info->mtds = kmalloc(size, GFP_KERNEL);
>>       if (info->mtds == NULL) {
>>               dev_err(&pdev->dev, "failed to allocate mtd storage\n");
>> @@ -888,10 +903,15 @@ static int s3c24xx_nand_probe(struct
>> platform_device *pdev,
>>       /* initialise all possible chips */
>>
>>       nmtd = info->mtds;
>> +     name = (char *)(info->mtds + nr_sets);
>>
>>       for (setno = 0; setno < nr_sets; setno++, nmtd++) {
>>               pr_debug("initialising set %d (%p, info %p)\n", setno, nmtd, info);
>>
>> +             sprintf(name, "s3c_nand%d", setno);
>> +             nmtd->mtd.name = name;
>> +             name += S3C_NAND_NAME_LEN;
>> +
>>               s3c2410_nand_init_chip(info, nmtd, sets);
>>
>>               nmtd->scan_res = nand_scan_ident(&nmtd->mtd,
>
> --
> Russell King
>  Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
>  maintainer of:
>
--
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/