diff options
author | Brian Norris <computersforpeace@gmail.com> | 2015-12-03 17:26:52 -0500 |
---|---|---|
committer | Brian Norris <computersforpeace@gmail.com> | 2015-12-03 20:50:36 -0500 |
commit | 8c62b4e118cfa7a3c906c01d4ba2c78a5bd97531 (patch) | |
tree | cb08edad2c1ff8a97f6c6e3d3c9ee1a88887a4b5 | |
parent | 1ddaa021b000220b5f2ad023e4f15ed44990974b (diff) |
mtd: ofpart: don't complain about missing 'partitions' node too loudly
The ofpart partition parser might be run on DT-enabled systems that
don't have any "ofpart" partition subnodes at all, since "ofpart" is in
the default parser list. So don't complain loudly on every boot.
Example: using m25p80.c with no intent to use ofpart:
&spi2 {
status = "okay";
flash@0 {
compatible = "jedec,spi-nor";
reg = <0>;
};
};
I see this warning:
[ 0.588471] m25p80 spi2.0: gd25q32 (4096 Kbytes)
[ 0.593091] spi2.0: 'partitions' subnode not found on /spi@ff130000/flash@0. Trying to parse direct subnodes as partitions.
Cc: Michal Suchanek <hramrach@gmail.com>
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
-rw-r--r-- | drivers/mtd/ofpart.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/drivers/mtd/ofpart.c b/drivers/mtd/ofpart.c index 669c3452f278..3e9c5857c991 100644 --- a/drivers/mtd/ofpart.c +++ b/drivers/mtd/ofpart.c | |||
@@ -46,8 +46,13 @@ static int parse_ofpart_partitions(struct mtd_info *master, | |||
46 | 46 | ||
47 | ofpart_node = of_get_child_by_name(mtd_node, "partitions"); | 47 | ofpart_node = of_get_child_by_name(mtd_node, "partitions"); |
48 | if (!ofpart_node) { | 48 | if (!ofpart_node) { |
49 | pr_warn("%s: 'partitions' subnode not found on %s. Trying to parse direct subnodes as partitions.\n", | 49 | /* |
50 | master->name, mtd_node->full_name); | 50 | * We might get here even when ofpart isn't used at all (e.g., |
51 | * when using another parser), so don't be louder than | ||
52 | * KERN_DEBUG | ||
53 | */ | ||
54 | pr_debug("%s: 'partitions' subnode not found on %s. Trying to parse direct subnodes as partitions.\n", | ||
55 | master->name, mtd_node->full_name); | ||
51 | ofpart_node = mtd_node; | 56 | ofpart_node = mtd_node; |
52 | dedicated = false; | 57 | dedicated = false; |
53 | } | 58 | } |