diff mbox series

[v2] Makefile: doesn't need check stack size when dtb is not built

Message ID 20200310002043.15523-1-takahiro.akashi@linaro.org
State Accepted
Commit dde1b75e9526d887a9aba9d4b30eb62ad2f22013
Headers show
Series [v2] Makefile: doesn't need check stack size when dtb is not built | expand

Commit Message

AKASHI Takahiro March 10, 2020, 12:20 a.m. UTC
The commit 5fed97af20da ("Makefile: ensure DTB doesn't overflow into
initial stack") adds an extra check for stack size in BSS if
CONFIG_SYS_INIT_SP_BSS_OFFSET is enabled.
This check, however, doesn't make sense under the configuration where
control dtb won't be built in and it should be void in such cases.

Signed-off-by: AKASHI Takahiro <takahiro.akashi at linaro.org>
Fixes: 5fed97af20da ("Makefile: ensure DTB doesn't overflow into
	initial stack")
---
v2:
* guard ALL-y += init_sp_bss_offset_check with simplified "if" condition
---
 Makefile | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Stephen Warren March 10, 2020, 3:56 p.m. UTC | #1
On 3/9/20 6:20 PM, AKASHI Takahiro wrote:
> The commit 5fed97af20da ("Makefile: ensure DTB doesn't overflow into
> initial stack") adds an extra check for stack size in BSS if
> CONFIG_SYS_INIT_SP_BSS_OFFSET is enabled.
> This check, however, doesn't make sense under the configuration where
> control dtb won't be built in and it should be void in such cases.
> 
> Signed-off-by: AKASHI Takahiro <takahiro.akashi at linaro.org>
> Fixes: 5fed97af20da ("Makefile: ensure DTB doesn't overflow into
> 	initial stack")

It would be best if that line wasn't wrapped; that will make it easier 
for anyone searching for the commit description, since they won't have 
to know to search for a substring of the the description instead of the 
whole thing.

Reviewed-by: Stephen Warren <swarren at nvidia.com>
AKASHI Takahiro March 11, 2020, 12:01 a.m. UTC | #2
Tom,

On Tue, Mar 10, 2020 at 09:56:42AM -0600, Stephen Warren wrote:
> On 3/9/20 6:20 PM, AKASHI Takahiro wrote:
> > The commit 5fed97af20da ("Makefile: ensure DTB doesn't overflow into
> > initial stack") adds an extra check for stack size in BSS if
> > CONFIG_SYS_INIT_SP_BSS_OFFSET is enabled.
> > This check, however, doesn't make sense under the configuration where
> > control dtb won't be built in and it should be void in such cases.
> > 
> > Signed-off-by: AKASHI Takahiro <takahiro.akashi at linaro.org>
> > Fixes: 5fed97af20da ("Makefile: ensure DTB doesn't overflow into
> > 	initial stack")
> 
> It would be best if that line wasn't wrapped; that will make it easier for
> anyone searching for the commit description, since they won't have to know
> to search for a substring of the the description instead of the whole thing.

Can you make this change for me when you merge this patch, please?

> Reviewed-by: Stephen Warren <swarren at nvidia.com>

Thanks,
-Takahiro Akashi
Tom Rini March 13, 2020, 5:25 p.m. UTC | #3
On Tue, Mar 10, 2020 at 09:20:43AM +0900, AKASHI Takahiro wrote:

> The commit 5fed97af20da ("Makefile: ensure DTB doesn't overflow into
> initial stack") adds an extra check for stack size in BSS if
> CONFIG_SYS_INIT_SP_BSS_OFFSET is enabled.
> This check, however, doesn't make sense under the configuration where
> control dtb won't be built in and it should be void in such cases.
> 
> Signed-off-by: AKASHI Takahiro <takahiro.akashi at linaro.org>
> Fixes: 5fed97af20da ("Makefile: ensure DTB doesn't overflow into initial stack")
> Reviewed-by: Stephen Warren <swarren at nvidia.com>

(I undid the linewrap as requested)

Applied to u-boot/master, thanks!
diff mbox series

Patch

diff --git a/Makefile b/Makefile
index 0af89e0a7881..93931c3cb51b 100644
--- a/Makefile
+++ b/Makefile
@@ -903,7 +903,7 @@  ifneq ($(CONFIG_BUILD_TARGET),)
 ALL-y += $(CONFIG_BUILD_TARGET:"%"=%)
 endif
 
-ifdef CONFIG_INIT_SP_RELATIVE
+ifeq ($(CONFIG_INIT_SP_RELATIVE)$(CONFIG_OF_SEPARATE),yy)
 ALL-y += init_sp_bss_offset_check
 endif
 
@@ -1208,7 +1208,7 @@  binary_size_check: u-boot-nodtb.bin FORCE
 		fi \
 	fi
 
-ifdef CONFIG_INIT_SP_RELATIVE
+ifeq ($(CONFIG_INIT_SP_RELATIVE)$(CONFIG_OF_SEPARATE),yy)
 ifneq ($(CONFIG_SYS_MALLOC_F_LEN),)
 subtract_sys_malloc_f_len = space=$$(($${space} - $(CONFIG_SYS_MALLOC_F_LEN)))
 else