Commit 59a8831d authored by Colin Ian King's avatar Colin Ian King Committed by Mark Brown

spi: loopback-test: make several module parameters static

Several module parameters are local to the source, so make them
static. Cleans up several sparse warnings such as:

"symbol 'loop_req' was not declared. Should it be static?"
Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent f12a616e
...@@ -32,45 +32,45 @@ ...@@ -32,45 +32,45 @@
#include "spi-test.h" #include "spi-test.h"
/* flag to only simulate transfers */ /* flag to only simulate transfers */
int simulate_only; static int simulate_only;
module_param(simulate_only, int, 0); module_param(simulate_only, int, 0);
MODULE_PARM_DESC(simulate_only, "if not 0 do not execute the spi message"); MODULE_PARM_DESC(simulate_only, "if not 0 do not execute the spi message");
/* dump spi messages */ /* dump spi messages */
int dump_messages; static int dump_messages;
module_param(dump_messages, int, 0); module_param(dump_messages, int, 0);
MODULE_PARM_DESC(dump_messages, MODULE_PARM_DESC(dump_messages,
"=1 dump the basic spi_message_structure, " \ "=1 dump the basic spi_message_structure, " \
"=2 dump the spi_message_structure including data, " \ "=2 dump the spi_message_structure including data, " \
"=3 dump the spi_message structure before and after execution"); "=3 dump the spi_message structure before and after execution");
/* the device is jumpered for loopback - enabling some rx_buf tests */ /* the device is jumpered for loopback - enabling some rx_buf tests */
int loopback; static int loopback;
module_param(loopback, int, 0); module_param(loopback, int, 0);
MODULE_PARM_DESC(loopback, MODULE_PARM_DESC(loopback,
"if set enable loopback mode, where the rx_buf " \ "if set enable loopback mode, where the rx_buf " \
"is checked to match tx_buf after the spi_message " \ "is checked to match tx_buf after the spi_message " \
"is executed"); "is executed");
int loop_req; static int loop_req;
module_param(loop_req, int, 0); module_param(loop_req, int, 0);
MODULE_PARM_DESC(loop_req, MODULE_PARM_DESC(loop_req,
"if set controller will be asked to enable test loop mode. " \ "if set controller will be asked to enable test loop mode. " \
"If controller supported it, MISO and MOSI will be connected"); "If controller supported it, MISO and MOSI will be connected");
/* run only a specific test */ /* run only a specific test */
int run_only_test = -1; static int run_only_test = -1;
module_param(run_only_test, int, 0); module_param(run_only_test, int, 0);
MODULE_PARM_DESC(run_only_test, MODULE_PARM_DESC(run_only_test,
"only run the test with this number (0-based !)"); "only run the test with this number (0-based !)");
/* use vmalloc'ed buffers */ /* use vmalloc'ed buffers */
int use_vmalloc; static int use_vmalloc;
module_param(use_vmalloc, int, 0644); module_param(use_vmalloc, int, 0644);
MODULE_PARM_DESC(use_vmalloc, MODULE_PARM_DESC(use_vmalloc,
"use vmalloc'ed buffers instead of kmalloc'ed"); "use vmalloc'ed buffers instead of kmalloc'ed");
/* check rx ranges */ /* check rx ranges */
int check_ranges = 1; static int check_ranges = 1;
module_param(check_ranges, int, 0644); module_param(check_ranges, int, 0644);
MODULE_PARM_DESC(check_ranges, MODULE_PARM_DESC(check_ranges,
"checks rx_buffer pattern are valid"); "checks rx_buffer pattern are valid");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment