diff options
| author |   권오훈 <ohoono.kwon@samsung.com> | 2021-07-01 23:04:57 +0900 |
|---|---|---|
| committer |   Rob Herring <robh@kernel.org> | 2021-07-15 07:35:49 -0600 |
| commit | eb7173988caf6fc68ef00065b9defb5ac3467f21 (patch) | |
| tree | 10e718e32e5fe7d6c62cd16bee915856d4ff64d5 /tools/perf/scripts/python/export-to-postgresql.py | |
| parent | of: base: remove unnecessary for loop (diff) | |
| download | wireguard-linux-eb7173988caf6fc68ef00065b9defb5ac3467f21.tar.xz wireguard-linux-eb7173988caf6fc68ef00065b9defb5ac3467f21.zip | |
of: fdt: remove unnecessary codes
While unflattening the device tree, we try to populate dt nodes and
properties into tree-shaped data structure.
In populate_properties function, pprev is initially set to
&np->properties, and then updated to &pp->next.
In both scenarios *pprev is NULL, since the memory area that we are
allocating from is initially zeroed.
I tested the code as below, and it showed that BUG was never called.
- if (!dryrun)
+ if (!dryrun) {
+ if (*pprev)
+ BUG();
*pprev = NULL;
+ }
Let's remove unnecessary code.
Signed-off-by: Ohhoon Kwon <ohoono.kwon@samsung.com>
Link: https://lore.kernel.org/r/20210701140457epcms1p2cc43a7c62150f012619feab913f017af@epcms1p2
Signed-off-by: Rob Herring <robh@kernel.org>
Diffstat (limited to 'tools/perf/scripts/python/export-to-postgresql.py')
0 files changed, 0 insertions, 0 deletions