kernel: remove compile warnings.

remove warnings at virtio-keyboard, virtio-esm, virtio-gl, maru_codec.

Signed-off-by: Kitae Kim <kt920.kim@samsung.com>

diff --git a/drivers/char/virtio-gl.c b/drivers/char/virtio-gl.c
index b71a2b50dccbc92bc25262ccf5288de0fdcc3c87..647ff691c2b02905428a0174f944effe43cc37af 100755 (executable)
--- a/drivers/char/virtio-gl.c
+++ b/drivers/char/virtio-gl.c
@@ -223,8 +223,7 @@ static int glmem_mmap(struct file *filp, struct vm_area_struct *vma)
        return 0;\r
 }\r
 \r
-// FIX ME : fsync parameter 2,3 parameter missing\r
-static int glmem_fsync(struct file *filp, int datasync)\r
+static int glmem_fsync(struct file *filp, loff_t *start, loff_t *end, int datasync)\r
 {\r
        struct virtio_gl_data *gldata = to_virtio_gl_data(filp);\r
 \r

diff --git a/drivers/maru/maru_codec.c b/drivers/maru/maru_codec.c
index ec691faa17935ecb3543abfb2345c1fc8f32c036..04bbf7cfc765caa308604d5e60f0e03c626a1297 100644 (file)
--- a/drivers/maru/maru_codec.c
+++ b/drivers/maru/maru_codec.c
@@ -143,7 +143,7 @@ static long svcodec_ioctl(struct file *file,
        } else if (cmd == CODEC_CMD_GET_MMAP_OFFSET) {
                value = readl(svcodec->ioaddr + cmd);
                CODEC_LOG(KERN_DEBUG,
-                               "ioctl: get mmap offset: %d.\n", value);
+                               "ioctl: get mmap offset: %ld.\n", value);
        } else {
                CODEC_LOG(KERN_INFO,
                                "ioctl: no command available.\n");

diff --git a/drivers/maru/maru_virtio_esm.c b/drivers/maru/maru_virtio_esm.c
index f2cb2d54b929b556f4b2697fb4b40b7bc4f4eb6a..27b6b88804dd202975f711ea82f04d34c9b010c4 100644 (file)
--- a/drivers/maru/maru_virtio_esm.c
+++ b/drivers/maru/maru_virtio_esm.c
@@ -100,7 +100,7 @@ static ssize_t esm_read(struct file *f, char __user *buf, size_t len, loff_t *of
        return 0;
 }
 
-static ssize_t esm_write(struct inode *i, const char __user *ubuf, size_t len, loff_t *off)
+static ssize_t esm_write(struct file *f, const char __user *ubuf, size_t len, loff_t *off)
 {
        int err = 0;
        ssize_t ret = 0;
@@ -120,7 +120,11 @@ static ssize_t esm_write(struct inode *i, const char __user *ubuf, size_t len, l
        }
        buf[len - 1] = '\0';
 
-       kstrtou16(buf, 10, &vesm->progress.percentage);
+       ret = kstrtou16(buf, 10, &vesm->progress.percentage);
+       if (ret < 0) {
+               VESM_LOG(KERN_ERR, "failed to convert string to integer.\n");
+               return ret;
+       }
 
        VESM_LOG(KERN_DEBUG, "boot up progress is [%u] percent done.\n", vesm->progress.percentage);
 

diff --git a/drivers/maru/maru_virtio_keyboard.c b/drivers/maru/maru_virtio_keyboard.c
index 15af27a70cbc7419cb32d1cbe5d166d237b41612..be2f68d925ff5075e47f8ec215d560ddaf3efbb0 100644 (file)
--- a/drivers/maru/maru_virtio_keyboard.c
+++ b/drivers/maru/maru_virtio_keyboard.c
@@ -175,7 +175,7 @@ static int virtio_keyboard_probe(struct virtio_device *vdev)
                                sizeof(struct EmulKbdEvent));
        }
 
-       ret = virtqueue_add_buf(vkbd->vq, &vkbd->sg, 0, 10, (void *)(10), GFP_ATOMIC);
+       ret = virtqueue_add_buf(vkbd->vq, vkbd->sg, 0, 10, (void *)(10), GFP_ATOMIC);
        if (ret < 0) {
                VKBD_LOG(KERN_ERR, "failed to add buffer to virtqueue.\n");
                kfree(vkbd);