[PATCH]: b88c186a31 adapter/parport: Drop 'parport cable' command

[<ge...@op...>]

This is an automated email from Gerrit.

"zapb <de...@za...>" just uploaded a new patch set to Gerrit, which you can find at https://review.openocd.org/c/openocd/+/9153

-- gerrit

commit b88c186a31c2346d840587b590fbed64bbaffe16
Author: Marc Schink <de...@za...>
Date:   Wed Aug 20 19:18:47 2025 +0000

    adapter/parport: Drop 'parport cable' command
    
    The 'parport cable' command has been deprecated for one release and can
    now be dropped.
    
    Change-Id: Iadfce444e769510c911a3b7c9420455baf9f4b83
    Signed-off-by: Marc Schink <de...@za...>

diff --git a/src/jtag/drivers/parport.c b/src/jtag/drivers/parport.c
index 89a70e9f9c..642d8ba850 100644
--- a/src/jtag/drivers/parport.c
+++ b/src/jtag/drivers/parport.c
@@ -463,16 +463,6 @@ COMMAND_HANDLER(parport_handle_port_command)
 	return ERROR_OK;
 }
 
-// This command is only for backward compatibility and will be removed in the
-// future.
-COMMAND_HANDLER(parport_handle_cable_command)
-{
-	if (CMD_ARGC != 1)
-		return ERROR_COMMAND_SYNTAX_ERROR;
-
-	return command_run_linef(CMD_CTX, "parport_select_cable %s", CMD_ARGV[0]);
-}
-
 COMMAND_HANDLER(parport_handle_write_on_exit_command)
 {
 	if (CMD_ARGC != 1)
@@ -523,14 +513,6 @@ static const struct command_registration parport_subcommand_handlers[] = {
 		.help = "Specify the device file of he parallel port",
 		.usage = "file",
 	},
-	{
-		.name = "cable",
-		.handler = parport_handle_cable_command,
-		.mode = COMMAND_CONFIG,
-		.help = "Set the layout of the parallel port cable "
-			"used to connect to the target",
-		.usage = "cable",
-	},
 	{
 		.name = "write_on_exit",
 		.handler = parport_handle_write_on_exit_command,
diff --git a/src/jtag/startup.tcl b/src/jtag/startup.tcl
index 5c466c628a..c06b7e0343 100644
--- a/src/jtag/startup.tcl
+++ b/src/jtag/startup.tcl
@@ -353,33 +353,6 @@ proc parport_port args {
 	eval parport port $args
 }
 
-lappend _telnet_autocomplete_skip parport_cable
-proc parport_cable args {
-	echo "DEPRECATED! use 'parport cable' not 'parport_cable'"
-	eval parport cable $args
-}
-
-lappend _telnet_autocomplete_skip parport_select_cable
-proc parport_select_cable {cable} {
-	echo "DEPRECATED! Do not use 'parport cable' but use a cable configuration file in interface/parport"
-
-	switch $cable {
-		"wiggler" { source [find interface/parport/wiggler.cfg] }
-		"wiggler2" { source [find interface/parport/wiggler2.cfg] }
-		"wiggler_ntrst_inverted" { source [find interface/parport/wiggler-ntrst-inverted.cfg] }
-		"old_amt_wiggler" { source [find interface/parport/amt-wiggler-old.cfg ] }
-		"arm-jtag" { source [find interface/parport/arm-jtag.cfg] }
-		"chameleon" { source [find interface/parport/chameleon.cfg] }
-		"dlc5" { source [find interface/parport/dlc5.cfg] }
-		"triton" { source [find interface/parport/triton.cfg] }
-		"lattice" { source [find interface/parport/lattice.cfg] }
-		"flashlink" { source [find interface/parport/flashlink.cfg] }
-		"altium" { source [find interface/parport/altium.cfg] }
-		"aspo" { source [find interface/parport/aspo.cfg] }
-		default { error "invalid parallel port cable '$cable'" }
-	}
-}
-
 lappend _telnet_autocomplete_skip parport_write_on_exit
 proc parport_write_on_exit args {
 	echo "DEPRECATED! use 'parport write_on_exit' not 'parport_write_on_exit'"

--