LM
/
u-boot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

of: addr: Remove call to dev_count_cells() in of_get_address() 

In of_get_address(), there is:

  dev_count_cells(dev, &na, &ns);

followed by:

  bus->count_cells(dev, &na, &ns);

but no codes in between use na/ns, hence the first call is useless.
By dropping the first call, dev_count_cells() is now useless too.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Bin Meng 2021-04-30 21:16:59 +08:00 committed by Simon Glass
parent 139eaa7a2b
commit 0e35b937b9
1 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
drivers/core/of_addr.c
View File

@ -118,11 +118,6 @@ static struct of_bus *of_match_bus(struct device_node *np)
return NULL; return NULL;
} }
static void dev_count_cells(const struct device_node *np, int *nap, int *nsp)
{
of_bus_default_count_cells(np, nap, nsp);
}
const __be32 *of_get_address(const struct device_node *dev, int index, const __be32 *of_get_address(const struct device_node *dev, int index,
u64 *size, unsigned int *flags) u64 *size, unsigned int *flags)
{ {
@ -136,7 +131,6 @@ const __be32 *of_get_address(const struct device_node *dev, int index,
parent = of_get_parent(dev); parent = of_get_parent(dev);
if (parent == NULL) if (parent == NULL)
return NULL; return NULL;
dev_count_cells(dev, &na, &ns);
bus = of_match_bus(parent); bus = of_match_bus(parent);
bus->count_cells(dev, &na, &ns); bus->count_cells(dev, &na, &ns);
of_node_put(parent); of_node_put(parent);