Remove useless typedef from struct jtag_command.

Remove misleading typedef from union jtag_container.

Remove misleading typedef from struct sleep_command.

Remove misleading typedef from struct end_state_command.

Remove misleading typedef from struct reset_command.

Remove misleading typedef from struct stableclocks_command.

Remove misleading typedef from struct runtest_command.

Remove misleading typedef from struct pathmove_command.

Remove misleading typedef from struct statemove_command.

Remove misleading typedef from struct cmd_queue_page.

Remove misleading typedef on struct scan_command.

Remove useless typedef for struct insert_insignification_operation.

Remove misleading typedef and redundant suffix.

Remove useless typedef.

Remove misleading typedef from ft2232_device; it was barely used.

Remove misleading typedef and redundant suffix.

Remove useless typedef and redundant suffix from parport struct cable.

Remove misleading typedef from struct struct pending_scan_result.
Future patches need to remove these duplicated types and code.

Remove useless typedef and redundant suffix from struct armjtatew_jtag.

Remove useless typedef and redundant suffix from jtag_tap_event_action.

Remove useless typedef and redundant suffix from jtag_event_callback.
Add documentation for the structure.

Remove useless typedef and redundant suffix from struct jtag_interface.

Removes another useless typedef and suffix.

Remove useless typedef and redundant suffix.

Move the bitq_state structure from the header to source file, as it
is used internally.

Also removes its silly typedef.

Remove useless structure typedef.

Search and destroy the jtag_tap_t typedef.  This also cleans up a
layering violation, removing the declaration from types.h.

The number of command arguments will always be 0 or more, so use
the right type in handlers.  This has a cascading effect up through
the layers, but the new COMMAND_HANDLER macros prevented total chaos.

By introducing the CMD_NAME macro, this parameter may be integrated
as args[-1] in command.[ch], without touching any other call sites.

By using CALL_COMMAND_HANDLER, parameters can be reordered, added, or
even removed in inherited signatures, without requiring revisiting
all of the various call sites.

Define the numerous helpers that inherit command handler parameters
using the COMMAND_HELPER macro.

Add the khz and speed_div functions to the parport interface driver.
Add the parport_toggling_time function that tells the parport driver
how long (in nanoseconds) it takes for the hardware to toggle TCK.

[dbrownell@users.sourceforge.net: tweak doc for clarity, mention
multimeter, and whitespace fixes]

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>

Without this patch, running "openocd -c 'jtag interface'" segfaults.
Now, it returns the string "undefined" when the interface is unset.

Contrary to my previous assessment, some opportunities to remove forward
declarations were overlooked.  Remove them by moving the definitions
of the command registration and interface structure to the end of files.

It's been about a year since these were deprecated and, in most
cases, removed.  There's no point in carrying that documentation,
or backwards compatibility for "jtag_device" and "jtag_speed",
around forever.  (Or a few remnants of obsolete code...)

Removed a few obsolete uses of "jtag_speed":

 - The Calao stuff hasn't worked since July 2008.  (Those Atmel
   targets need to work with a 32KHz core clock after reset until
   board-specific init-reset code sets up the PLL and enables a
   faster JTAg clock.)
 - Parport speed controls don't actually work (tops out at about
   1 MHz on typical HW).
 - In general, speed controls need to live in board.cfg files (or
   sometimes target.cfg files), not interface.cfg ...

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>

Removes the 'extern' keyword from function declarations.
Wraps long prototypes to fit into 80 columns.

Fixes documentation for jtag_tap_s::{,has}idcode fields.

Previous patch somehow made GCC lose some of its cookies;
work around, zero-init that struct.

Clean up code from the previous patch.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>