This is an automated email from Gerrit.

"zapb <[email protected]>" just uploaded a new patch set to Gerrit, which you can 
find at https://review.openocd.org/c/openocd/+/9157

-- gerrit

commit 6301325c772053866f836526bd9be8a57eb2d3d0
Author: Marc Schink <[email protected]>
Date:   Mon Sep 29 09:16:06 2025 +0000

    adapter/parport: Require device file specification
    
    Require that a device file is specified to select the parallel port.
    This simplifies the code and removes hard-coded device files. Also, it
    ensures that a device is not selected arbitrarily. In the case of
    multiple availables devices, this can lead to undesirable behavior.
    Other drivers such as 'jlink' already implement this behavior.
    
    Change-Id: I5a8fd477ef8027ce2a4cfc1e541b34c6436e321b
    Signed-off-by: Marc Schink <[email protected]>

diff --git a/src/jtag/drivers/parport.c b/src/jtag/drivers/parport.c
index 3fce3129d9..fb06e39b05 100644
--- a/src/jtag/drivers/parport.c
+++ b/src/jtag/drivers/parport.c
@@ -239,13 +239,8 @@ static int parport_init(void)
        }
 
        if (!parport_device_file) {
-#if defined(__FreeBSD__) || defined(__FreeBSD_kernel__)
-               parport_device_file = strdup("/dev/ppi0");
-#else
-               parport_device_file = strdup("/dev/parport0");
-#endif
-               LOG_WARNING("No parallel port specified, using %s", 
parport_device_file);
-               LOG_WARNING("DEPRECATED! The lack of a parallel port 
specification is deprecated and will no longer work in the future");
+               LOG_ERROR("No parallel port device file specified");
+               goto init_fail;
        }
 
        LOG_DEBUG("Using parallel port %s", parport_device_file);

-- 

Reply via email to