usb: Tidy up the usb_start flag

This should be declared in a header file so that type-checking works
correctly.

Add a single declaration to usb.h and remove the others.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Marek Vasut <marex@denx.de>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>

diff --git a/cmd/usb.c b/cmd/usb.c
index 73addb04c498de4d44366587e2199a07983ea952..6193728384097f997d2b5ca044d74f596887ef54 100644 (file)
--- a/cmd/usb.c
+++ b/cmd/usb.c
@@ -620,7 +620,6 @@ static int do_usb(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
 {
        struct usb_device *udev = NULL;
        int i;
-       extern char usb_started;
 
        if (argc < 2)
                return CMD_RET_USAGE;

diff --git a/common/usb.c b/common/usb.c
index ae9253dfc0edfb681776b986eb871b87993ebf1d..836506dcd9e9f7ba462dea115f0e4fb4d3f74d80 100644 (file)
--- a/common/usb.c
+++ b/common/usb.c
@@ -43,7 +43,7 @@
 #define USB_BUFSIZ     512
 
 static int asynch_allowed;
-char usb_started; /* flag for the started/stopped USB status */
+bool usb_started; /* flag for the started/stopped USB status */
 
 #if !CONFIG_IS_ENABLED(DM_USB)
 static struct usb_device usb_dev[USB_MAX_DEVICE];

diff --git a/drivers/usb/host/usb-uclass.c b/drivers/usb/host/usb-uclass.c
index 28f7ca9654db2dd1c9e05d50d44b25f1ebf7ba19..02c0138a2065a824f12c8def75556cb5bfc06153 100644 (file)
--- a/drivers/usb/host/usb-uclass.c
+++ b/drivers/usb/host/usb-uclass.c
@@ -18,7 +18,6 @@
 #include <dm/lists.h>
 #include <dm/uclass-internal.h>
 
-extern bool usb_started; /* flag for the started/stopped USB status */
 static bool asynch_allowed;
 
 struct usb_uclass_priv {

diff --git a/include/usb.h b/include/usb.h
index 80cb8467203047cc34d1d518ab114c8e2ac831eb..42b001c3dd5efac3967b9166894b2feab38f7680 100644 (file)
--- a/include/usb.h
+++ b/include/usb.h
@@ -11,12 +11,15 @@
 #ifndef _USB_H_
 #define _USB_H_
 
+#include <stdbool.h>
 #include <fdtdec.h>
 #include <usb_defs.h>
 #include <linux/usb/ch9.h>
 #include <asm/cache.h>
 #include <part.h>
 
+extern bool usb_started; /* flag for the started/stopped USB status */
+
 /*
  * The EHCI spec says that we must align to at least 32 bytes.  However,
  * some platforms require larger alignment.

diff --git a/test/boot/bootdev.c b/test/boot/bootdev.c
index 8cf3f30e0f75ca44754ad2ad7098c69dbcd82707..606bf4fcc1c162a96c42fe3943235b0540316ee9 100644 (file)
--- a/test/boot/bootdev.c
+++ b/test/boot/bootdev.c
@@ -19,9 +19,9 @@
 
 /* Allow reseting the USB-started flag */
 #if defined(CONFIG_USB_HOST) || defined(CONFIG_USB_GADGET)
-extern char usb_started;
+extern bool usb_started;
 #else
-char usb_started;
+#include <usb.h>
 #endif
 
 /* Check 'bootdev list' command */

diff --git a/test/dm/blk.c b/test/dm/blk.c
index 0aa04c64ef7effa306966f8bd245591a8123bb57..446c4423e6f5bf0d8557dc75da71db5aeb7a7d77 100644 (file)
--- a/test/dm/blk.c
+++ b/test/dm/blk.c
@@ -16,9 +16,6 @@
 
 DECLARE_GLOBAL_DATA_PTR;
 
-/* Allow resetting the USB-started flag */
-extern char usb_started;
-
 /* Test that block devices can be created */
 static int dm_test_blk_base(struct unit_test_state *uts)
 {