diff options
author | Heikki Krogerus <heikki.krogerus@linux.intel.com> | 2016-01-25 04:03:47 -0500 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2016-02-24 03:13:08 -0500 |
commit | 7d5e9737efda16535e5b54bd627ef4881d11d31f (patch) | |
tree | 3dc55914d843942bc812703c6554f7b7449703b9 /net/rfkill/rfkill-gpio.c | |
parent | 648b50dd6abf8e6e5b589bb8e6873a4596389dbe (diff) |
net: rfkill: gpio: get the name and type from device property
This prepares the driver for removal of platform data.
Signed-off-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/rfkill/rfkill-gpio.c')
-rw-r--r-- | net/rfkill/rfkill-gpio.c | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/net/rfkill/rfkill-gpio.c b/net/rfkill/rfkill-gpio.c index 4b1e3f35f06c..1a9c0316aad1 100644 --- a/net/rfkill/rfkill-gpio.c +++ b/net/rfkill/rfkill-gpio.c | |||
@@ -81,7 +81,6 @@ static int rfkill_gpio_acpi_probe(struct device *dev, | |||
81 | if (!id) | 81 | if (!id) |
82 | return -ENODEV; | 82 | return -ENODEV; |
83 | 83 | ||
84 | rfkill->name = dev_name(dev); | ||
85 | rfkill->type = (unsigned)id->driver_data; | 84 | rfkill->type = (unsigned)id->driver_data; |
86 | 85 | ||
87 | return acpi_dev_add_driver_gpios(ACPI_COMPANION(dev), | 86 | return acpi_dev_add_driver_gpios(ACPI_COMPANION(dev), |
@@ -93,12 +92,21 @@ static int rfkill_gpio_probe(struct platform_device *pdev) | |||
93 | struct rfkill_gpio_platform_data *pdata = pdev->dev.platform_data; | 92 | struct rfkill_gpio_platform_data *pdata = pdev->dev.platform_data; |
94 | struct rfkill_gpio_data *rfkill; | 93 | struct rfkill_gpio_data *rfkill; |
95 | struct gpio_desc *gpio; | 94 | struct gpio_desc *gpio; |
95 | const char *type_name; | ||
96 | int ret; | 96 | int ret; |
97 | 97 | ||
98 | rfkill = devm_kzalloc(&pdev->dev, sizeof(*rfkill), GFP_KERNEL); | 98 | rfkill = devm_kzalloc(&pdev->dev, sizeof(*rfkill), GFP_KERNEL); |
99 | if (!rfkill) | 99 | if (!rfkill) |
100 | return -ENOMEM; | 100 | return -ENOMEM; |
101 | 101 | ||
102 | device_property_read_string(&pdev->dev, "name", &rfkill->name); | ||
103 | device_property_read_string(&pdev->dev, "type", &type_name); | ||
104 | |||
105 | if (!rfkill->name) | ||
106 | rfkill->name = dev_name(&pdev->dev); | ||
107 | |||
108 | rfkill->type = rfkill_find_type(type_name); | ||
109 | |||
102 | if (ACPI_HANDLE(&pdev->dev)) { | 110 | if (ACPI_HANDLE(&pdev->dev)) { |
103 | ret = rfkill_gpio_acpi_probe(&pdev->dev, rfkill); | 111 | ret = rfkill_gpio_acpi_probe(&pdev->dev, rfkill); |
104 | if (ret) | 112 | if (ret) |
@@ -124,10 +132,8 @@ static int rfkill_gpio_probe(struct platform_device *pdev) | |||
124 | 132 | ||
125 | rfkill->shutdown_gpio = gpio; | 133 | rfkill->shutdown_gpio = gpio; |
126 | 134 | ||
127 | /* Make sure at-least one of the GPIO is defined and that | 135 | /* Make sure at-least one GPIO is defined for this instance */ |
128 | * a name is specified for this instance | 136 | if (!rfkill->reset_gpio && !rfkill->shutdown_gpio) { |
129 | */ | ||
130 | if ((!rfkill->reset_gpio && !rfkill->shutdown_gpio) || !rfkill->name) { | ||
131 | dev_err(&pdev->dev, "invalid platform data\n"); | 137 | dev_err(&pdev->dev, "invalid platform data\n"); |
132 | return -EINVAL; | 138 | return -EINVAL; |
133 | } | 139 | } |