No need to use the "&" prefix and, since you're calling nonseekable_open(),
there is no need to use no_llseek().
Signed-off-by: Robert P. J. Day <rpjday@mindspring.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
 
 static const struct file_operations vmcp_fops = {
        .owner          = THIS_MODULE,
-       .open           = &vmcp_open,
-       .release        = &vmcp_release,
-       .read           = &vmcp_read,
-       .llseek         = &no_llseek,
-       .write          = &vmcp_write,
-       .unlocked_ioctl = &vmcp_ioctl,
-       .compat_ioctl   = &vmcp_ioctl
+       .open           = vmcp_open,
+       .release        = vmcp_release,
+       .read           = vmcp_read,
+       .write          = vmcp_write,
+       .unlocked_ioctl = vmcp_ioctl,
+       .compat_ioctl   = vmcp_ioctl
 };
 
 static struct miscdevice vmcp_dev = {