From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 8965 invoked by alias); 17 May 2006 14:42:11 -0000 Received: (qmail 8957 invoked by uid 22791); 17 May 2006 14:42:11 -0000 X-Spam-Check-By: sourceware.org Received: from nevyn.them.org (HELO nevyn.them.org) (66.93.172.17) by sourceware.org (qpsmtpd/0.31.1) with ESMTP; Wed, 17 May 2006 14:42:08 +0000 Received: from drow by nevyn.them.org with local (Exim 4.54) id 1FgNDy-00018v-7q; Wed, 17 May 2006 10:42:06 -0400 Date: Wed, 17 May 2006 14:53:00 -0000 From: Daniel Jacobowitz To: gdb-patches@sourceware.org, Mark Mitchell Subject: Re: PATCH: Disable ARM RDP Message-ID: <20060517144206.GA4118@nevyn.them.org> Mail-Followup-To: gdb-patches@sourceware.org, Mark Mitchell References: <200603172225.k2HMPgPG024087@sethra.codesourcery.com> <20060324203053.GC26748@nevyn.them.org> <20060411200009.GA3235@nevyn.them.org> <20060505195846.GP31029@nevyn.them.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20060505195846.GP31029@nevyn.them.org> User-Agent: Mutt/1.5.11+cvs20060403 X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2006-05/txt/msg00377.txt.bz2 On Fri, May 05, 2006 at 03:58:46PM -0400, Daniel Jacobowitz wrote: > Lacking objections, I plan to apply this patch next week. Does anyone > have a last call for RDP? NEWS addition OK? > 2006-05-05 Daniel Jacobowitz > > * remote-rdp.c: Deleted. > * NEWS: Mention removal of remote-rdp.c. > > * Makefile.in (ALLDEPFILES): Remove remote-rdp.c. > (remote-rdp.o): Delete. > * README: Remove description of remote-rdp.c. > * arm-tdep.c (arm_breakpoint_from_pc): Remove obsolete comment. > * config/arm/embed.mt (TDEPFILES): Remove remote-rdp.o. Let's have a moment of silence for RDP... I have applied this to HEAD. I don't have any strong preference as to whether it goes to 6.5 also, so for now I'm not going to touch the branch. Here's the version as committed. (Eli, sorry for not copying you on the original. If you would like any changes to the NEWS entry, please let me know!) -- Daniel Jacobowitz CodeSourcery 2006-05-17 Daniel Jacobowitz * remote-rdp.c: Deleted. * NEWS: Mention removal of remote-rdp.c. * Makefile.in (ALLDEPFILES): Remove remote-rdp.c. (remote-rdp.o): Delete. * README: Remove description of remote-rdp.c. * arm-tdep.c (arm_breakpoint_from_pc): Remove obsolete comment. * config/arm/embed.mt (TDEPFILES): Remove remote-rdp.o. Index: Makefile.in =================================================================== RCS file: /cvs/src/src/gdb/Makefile.in,v retrieving revision 1.816 diff -u -p -r1.816 Makefile.in --- Makefile.in 15 May 2006 21:53:20 -0000 1.816 +++ Makefile.in 17 May 2006 14:39:36 -0000 @@ -1468,7 +1468,7 @@ ALLDEPFILES = \ procfs.c \ remote-e7000.c \ remote-hms.c remote-m32r-sdi.c remote-mips.c \ - remote-rdp.c remote-sim.c \ + remote-sim.c \ remote-st.c remote-utils.c dcache.c \ rs6000-nat.c rs6000-tdep.c \ s390-tdep.c s390-nat.c \ @@ -2504,9 +2504,6 @@ remote-mips.o: remote-mips.c $(defs_h) $ $(gdbcmd_h) $(gdbcore_h) $(serial_h) $(target_h) $(exceptions_h) \ $(remote_utils_h) $(gdb_string_h) $(gdb_stat_h) $(regcache_h) \ $(mips_tdep_h) -remote-rdp.o: remote-rdp.c $(defs_h) $(inferior_h) $(value_h) \ - $(gdb_callback_h) $(command_h) $(symfile_h) $(remote_utils_h) \ - $(gdb_string_h) $(gdbcore_h) $(regcache_h) $(serial_h) $(arm_tdep_h) remote-sds.o: remote-sds.c $(defs_h) $(gdb_string_h) $(frame_h) \ $(inferior_h) $(exceptions_h) $(bfd_h) $(symfile_h) $(target_h) \ $(gdbcmd_h) $(objfiles_h) $(gdb_stabs_h) $(gdbthread_h) $(gdbcore_h) \ Index: NEWS =================================================================== RCS file: /cvs/src/src/gdb/NEWS,v retrieving revision 1.187 diff -u -p -r1.187 NEWS --- NEWS 16 May 2006 19:43:57 -0000 1.187 +++ NEWS 17 May 2006 14:39:37 -0000 @@ -3,6 +3,10 @@ *** Changes since GDB 6.5 +* REMOVED features + +The ARM Demon monitor support (RDP protocol, "target rdp"). + *** Changes in GDB 6.5 * New targets Index: README =================================================================== RCS file: /cvs/src/src/gdb/README,v retrieving revision 1.35 diff -u -p -r1.35 README --- README 21 Jan 2006 01:29:03 -0000 1.35 +++ README 17 May 2006 14:39:37 -0000 @@ -454,7 +454,6 @@ monitors and other hardware: remote-est.c EST emulator remote-hms.c Renesas Micro Systems H8/300 monitor remote-mips.c MIPS remote debugging protocol - remote-rdp.c ARM with Demon monitor remote-sds.c PowerPC SDS monitor remote-sim.c Generalized simulator protocol remote-st.c Tandem ST-2000 monitor Index: arm-tdep.c =================================================================== RCS file: /cvs/src/src/gdb/arm-tdep.c,v retrieving revision 1.208 diff -u -p -r1.208 arm-tdep.c --- arm-tdep.c 18 Apr 2006 19:20:05 -0000 1.208 +++ arm-tdep.c 17 May 2006 14:39:37 -0000 @@ -1964,11 +1964,6 @@ static const char arm_default_thumb_be_b necessary) to point to the actual memory location where the breakpoint should be inserted. */ -/* XXX ??? from old tm-arm.h: if we're using RDP, then we're inserting - breakpoints and storing their handles instread of what was in - memory. It is nice that this is the same size as a handle - - otherwise remote-rdp will have to change. */ - static const unsigned char * arm_breakpoint_from_pc (CORE_ADDR *pcptr, int *lenptr) { Index: config/arm/embed.mt =================================================================== RCS file: /cvs/src/src/gdb/config/arm/embed.mt,v retrieving revision 1.3 diff -u -p -r1.3 embed.mt --- config/arm/embed.mt 21 Jan 2006 01:29:04 -0000 1.3 +++ config/arm/embed.mt 17 May 2006 14:39:38 -0000 @@ -1,5 +1,5 @@ # Target: ARM embedded system -TDEPFILES= arm-tdep.o remote-rdp.o +TDEPFILES= arm-tdep.o DEPRECATED_TM_FILE= tm-embed.h SIM_OBS = remote-sim.o