| Message ID | 832f494b89b80c97702bee084bab7000a3ec0854.1523946361.git.viresh.kumar@linaro.org |
|---|---|
| State | Accepted, archived |
| Headers | show |
| Series | [V3] of: Don't create device for OPP tables | expand |
On Tue, Apr 17, 2018 at 11:57:34AM +0530, Viresh Kumar wrote: > The OPP tables are present as separate nodes, whose phandle is used in > the "operating-points-v2" property of devices. Currently the OF core > creates a device for the OPP table unconditionally, which is not used by > any kernel code. > > This patch creates another OF device_id table for the nodes which must > be skipped while creating devices and OPP is the only user of it for now. > > Reported-by: Stephen Boyd <sboyd@kernel.org> > Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> > Reviewed-by: Stephen Boyd <sboyd@kernel.org> > --- > V2->V3: > - Staticize of_skipped_node_table > > V1->V2: > - Added Reviewed-by from Stephen. > - Created a table for nodes to be skipped. > > drivers/of/platform.c | 11 +++++++++++ > 1 file changed, 11 insertions(+) Applied, thanks. Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/of/platform.c b/drivers/of/platform.c index c00d81dfac0b..d75050b85d0c 100644 --- a/drivers/of/platform.c +++ b/drivers/of/platform.c @@ -32,6 +32,11 @@ const struct of_device_id of_default_bus_match_table[] = { {} /* Empty terminated list */ }; +static const struct of_device_id of_skipped_node_table[] = { + { .compatible = "operating-points-v2", }, + {} /* Empty terminated list */ +}; + static int of_dev_node_match(struct device *dev, void *data) { return dev->of_node == data; @@ -356,6 +361,12 @@ static int of_platform_bus_create(struct device_node *bus, return 0; } + /* Skip nodes for which we don't want to create devices */ + if (unlikely(of_match_node(of_skipped_node_table, bus))) { + pr_debug("%s() - skipping %pOF node\n", __func__, bus); + return 0; + } + if (of_node_check_flag(bus, OF_POPULATED_BUS)) { pr_debug("%s() - skipping %pOF, already populated\n", __func__, bus);