mailbox: Fix a couple of trivial static checker issues
authorLee Jones <lee.jones@linaro.org>
Fri, 16 Oct 2015 12:32:47 +0000 (13:32 +0100)
committerJassi Brar <jaswinder.singh@linaro.org>
Sat, 17 Oct 2015 05:06:56 +0000 (10:36 +0530)
This patch deals with a few spelling, white space and type
warnings reported by Intel's Kbuild Test Robot.

Reported-by: kbuild test robot <lkp@intel.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Signed-off-by: Jassi Brar <jaswinder.singh@linaro.org>
drivers/mailbox/mailbox-sti.c
drivers/mailbox/mailbox-test.c

index f11d4e8feabf62c9dca6f1f31a6acb0b97de4efa..4835817c53651dbfeeab4c4d7e6c86bf5f06aa58 100644 (file)
@@ -64,7 +64,7 @@ struct sti_mbox_device {
 };
 
 /**
- * STi Mailbox platform specfic configuration
+ * STi Mailbox platform specific configuration
  *
  * @num_inst:  Maximum number of instances in one HW Mailbox
  * @num_chan:  Maximum number of channel per instance
index cac1ba2f3e22a39ba08c9b9fb40e6c36d2495721..22f2e40476b9031aefb0af7a51be00b8dea067cd 100644 (file)
@@ -22,7 +22,7 @@
 #define MBOX_MAX_SIG_LEN       8
 #define MBOX_MAX_MSG_LEN       128
 #define MBOX_BYTES_PER_LINE    16
-#define MBOX_HEXDUMP_LINE_LEN  ((MBOX_BYTES_PER_LINE * 4) + 2)
+#define MBOX_HEXDUMP_LINE_LEN  ((MBOX_BYTES_PER_LINE * 4) + 2)
 #define MBOX_HEXDUMP_MAX_LEN   (MBOX_HEXDUMP_LINE_LEN *                \
                                 (MBOX_MAX_MSG_LEN / MBOX_BYTES_PER_LINE))
 
@@ -30,7 +30,7 @@ static struct dentry *root_debugfs_dir;
 
 struct mbox_test_device {
        struct device           *dev;
-       void __iomem            *mmio;
+       void                    *mmio;
        struct mbox_chan        *tx_channel;
        struct mbox_chan        *rx_channel;
        char                    *rx_buffer;
@@ -53,7 +53,7 @@ static ssize_t mbox_test_signal_write(struct file *filp,
 
        if (count > MBOX_MAX_SIG_LEN) {
                dev_err(tdev->dev,
-                       "Signal length %d greater than max allowed %d\n",
+                       "Signal length %zd greater than max allowed %d\n",
                        count, MBOX_MAX_SIG_LEN);
                return -EINVAL;
        }
@@ -92,7 +92,7 @@ static ssize_t mbox_test_message_write(struct file *filp,
 
        if (count > MBOX_MAX_MSG_LEN) {
                dev_err(tdev->dev,
-                       "Message length %d greater than max allowed %d\n",
+                       "Message length %zd greater than max allowed %d\n",
                        count, MBOX_MAX_MSG_LEN);
                return -EINVAL;
        }
@@ -303,7 +303,7 @@ static int mbox_test_probe(struct platform_device *pdev)
        tdev->tx_channel = mbox_test_request_channel(pdev, "tx");
        tdev->rx_channel = mbox_test_request_channel(pdev, "rx");
 
-       if (!tdev->tx_channel && !tdev->tx_channel)
+       if (!tdev->tx_channel && !tdev->rx_channel)
                return -EPROBE_DEFER;
 
        tdev->dev = &pdev->dev;