From mboxrd@z Thu Jan  1 00:00:00 1970
From: Scott Wood <scottwood@freescale.com>
Date: Tue, 05 May 2009 14:30:39 -0500
Subject: [U-Boot] [PATCH] arm920t kgdb serial support
In-Reply-To: <20090505212121.82375u7m9i47arok@webmail.igi.cnr.it>
References: <20090505212121.82375u7m9i47arok@webmail.igi.cnr.it>
Message-ID: <4A0093DF.3070500@freescale.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

barbalace at igi.cnr.it wrote:
> This patch add kgdb to ARM920T cpus (full support to s3c2440).
> The patch lets debugging remote ARM targets using the serial
> line and GDB.
> 
> Signed-off-by: Antonio Barbalace <antonio.barbalace@igi.cnr.it>
> 
> diff -r -u u-boot/common/kgdb.c u-boot-UniPd-denx.de/common/kgdb.c
> --- u-boot/common/kgdb.c	2007-11-06 18:07:07.000000000 +0100
> +++ u-boot-UniPd-denx.de/common/kgdb.c	2009-05-05 18:44:25.000000000 +0200
> @@ -458,6 +458,7 @@
> 
>   			goto doexit;
> 
> +#ifdef _PPC

CONFIG_PPC -- and explain in a comment why this is powerpc-specific (and 
keep in mind that ARM and PowerPC are not the only possibilities), and 
what happens on other architectures if those commands are used.

> @@ -496,8 +497,13 @@
>   			} else {
>   				kgdb_error(KGDBERR_BADPARAMS);
>   			}
> +
> +		case 'q':
> +			if (!(strcmp(ptr, "Supported"))) {
> +				sprintf(remcomOutBuffer, "PacketSize=%d", BUFMAX);
> +			}
>   			break;

Explain what this change does -- what does it have to do with arm920t?

-Scott