staging: greybus: Remove unnecessary platform_set_drvdata
authorAmitoj Kaur Chawla <amitoj1606@gmail.com>
Thu, 27 Jul 2017 03:19:07 +0000 (23:19 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 28 Jul 2017 04:48:06 +0000 (21:48 -0700)
Unnecessary platform_set_drvdata() has been removed since the driver
core clears the driver data to NULL after device release or on
probe failure. There is no need to manually clear the device
driver data to NULL.

The Coccinelle semantic patch used to make this change is as follows:
//<smpl>
@@
struct platform_device *pdev;
@@
- platform_set_drvdata(pdev, NULL);
//</smpl>

Signed-off-by: Amitoj Kaur Chawla <amitoj1606@gmail.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/greybus/arche-platform.c

index 625990400fa94f32fa7223917b48fd1e6582764b..4837aca41389bcd07b3c04870fb56d7d53b54f2d 100644 (file)
@@ -610,7 +610,6 @@ static int arche_platform_remove(struct platform_device *pdev)
        device_remove_file(&pdev->dev, &dev_attr_state);
        device_for_each_child(&pdev->dev, NULL, arche_remove_child);
        arche_platform_poweroff_seq(arche_pdata);
-       platform_set_drvdata(pdev, NULL);
 
        if (usb3613_hub_mode_ctrl(false))
                dev_warn(arche_pdata->dev, "failed to control hub device\n");