diff options
author | Richard Spiegel <richard.spiegel@amd.corp-partner.google.com> | 2018-04-17 14:25:26 -0700 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2018-04-19 15:05:08 +0000 |
commit | 51895d183861234db9500bd1ee33634776e91e93 (patch) | |
tree | 70ce98b4a168c8cc6fdfe867d945c353638990c1 /util/superiotool/winbond.c | |
parent | 26be35a50720f7dc60aa6a28dae151b47cb782ce (diff) |
soc/amd/common/block/pi/heapmanager.c: Simplify code
There are sections of code that are almost identical and they can be
converted to auxiliary procedures. For allocating heap, 3 sizes (the
buffer size of currently being examined node, the buffer size of the
current best fit node and the minimum size for a buffer that will need
to be split if selected as the best fit) are used often so they could
be stored in temporary variables. These 2 changes will make code shorter,
with less indentation problems and overall easier to read. The actual
logic of the code is not changed.
BUG=b:77940747
TEST=Build and boot grunt.
Change-Id: Ib4c69981eab7452228ccae9ed9bc288c8baceffe
Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com>
Reviewed-on: https://review.coreboot.org/25703
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'util/superiotool/winbond.c')
0 files changed, 0 insertions, 0 deletions