Commit db3e50f3 authored by Sakari Ailus's avatar Sakari Ailus Committed by Rafael J. Wysocki

device property: Get rid of struct fwnode_handle type field

Instead of relying on the struct fwnode_handle type field, define
fwnode_operations structs for all separate types of fwnodes. To find out
the type, compare to the ops field to relevant ops structs.

This change has two benefits:

1. it avoids adding the type field to each and every instance of struct
fwnode_handle, thus saving memory and

2. makes the ops field the single factor that defines both the types of
the fwnode as well as defines the implementation of its operations,
decreasing the possibility of bugs when developing code dealing with
fwnode internals.
Suggested-by: default avatarRob Herring <robh@kernel.org>
Signed-off-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
Reviewed-by: default avatarMika Westerberg <mika.westerberg@linux.intel.com>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent b81b7291
...@@ -56,8 +56,7 @@ static bool acpi_nondev_subnode_extract(const union acpi_object *desc, ...@@ -56,8 +56,7 @@ static bool acpi_nondev_subnode_extract(const union acpi_object *desc,
return false; return false;
dn->name = link->package.elements[0].string.pointer; dn->name = link->package.elements[0].string.pointer;
dn->fwnode.type = FWNODE_ACPI_DATA; dn->fwnode.ops = &acpi_data_fwnode_ops;
dn->fwnode.ops = &acpi_fwnode_ops;
dn->parent = parent; dn->parent = parent;
INIT_LIST_HEAD(&dn->data.subnodes); INIT_LIST_HEAD(&dn->data.subnodes);
...@@ -469,10 +468,10 @@ EXPORT_SYMBOL_GPL(acpi_dev_get_property); ...@@ -469,10 +468,10 @@ EXPORT_SYMBOL_GPL(acpi_dev_get_property);
static struct acpi_device_data *acpi_device_data_of_node(struct fwnode_handle *fwnode) static struct acpi_device_data *acpi_device_data_of_node(struct fwnode_handle *fwnode)
{ {
if (fwnode->type == FWNODE_ACPI) { if (is_acpi_device_node(fwnode)) {
struct acpi_device *adev = to_acpi_device_node(fwnode); struct acpi_device *adev = to_acpi_device_node(fwnode);
return &adev->data; return &adev->data;
} else if (fwnode->type == FWNODE_ACPI_DATA) { } else if (is_acpi_data_node(fwnode)) {
struct acpi_data_node *dn = to_acpi_data_node(fwnode); struct acpi_data_node *dn = to_acpi_data_node(fwnode);
return &dn->data; return &dn->data;
} }
...@@ -903,7 +902,7 @@ struct fwnode_handle *acpi_get_next_subnode(struct fwnode_handle *fwnode, ...@@ -903,7 +902,7 @@ struct fwnode_handle *acpi_get_next_subnode(struct fwnode_handle *fwnode,
struct acpi_device *adev = to_acpi_device_node(fwnode); struct acpi_device *adev = to_acpi_device_node(fwnode);
struct list_head *head, *next; struct list_head *head, *next;
if (!child || child->type == FWNODE_ACPI) { if (!child || is_acpi_device_node(child)) {
if (adev) if (adev)
head = &adev->children; head = &adev->children;
else else
...@@ -927,7 +926,7 @@ struct fwnode_handle *acpi_get_next_subnode(struct fwnode_handle *fwnode, ...@@ -927,7 +926,7 @@ struct fwnode_handle *acpi_get_next_subnode(struct fwnode_handle *fwnode,
} }
nondev: nondev:
if (!child || child->type == FWNODE_ACPI_DATA) { if (!child || is_acpi_data_node(child)) {
struct acpi_data_node *data = to_acpi_data_node(fwnode); struct acpi_data_node *data = to_acpi_data_node(fwnode);
struct acpi_data_node *dn; struct acpi_data_node *dn;
...@@ -1223,16 +1222,26 @@ static int acpi_fwnode_graph_parse_endpoint(struct fwnode_handle *fwnode, ...@@ -1223,16 +1222,26 @@ static int acpi_fwnode_graph_parse_endpoint(struct fwnode_handle *fwnode,
return 0; return 0;
} }
const struct fwnode_operations acpi_fwnode_ops = { #define DECLARE_ACPI_FWNODE_OPS(ops) \
.device_is_available = acpi_fwnode_device_is_available, const struct fwnode_operations ops = { \
.property_present = acpi_fwnode_property_present, .device_is_available = acpi_fwnode_device_is_available, \
.property_read_int_array = acpi_fwnode_property_read_int_array, .property_present = acpi_fwnode_property_present, \
.property_read_string_array = acpi_fwnode_property_read_string_array, .property_read_int_array = \
.get_parent = acpi_node_get_parent, acpi_fwnode_property_read_int_array, \
.get_next_child_node = acpi_get_next_subnode, .property_read_string_array = \
.get_named_child_node = acpi_fwnode_get_named_child_node, acpi_fwnode_property_read_string_array, \
.graph_get_next_endpoint = acpi_fwnode_graph_get_next_endpoint, .get_parent = acpi_node_get_parent, \
.graph_get_remote_endpoint = acpi_fwnode_graph_get_remote_endpoint, .get_next_child_node = acpi_get_next_subnode, \
.graph_get_port_parent = acpi_node_get_parent, .get_named_child_node = acpi_fwnode_get_named_child_node, \
.graph_parse_endpoint = acpi_fwnode_graph_parse_endpoint, .graph_get_next_endpoint = \
}; acpi_fwnode_graph_get_next_endpoint, \
.graph_get_remote_endpoint = \
acpi_fwnode_graph_get_remote_endpoint, \
.graph_get_port_parent = acpi_node_get_parent, \
.graph_parse_endpoint = acpi_fwnode_graph_parse_endpoint, \
}; \
EXPORT_SYMBOL_GPL(ops)
DECLARE_ACPI_FWNODE_OPS(acpi_device_fwnode_ops);
DECLARE_ACPI_FWNODE_OPS(acpi_data_fwnode_ops);
const struct fwnode_operations acpi_static_fwnode_ops;
...@@ -1467,8 +1467,7 @@ void acpi_init_device_object(struct acpi_device *device, acpi_handle handle, ...@@ -1467,8 +1467,7 @@ void acpi_init_device_object(struct acpi_device *device, acpi_handle handle,
device->device_type = type; device->device_type = type;
device->handle = handle; device->handle = handle;
device->parent = acpi_bus_get_parent(handle); device->parent = acpi_bus_get_parent(handle);
device->fwnode.type = FWNODE_ACPI; device->fwnode.ops = &acpi_device_fwnode_ops;
device->fwnode.ops = &acpi_fwnode_ops;
acpi_set_device_status(device, sta); acpi_set_device_status(device, sta);
acpi_device_get_busid(device); acpi_device_get_busid(device);
acpi_set_pnp_ids(handle, &device->pnp, type); acpi_set_pnp_ids(handle, &device->pnp, type);
......
...@@ -25,9 +25,11 @@ struct property_set { ...@@ -25,9 +25,11 @@ struct property_set {
const struct property_entry *properties; const struct property_entry *properties;
}; };
static const struct fwnode_operations pset_fwnode_ops;
static inline bool is_pset_node(struct fwnode_handle *fwnode) static inline bool is_pset_node(struct fwnode_handle *fwnode)
{ {
return !IS_ERR_OR_NULL(fwnode) && fwnode->type == FWNODE_PDATA; return !IS_ERR_OR_NULL(fwnode) && fwnode->ops == &pset_fwnode_ops;
} }
static inline struct property_set *to_pset_node(struct fwnode_handle *fwnode) static inline struct property_set *to_pset_node(struct fwnode_handle *fwnode)
...@@ -900,7 +902,6 @@ int device_add_properties(struct device *dev, ...@@ -900,7 +902,6 @@ int device_add_properties(struct device *dev,
if (IS_ERR(p)) if (IS_ERR(p))
return PTR_ERR(p); return PTR_ERR(p);
p->fwnode.type = FWNODE_PDATA;
p->fwnode.ops = &pset_fwnode_ops; p->fwnode.ops = &pset_fwnode_ops;
set_secondary_fwnode(dev, &p->fwnode); set_secondary_fwnode(dev, &p->fwnode);
return 0; return 0;
......
...@@ -952,3 +952,4 @@ const struct fwnode_operations of_fwnode_ops = { ...@@ -952,3 +952,4 @@ const struct fwnode_operations of_fwnode_ops = {
.graph_get_port_parent = of_fwnode_graph_get_port_parent, .graph_get_port_parent = of_fwnode_graph_get_port_parent,
.graph_parse_endpoint = of_fwnode_graph_parse_endpoint, .graph_parse_endpoint = of_fwnode_graph_parse_endpoint,
}; };
EXPORT_SYMBOL_GPL(of_fwnode_ops);
...@@ -395,15 +395,21 @@ struct acpi_data_node { ...@@ -395,15 +395,21 @@ struct acpi_data_node {
struct completion kobj_done; struct completion kobj_done;
}; };
extern const struct fwnode_operations acpi_device_fwnode_ops;
extern const struct fwnode_operations acpi_data_fwnode_ops;
extern const struct fwnode_operations acpi_static_fwnode_ops;
static inline bool is_acpi_node(struct fwnode_handle *fwnode) static inline bool is_acpi_node(struct fwnode_handle *fwnode)
{ {
return !IS_ERR_OR_NULL(fwnode) && (fwnode->type == FWNODE_ACPI return !IS_ERR_OR_NULL(fwnode) &&
|| fwnode->type == FWNODE_ACPI_DATA); (fwnode->ops == &acpi_device_fwnode_ops
|| fwnode->ops == &acpi_data_fwnode_ops);
} }
static inline bool is_acpi_device_node(struct fwnode_handle *fwnode) static inline bool is_acpi_device_node(struct fwnode_handle *fwnode)
{ {
return !IS_ERR_OR_NULL(fwnode) && fwnode->type == FWNODE_ACPI; return !IS_ERR_OR_NULL(fwnode) &&
fwnode->ops == &acpi_device_fwnode_ops;
} }
static inline struct acpi_device *to_acpi_device_node(struct fwnode_handle *fwnode) static inline struct acpi_device *to_acpi_device_node(struct fwnode_handle *fwnode)
...@@ -414,7 +420,7 @@ static inline struct acpi_device *to_acpi_device_node(struct fwnode_handle *fwno ...@@ -414,7 +420,7 @@ static inline struct acpi_device *to_acpi_device_node(struct fwnode_handle *fwno
static inline bool is_acpi_data_node(struct fwnode_handle *fwnode) static inline bool is_acpi_data_node(struct fwnode_handle *fwnode)
{ {
return !IS_ERR_OR_NULL(fwnode) && fwnode->type == FWNODE_ACPI_DATA; return !IS_ERR_OR_NULL(fwnode) && fwnode->ops == &acpi_data_fwnode_ops;
} }
static inline struct acpi_data_node *to_acpi_data_node(struct fwnode_handle *fwnode) static inline struct acpi_data_node *to_acpi_data_node(struct fwnode_handle *fwnode)
...@@ -423,6 +429,12 @@ static inline struct acpi_data_node *to_acpi_data_node(struct fwnode_handle *fwn ...@@ -423,6 +429,12 @@ static inline struct acpi_data_node *to_acpi_data_node(struct fwnode_handle *fwn
container_of(fwnode, struct acpi_data_node, fwnode) : NULL; container_of(fwnode, struct acpi_data_node, fwnode) : NULL;
} }
static inline bool is_acpi_static_node(struct fwnode_handle *fwnode)
{
return !IS_ERR_OR_NULL(fwnode) &&
fwnode->ops == &acpi_static_fwnode_ops;
}
static inline bool acpi_data_node_match(struct fwnode_handle *fwnode, static inline bool acpi_data_node_match(struct fwnode_handle *fwnode,
const char *name) const char *name)
{ {
......
...@@ -57,9 +57,6 @@ static inline acpi_handle acpi_device_handle(struct acpi_device *adev) ...@@ -57,9 +57,6 @@ static inline acpi_handle acpi_device_handle(struct acpi_device *adev)
acpi_fwnode_handle(adev) : NULL) acpi_fwnode_handle(adev) : NULL)
#define ACPI_HANDLE(dev) acpi_device_handle(ACPI_COMPANION(dev)) #define ACPI_HANDLE(dev) acpi_device_handle(ACPI_COMPANION(dev))
extern const struct fwnode_operations acpi_fwnode_ops;
static inline struct fwnode_handle *acpi_alloc_fwnode_static(void) static inline struct fwnode_handle *acpi_alloc_fwnode_static(void)
{ {
struct fwnode_handle *fwnode; struct fwnode_handle *fwnode;
...@@ -68,15 +65,14 @@ static inline struct fwnode_handle *acpi_alloc_fwnode_static(void) ...@@ -68,15 +65,14 @@ static inline struct fwnode_handle *acpi_alloc_fwnode_static(void)
if (!fwnode) if (!fwnode)
return NULL; return NULL;
fwnode->type = FWNODE_ACPI_STATIC; fwnode->ops = &acpi_static_fwnode_ops;
fwnode->ops = &acpi_fwnode_ops;
return fwnode; return fwnode;
} }
static inline void acpi_free_fwnode_static(struct fwnode_handle *fwnode) static inline void acpi_free_fwnode_static(struct fwnode_handle *fwnode)
{ {
if (WARN_ON(!fwnode || fwnode->type != FWNODE_ACPI_STATIC)) if (WARN_ON(!is_acpi_static_node(fwnode)))
return; return;
kfree(fwnode); kfree(fwnode);
......
...@@ -14,20 +14,9 @@ ...@@ -14,20 +14,9 @@
#include <linux/types.h> #include <linux/types.h>
enum fwnode_type {
FWNODE_INVALID = 0,
FWNODE_OF,
FWNODE_ACPI,
FWNODE_ACPI_DATA,
FWNODE_ACPI_STATIC,
FWNODE_PDATA,
FWNODE_IRQCHIP
};
struct fwnode_operations; struct fwnode_operations;
struct fwnode_handle { struct fwnode_handle {
enum fwnode_type type;
struct fwnode_handle *secondary; struct fwnode_handle *secondary;
const struct fwnode_operations *ops; const struct fwnode_operations *ops;
}; };
......
...@@ -265,9 +265,11 @@ static inline struct fwnode_handle *of_node_to_fwnode(struct device_node *node) ...@@ -265,9 +265,11 @@ static inline struct fwnode_handle *of_node_to_fwnode(struct device_node *node)
return node ? &node->fwnode : NULL; return node ? &node->fwnode : NULL;
} }
extern const struct fwnode_operations irqchip_fwnode_ops;
static inline bool is_fwnode_irqchip(struct fwnode_handle *fwnode) static inline bool is_fwnode_irqchip(struct fwnode_handle *fwnode)
{ {
return fwnode && fwnode->type == FWNODE_IRQCHIP; return fwnode && fwnode->ops == &irqchip_fwnode_ops;
} }
extern void irq_domain_update_bus_token(struct irq_domain *domain, extern void irq_domain_update_bus_token(struct irq_domain *domain,
......
...@@ -104,7 +104,6 @@ extern const struct fwnode_operations of_fwnode_ops; ...@@ -104,7 +104,6 @@ extern const struct fwnode_operations of_fwnode_ops;
static inline void of_node_init(struct device_node *node) static inline void of_node_init(struct device_node *node)
{ {
kobject_init(&node->kobj, &of_node_ktype); kobject_init(&node->kobj, &of_node_ktype);
node->fwnode.type = FWNODE_OF;
node->fwnode.ops = &of_fwnode_ops; node->fwnode.ops = &of_fwnode_ops;
} }
...@@ -152,7 +151,7 @@ void of_core_init(void); ...@@ -152,7 +151,7 @@ void of_core_init(void);
static inline bool is_of_node(const struct fwnode_handle *fwnode) static inline bool is_of_node(const struct fwnode_handle *fwnode)
{ {
return !IS_ERR_OR_NULL(fwnode) && fwnode->type == FWNODE_OF; return !IS_ERR_OR_NULL(fwnode) && fwnode->ops == &of_fwnode_ops;
} }
#define to_of_node(__fwnode) \ #define to_of_node(__fwnode) \
......
...@@ -41,6 +41,8 @@ static inline void debugfs_add_domain_dir(struct irq_domain *d) { } ...@@ -41,6 +41,8 @@ static inline void debugfs_add_domain_dir(struct irq_domain *d) { }
static inline void debugfs_remove_domain_dir(struct irq_domain *d) { } static inline void debugfs_remove_domain_dir(struct irq_domain *d) { }
#endif #endif
const struct fwnode_operations irqchip_fwnode_ops;
/** /**
* irq_domain_alloc_fwnode - Allocate a fwnode_handle suitable for * irq_domain_alloc_fwnode - Allocate a fwnode_handle suitable for
* identifying an irq domain * identifying an irq domain
...@@ -86,7 +88,7 @@ struct fwnode_handle *__irq_domain_alloc_fwnode(unsigned int type, int id, ...@@ -86,7 +88,7 @@ struct fwnode_handle *__irq_domain_alloc_fwnode(unsigned int type, int id,
fwid->type = type; fwid->type = type;
fwid->name = n; fwid->name = n;
fwid->data = data; fwid->data = data;
fwid->fwnode.type = FWNODE_IRQCHIP; fwid->fwnode.ops = &irqchip_fwnode_ops;
return &fwid->fwnode; return &fwid->fwnode;
} }
EXPORT_SYMBOL_GPL(__irq_domain_alloc_fwnode); EXPORT_SYMBOL_GPL(__irq_domain_alloc_fwnode);
...@@ -193,10 +195,8 @@ struct irq_domain *__irq_domain_add(struct fwnode_handle *fwnode, int size, ...@@ -193,10 +195,8 @@ struct irq_domain *__irq_domain_add(struct fwnode_handle *fwnode, int size,
} }
if (!domain->name) { if (!domain->name) {
if (fwnode) { if (fwnode)
pr_err("Invalid fwnode type (%d) for irqdomain\n", pr_err("Invalid fwnode type for irqdomain\n");
fwnode->type);
}
domain->name = kasprintf(GFP_KERNEL, "unknown-%d", domain->name = kasprintf(GFP_KERNEL, "unknown-%d",
atomic_inc_return(&unknown_domains)); atomic_inc_return(&unknown_domains));
if (!domain->name) { if (!domain->name) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment