staging/mei: mei-amt-version - make all function static and used
authorTomas Winkler <tomas.winkler@intel.com>
Mon, 20 Feb 2012 12:46:03 +0000 (14:46 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 24 Feb 2012 19:59:56 +0000 (11:59 -0800)
This patch eliminates following type of warnings
warning: no previous prototype for '...func...' [-Wmissing-prototypes]

For this is a single file example lets make all API-like functions
be static.
Since all static functions should be used so let's call
to amt_host_if_deinit() even if in this example it's not really necessary

Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Acked-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/mei/mei-amt-version.c

index be04934cd3295cffbcb03e5ca4f0f8cb7e4e1611..970f2d3316aafb3bf721502cece39ec1aeab2ffa 100644 (file)
@@ -293,7 +293,7 @@ struct amt_host_if {
 };
 
 
-bool amt_host_if_init(struct amt_host_if *acmd,
+static bool amt_host_if_init(struct amt_host_if *acmd,
                      unsigned long send_timeout, bool verbose)
 {
        acmd->send_timeout = (send_timeout) ? send_timeout : 20000;
@@ -301,13 +301,13 @@ bool amt_host_if_init(struct amt_host_if *acmd,
        return acmd->initialized;
 }
 
-void amt_host_if_deinit(struct amt_host_if *acmd)
+static void amt_host_if_deinit(struct amt_host_if *acmd)
 {
        mei_deinit(&acmd->mei_cl);
        acmd->initialized = false;
 }
 
-uint32_t amt_verify_code_versions(const struct amt_host_if_resp_header *resp)
+static uint32_t amt_verify_code_versions(const struct amt_host_if_resp_header *resp)
 {
        uint32_t status = AMT_STATUS_SUCCESS;
        struct amt_code_versions *code_ver;
@@ -346,7 +346,7 @@ out:
        return status;
 }
 
-uint32_t amt_verify_response_header(uint32_t command,
+static uint32_t amt_verify_response_header(uint32_t command,
                                const struct amt_host_if_msg_header *resp_hdr,
                                uint32_t response_size)
 {
@@ -409,7 +409,7 @@ static uint32_t amt_host_if_call(struct amt_host_if *acmd,
 }
 
 
-uint32_t amt_get_code_versions(struct amt_host_if *cmd,
+static uint32_t amt_get_code_versions(struct amt_host_if *cmd,
                               struct amt_code_versions *versions)
 {
        struct amt_host_if_resp_header *response = NULL;
@@ -455,6 +455,8 @@ int main(int argc, char **argv)
 
        status = amt_get_code_versions(&acmd, &ver);
 
+       amt_host_if_deinit(&acmd);
+
        switch (status) {
        case AMT_STATUS_HOST_IF_EMPTY_RESPONSE:
                printf("Intel AMT: DISABLED\n");