Hi Mateusz,
Preprocessor definitions and hardcoded implementation selection in g_dnl core were replaced by a linker list made of (usb_function_name, bind_callback) pairs.
Signed-off-by: Mateusz Zalega m.zalega@samsung.com Cc: Lukasz Majewski l.majewski@samsung.com Cc: Marek Vasut marex@denx.de
common/cmd_dfu.c | 3 +- common/cmd_thordown.c | 3 +- common/cmd_usb_mass_storage.c | 2 +- drivers/usb/gadget/f_dfu.c | 3 ++ drivers/usb/gadget/f_mass_storage.c | 3 ++ drivers/usb/gadget/f_thor.c | 2 ++ drivers/usb/gadget/g_dnl.c | 64 ++++++++++++++++--------------------- include/dfu.h | 7 ---- include/g_dnl.h | 24 ++++++++++++++ include/thor.h | 8 ----- include/usb_mass_storage.h | 8 ----- 11 files changed, 63 insertions(+), 64 deletions(-)
diff --git a/common/cmd_dfu.c b/common/cmd_dfu.c index 5547678..a03538d 100644 --- a/common/cmd_dfu.c +++ b/common/cmd_dfu.c @@ -22,7 +22,6 @@ static int do_dfu(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) char *interface = argv[2]; char *devstring = argv[3];
char *s = "dfu"; int ret, i = 0;
ret = dfu_init_env_entities(interface,
simple_strtoul(devstring, @@ -38,7 +37,7 @@ static int do_dfu(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) int controller_index = simple_strtoul(usb_controller, NULL, 0); board_usb_init(controller_index, USB_INIT_DEVICE);
- g_dnl_register(s);
- g_dnl_register("usb_dnl_dfu"); while (1) { if (dfu_reset()) /*
diff --git a/common/cmd_thordown.c b/common/cmd_thordown.c index c4b3511..2dd7509 100644 --- a/common/cmd_thordown.c +++ b/common/cmd_thordown.c @@ -22,7 +22,6 @@ int do_thor_down(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) char *interface = argv[2]; char *devstring = argv[3];
const char *s = "thor"; int ret;
puts("TIZEN "THOR" Downloader\n");
@@ -40,7 +39,7 @@ int do_thor_down(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) goto exit; }
- g_dnl_register(s);
g_dnl_register("usb_dnl_thor");
ret = thor_init(); if (ret) {
diff --git a/common/cmd_usb_mass_storage.c b/common/cmd_usb_mass_storage.c index 5175bd5..4c2de48 100644 --- a/common/cmd_usb_mass_storage.c +++ b/common/cmd_usb_mass_storage.c @@ -40,7 +40,7 @@ int do_usb_mass_storage(cmd_tbl_t *cmdtp, int flag, return CMD_RET_FAILURE; }
- g_dnl_register("ums");
g_dnl_register("usb_dnl_ums");
/* Timeout unit: seconds */ int cable_ready_timeout = UMS_CABLE_READY_TIMEOUT;
diff --git a/drivers/usb/gadget/f_dfu.c b/drivers/usb/gadget/f_dfu.c index a045864..2f822c4 100644 --- a/drivers/usb/gadget/f_dfu.c +++ b/drivers/usb/gadget/f_dfu.c @@ -24,6 +24,7 @@ #include <linux/usb/composite.h>
#include <dfu.h> +#include <g_dnl.h> #include "f_dfu.h"
struct f_dfu { @@ -781,3 +782,5 @@ int dfu_add(struct usb_configuration *c)
return dfu_bind_config(c); }
+DECLARE_GADGET_BIND_CALLBACK(usb_dnl_dfu, dfu_add); diff --git a/drivers/usb/gadget/f_mass_storage.c b/drivers/usb/gadget/f_mass_storage.c index f896169..f88bb12 100644 --- a/drivers/usb/gadget/f_mass_storage.c +++ b/drivers/usb/gadget/f_mass_storage.c @@ -255,6 +255,7 @@ #include <linux/usb/gadget.h> #include <linux/usb/composite.h> #include <usb/lin_gadget_compat.h> +#include <g_dnl.h>
/*------------------------------------------------------------------------*/
@@ -2778,3 +2779,5 @@ int fsg_init(struct ums *ums_dev)
return 0; }
+DECLARE_GADGET_BIND_CALLBACK(usb_dnl_ums, fsg_add); diff --git a/drivers/usb/gadget/f_thor.c b/drivers/usb/gadget/f_thor.c index f5c0224..59d246d 100644 --- a/drivers/usb/gadget/f_thor.c +++ b/drivers/usb/gadget/f_thor.c @@ -999,3 +999,5 @@ int thor_add(struct usb_configuration *c) debug("%s:\n", __func__); return thor_func_init(c); }
+DECLARE_GADGET_BIND_CALLBACK(usb_dnl_thor, thor_add); diff --git a/drivers/usb/gadget/g_dnl.c b/drivers/usb/gadget/g_dnl.c index dd95afe..3575aca 100644 --- a/drivers/usb/gadget/g_dnl.c +++ b/drivers/usb/gadget/g_dnl.c @@ -41,7 +41,6 @@
#define DRIVER_VERSION "usb_dnl 2.0"
-static const char shortname[] = "usb_dnl_"; static const char product[] = "USB download gadget"; static char g_dnl_serial[MAX_STRING_SERIAL]; static const char manufacturer[] = CONFIG_G_DNL_MANUFACTURER; @@ -96,29 +95,36 @@ static int g_dnl_unbind(struct usb_composite_dev *cdev) free(cdev->config); cdev->config = NULL; debug("%s: calling usb_gadget_disconnect for "
"controller '%s'\n", shortname,gadget->name);
"controller '%s'\n", __func__, gadget->name);usb_gadget_disconnect(gadget);
return 0;
}
+static inline struct g_dnl_bind_callback * g_dnl_bind_callback_first(void) +{
- return ll_entry_start(struct g_dnl_bind_callback,
g_dnl_bind_callbacks);+}
+static inline struct g_dnl_bind_callback * g_dnl_bind_callback_end(void) +{
- return ll_entry_end(struct g_dnl_bind_callback,
g_dnl_bind_callbacks);+}
static int g_dnl_do_config(struct usb_configuration *c) { const char *s = c->cdev->driver->name;
- int ret = -1;
- struct g_dnl_bind_callback *callback =
g_dnl_bind_callback_first(); debug("%s: configuration: 0x%p composite dev: 0x%p\n", __func__, c, c->cdev);
- printf("GADGET DRIVER: %s\n", s);
- if (!strcmp(s, "usb_dnl_dfu"))
ret = dfu_add(c);- else if (!strcmp(s, "usb_dnl_ums"))
ret = fsg_add(c);- else if (!strcmp(s, "usb_dnl_thor"))
ret = thor_add(c);- return ret;
- for (; callback != g_dnl_bind_callback_end(); callback++)
if (!strcmp(s, callback->usb_function_name))return callback->fptr(c);- return -ENODEV;
}
static int g_dnl_config_register(struct usb_composite_dev *cdev) @@ -203,12 +209,12 @@ static int g_dnl_bind(struct usb_composite_dev *cdev) device_desc.bcdDevice = cpu_to_le16(gcnum); else { debug("%s: controller '%s' not recognized\n",
shortname, gadget->name);
__func__, gadget->name);__constant_cpu_to_le16(0x9999); }
debug("%s: calling usb_gadget_connect for "
"controller '%s'\n", shortname,gadget->name);
"controller '%s'\n", __func__, gadget->name);usb_gadget_connect(gadget);
return 0;
@@ -227,36 +233,22 @@ static struct usb_composite_driver g_dnl_driver = { .unbind = g_dnl_unbind, };
-int g_dnl_register(const char *type) +/*
- NOTICE:
- Registering via USB function name won't be necessary after
rewriting
- g_dnl to support multiple USB functions.
- */
+int g_dnl_register(const char *name) {
- /* The largest function name is 4 */
- static char name[sizeof(shortname) + 4];
- int ret;
- if (!strcmp(type, "dfu")) {
strcpy(name, shortname);strcat(name, type);- } else if (!strcmp(type, "ums")) {
strcpy(name, shortname);strcat(name, type);- } else if (!strcmp(type, "thor")) {
strcpy(name, shortname);strcat(name, type);- } else {
printf("%s: unknown command: %s\n", __func__, type);return -EINVAL;- }
int ret = usb_composite_register(&g_dnl_driver);
debug("%s: g_dnl_driver.name = %s\n", __func__, name); g_dnl_driver.name = name;
- debug("%s: g_dnl_driver.name: %s\n", __func__,
g_dnl_driver.name);
- ret = usb_composite_register(&g_dnl_driver);
- if (ret) { printf("%s: failed!, error: %d\n", __func__, ret); return ret; }
- return 0;
}
diff --git a/include/dfu.h b/include/dfu.h index f973426..9956636 100644 --- a/include/dfu.h +++ b/include/dfu.h @@ -169,12 +169,5 @@ static inline int dfu_fill_entity_ram(struct dfu_entity *dfu, char *s) } #endif
-#ifdef CONFIG_DFU_FUNCTION int dfu_add(struct usb_configuration *c); -#else -int dfu_add(struct usb_configuration *c) -{
- return 0;
-} -#endif #endif /* __DFU_ENTITY_H_ */ diff --git a/include/g_dnl.h b/include/g_dnl.h index 8f813c2..fb20bf0 100644 --- a/include/g_dnl.h +++ b/include/g_dnl.h @@ -10,6 +10,30 @@
#include <linux/usb/ch9.h> #include <linux/usb/gadget.h> +#include <linux/usb/composite.h> +#include <linker_lists.h>
+/*
- @usb_fname: unescaped USB function name
- @callback_ptr: bind callback, one per function name
- */
+#define DECLARE_GADGET_BIND_CALLBACK(usb_fname, callback_ptr) \
- ll_entry_declare(struct g_dnl_bind_callback, \
__usb_function_name_##usb_fname, \g_dnl_bind_callbacks) = { \.usb_function_name = #usb_fname, \.fptr = callback_ptr \}+typedef int (*g_dnl_bind_callback_f)(struct usb_configuration *);
+/* used in Gadget downloader callback linker list */ +struct g_dnl_bind_callback +{
- const char *usb_function_name;
- g_dnl_bind_callback_f fptr;
+};
int g_dnl_bind_fixup(struct usb_device_descriptor *, const char *); int g_dnl_register(const char *s); void g_dnl_unregister(void); diff --git a/include/thor.h b/include/thor.h index afeade4..5051be7 100644 --- a/include/thor.h +++ b/include/thor.h @@ -15,13 +15,5 @@
int thor_handle(void); int thor_init(void);
-#ifdef CONFIG_THOR_FUNCTION int thor_add(struct usb_configuration *c); -#else -int thor_add(struct usb_configuration *c) -{
- return 0;
-} -#endif #endif /* __THOR_H_ */ diff --git a/include/usb_mass_storage.h b/include/usb_mass_storage.h index 058dcf1..ed46064 100644 --- a/include/usb_mass_storage.h +++ b/include/usb_mass_storage.h @@ -40,13 +40,5 @@ int fsg_init(struct ums *); void fsg_cleanup(void); struct ums *ums_init(unsigned int); int fsg_main_thread(void *);
-#ifdef CONFIG_USB_GADGET_MASS_STORAGE int fsg_add(struct usb_configuration *c); -#else -int fsg_add(struct usb_configuration *c) -{
- return 0;
-} -#endif #endif /* __USB_MASS_STORAGE_H__ */
Acked-by: Lukasz Majewski l.majewski@samsung.com