Remove the function 'finish_driver' since a direct call to
platform_device_unregister() is intuitive enough to signify the original
intention of the function being removed. Coccinelle semantic patch used to
detect this:
@@
identifier wrapper, func;
type T;
@@
*T wrapper(...)
{
(
	return func(...);
|
	func(...);
)
}
Signed-off-by: Eva Rachel Retuya <eraretuya@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
        return PTR_ERR_OR_ZERO(firmware_pdev);
 }
 
-static void finish_driver(void)
-{
-       platform_device_unregister(firmware_pdev);
-}
-
 static int gs_fpgaboot(void)
 {
        int err;
        return 0;
 
 errout:
-       finish_driver();
+       platform_device_unregister(firmware_pdev);
 
        return err;
 }
 
 static void __exit gs_fpgaboot_exit(void)
 {
-       finish_driver();
+       platform_device_unregister(firmware_pdev);
        pr_info("FPGA image download module removed\n");
 }