[PATCH]: 48c20b257a target/semihosting_common: remove redundant NULL check

[gerrit]

This is an automated email from Gerrit.

"Mark O'Donovan <shif...@posteo.net>" just uploaded a new patch set to Gerrit, 
which you can find at https://review.openocd.org/c/openocd/+/9503

-- gerrit

commit 48c20b257a6a8f3c9c25733bde5139880a83a0db
Author: Mark O'Donovan <shif...@posteo.net>
Date:   Tue Feb 24 20:49:50 2026 +0000

    target/semihosting_common: remove redundant NULL check
    
    The value semihosting can never be NULL here.
    Found by cppcheck.
    
    Change-Id: Iff5cef0ffc78ec19bc7630bc2cc9bc4eea002e47
    Signed-off-by: Mark O'Donovan <shif...@posteo.net>

diff --git a/src/target/semihosting_common.c b/src/target/semihosting_common.c
index 345e542c3c..bcf37b6ad3 100644
--- a/src/target/semihosting_common.c
+++ b/src/target/semihosting_common.c
@@ -1857,7 +1857,7 @@ COMMAND_HANDLER(handle_common_semihosting_command)
                        return ERROR_FAIL;
                }
 
-               if (semihosting && semihosting->setup(target, is_active) != 
ERROR_OK) {
+               if (semihosting->setup(target, is_active) != ERROR_OK) {
                        LOG_ERROR("Failed to Configure semihosting");
                        return ERROR_FAIL;
                }

--