diff options
| author |   Bjorn Andersson <bjorn.andersson@linaro.org> | 2017-10-10 22:08:53 -0700 |
|---|---|---|
| committer |   Andy Gross <andy.gross@linaro.org> | 2017-10-22 05:06:33 -0500 |
| commit | a50ff19d0658d8a09026158448894f06a4521f33 (patch) | |
| tree | f9f4927d4bb7dd10ea5cbcdce4abe8a0a4e61679 /drivers/of | |
| parent | firmware: qcom: scm: Expose download-mode control (diff) | |
| download | linux-dev-a50ff19d0658d8a09026158448894f06a4521f33.tar.xz linux-dev-a50ff19d0658d8a09026158448894f06a4521f33.zip | |
of/platform: Generalize /reserved-memory handling
By iterating over all /reserved-memory child nodes and match each one to
a list of compatibles that we want to treat specially, we can easily
extend the list of compatibles to handle - without having to resort to
of_platform_populate() that would create unnecessary platform_devices.
Reviewed-by: Rob Herring <robh@kernel.org>
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Signed-off-by: Andy Gross <andy.gross@linaro.org>
Diffstat (limited to 'drivers/of')
| -rw-r--r-- | drivers/of/platform.c | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/drivers/of/platform.c b/drivers/of/platform.c index ac15d0e3d27d..922b46646bcc 100644 --- a/drivers/of/platform.c +++ b/drivers/of/platform.c @@ -497,6 +497,11 @@ int of_platform_default_populate(struct device_node *root, EXPORT_SYMBOL_GPL(of_platform_default_populate); #ifndef CONFIG_PPC +static const struct of_device_id reserved_mem_matches[] = { + { .compatible = "ramoops" }, + {} +}; + static int __init of_platform_default_populate_init(void) { struct device_node *node; @@ -505,15 +510,12 @@ static int __init of_platform_default_populate_init(void) return -ENODEV; /* - * Handle ramoops explicitly, since it is inside /reserved-memory, - * which lacks a "compatible" property. + * Handle certain compatibles explicitly, since we don't want to create + * platform_devices for every node in /reserved-memory with a + * "compatible", */ - node = of_find_node_by_path("/reserved-memory"); - if (node) { - node = of_find_compatible_node(node, NULL, "ramoops"); - if (node) - of_platform_device_create(node, NULL, NULL); - } + for_each_matching_node(node, reserved_mem_matches) + of_platform_device_create(node, NULL, NULL); /* Populate everything else. */ of_platform_default_populate(NULL, NULL, NULL); |