All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/1] riscv: CONFIG_SPL_FRAMEPOINTER must depend on CONFIG_SPL
@ 2024-08-11  9:51 Heinrich Schuchardt
  2024-08-13  3:17 ` Leo Liang
  2024-08-13  7:39 ` Ben Dooks
  0 siblings, 2 replies; 3+ messages in thread
From: Heinrich Schuchardt @ 2024-08-11  9:51 UTC (permalink / raw)
  To: Rick Chen, Leo; +Cc: Ben Dooks, u-boot, Heinrich Schuchardt

The CONFIG_SPL_FRAMEPOINTER symbol is only relevant in SPL.

Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
---
 arch/riscv/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
index fa3b016c527..4e228a659f2 100644
--- a/arch/riscv/Kconfig
+++ b/arch/riscv/Kconfig
@@ -133,6 +133,7 @@ config FRAMEPOINTER
 
 config SPL_FRAMEPOINTER
 	bool "Build SPL with frame pointer for stack unwinding"
+	depends on SPL
 	help
 	  Choose this option to use the frame pointer so the stack can be
 	  unwound if needed. This is useful for tracing where faults came
-- 
2.45.2


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/1] riscv: CONFIG_SPL_FRAMEPOINTER must depend on CONFIG_SPL
  2024-08-11  9:51 [PATCH 1/1] riscv: CONFIG_SPL_FRAMEPOINTER must depend on CONFIG_SPL Heinrich Schuchardt
@ 2024-08-13  3:17 ` Leo Liang
  2024-08-13  7:39 ` Ben Dooks
  1 sibling, 0 replies; 3+ messages in thread
From: Leo Liang @ 2024-08-13  3:17 UTC (permalink / raw)
  To: Heinrich Schuchardt; +Cc: Rick Chen, Ben Dooks, u-boot

On Sun, Aug 11, 2024 at 11:51:09AM +0200, Heinrich Schuchardt wrote:
> [EXTERNAL MAIL]
> 
> The CONFIG_SPL_FRAMEPOINTER symbol is only relevant in SPL.
> 
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
> ---
>  arch/riscv/Kconfig | 1 +
>  1 file changed, 1 insertion(+)

Reviewed-by: Leo Yu-Chi Liang <ycliang@andestech.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/1] riscv: CONFIG_SPL_FRAMEPOINTER must depend on CONFIG_SPL
  2024-08-11  9:51 [PATCH 1/1] riscv: CONFIG_SPL_FRAMEPOINTER must depend on CONFIG_SPL Heinrich Schuchardt
  2024-08-13  3:17 ` Leo Liang
@ 2024-08-13  7:39 ` Ben Dooks
  1 sibling, 0 replies; 3+ messages in thread
From: Ben Dooks @ 2024-08-13  7:39 UTC (permalink / raw)
  To: Heinrich Schuchardt, Rick Chen, Leo; +Cc: u-boot

On 11/08/2024 10:51, Heinrich Schuchardt wrote:
> The CONFIG_SPL_FRAMEPOINTER symbol is only relevant in SPL.
> 
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>

Thanks for spotting

Reviewed-by: Ben Dooks <ben.dooks@codethink.co.uk>

> ---
>   arch/riscv/Kconfig | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig
> index fa3b016c527..4e228a659f2 100644
> --- a/arch/riscv/Kconfig
> +++ b/arch/riscv/Kconfig
> @@ -133,6 +133,7 @@ config FRAMEPOINTER
>   
>   config SPL_FRAMEPOINTER
>   	bool "Build SPL with frame pointer for stack unwinding"
> +	depends on SPL
>   	help
>   	  Choose this option to use the frame pointer so the stack can be
>   	  unwound if needed. This is useful for tracing where faults came

-- 
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius

https://www.codethink.co.uk/privacy.html


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-08-13  7:39 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-08-11  9:51 [PATCH 1/1] riscv: CONFIG_SPL_FRAMEPOINTER must depend on CONFIG_SPL Heinrich Schuchardt
2024-08-13  3:17 ` Leo Liang
2024-08-13  7:39 ` Ben Dooks

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.