aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlexander Sverdlin <alexander.sverdlin@gmail.com>2015-06-22 16:38:53 -0400
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2015-06-24 14:03:10 -0400
commitc181fb3e723351e2f7a1f76b6c0627a4b8ad1723 (patch)
tree38b3c10bbb513fd795d166551a291e9fad852ee9
parentb953c0d234bc72e8489d3bf51a276c5c4ec85345 (diff)
ACPI / OF: Rename of_node() and acpi_node() to to_of_node() and to_acpi_node()
Commit 8a0662d9 introduced of_node and acpi_node symbols in global namespace but there were already ~63 of_node local variables or function parameters (no single acpi_node though, but anyway). After debugging undefined but used of_node local varible (which turned out to reference static function of_node() instead) it became clear that the names for the functions are too short and too generic for global scope. Signed-off-by: Alexander Sverdlin <alexander.sverdlin@gmail.com> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
-rw-r--r--drivers/base/property.c26
-rw-r--r--drivers/gpio/gpiolib.c4
-rw-r--r--drivers/leds/leds-gpio.c2
-rw-r--r--include/acpi/acpi_bus.h2
-rw-r--r--include/linux/acpi.h4
-rw-r--r--include/linux/of.h4
6 files changed, 21 insertions, 21 deletions
diff --git a/drivers/base/property.c b/drivers/base/property.c
index 1d0b116cae95..dfd4de69b67b 100644
--- a/drivers/base/property.c
+++ b/drivers/base/property.c
@@ -128,9 +128,9 @@ EXPORT_SYMBOL_GPL(device_property_present);
128bool fwnode_property_present(struct fwnode_handle *fwnode, const char *propname) 128bool fwnode_property_present(struct fwnode_handle *fwnode, const char *propname)
129{ 129{
130 if (is_of_node(fwnode)) 130 if (is_of_node(fwnode))
131 return of_property_read_bool(of_node(fwnode), propname); 131 return of_property_read_bool(to_of_node(fwnode), propname);
132 else if (is_acpi_node(fwnode)) 132 else if (is_acpi_node(fwnode))
133 return !acpi_dev_prop_get(acpi_node(fwnode), propname, NULL); 133 return !acpi_dev_prop_get(to_acpi_node(fwnode), propname, NULL);
134 134
135 return !!pset_prop_get(to_pset(fwnode), propname); 135 return !!pset_prop_get(to_pset(fwnode), propname);
136} 136}
@@ -285,10 +285,10 @@ EXPORT_SYMBOL_GPL(device_property_read_string);
285({ \ 285({ \
286 int _ret_; \ 286 int _ret_; \
287 if (is_of_node(_fwnode_)) \ 287 if (is_of_node(_fwnode_)) \
288 _ret_ = OF_DEV_PROP_READ_ARRAY(of_node(_fwnode_), _propname_, \ 288 _ret_ = OF_DEV_PROP_READ_ARRAY(to_of_node(_fwnode_), _propname_, \
289 _type_, _val_, _nval_); \ 289 _type_, _val_, _nval_); \
290 else if (is_acpi_node(_fwnode_)) \ 290 else if (is_acpi_node(_fwnode_)) \
291 _ret_ = acpi_dev_prop_read(acpi_node(_fwnode_), _propname_, \ 291 _ret_ = acpi_dev_prop_read(to_acpi_node(_fwnode_), _propname_, \
292 _proptype_, _val_, _nval_); \ 292 _proptype_, _val_, _nval_); \
293 else \ 293 else \
294 _ret_ = pset_prop_read_array(to_pset(_fwnode_), _propname_, \ 294 _ret_ = pset_prop_read_array(to_pset(_fwnode_), _propname_, \
@@ -424,11 +424,11 @@ int fwnode_property_read_string_array(struct fwnode_handle *fwnode,
424{ 424{
425 if (is_of_node(fwnode)) 425 if (is_of_node(fwnode))
426 return val ? 426 return val ?
427 of_property_read_string_array(of_node(fwnode), propname, 427 of_property_read_string_array(to_of_node(fwnode),
428 val, nval) : 428 propname, val, nval) :
429 of_property_count_strings(of_node(fwnode), propname); 429 of_property_count_strings(to_of_node(fwnode), propname);
430 else if (is_acpi_node(fwnode)) 430 else if (is_acpi_node(fwnode))
431 return acpi_dev_prop_read(acpi_node(fwnode), propname, 431 return acpi_dev_prop_read(to_acpi_node(fwnode), propname,
432 DEV_PROP_STRING, val, nval); 432 DEV_PROP_STRING, val, nval);
433 433
434 return pset_prop_read_array(to_pset(fwnode), propname, 434 return pset_prop_read_array(to_pset(fwnode), propname,
@@ -455,9 +455,9 @@ int fwnode_property_read_string(struct fwnode_handle *fwnode,
455 const char *propname, const char **val) 455 const char *propname, const char **val)
456{ 456{
457 if (is_of_node(fwnode)) 457 if (is_of_node(fwnode))
458 return of_property_read_string(of_node(fwnode), propname, val); 458 return of_property_read_string(to_of_node(fwnode), propname, val);
459 else if (is_acpi_node(fwnode)) 459 else if (is_acpi_node(fwnode))
460 return acpi_dev_prop_read(acpi_node(fwnode), propname, 460 return acpi_dev_prop_read(to_acpi_node(fwnode), propname,
461 DEV_PROP_STRING, val, 1); 461 DEV_PROP_STRING, val, 1);
462 462
463 return -ENXIO; 463 return -ENXIO;
@@ -475,13 +475,13 @@ struct fwnode_handle *device_get_next_child_node(struct device *dev,
475 if (IS_ENABLED(CONFIG_OF) && dev->of_node) { 475 if (IS_ENABLED(CONFIG_OF) && dev->of_node) {
476 struct device_node *node; 476 struct device_node *node;
477 477
478 node = of_get_next_available_child(dev->of_node, of_node(child)); 478 node = of_get_next_available_child(dev->of_node, to_of_node(child));
479 if (node) 479 if (node)
480 return &node->fwnode; 480 return &node->fwnode;
481 } else if (IS_ENABLED(CONFIG_ACPI)) { 481 } else if (IS_ENABLED(CONFIG_ACPI)) {
482 struct acpi_device *node; 482 struct acpi_device *node;
483 483
484 node = acpi_get_next_child(dev, acpi_node(child)); 484 node = acpi_get_next_child(dev, to_acpi_node(child));
485 if (node) 485 if (node)
486 return acpi_fwnode_handle(node); 486 return acpi_fwnode_handle(node);
487 } 487 }
@@ -500,7 +500,7 @@ EXPORT_SYMBOL_GPL(device_get_next_child_node);
500void fwnode_handle_put(struct fwnode_handle *fwnode) 500void fwnode_handle_put(struct fwnode_handle *fwnode)
501{ 501{
502 if (is_of_node(fwnode)) 502 if (is_of_node(fwnode))
503 of_node_put(of_node(fwnode)); 503 of_node_put(to_of_node(fwnode));
504} 504}
505EXPORT_SYMBOL_GPL(fwnode_handle_put); 505EXPORT_SYMBOL_GPL(fwnode_handle_put);
506 506
diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c
index 6bc612b8a49f..5d8b2b35e2a2 100644
--- a/drivers/gpio/gpiolib.c
+++ b/drivers/gpio/gpiolib.c
@@ -2040,14 +2040,14 @@ struct gpio_desc *fwnode_get_named_gpiod(struct fwnode_handle *fwnode,
2040 if (is_of_node(fwnode)) { 2040 if (is_of_node(fwnode)) {
2041 enum of_gpio_flags flags; 2041 enum of_gpio_flags flags;
2042 2042
2043 desc = of_get_named_gpiod_flags(of_node(fwnode), propname, 0, 2043 desc = of_get_named_gpiod_flags(to_of_node(fwnode), propname, 0,
2044 &flags); 2044 &flags);
2045 if (!IS_ERR(desc)) 2045 if (!IS_ERR(desc))
2046 active_low = flags & OF_GPIO_ACTIVE_LOW; 2046 active_low = flags & OF_GPIO_ACTIVE_LOW;
2047 } else if (is_acpi_node(fwnode)) { 2047 } else if (is_acpi_node(fwnode)) {
2048 struct acpi_gpio_info info; 2048 struct acpi_gpio_info info;
2049 2049
2050 desc = acpi_get_gpiod_by_index(acpi_node(fwnode), propname, 0, 2050 desc = acpi_get_gpiod_by_index(to_acpi_node(fwnode), propname, 0,
2051 &info); 2051 &info);
2052 if (!IS_ERR(desc)) 2052 if (!IS_ERR(desc))
2053 active_low = info.active_low; 2053 active_low = info.active_low;
diff --git a/drivers/leds/leds-gpio.c b/drivers/leds/leds-gpio.c
index 15eb3f86f670..d2d54d62afee 100644
--- a/drivers/leds/leds-gpio.c
+++ b/drivers/leds/leds-gpio.c
@@ -191,7 +191,7 @@ static struct gpio_leds_priv *gpio_leds_create(struct platform_device *pdev)
191 goto err; 191 goto err;
192 } 192 }
193 193
194 np = of_node(child); 194 np = to_of_node(child);
195 195
196 if (fwnode_property_present(child, "label")) { 196 if (fwnode_property_present(child, "label")) {
197 fwnode_property_read_string(child, "label", &led.name); 197 fwnode_property_read_string(child, "label", &led.name);
diff --git a/include/acpi/acpi_bus.h b/include/acpi/acpi_bus.h
index 8de4fa90e8c4..1224be8184fc 100644
--- a/include/acpi/acpi_bus.h
+++ b/include/acpi/acpi_bus.h
@@ -385,7 +385,7 @@ static inline bool is_acpi_node(struct fwnode_handle *fwnode)
385 return fwnode && fwnode->type == FWNODE_ACPI; 385 return fwnode && fwnode->type == FWNODE_ACPI;
386} 386}
387 387
388static inline struct acpi_device *acpi_node(struct fwnode_handle *fwnode) 388static inline struct acpi_device *to_acpi_node(struct fwnode_handle *fwnode)
389{ 389{
390 return is_acpi_node(fwnode) ? 390 return is_acpi_node(fwnode) ?
391 container_of(fwnode, struct acpi_device, fwnode) : NULL; 391 container_of(fwnode, struct acpi_device, fwnode) : NULL;
diff --git a/include/linux/acpi.h b/include/linux/acpi.h
index e4da5e35e29c..ec3c98ed9dca 100644
--- a/include/linux/acpi.h
+++ b/include/linux/acpi.h
@@ -53,7 +53,7 @@ static inline acpi_handle acpi_device_handle(struct acpi_device *adev)
53 return adev ? adev->handle : NULL; 53 return adev ? adev->handle : NULL;
54} 54}
55 55
56#define ACPI_COMPANION(dev) acpi_node((dev)->fwnode) 56#define ACPI_COMPANION(dev) to_acpi_node((dev)->fwnode)
57#define ACPI_COMPANION_SET(dev, adev) set_primary_fwnode(dev, (adev) ? \ 57#define ACPI_COMPANION_SET(dev, adev) set_primary_fwnode(dev, (adev) ? \
58 acpi_fwnode_handle(adev) : NULL) 58 acpi_fwnode_handle(adev) : NULL)
59#define ACPI_HANDLE(dev) acpi_device_handle(ACPI_COMPANION(dev)) 59#define ACPI_HANDLE(dev) acpi_device_handle(ACPI_COMPANION(dev))
@@ -473,7 +473,7 @@ static inline bool is_acpi_node(struct fwnode_handle *fwnode)
473 return false; 473 return false;
474} 474}
475 475
476static inline struct acpi_device *acpi_node(struct fwnode_handle *fwnode) 476static inline struct acpi_device *to_acpi_node(struct fwnode_handle *fwnode)
477{ 477{
478 return NULL; 478 return NULL;
479} 479}
diff --git a/include/linux/of.h b/include/linux/of.h
index b871ff9d81d7..f05fdcea4e66 100644
--- a/include/linux/of.h
+++ b/include/linux/of.h
@@ -128,7 +128,7 @@ static inline bool is_of_node(struct fwnode_handle *fwnode)
128 return fwnode && fwnode->type == FWNODE_OF; 128 return fwnode && fwnode->type == FWNODE_OF;
129} 129}
130 130
131static inline struct device_node *of_node(struct fwnode_handle *fwnode) 131static inline struct device_node *to_of_node(struct fwnode_handle *fwnode)
132{ 132{
133 return fwnode ? container_of(fwnode, struct device_node, fwnode) : NULL; 133 return fwnode ? container_of(fwnode, struct device_node, fwnode) : NULL;
134} 134}
@@ -387,7 +387,7 @@ static inline bool is_of_node(struct fwnode_handle *fwnode)
387 return false; 387 return false;
388} 388}
389 389
390static inline struct device_node *of_node(struct fwnode_handle *fwnode) 390static inline struct device_node *to_of_node(struct fwnode_handle *fwnode)
391{ 391{
392 return NULL; 392 return NULL;
393} 393}