diff options
| author | Grant Likely <grant.likely@linaro.org> | 2014-06-04 16:42:26 +0100 |
|---|---|---|
| committer | Grant Likely <grant.likely@linaro.org> | 2014-06-26 17:12:24 +0100 |
| commit | ccdb8ed3b3c739fe99a6f2f474f7ffad3203485d (patch) | |
| tree | f556b308a0fb81e00a39155596fab68374159cc5 /drivers/edac | |
| parent | a752ee56ad84bf9a35b8323af1ad22b03c1df2c4 (diff) | |
| download | talos-op-linux-ccdb8ed3b3c739fe99a6f2f474f7ffad3203485d.tar.gz talos-op-linux-ccdb8ed3b3c739fe99a6f2f474f7ffad3203485d.zip | |
of: Migrate of_find_node_by_name() users to for_each_node_by_name()
There are a bunch of users open coding the for_each_node_by_name() by
calling of_find_node_by_name() directly instead of using the macro. This
is getting in the way of some cleanups, and the possibility of removing
of_find_node_by_name() entirely. Clean it up so that all the users are
consistent.
Signed-off-by: Grant Likely <grant.likely@linaro.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'drivers/edac')
| -rw-r--r-- | drivers/edac/cell_edac.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/edac/cell_edac.c b/drivers/edac/cell_edac.c index 374b57fc596d..a12c8552f6a6 100644 --- a/drivers/edac/cell_edac.c +++ b/drivers/edac/cell_edac.c @@ -134,8 +134,7 @@ static void cell_edac_init_csrows(struct mem_ctl_info *mci) int j; u32 nr_pages; - for (np = NULL; - (np = of_find_node_by_name(np, "memory")) != NULL;) { + for_each_node_by_name(np, "memory") { struct resource r; /* We "know" that the Cell firmware only creates one entry |
