Commit 5b9ff0ba authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Greg Kroah-Hartman
Browse files

device property: Constify a few fwnode APIs 



The fwnode parameter is not altered in the following APIs:

- fwnode_get_next_parent_dev()
- fwnode_is_ancestor_of()
- fwnode_graph_get_endpoint_count()

so constify them.

Reported-by: default avatarRussell King (Oracle) <rmk+kernel@armlinux.org.uk>
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20230324112720.71315-1-andriy.shevchenko@linux.intel.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 0a392354

drivers/base/property.c

+3 −3
Original line number Diff line number Diff line
@@ -648,7 +648,7 @@ EXPORT_SYMBOL_GPL(fwnode_get_next_parent); 
 *

 * Return: a pointer to the device of the @fwnode's closest ancestor.

 */

struct device *fwnode_get_next_parent_dev(struct fwnode_handle *fwnode)

struct device *fwnode_get_next_parent_dev(const struct fwnode_handle *fwnode)

{

	struct fwnode_handle *parent;

	struct device *dev;
@@ -718,7 +718,7 @@ EXPORT_SYMBOL_GPL(fwnode_get_nth_parent); 
 *

 * Return: true if @ancestor is an ancestor of @child. Otherwise, returns false.

 */

bool fwnode_is_ancestor_of(struct fwnode_handle *ancestor, struct fwnode_handle *child)

bool fwnode_is_ancestor_of(const struct fwnode_handle *ancestor, const struct fwnode_handle *child)

{

	struct fwnode_handle *parent;
@@ -1235,7 +1235,7 @@ EXPORT_SYMBOL_GPL(fwnode_graph_get_endpoint_by_id); 
 * If FWNODE_GRAPH_DEVICE_DISABLED flag is specified, also unconnected endpoints

 * and endpoints connected to disabled devices are counted.

 */

unsigned int fwnode_graph_get_endpoint_count(struct fwnode_handle *fwnode,

unsigned int fwnode_graph_get_endpoint_count(const struct fwnode_handle *fwnode,

					     unsigned long flags)

{

	struct fwnode_handle *ep;

include/linux/property.h

+3 −3
Original line number Diff line number Diff line
@@ -105,11 +105,11 @@ struct fwnode_handle *fwnode_get_next_parent(struct fwnode_handle *fwnode); 
	for (parent = fwnode_get_parent(fwnode); parent;	\

	     parent = fwnode_get_next_parent(parent))



struct device *fwnode_get_next_parent_dev(struct fwnode_handle *fwnode);

struct device *fwnode_get_next_parent_dev(const struct fwnode_handle *fwnode);

unsigned int fwnode_count_parents(const struct fwnode_handle *fwn);

struct fwnode_handle *fwnode_get_nth_parent(struct fwnode_handle *fwn,

					    unsigned int depth);

bool fwnode_is_ancestor_of(struct fwnode_handle *ancestor, struct fwnode_handle *child);

bool fwnode_is_ancestor_of(const struct fwnode_handle *ancestor, const struct fwnode_handle *child);

struct fwnode_handle *fwnode_get_next_child_node(

	const struct fwnode_handle *fwnode, struct fwnode_handle *child);

struct fwnode_handle *fwnode_get_next_available_child_node(
@@ -433,7 +433,7 @@ static inline bool fwnode_graph_is_endpoint(const struct fwnode_handle *fwnode) 
struct fwnode_handle *

fwnode_graph_get_endpoint_by_id(const struct fwnode_handle *fwnode,

				u32 port, u32 endpoint, unsigned long flags);

unsigned int fwnode_graph_get_endpoint_count(struct fwnode_handle *fwnode,

unsigned int fwnode_graph_get_endpoint_count(const struct fwnode_handle *fwnode,

					     unsigned long flags);



#define fwnode_graph_for_each_endpoint(fwnode, child)				\