[PATCH 1/5] kgdb: Remove unused serial related options

Peng Fan (OSS) peng.fan at oss.nxp.com
Wed Sep 15 04:07:23 CEST 2021


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 at 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
>    */
> 


More information about the U-Boot mailing list