staging:ced1401: remove read write callbacks from fops
authorDevendra Naga <devendra.aaru@gmail.com>
Thu, 20 Sep 2012 16:10:57 +0000 (12:10 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 21 Sep 2012 15:56:41 +0000 (08:56 -0700)
As the driver says that read and writes should not be performed
and instead the user to kernel transactions are performed through
ioctl interface, remove these functions as they are not required

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/ced1401/usb1401.c

index 59d69b406a2122c4d697b5bf6d46f36b1665c20d..6ba0ef6525611f0de97284ad744de1e0cfa437fd 100644 (file)
@@ -261,24 +261,6 @@ static int ced_flush(struct file *file, fl_owner_t id)
        return res;
 }
 
-static ssize_t ced_read(struct file *file, char *buffer, size_t count,
-                       loff_t * ppos)
-{
-       DEVICE_EXTENSION *pdx = file->private_data;
-       dev_err(&pdx->interface->dev, "%s called: use ioctl for cedusb",
-               __func__);
-       return 0;               // as we do not do reads this way
-}
-
-static ssize_t ced_write(struct file *file, const char *user_buffer,
-                        size_t count, loff_t * ppos)
-{
-       DEVICE_EXTENSION *pdx = file->private_data;
-       dev_err(&pdx->interface->dev, "%s called: use ioctl for cedusb",
-               __func__);
-       return 0;
-}
-
 /***************************************************************************
 ** CanAcceptIoRequests
 ** If the device is removed, interface is set NULL. We also clear our pointer
@@ -1402,8 +1384,6 @@ static int ced_ioctl(struct inode *node, struct file *file, unsigned int cmd,
 
 static const struct file_operations ced_fops = {
        .owner = THIS_MODULE,
-       .read = ced_read,
-       .write = ced_write,
        .open = ced_open,
        .release = ced_release,
        .flush = ced_flush,