This is an automated email from Gerrit.

"Antonio Borneo <borneo.anto...@gmail.com>" just uploaded a new patch set to 
Gerrit, which you can find at https://review.openocd.org/c/openocd/+/7663

-- gerrit

commit 231fdf4bf8887a0171a596d90c79b112b724e608
Author: Antonio Borneo <borneo.anto...@gmail.com>
Date:   Mon May 1 00:24:37 2023 +0200

    pld: move in pld.h the pld_driver's declaration
    
    The static analyser 'sparse' complains, while compiling a pld
    driver, that the struct pld_driver is declared in the file as
    non static, but it is not exposed through an include file.
    The message is:
            warning: symbol 'XXX' was not declared. Should it be static?
    
    Move the list of pld_driver's declaration in pld.h
    
    Change-Id: I0f917aecc7534c1b51af0afa9b32ccfd33db3511
    Signed-off-by: Antonio Borneo <borneo.anto...@gmail.com>

diff --git a/src/pld/pld.c b/src/pld/pld.c
index dd8f8263fe..7dd2cec18f 100644
--- a/src/pld/pld.c
+++ b/src/pld/pld.c
@@ -16,15 +16,6 @@
 #include <helper/time_support.h>
 
 
-/* pld drivers
- */
-extern struct pld_driver efinix_pld;
-extern struct pld_driver gatemate_pld;
-extern struct pld_driver gowin_pld;
-extern struct pld_driver intel_pld;
-extern struct pld_driver lattice_pld;
-extern struct pld_driver virtex2_pld;
-
 static struct pld_driver *pld_drivers[] = {
        &efinix_pld,
        &gatemate_pld,
diff --git a/src/pld/pld.h b/src/pld/pld.h
index a7cd20f641..322b96ec6e 100644
--- a/src/pld/pld.h
+++ b/src/pld/pld.h
@@ -38,4 +38,11 @@ struct pld_device *get_pld_device_by_num(int num);
 #define ERROR_PLD_DEVICE_INVALID        (-1000)
 #define ERROR_PLD_FILE_LOAD_FAILED      (-1001)
 
+extern struct pld_driver efinix_pld;
+extern struct pld_driver gatemate_pld;
+extern struct pld_driver gowin_pld;
+extern struct pld_driver intel_pld;
+extern struct pld_driver lattice_pld;
+extern struct pld_driver virtex2_pld;
+
 #endif /* OPENOCD_PLD_PLD_H */

-- 

Reply via email to