public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Leo Liang <ycliang@andestech.com>
To: u-boot@lists.denx.de
Subject: [PATCH 1/1] sandbox: Make tracing totally depends on Kconfig
Date: Wed, 25 Nov 2020 11:24:11 +0800	[thread overview]
Message-ID: <20201125032407.GA423@andestech.com> (raw)

 With this patch, 'make sandbox_defconfig', choose tracing support option then 'make',
 would be enough when trying tracing feature with sandbox.
 
 No need to add FTRACE=1
 
 Signed-off-by: Leo Yu-Chi Liang <ycliang@andestech.com>
 ---
  config.mk                 | 2 +-
  include/configs/sandbox.h | 7 -------
  lib/Kconfig               | 2 +-
  3 files changed, 2 insertions(+), 9 deletions(-)
 
 diff --git a/config.mk b/config.mk
 index 7bb1fd4ed1..4ca1774280 100644
 --- a/config.mk
 +++ b/config.mk
 @@ -58,7 +58,7 @@ ifdef	BOARD
  sinclude $(srctree)/board/$(BOARDDIR)/config.mk	# include board specific rules
  endif
  
 -ifdef FTRACE
 +ifdef CONFIG_TRACE
  PLATFORM_CPPFLAGS += -finstrument-functions -DFTRACE
  endif
  
 diff --git a/include/configs/sandbox.h b/include/configs/sandbox.h
 index e0708fe573..7589162b40 100644
 --- a/include/configs/sandbox.h
 +++ b/include/configs/sandbox.h
 @@ -6,13 +6,6 @@
  #ifndef __CONFIG_H
  #define __CONFIG_H
  
 -#ifdef FTRACE
 -#define CONFIG_TRACE
 -#define CONFIG_TRACE_BUFFER_SIZE	(16 << 20)
 -#define CONFIG_TRACE_EARLY_SIZE		(16 << 20)
 -#define CONFIG_TRACE_EARLY
 -#define CONFIG_TRACE_EARLY_ADDR		0x00100000
 -#endif
  
  #ifndef CONFIG_SPL_BUILD
  #define CONFIG_IO_TRACE
 diff --git a/lib/Kconfig b/lib/Kconfig
 index 7673d2e4e0..a6bd7fd3ff 100644
 --- a/lib/Kconfig
 +++ b/lib/Kconfig
 @@ -209,7 +209,7 @@ config BITREVERSE
  
  config TRACE
  	bool "Support for tracing of function calls and timing"
 -	imply CMD_TRACE
 +	select CMD_TRACE
  	help
  	  Enables function tracing within U-Boot. This allows recording of call
  	  traces including timing information. The command can write data to
 -- 
 2.17.0

                 reply	other threads:[~2020-11-25  3:24 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20201125032407.GA423@andestech.com \
    --to=ycliang@andestech.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox