From: Vaibhav Hiremath Date: Fri, 12 Feb 2016 20:34:19 +0000 (+0530) Subject: greybus: arche-apb-ctrl: Do not export any gpios X-Git-Tag: v4.9.8~1233^2~378^2~21^2~693 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=bc8eadd46c05280f7ddcf2ab7cf97d1cd15a578c;p=platform%2Fkernel%2Flinux-rpi3.git greybus: arche-apb-ctrl: Do not export any gpios With addition of operational state in driver, user/developer can switch to FW flashing state through sysfs. So no need to export any gpios to userspace now. Testing Done: Tested on EVT1.2 and DB3.5 platform Signed-off-by: Vaibhav Hiremath Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/greybus/arche-apb-ctrl.c b/drivers/staging/greybus/arche-apb-ctrl.c index e1df998..b888da3 100644 --- a/drivers/staging/greybus/arche-apb-ctrl.c +++ b/drivers/staging/greybus/arche-apb-ctrl.c @@ -58,17 +58,6 @@ static inline void assert_reset(unsigned int gpio) gpio_set_value(gpio, 0); } -/* Export gpio's to user space */ -static void export_gpios(struct arche_apb_ctrl_drvdata *apb) -{ - gpio_export(apb->resetn_gpio, false); -} - -static void unexport_gpios(struct arche_apb_ctrl_drvdata *apb) -{ - gpio_unexport(apb->resetn_gpio); -} - /* * Note: Please do not modify the below sequence, as it is as per the spec */ @@ -407,20 +396,15 @@ int arche_apb_ctrl_probe(struct platform_device *pdev) return ret; } - export_gpios(apb); - dev_info(&pdev->dev, "Device registered successfully\n"); return 0; } int arche_apb_ctrl_remove(struct platform_device *pdev) { - struct arche_apb_ctrl_drvdata *apb = platform_get_drvdata(pdev); - device_remove_file(&pdev->dev, &dev_attr_state); poweroff_seq(pdev); platform_set_drvdata(pdev, NULL); - unexport_gpios(apb); return 0; }