efirtc: remove unnecessary code efi_rtc_open & efi_rtc_close
authorNaveen Kumar Parna <parna.naveenkumar@gmail.com>
Tue, 22 Jan 2019 08:25:10 +0000 (13:55 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 22 Jan 2019 12:19:12 +0000 (13:19 +0100)
There is no advantage to keep 'struct file_operations.open & .close'
API's. So removed the unnecessary code efi_rtc_open & efi_rtc_close.

Signed-off-by: Naveen Kumar Parna <parna.naveenkumar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/char/efirtc.c

index d9aab643997e9f253a0abf4e2c02fefccc917f13..11781ebffbf725ed07b15148a1628d38d0563b59 100644 (file)
@@ -254,27 +254,6 @@ static long efi_rtc_ioctl(struct file *file, unsigned int cmd,
        return -ENOTTY;
 }
 
-/*
- *     We enforce only one user at a time here with the open/close.
- *     Also clear the previous interrupt data on an open, and clean
- *     up things on a close.
- */
-
-static int efi_rtc_open(struct inode *inode, struct file *file)
-{
-       /*
-        * nothing special to do here
-        * We do accept multiple open files at the same time as we
-        * synchronize on the per call operation.
-        */
-       return 0;
-}
-
-static int efi_rtc_close(struct inode *inode, struct file *file)
-{
-       return 0;
-}
-
 /*
  *     The various file operations we support.
  */
@@ -282,8 +261,6 @@ static int efi_rtc_close(struct inode *inode, struct file *file)
 static const struct file_operations efi_rtc_fops = {
        .owner          = THIS_MODULE,
        .unlocked_ioctl = efi_rtc_ioctl,
-       .open           = efi_rtc_open,
-       .release        = efi_rtc_close,
        .llseek         = no_llseek,
 };