All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Peng Fan (OSS)" <peng.fan@oss.nxp.com>
To: Tom Rini <trini@konsulko.com>, u-boot@lists.denx.de
Subject: Re: [PATCH 1/5] kgdb: Remove unused serial related options
Date: Wed, 15 Sep 2021 10:07:23 +0800	[thread overview]
Message-ID: <42148ee4-6a4e-5be1-2cb1-927c6bdd1762@oss.nxp.com> (raw)
In-Reply-To: <20210913212455.29165-1-trini@konsulko.com>

Just wonder anyone using kgdb in U-Boot?
And sharing the same uart with normal output?

Thanks,
Peng

On 2021/9/14 5:24, Tom Rini wrote:
> We have a few CONFIG options for KGDB that are not referenced, remove
> them.
> 
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
>   README                              | 5 -----
>   include/configs/MPC8349EMDS.h       | 4 ----
>   include/configs/MPC8349EMDS_SDRAM.h | 4 ----
>   include/configs/MPC837XERDB.h       | 4 ----
>   include/configs/MPC8540ADS.h        | 4 ----
>   include/configs/MPC8548CDS.h        | 4 ----
>   include/configs/MPC8560ADS.h        | 4 ----
>   include/configs/P1010RDB.h          | 4 ----
>   include/configs/P2041RDB.h          | 4 ----
>   include/configs/T102xRDB.h          | 4 ----
>   include/configs/T104xRDB.h          | 4 ----
>   include/configs/T208xQDS.h          | 5 -----
>   include/configs/T208xRDB.h          | 5 -----
>   include/configs/T4240RDB.h          | 4 ----
>   include/configs/UCP1020.h           | 5 -----
>   include/configs/corenet_ds.h        | 4 ----
>   include/configs/p1_p2_rdb_pc.h      | 4 ----
>   include/configs/pxa-common.h        | 7 -------
>   include/configs/socrates.h          | 4 ----
>   include/configs/x86-common.h        | 4 ----
>   20 files changed, 87 deletions(-)
> 
> diff --git a/README b/README
> index 6df06c79bc83..e9a6df4cd1de 100644
> --- a/README
> +++ b/README
> @@ -648,11 +648,6 @@ The following options need to be configured:
>   		time on others. This setting #define's the initial
>   		value of the "loads_echo" environment variable.
>   
> -- Kgdb Serial Baudrate: (if CONFIG_CMD_KGDB is defined)
> -		CONFIG_KGDB_BAUDRATE
> -		Select one of the baudrates listed in
> -		CONFIG_SYS_BAUDRATE_TABLE, see below.
> -
>   - Removal of commands
>   		If no commands are needed to boot, you can disable
>   		CONFIG_CMDLINE to remove them. In this case, the command line
> diff --git a/include/configs/MPC8349EMDS.h b/include/configs/MPC8349EMDS.h
> index 2f1fc6a6a2b9..d96d3ce6967b 100644
> --- a/include/configs/MPC8349EMDS.h
> +++ b/include/configs/MPC8349EMDS.h
> @@ -255,10 +255,6 @@
>   #define CONFIG_PCI_INDIRECT_BRIDGE
>   #endif
>   
> -#if defined(CONFIG_CMD_KGDB)
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed of kgdb serial port */
> -#endif
> -
>   /*
>    * Environment Configuration
>    */
> diff --git a/include/configs/MPC8349EMDS_SDRAM.h b/include/configs/MPC8349EMDS_SDRAM.h
> index d6a151d20ce6..0f8db35ae90e 100644
> --- a/include/configs/MPC8349EMDS_SDRAM.h
> +++ b/include/configs/MPC8349EMDS_SDRAM.h
> @@ -312,10 +312,6 @@
>   #define CONFIG_PCI_INDIRECT_BRIDGE
>   #endif
>   
> -#if defined(CONFIG_CMD_KGDB)
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed of kgdb serial port */
> -#endif
> -
>   /*
>    * Environment Configuration
>    */
> diff --git a/include/configs/MPC837XERDB.h b/include/configs/MPC837XERDB.h
> index 0e2e034dcd9b..ce810e5f2cbe 100644
> --- a/include/configs/MPC837XERDB.h
> +++ b/include/configs/MPC837XERDB.h
> @@ -323,10 +323,6 @@
>   #define CONFIG_SYS_BOOTMAPSZ	(256 << 20) /* Initial Memory map for Linux */
>   #define CONFIG_SYS_BOOTM_LEN	(64 << 20)	/* Increase max gunzip size */
>   
> -#if defined(CONFIG_CMD_KGDB)
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed of kgdb serial port */
> -#endif
> -
>   /*
>    * Environment Configuration
>    */
> diff --git a/include/configs/MPC8540ADS.h b/include/configs/MPC8540ADS.h
> index 66cf2053f980..b73a7c9ac955 100644
> --- a/include/configs/MPC8540ADS.h
> +++ b/include/configs/MPC8540ADS.h
> @@ -289,10 +289,6 @@
>   #define CONFIG_SYS_BOOTMAPSZ	(64 << 20)	/* Initial Memory map for Linux*/
>   #define CONFIG_SYS_BOOTM_LEN	(64 << 20)	/* Increase max gunzip size */
>   
> -#if defined(CONFIG_CMD_KGDB)
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed to run kgdb serial port */
> -#endif
> -
>   /*
>    * Environment Configuration
>    */
> diff --git a/include/configs/MPC8548CDS.h b/include/configs/MPC8548CDS.h
> index f7d05ffa89a1..5c54bad4964f 100644
> --- a/include/configs/MPC8548CDS.h
> +++ b/include/configs/MPC8548CDS.h
> @@ -415,10 +415,6 @@ extern unsigned long get_clock_freq(void);
>   #define CONFIG_SYS_BOOTMAPSZ	(64 << 20)	/* Initial Memory map for Linux*/
>   #define CONFIG_SYS_BOOTM_LEN	(64 << 20)	/* Increase max gunzip size */
>   
> -#if defined(CONFIG_CMD_KGDB)
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed to run kgdb serial port */
> -#endif
> -
>   /*
>    * Environment Configuration
>    */
> diff --git a/include/configs/MPC8560ADS.h b/include/configs/MPC8560ADS.h
> index 224314366439..23ac9f8e6ece 100644
> --- a/include/configs/MPC8560ADS.h
> +++ b/include/configs/MPC8560ADS.h
> @@ -279,10 +279,6 @@
>   #define CONFIG_SYS_BOOTMAPSZ	(64 << 20)	/* Initial Memory map for Linux*/
>   #define CONFIG_SYS_BOOTM_LEN	(64 << 20)	/* Increase max gunzip size */
>   
> -#if defined(CONFIG_CMD_KGDB)
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed to run kgdb serial port */
> -#endif
> -
>   /*
>    * Environment Configuration
>    */
> diff --git a/include/configs/P1010RDB.h b/include/configs/P1010RDB.h
> index 02b0b71ada56..922f0661aba2 100644
> --- a/include/configs/P1010RDB.h
> +++ b/include/configs/P1010RDB.h
> @@ -619,10 +619,6 @@ extern unsigned long get_sdram_size(void);
>   #define CONFIG_SYS_BOOTMAPSZ	(64 << 20) /* Initial Memory map for Linux */
>   #define CONFIG_SYS_BOOTM_LEN	(64 << 20) /* Increase max gunzip size */
>   
> -#if defined(CONFIG_CMD_KGDB)
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed to run kgdb serial port */
> -#endif
> -
>   /*
>    * Environment Configuration
>    */
> diff --git a/include/configs/P2041RDB.h b/include/configs/P2041RDB.h
> index 6da44e7ab828..1e75066eb1c9 100644
> --- a/include/configs/P2041RDB.h
> +++ b/include/configs/P2041RDB.h
> @@ -466,10 +466,6 @@ unsigned long get_board_sys_clk(unsigned long dummy);
>   #define CONFIG_SYS_BOOTMAPSZ	(64 << 20)	/* Initial Memory for Linux */
>   #define CONFIG_SYS_BOOTM_LEN	(64 << 20)	/* Increase max gunzip size */
>   
> -#ifdef CONFIG_CMD_KGDB
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed to run kgdb serial port */
> -#endif
> -
>   /*
>    * Environment Configuration
>    */
> diff --git a/include/configs/T102xRDB.h b/include/configs/T102xRDB.h
> index 3ae8a14c9445..e4629946ddb1 100644
> --- a/include/configs/T102xRDB.h
> +++ b/include/configs/T102xRDB.h
> @@ -584,10 +584,6 @@ unsigned long get_board_sys_clk(void);
>   #define CONFIG_SYS_BOOTMAPSZ	(64 << 20)	/* Initial map for Linux*/
>   #define CONFIG_SYS_BOOTM_LEN	(64 << 20)	/* Increase max gunzip size */
>   
> -#ifdef CONFIG_CMD_KGDB
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed to run kgdb serial port */
> -#endif
> -
>   /*
>    * Environment Configuration
>    */
> diff --git a/include/configs/T104xRDB.h b/include/configs/T104xRDB.h
> index 910baef00c47..7c48c3af71fc 100644
> --- a/include/configs/T104xRDB.h
> +++ b/include/configs/T104xRDB.h
> @@ -592,10 +592,6 @@
>   #define CONFIG_SYS_BOOTMAPSZ	(64 << 20)	/* Initial map for Linux*/
>   #define CONFIG_SYS_BOOTM_LEN	(64 << 20)	/* Increase max gunzip size */
>   
> -#ifdef CONFIG_CMD_KGDB
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed to run kgdb serial port */
> -#endif
> -
>   /*
>    * Dynamic MTD Partition support with mtdparts
>    */
> diff --git a/include/configs/T208xQDS.h b/include/configs/T208xQDS.h
> index 315d1f7b01a5..57640acfc90b 100644
> --- a/include/configs/T208xQDS.h
> +++ b/include/configs/T208xQDS.h
> @@ -585,11 +585,6 @@ unsigned long get_board_sys_clk(void);
>   #define CONFIG_SYS_BOOTMAPSZ	(64 << 20)	/* Initial map for Linux*/
>   #define CONFIG_SYS_BOOTM_LEN	(64 << 20)	/* Increase max gunzip size */
>   
> -#ifdef CONFIG_CMD_KGDB
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed to run kgdb serial port */
> -#define CONFIG_KGDB_SER_INDEX	2	/* which serial port to use */
> -#endif
> -
>   /*
>    * Environment Configuration
>    */
> diff --git a/include/configs/T208xRDB.h b/include/configs/T208xRDB.h
> index 6824be9e758f..6f27386d2bac 100644
> --- a/include/configs/T208xRDB.h
> +++ b/include/configs/T208xRDB.h
> @@ -540,11 +540,6 @@ unsigned long get_board_sys_clk(void);
>   #define CONFIG_SYS_BOOTMAPSZ	(64 << 20)	/* Initial map for Linux*/
>   #define CONFIG_SYS_BOOTM_LEN	(64 << 20)	/* Increase max gunzip size */
>   
> -#ifdef CONFIG_CMD_KGDB
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed to run kgdb serial port */
> -#define CONFIG_KGDB_SER_INDEX	2	/* which serial port to use */
> -#endif
> -
>   /*
>    * Environment Configuration
>    */
> diff --git a/include/configs/T4240RDB.h b/include/configs/T4240RDB.h
> index 54db021ec9ad..b7cbf87d3062 100644
> --- a/include/configs/T4240RDB.h
> +++ b/include/configs/T4240RDB.h
> @@ -214,10 +214,6 @@
>   #define CONFIG_SYS_BOOTMAPSZ	(64 << 20)	/* Initial map for Linux*/
>   #define CONFIG_SYS_BOOTM_LEN	(64 << 20)	/* Increase max gunzip size */
>   
> -#ifdef CONFIG_CMD_KGDB
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed to run kgdb serial port */
> -#endif
> -
>   /*
>    * Environment Configuration
>    */
> diff --git a/include/configs/UCP1020.h b/include/configs/UCP1020.h
> index 22b8c3f4e012..fceccc6d52a6 100644
> --- a/include/configs/UCP1020.h
> +++ b/include/configs/UCP1020.h
> @@ -386,11 +386,6 @@
>   #define CONFIG_SYS_BOOTMAPSZ	(64 << 20)	/* Initial Memory for Linux*/
>   #define CONFIG_SYS_BOOTM_LEN	(64 << 20)	/* Increase max gunzip size */
>   
> -#if defined(CONFIG_CMD_KGDB)
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed to run kgdb serial port */
> -#define CONFIG_KGDB_SER_INDEX	2	/* which serial port to use */
> -#endif
> -
>   /*
>    * Environment Configuration
>    */
> diff --git a/include/configs/corenet_ds.h b/include/configs/corenet_ds.h
> index 2a0a6beaf629..e311ced14bc1 100644
> --- a/include/configs/corenet_ds.h
> +++ b/include/configs/corenet_ds.h
> @@ -466,10 +466,6 @@
>   #define CONFIG_SYS_BOOTMAPSZ	(64 << 20)	/* Initial Memory map for Linux*/
>   #define CONFIG_SYS_BOOTM_LEN	(64 << 20)	/* Increase max gunzip size */
>   
> -#ifdef CONFIG_CMD_KGDB
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed to run kgdb serial port */
> -#endif
> -
>   /*
>    * Environment Configuration
>    */
> diff --git a/include/configs/p1_p2_rdb_pc.h b/include/configs/p1_p2_rdb_pc.h
> index 49dbbf07f889..58336110568c 100644
> --- a/include/configs/p1_p2_rdb_pc.h
> +++ b/include/configs/p1_p2_rdb_pc.h
> @@ -597,10 +597,6 @@
>   #define CONFIG_SYS_BOOTMAPSZ	(64 << 20)	/* Initial Memory for Linux*/
>   #define CONFIG_SYS_BOOTM_LEN	(64 << 20)	/* Increase max gunzip size */
>   
> -#if defined(CONFIG_CMD_KGDB)
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed to run kgdb serial port */
> -#endif
> -
>   /*
>    * Environment Configuration
>    */
> diff --git a/include/configs/pxa-common.h b/include/configs/pxa-common.h
> index 52d77e06acf4..7a78f98eb15f 100644
> --- a/include/configs/pxa-common.h
> +++ b/include/configs/pxa-common.h
> @@ -8,13 +8,6 @@
>   #ifndef	__CONFIG_PXA_COMMON_H__
>   #define	__CONFIG_PXA_COMMON_H__
>   
> -/*
> - * KGDB
> - */
> -#ifdef	CONFIG_CMD_KGDB
> -#define	CONFIG_KGDB_BAUDRATE		230400
> -#endif
> -
>   /*
>    * OHCI USB
>    */
> diff --git a/include/configs/socrates.h b/include/configs/socrates.h
> index 400cea479ef9..018bc6918ce7 100644
> --- a/include/configs/socrates.h
> +++ b/include/configs/socrates.h
> @@ -203,10 +203,6 @@
>    */
>   #define CONFIG_SYS_BOOTMAPSZ	(8 << 20)	/* Initial Memory map for Linux	*/
>   
> -#if defined(CONFIG_CMD_KGDB)
> -#define CONFIG_KGDB_BAUDRATE	230400	/* speed to run kgdb serial port*/
> -#endif
> -
>   
>   #define	CONFIG_EXTRA_ENV_SETTINGS					\
>   	"netdev=eth0\0"							\
> diff --git a/include/configs/x86-common.h b/include/configs/x86-common.h
> index 4b39faababd3..486b5ca77658 100644
> --- a/include/configs/x86-common.h
> +++ b/include/configs/x86-common.h
> @@ -44,10 +44,6 @@
>   	"ext2load scsi 0:3 01000000 /boot/vmlinuz; zboot 01000000"
>   #endif
>   
> -#if defined(CONFIG_CMD_KGDB)
> -#define CONFIG_KGDB_BAUDRATE			115200
> -#endif
> -
>   /*
>    * Miscellaneous configurable options
>    */
> 

  parent reply	other threads:[~2021-09-15  2:07 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-13 21:24 [PATCH 1/5] kgdb: Remove unused serial related options Tom Rini
2021-09-13 21:24 ` [PATCH 2/5] Convert CONFIG_BAUDRATE to Kconfig Tom Rini
2021-10-02 21:09   ` Tom Rini
2021-09-13 21:24 ` [PATCH 3/5] serial: Use the default CONFIG_SYS_BAUDRATE_TABLE in more platforms Tom Rini
2021-10-02 21:09   ` Tom Rini
2021-09-13 21:24 ` [PATCH 4/5] serial: Remove extraneous SYS_MALLOC_F check Tom Rini
2021-10-02 21:09   ` Tom Rini
2021-09-13 21:24 ` [PATCH 5/5] serial: Rework CONFIG_SYS_BAUDRATE_TABLE Tom Rini
2021-09-13 22:03   ` Alex G.
2021-09-13 22:11     ` Tom Rini
2021-09-24 19:08       ` Pali Rohár
2021-09-24 22:07         ` Tom Rini
2021-09-25 12:22           ` Pali Rohár
2021-09-13 22:14     ` Tom Rini
2021-09-15  2:07 ` Peng Fan (OSS) [this message]
2021-09-15  3:15   ` [PATCH 1/5] kgdb: Remove unused serial related options Tom Rini
2021-09-26  9:54     ` Peng Fan (OSS)
2021-09-26 16:54       ` Tom Rini
2021-10-02 21:09 ` Tom Rini

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=42148ee4-6a4e-5be1-2cb1-927c6bdd1762@oss.nxp.com \
    --to=peng.fan@oss.nxp.com \
    --cc=trini@konsulko.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 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.