From e1986bc2eff29ecf08dc42c4081d4a0be85f262e Mon Sep 17 00:00:00 2001 From: Kevin Schoedel <67607049+kpschoedel@users.noreply.github.com> Date: Wed, 15 Dec 2021 17:22:09 -0500 Subject: [PATCH] [Size reports] Remove .heap from reports (#13056) #### Problem For platforms that use it, the `.heap` section indicates memory available for the heap, that is, RAM not otherwise in use. Including it in size reports is confusing because a `.heap` increase is good. #### Change overview Remove `.heap` from the default section list of the platforms where it currently appears, namely Mbed and P6. #### Testing Manually checked on a P6 binary. --- scripts/tools/memory/platform/mbed.cfg | 2 +- scripts/tools/memory/platform/p6.cfg | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/tools/memory/platform/mbed.cfg b/scripts/tools/memory/platform/mbed.cfg index f261ef24a6f9d3..ca5d43d692eb7f 100644 --- a/scripts/tools/memory/platform/mbed.cfg +++ b/scripts/tools/memory/platform/mbed.cfg @@ -18,7 +18,7 @@ 'section': { # By default, only these sections will be included # when operating by sections. - 'default': ['.text', '.data', '.bss', '.heap'], + 'default': ['.text', '.data', '.bss'], }, # 'symbol': { # 'free': { diff --git a/scripts/tools/memory/platform/p6.cfg b/scripts/tools/memory/platform/p6.cfg index 96990a3cc7b5b6..d1870ee8e1fb79 100644 --- a/scripts/tools/memory/platform/p6.cfg +++ b/scripts/tools/memory/platform/p6.cfg @@ -18,7 +18,7 @@ 'section': { # By default, only these sections will be included # when operating by sections. - 'default': ['.text', '.data', '.bss', '.heap'], + 'default': ['.text', '.data', '.bss'], }, # 'symbol': { # 'free': {