Fix misc issues with unused parameters and check-names.sh

Fix unused parameter warnings when MBEDTLS_TEST_HOOKS is not enabled.

A few issues were caught by check-names.sh namely:

- mbedtls_error_add was not capitalised.
- mbedtls_test_hook_error_add was being defined multiple times as the
  definition was in a header.

Signed-off-by: Chris Jones <christopher.jones@arm.com>
diff --git a/scripts/data_files/error.fmt b/scripts/data_files/error.fmt
index 9e479bb..7fed598 100644
--- a/scripts/data_files/error.fmt
+++ b/scripts/data_files/error.fmt
@@ -82,6 +82,8 @@
     return( NULL );
 }
 
+void (*mbedtls_test_hook_error_add)( int, int, const char *, int );
+
 void mbedtls_strerror( int ret, char *buf, size_t buflen )
 {
     size_t len;