X-Git-Url: https://review.openocd.org/gitweb?p=openocd.git;a=blobdiff_plain;f=src%2Fpld%2Fpld.c;h=5210b97b7d300c51109c7066b3d63f18dfa456cd;hp=e8cd075ef02720abdb6b1fafe3e78b3bc189747c;hb=bcaf775fc10d88d2c63c06bafada141895318b34;hpb=5b6df55a1e5e4c0f531bc336691bc7c9a6a0df87
diff --git a/src/pld/pld.c b/src/pld/pld.c
index e8cd075ef0..5210b97b7d 100644
--- a/src/pld/pld.c
+++ b/src/pld/pld.c
@@ -13,43 +13,37 @@
* GNU General Public License for more details. *
* *
* You should have received a copy of the GNU General Public License *
- * along with this program; if not, write to the *
- * Free Software Foundation, Inc., *
- * 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. *
+ * along with this program. If not, see . *
***************************************************************************/
+
#ifdef HAVE_CONFIG_H
#include "config.h"
#endif
#include "pld.h"
-#include "log.h"
-#include "time_support.h"
+#include
+#include
/* pld drivers
*/
-extern pld_driver_t virtex2_pld;
+extern struct pld_driver virtex2_pld;
-static pld_driver_t *pld_drivers[] =
-{
+static struct pld_driver *pld_drivers[] = {
&virtex2_pld,
NULL,
};
-static pld_device_t *pld_devices;
-static command_t *pld_cmd;
+static struct pld_device *pld_devices;
-pld_device_t *get_pld_device_by_num(int num)
+struct pld_device *get_pld_device_by_num(int num)
{
- pld_device_t *p;
+ struct pld_device *p;
int i = 0;
- for (p = pld_devices; p; p = p->next)
- {
+ for (p = pld_devices; p; p = p->next) {
if (i++ == num)
- {
return p;
- }
}
return NULL;
@@ -62,58 +56,54 @@ COMMAND_HANDLER(handle_pld_device_command)
int i;
int found = 0;
- if (argc < 1)
- {
- LOG_WARNING("incomplete 'pld device' command");
- return ERROR_OK;
- }
+ if (CMD_ARGC < 1)
+ return ERROR_COMMAND_SYNTAX_ERROR;
- for (i = 0; pld_drivers[i]; i++)
- {
- if (strcmp(args[0], pld_drivers[i]->name) == 0)
- {
- pld_device_t *p, *c;
+ for (i = 0; pld_drivers[i]; i++) {
+ if (strcmp(CMD_ARGV[0], pld_drivers[i]->name) == 0) {
+ struct pld_device *p, *c;
/* register pld specific commands */
- if (pld_drivers[i]->register_commands(cmd_ctx) != ERROR_OK)
- {
- LOG_ERROR("couldn't register '%s' commands", args[0]);
- exit(-1);
+ int retval;
+ if (pld_drivers[i]->commands) {
+ retval = register_commands(CMD_CTX, NULL,
+ pld_drivers[i]->commands);
+ if (ERROR_OK != retval) {
+ LOG_ERROR("couldn't register '%s' commands", CMD_ARGV[0]);
+ return ERROR_FAIL;
+ }
}
- c = malloc(sizeof(pld_device_t));
+ c = malloc(sizeof(struct pld_device));
c->driver = pld_drivers[i];
c->next = NULL;
- int retval = CALL_COMMAND_HANDLER(pld_drivers[i]->pld_device_command, c);
- if (ERROR_OK != retval)
- {
- LOG_ERROR("'%s' driver rejected pld device", args[0]);
+ retval = CALL_COMMAND_HANDLER(
+ pld_drivers[i]->pld_device_command, c);
+ if (ERROR_OK != retval) {
+ LOG_ERROR("'%s' driver rejected pld device",
+ CMD_ARGV[0]);
free(c);
return ERROR_OK;
}
/* put pld device in linked list */
- if (pld_devices)
- {
+ if (pld_devices) {
/* find last pld device */
- for (p = pld_devices; p && p->next; p = p->next);
+ for (p = pld_devices; p && p->next; p = p->next)
+ ;
if (p)
p->next = c;
- }
- else
- {
+ } else
pld_devices = c;
- }
found = 1;
}
}
/* no matching pld driver found */
- if (!found)
- {
- LOG_ERROR("pld driver '%s' not found", args[0]);
+ if (!found) {
+ LOG_ERROR("pld driver '%s' not found", CMD_ARGV[0]);
exit(-1);
}
@@ -122,19 +112,16 @@ COMMAND_HANDLER(handle_pld_device_command)
COMMAND_HANDLER(handle_pld_devices_command)
{
- pld_device_t *p;
+ struct pld_device *p;
int i = 0;
- if (!pld_devices)
- {
- command_print(cmd_ctx, "no pld devices configured");
+ if (!pld_devices) {
+ command_print(CMD_CTX, "no pld devices configured");
return ERROR_OK;
}
for (p = pld_devices; p; p = p->next)
- {
- command_print(cmd_ctx, "#%i: %s", i++, p->driver->name);
- }
+ command_print(CMD_CTX, "#%i: %s", i++, p->driver->name);
return ERROR_OK;
}
@@ -143,67 +130,110 @@ COMMAND_HANDLER(handle_pld_load_command)
{
int retval;
struct timeval start, end, duration;
- pld_device_t *p;
+ struct pld_device *p;
gettimeofday(&start, NULL);
- if (argc < 2)
- {
- command_print(cmd_ctx, "usage: pld load ");
- return ERROR_OK;
- }
+ if (CMD_ARGC < 2)
+ return ERROR_COMMAND_SYNTAX_ERROR;
unsigned dev_id;
- COMMAND_PARSE_NUMBER(uint, args[0], dev_id);
+ COMMAND_PARSE_NUMBER(uint, CMD_ARGV[0], dev_id);
p = get_pld_device_by_num(dev_id);
- if (!p)
- {
- command_print(cmd_ctx, "pld device '#%s' is out of bounds", args[0]);
+ if (!p) {
+ command_print(CMD_CTX, "pld device '#%s' is out of bounds", CMD_ARGV[0]);
return ERROR_OK;
}
- if ((retval = p->driver->load(p, args[1])) != ERROR_OK)
- {
- command_print(cmd_ctx, "failed loading file %s to pld device %u",
- args[1], dev_id);
- switch (retval)
- {
+ retval = p->driver->load(p, CMD_ARGV[1]);
+ if (retval != ERROR_OK) {
+ command_print(CMD_CTX, "failed loading file %s to pld device %u",
+ CMD_ARGV[1], dev_id);
+ switch (retval) {
}
return retval;
- }
- else
- {
+ } else {
gettimeofday(&end, NULL);
timeval_subtract(&duration, &end, &start);
- command_print(cmd_ctx, "loaded file %s to pld device %u in %jis %jius",
- args[1], dev_id,
+ command_print(CMD_CTX, "loaded file %s to pld device %u in %jis %jius",
+ CMD_ARGV[1], dev_id,
(intmax_t)duration.tv_sec, (intmax_t)duration.tv_usec);
}
return ERROR_OK;
}
-int pld_init(struct command_context_s *cmd_ctx)
+static const struct command_registration pld_exec_command_handlers[] = {
+ {
+ .name = "devices",
+ .handler = handle_pld_devices_command,
+ .mode = COMMAND_EXEC,
+ .help = "list configured pld devices",
+ },
+ {
+ .name = "load",
+ .handler = handle_pld_load_command,
+ .mode = COMMAND_EXEC,
+ .help = "load configuration file into PLD",
+ .usage = "pld_num filename",
+ },
+ COMMAND_REGISTRATION_DONE
+};
+
+static int pld_init(struct command_context *cmd_ctx)
{
if (!pld_devices)
return ERROR_OK;
- register_command(cmd_ctx, pld_cmd, "devices",
- handle_pld_devices_command, COMMAND_EXEC,
- "list configured pld devices");
- register_command(cmd_ctx, pld_cmd, "load",
- handle_pld_load_command, COMMAND_EXEC,
- "load configuration into programmable logic device");
-
- return ERROR_OK;
+ struct command *parent = command_find_in_context(cmd_ctx, "pld");
+ return register_commands(cmd_ctx, parent, pld_exec_command_handlers);
}
-int pld_register_commands(struct command_context_s *cmd_ctx)
+COMMAND_HANDLER(handle_pld_init_command)
{
- pld_cmd = register_command(cmd_ctx, NULL, "pld", NULL, COMMAND_ANY, "programmable logic device commands");
+ if (CMD_ARGC != 0)
+ return ERROR_COMMAND_SYNTAX_ERROR;
+
+ static bool pld_initialized;
+ if (pld_initialized) {
+ LOG_INFO("'pld init' has already been called");
+ return ERROR_OK;
+ }
+ pld_initialized = true;
- register_command(cmd_ctx, pld_cmd, "device", handle_pld_device_command, COMMAND_CONFIG, NULL);
+ LOG_DEBUG("Initializing PLDs...");
+ return pld_init(CMD_CTX);
+}
- return ERROR_OK;
+static const struct command_registration pld_config_command_handlers[] = {
+ {
+ .name = "device",
+ .mode = COMMAND_CONFIG,
+ .handler = handle_pld_device_command,
+ .help = "configure a PLD device",
+ .usage = "driver_name [driver_args ... ]",
+ },
+ {
+ .name = "init",
+ .mode = COMMAND_CONFIG,
+ .handler = handle_pld_init_command,
+ .help = "initialize PLD devices",
+ .usage = ""
+ },
+ COMMAND_REGISTRATION_DONE
+};
+static const struct command_registration pld_command_handler[] = {
+ {
+ .name = "pld",
+ .mode = COMMAND_ANY,
+ .help = "programmable logic device commands",
+ .usage = "",
+ .chain = pld_config_command_handlers,
+ },
+ COMMAND_REGISTRATION_DONE
+};
+int pld_register_commands(struct command_context *cmd_ctx)
+{
+ return register_commands(cmd_ctx, NULL, pld_command_handler);
}