Commit 01f9c724 authored by Brian Norris's avatar Brian Norris
Browse files

mtd: partitions: factor out code calling parser



This code is going to be reused for parsers matched using OF so let's
factor it out to make this easier.

Signed-off-by: default avatarBrian Norris <computersforpeace@gmail.com>
Signed-off-by: default avatarRafał Miłecki <rafal@milecki.pl>
Acked-by: default avatarBrian Norris <computersforpeace@gmail.com>
parent df1d1788
Loading
Loading
Loading
Loading
+24 −9
Original line number Diff line number Diff line
@@ -807,6 +807,27 @@ static const char * const default_mtd_part_types[] = {
	NULL
};

static int mtd_part_do_parse(struct mtd_part_parser *parser,
			     struct mtd_info *master,
			     struct mtd_partitions *pparts,
			     struct mtd_part_parser_data *data)
{
	int ret;

	ret = (*parser->parse_fn)(master, &pparts->parts, data);
	pr_debug("%s: parser %s: %i\n", master->name, parser->name, ret);
	if (ret <= 0)
		return ret;

	pr_notice("%d %s partitions found on MTD device %s\n", ret,
		  parser->name, master->name);

	pparts->nr_parts = ret;
	pparts->parser = parser;

	return ret;
}

/**
 * parse_mtd_partitions - parse MTD partitions
 * @master: the master partition (describes whole MTD device)
@@ -847,16 +868,10 @@ int parse_mtd_partitions(struct mtd_info *master, const char *const *types,
			 parser ? parser->name : NULL);
		if (!parser)
			continue;
		ret = (*parser->parse_fn)(master, &pparts->parts, data);
		pr_debug("%s: parser %s: %i\n",
			 master->name, parser->name, ret);
		if (ret > 0) {
			printk(KERN_NOTICE "%d %s partitions found on MTD device %s\n",
			       ret, parser->name, master->name);
			pparts->nr_parts = ret;
			pparts->parser = parser;
		ret = mtd_part_do_parse(parser, master, pparts, data);
		/* Found partitions! */
		if (ret > 0)
			return 0;
		}
		mtd_part_parser_put(parser);
		/*
		 * Stash the first error we see; only report it if no parser