From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Brezillon Subject: Re: [PATCH V3 3/3] mtd: physmap_of: drop duplicated support for linux,part-probe property Date: Sun, 9 Apr 2017 13:04:46 +0200 Message-ID: <20170409130446.107a3a32@bbrezillon> References: <20170331114016.26858-1-zajec5@gmail.com> <20170331114016.26858-3-zajec5@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Return-path: In-Reply-To: <20170331114016.26858-3-zajec5-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= Cc: David Woodhouse , Brian Norris , Marek Vasut , Richard Weinberger , Cyrille Pitchen , Rob Herring , Mark Rutland , Frank Rowand , Linus Walleij , linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= List-Id: devicetree@vger.kernel.org On Fri, 31 Mar 2017 13:40:16 +0200 Rafał Miłecki wrote: > From: Rafał Miłecki > > Now support for linux,part-probe has been added to the MTD core there is > no need to duplicate support for it in physmap_of. > > Signed-off-by: Rafał Miłecki Acked-by: Boris Brezillon > --- > drivers/mtd/maps/physmap_of.c | 46 +------------------------------------------ > 1 file changed, 1 insertion(+), 45 deletions(-) > > diff --git a/drivers/mtd/maps/physmap_of.c b/drivers/mtd/maps/physmap_of.c > index 14e8909c9955..49dbb7235848 100644 > --- a/drivers/mtd/maps/physmap_of.c > +++ b/drivers/mtd/maps/physmap_of.c > @@ -114,47 +114,9 @@ static struct mtd_info *obsolete_probe(struct platform_device *dev, > static const char * const part_probe_types_def[] = { > "cmdlinepart", "RedBoot", "ofpart", "ofoldpart", NULL }; > > -static const char * const *of_get_probes(struct device_node *dp) > -{ > - const char *cp; > - int cplen; > - unsigned int l; > - unsigned int count; > - const char **res; > - > - cp = of_get_property(dp, "linux,part-probe", &cplen); > - if (cp == NULL) > - return part_probe_types_def; > - > - count = 0; > - for (l = 0; l != cplen; l++) > - if (cp[l] == 0) > - count++; > - > - res = kzalloc((count + 1)*sizeof(*res), GFP_KERNEL); > - if (!res) > - return NULL; > - count = 0; > - while (cplen > 0) { > - res[count] = cp; > - l = strlen(cp) + 1; > - cp += l; > - cplen -= l; > - count++; > - } > - return res; > -} > - > -static void of_free_probes(const char * const *probes) > -{ > - if (probes != part_probe_types_def) > - kfree(probes); > -} > - > static const struct of_device_id of_flash_match[]; > static int of_flash_probe(struct platform_device *dev) > { > - const char * const *part_probe_types; > const struct of_device_id *match; > struct device_node *dp = dev->dev.of_node; > struct resource res; > @@ -320,14 +282,8 @@ static int of_flash_probe(struct platform_device *dev) > > info->cmtd->dev.parent = &dev->dev; > mtd_set_of_node(info->cmtd, dp); > - part_probe_types = of_get_probes(dp); > - if (!part_probe_types) { > - err = -ENOMEM; > - goto err_out; > - } > - mtd_device_parse_register(info->cmtd, part_probe_types, NULL, > + mtd_device_parse_register(info->cmtd, part_probe_types_def, NULL, > NULL, 0); > - of_free_probes(part_probe_types); > > kfree(mtd_list); > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html