diff for duplicates of <1305619989.2630.31.camel@localhost> diff --git a/a/1.txt b/N1/1.txt index 65c74ba..b206c01 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -17,50 +17,3 @@ David should answer, but from my POW the patches are OK. The "#ifdef CONFIG_MTD_AFS_PARTS" is ugly. I think it can be just killed, it just needs an additional patch to silence the kernel (completely untested, Marc, could you please check it?): - - ->From 8dd2f442c7acd5d45e2f35471846c68b26619eb2 Mon Sep 17 00:00:00 2001 -From: Artem Bityutskiy <Artem.Bityutskiy@nokia.com> -Date: Tue, 17 May 2011 11:13:17 +0300 -Subject: [PATCH] mtd: be silent when mtd partition parser cannot be found - -Currently when we register partitions in 'parse_mtd_partitions()' we accept the -list of parsers we should try. And if one of the parsers was not found we print -a message. Well, first of all this whole idea is bad - look at how many -'part_probes' and 'part_probe_types' variables we have - nearly every driver -defines one. Instead, we should just go through all registered parsers all the -time. But this needs to be worked on separately. - -This patch makes life of MTD partitions' users a bit simpler and allows them to -safely request parsers which have not been registered - -'parse_mtd_partitions()' will not print a "not available" message in this -case. - -Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com> ---- - drivers/mtd/mtdpart.c | 5 +---- - 1 files changed, 1 insertions(+), 4 deletions(-) - -diff --git a/drivers/mtd/mtdpart.c b/drivers/mtd/mtdpart.c -index 0a47601..cd631e7 100644 ---- a/drivers/mtd/mtdpart.c -+++ b/drivers/mtd/mtdpart.c -@@ -722,11 +722,8 @@ int parse_mtd_partitions(struct mtd_info *master, const char **types, - parser = get_partition_parser(*types); - if (!parser && !request_module("%s", *types)) - parser = get_partition_parser(*types); -- if (!parser) { -- printk(KERN_NOTICE "%s partition parsing not available\n", -- *types); -+ if (!parser) - continue; -- } - ret = (*parser->parse_fn)(master, pparts, origin); - if (ret > 0) { - printk(KERN_NOTICE "%d %s partitions found on MTD device %s\n", --- -1.7.2.3 - --- -Best Regards, -Artem Bityutskiy (Артём Битюцкий) diff --git a/a/content_digest b/N1/content_digest index a6055e6..ca4c0cd 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -3,14 +3,10 @@ "ref\01305610643.2630.15.camel@localhost\0" "ref\01305611178.2630.21.camel@localhost\0" "ref\020110517074348.GA26669@n2100.arm.linux.org.uk\0" - "From\0Artem Bityutskiy <dedekind1@gmail.com>\0" - "Subject\0Re: [PATCH v5 1/8] MTD: Add integrator-flash feature to physmap\0" + "From\0dedekind1@gmail.com (Artem Bityutskiy)\0" + "Subject\0[PATCH v5 1/8] MTD: Add integrator-flash feature to physmap\0" "Date\0Tue, 17 May 2011 11:13:09 +0300\0" - "To\0Russell King - ARM Linux <linux@arm.linux.org.uk>" - " Marc Zyngier <marc.zyngier@arm.com>\0" - "Cc\0linux-mtd@lists.infradead.org" - David Woodhouse <dwmw2@infradead.org> - " linux-arm-kernel@lists.infradead.org\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "On Tue, 2011-05-17 at 08:43 +0100, Russell King - ARM Linux wrote:\n" @@ -31,53 +27,6 @@ "David should answer, but from my POW the patches are OK. The \"#ifdef\n" "CONFIG_MTD_AFS_PARTS\" is ugly. I think it can be just killed, it just\n" "needs an additional patch to silence the kernel (completely untested,\n" - "Marc, could you please check it?):\n" - "\n" - "\n" - ">From 8dd2f442c7acd5d45e2f35471846c68b26619eb2 Mon Sep 17 00:00:00 2001\n" - "From: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>\n" - "Date: Tue, 17 May 2011 11:13:17 +0300\n" - "Subject: [PATCH] mtd: be silent when mtd partition parser cannot be found\n" - "\n" - "Currently when we register partitions in 'parse_mtd_partitions()' we accept the\n" - "list of parsers we should try. And if one of the parsers was not found we print\n" - "a message. Well, first of all this whole idea is bad - look at how many\n" - "'part_probes' and 'part_probe_types' variables we have - nearly every driver\n" - "defines one. Instead, we should just go through all registered parsers all the\n" - "time. But this needs to be worked on separately.\n" - "\n" - "This patch makes life of MTD partitions' users a bit simpler and allows them to\n" - "safely request parsers which have not been registered -\n" - "'parse_mtd_partitions()' will not print a \"not available\" message in this\n" - "case.\n" - "\n" - "Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>\n" - "---\n" - " drivers/mtd/mtdpart.c | 5 +----\n" - " 1 files changed, 1 insertions(+), 4 deletions(-)\n" - "\n" - "diff --git a/drivers/mtd/mtdpart.c b/drivers/mtd/mtdpart.c\n" - "index 0a47601..cd631e7 100644\n" - "--- a/drivers/mtd/mtdpart.c\n" - "+++ b/drivers/mtd/mtdpart.c\n" - "@@ -722,11 +722,8 @@ int parse_mtd_partitions(struct mtd_info *master, const char **types,\n" - " \t\tparser = get_partition_parser(*types);\n" - " \t\tif (!parser && !request_module(\"%s\", *types))\n" - " \t\t\t\tparser = get_partition_parser(*types);\n" - "-\t\tif (!parser) {\n" - "-\t\t\tprintk(KERN_NOTICE \"%s partition parsing not available\\n\",\n" - "-\t\t\t *types);\n" - "+\t\tif (!parser)\n" - " \t\t\tcontinue;\n" - "-\t\t}\n" - " \t\tret = (*parser->parse_fn)(master, pparts, origin);\n" - " \t\tif (ret > 0) {\n" - " \t\t\tprintk(KERN_NOTICE \"%d %s partitions found on MTD device %s\\n\",\n" - "-- \n" - "1.7.2.3\n" - "\n" - "-- \n" - "Best Regards,\n" - "Artem Bityutskiy (\320\220\321\200\321\202\321\221\320\274 \320\221\320\270\321\202\321\216\321\206\320\272\320\270\320\271)" + Marc, could you please check it?): -62ca9e06ca038b3e119bc82448ab3427ed806ac613dd43e3590bd7de63f9abcf +79496ac99169a5645d703882d7a2b00522e1ac31d6b08bc2f142b0861293127b
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.