* [PATCH v2 14/26] media: xilinx: fix a debug printk [not found] <c4389ab1c02bb08c1a55012fdb859c8b10bdc47e.1509569763.git.mchehab@s-opensource.com> @ 2017-11-01 21:05 ` Mauro Carvalho Chehab 2017-11-02 2:43 ` Laurent Pinchart 0 siblings, 1 reply; 7+ messages in thread From: Mauro Carvalho Chehab @ 2017-11-01 21:05 UTC (permalink / raw) To: linux-arm-kernel Two orthogonal changesets caused a breakage at several printk messages inside xilinx. Changeset 859969b38e2e ("[media] v4l: Switch from V4L2 OF not V4L2 fwnode API") made davinci to use struct fwnode_handle instead of struct device_node. Changeset 68d9c47b1679 ("media: Convert to using %pOF instead of full_name") changed the printk to not use ->full_name, but, instead, to rely on %pOF. With both patches applied, the Kernel will do the wrong thing, as warned by smatch: drivers/media/platform/xilinx/xilinx-vipp.c:108 xvip_graph_build_one() error: '%pOF' expects argument of type 'struct device_node*', argument 4 has type 'void*' drivers/media/platform/xilinx/xilinx-vipp.c:117 xvip_graph_build_one() error: '%pOF' expects argument of type 'struct device_node*', argument 4 has type 'void*' drivers/media/platform/xilinx/xilinx-vipp.c:126 xvip_graph_build_one() error: '%pOF' expects argument of type 'struct device_node*', argument 4 has type 'void*' drivers/media/platform/xilinx/xilinx-vipp.c:138 xvip_graph_build_one() error: '%pOF' expects argument of type 'struct device_node*', argument 3 has type 'void*' drivers/media/platform/xilinx/xilinx-vipp.c:148 xvip_graph_build_one() error: '%pOF' expects argument of type 'struct device_node*', argument 4 has type 'void*' drivers/media/platform/xilinx/xilinx-vipp.c:245 xvip_graph_build_dma() error: '%pOF' expects argument of type 'struct device_node*', argument 3 has type 'void*' drivers/media/platform/xilinx/xilinx-vipp.c:254 xvip_graph_build_dma() error: '%pOF' expects argument of type 'struct device_node*', argument 4 has type 'void*' So, change the logic to actually print the device name that was obtained before the print logic. Fixes: 68d9c47b1679 ("media: Convert to using %pOF instead of full_name") Fixes: 859969b38e2e ("[media] v4l: Switch from V4L2 OF not V4L2 fwnode API") Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com> --- drivers/media/platform/xilinx/xilinx-vipp.c | 31 ++++++++++++++++------------- 1 file changed, 17 insertions(+), 14 deletions(-) diff --git a/drivers/media/platform/xilinx/xilinx-vipp.c b/drivers/media/platform/xilinx/xilinx-vipp.c index d881cf09876d..dd777c834c43 100644 --- a/drivers/media/platform/xilinx/xilinx-vipp.c +++ b/drivers/media/platform/xilinx/xilinx-vipp.c @@ -65,6 +65,9 @@ xvip_graph_find_entity(struct xvip_composite_device *xdev, return NULL; } +#define LOCAL_NAME(link) to_of_node(link.local_node)->full_name +#define REMOTE_NAME(link) to_of_node(link.remote_node)->full_name + static int xvip_graph_build_one(struct xvip_composite_device *xdev, struct xvip_graph_entity *entity) { @@ -103,9 +106,9 @@ static int xvip_graph_build_one(struct xvip_composite_device *xdev, * the link. */ if (link.local_port >= local->num_pads) { - dev_err(xdev->dev, "invalid port number %u for %pOF\n", + dev_err(xdev->dev, "invalid port number %u for %s\n", link.local_port, - to_of_node(link.local_node)); + LOCAL_NAME(link)); v4l2_fwnode_put_link(&link); ret = -EINVAL; break; @@ -114,8 +117,8 @@ static int xvip_graph_build_one(struct xvip_composite_device *xdev, local_pad = &local->pads[link.local_port]; if (local_pad->flags & MEDIA_PAD_FL_SINK) { - dev_dbg(xdev->dev, "skipping sink port %pOF:%u\n", - to_of_node(link.local_node), + dev_dbg(xdev->dev, "skipping sink port %s:%u\n", + LOCAL_NAME(link), link.local_port); v4l2_fwnode_put_link(&link); continue; @@ -123,8 +126,8 @@ static int xvip_graph_build_one(struct xvip_composite_device *xdev, /* Skip DMA engines, they will be processed separately. */ if (link.remote_node == of_fwnode_handle(xdev->dev->of_node)) { - dev_dbg(xdev->dev, "skipping DMA port %pOF:%u\n", - to_of_node(link.local_node), + dev_dbg(xdev->dev, "skipping DMA port %s:%u\n", + REMOTE_NAME(link), link.local_port); v4l2_fwnode_put_link(&link); continue; @@ -134,8 +137,8 @@ static int xvip_graph_build_one(struct xvip_composite_device *xdev, ent = xvip_graph_find_entity(xdev, to_of_node(link.remote_node)); if (ent == NULL) { - dev_err(xdev->dev, "no entity found for %pOF\n", - to_of_node(link.remote_node)); + dev_err(xdev->dev, "no entity found for %s\n", + REMOTE_NAME(link)); v4l2_fwnode_put_link(&link); ret = -ENODEV; break; @@ -144,8 +147,8 @@ static int xvip_graph_build_one(struct xvip_composite_device *xdev, remote = ent->entity; if (link.remote_port >= remote->num_pads) { - dev_err(xdev->dev, "invalid port number %u on %pOF\n", - link.remote_port, to_of_node(link.remote_node)); + dev_err(xdev->dev, "invalid port number %u on %s\n", + link.remote_port, REMOTE_NAME(link)); v4l2_fwnode_put_link(&link); ret = -EINVAL; break; @@ -241,17 +244,17 @@ static int xvip_graph_build_dma(struct xvip_composite_device *xdev) ent = xvip_graph_find_entity(xdev, to_of_node(link.remote_node)); if (ent == NULL) { - dev_err(xdev->dev, "no entity found for %pOF\n", - to_of_node(link.remote_node)); + dev_err(xdev->dev, "no entity found for %s\n", + REMOTE_NAME(link)); v4l2_fwnode_put_link(&link); ret = -ENODEV; break; } if (link.remote_port >= ent->entity->num_pads) { - dev_err(xdev->dev, "invalid port number %u on %pOF\n", + dev_err(xdev->dev, "invalid port number %u on %s\n", link.remote_port, - to_of_node(link.remote_node)); + REMOTE_NAME(link)); v4l2_fwnode_put_link(&link); ret = -EINVAL; break; -- 2.13.6 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH v2 14/26] media: xilinx: fix a debug printk 2017-11-01 21:05 ` [PATCH v2 14/26] media: xilinx: fix a debug printk Mauro Carvalho Chehab @ 2017-11-02 2:43 ` Laurent Pinchart 2017-11-02 9:20 ` Sakari Ailus ` (2 more replies) 0 siblings, 3 replies; 7+ messages in thread From: Laurent Pinchart @ 2017-11-02 2:43 UTC (permalink / raw) To: linux-arm-kernel Hi Mauro, (CC'ing Rob and Sakari) Thank you for the patch. On Wednesday, 1 November 2017 23:05:51 EET Mauro Carvalho Chehab wrote: > Two orthogonal changesets caused a breakage at several printk > messages inside xilinx. Changeset 859969b38e2e > ("[media] v4l: Switch from V4L2 OF not V4L2 fwnode API") > made davinci to use struct fwnode_handle instead of > struct device_node. Changeset 68d9c47b1679 > ("media: Convert to using %pOF instead of full_name") > changed the printk to not use ->full_name, but, instead, > to rely on %pOF. > > With both patches applied, the Kernel will do the wrong > thing, as warned by smatch: > drivers/media/platform/xilinx/xilinx-vipp.c:108 xvip_graph_build_one() > error: '%pOF' expects argument of type 'struct device_node*', argument 4 > has type 'void*' > drivers/media/platform/xilinx/xilinx-vipp.c:117 xvip_graph_build_one() > error: '%pOF' expects argument of type 'struct device_node*', argument 4 has > type 'void*' > drivers/media/platform/xilinx/xilinx-vipp.c:126 xvip_graph_build_one() > error: '%pOF' expects argument of type 'struct device_node*', argument 4 > has type 'void*' > drivers/media/platform/xilinx/xilinx-vipp.c:138 xvip_graph_build_one() > error: '%pOF' expects argument of type 'struct device_node*', argument 3 has > type 'void*' > drivers/media/platform/xilinx/xilinx-vipp.c:148 xvip_graph_build_one() > error: '%pOF' expects argument of type 'struct device_node*', argument 4 > has type 'void*' > drivers/media/platform/xilinx/xilinx-vipp.c:245 xvip_graph_build_dma() > error: '%pOF' expects argument of type 'struct device_node*', argument 3 has > type 'void*' > drivers/media/platform/xilinx/xilinx-vipp.c:254 xvip_graph_build_dma() > error: '%pOF' expects argument of type 'struct device_node*', argument 4 > has type 'void*' > > So, change the logic to actually print the device name > that was obtained before the print logic. This doesn't seem like a good idea to me. The main point of commit 68d9c47b1679 is to avoid accessing full_name directly to prepare for removal of that field. to_of_node() is defined as #define to_of_node(__fwnode) \ ({ \ typeof(__fwnode) __to_of_node_fwnode = (__fwnode); \ \ is_of_node(__to_of_node_fwnode) ? \ container_of(__to_of_node_fwnode, \ struct device_node, fwnode) : \ NULL; \ }) when CONFIG_OF is set, and as static inline struct device_node *to_of_node(const struct fwnode_handle *fwnode) { return NULL; } otherwise. I wonder why smatch believes the value is a void *, but in any case that should be fixed either in smatch (if it's a smatch bug) or in the definition of the to_of_node() macro. > Fixes: 68d9c47b1679 ("media: Convert to using %pOF instead of full_name") > Fixes: 859969b38e2e ("[media] v4l: Switch from V4L2 OF not V4L2 fwnode API") When submitting fixes you should CC the author of the original commits. They should have more information about the context, and in this case I believe Rob would have pointed out that adding back references to full_name would break the refactoring he's working on. > Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com> > --- > drivers/media/platform/xilinx/xilinx-vipp.c | 31 +++++++++++++------------- > 1 file changed, 17 insertions(+), 14 deletions(-) > > diff --git a/drivers/media/platform/xilinx/xilinx-vipp.c > b/drivers/media/platform/xilinx/xilinx-vipp.c index > d881cf09876d..dd777c834c43 100644 > --- a/drivers/media/platform/xilinx/xilinx-vipp.c > +++ b/drivers/media/platform/xilinx/xilinx-vipp.c > @@ -65,6 +65,9 @@ xvip_graph_find_entity(struct xvip_composite_device *xdev, > return NULL; > } > > +#define LOCAL_NAME(link) to_of_node(link.local_node)->full_name > +#define REMOTE_NAME(link) to_of_node(link.remote_node)->full_name > + > static int xvip_graph_build_one(struct xvip_composite_device *xdev, > struct xvip_graph_entity *entity) > { > @@ -103,9 +106,9 @@ static int xvip_graph_build_one(struct > xvip_composite_device *xdev, > * the link. > */ > if (link.local_port >= local->num_pads) { > - dev_err(xdev->dev, "invalid port number %u for %pOF\n", > + dev_err(xdev->dev, "invalid port number %u for %s\n", > link.local_port, > - to_of_node(link.local_node)); > + LOCAL_NAME(link)); > v4l2_fwnode_put_link(&link); > ret = -EINVAL; > break; > @@ -114,8 +117,8 @@ static int xvip_graph_build_one(struct > xvip_composite_device *xdev, local_pad = &local->pads[link.local_port]; > > if (local_pad->flags & MEDIA_PAD_FL_SINK) { > - dev_dbg(xdev->dev, "skipping sink port %pOF:%u\n", > - to_of_node(link.local_node), > + dev_dbg(xdev->dev, "skipping sink port %s:%u\n", > + LOCAL_NAME(link), > link.local_port); > v4l2_fwnode_put_link(&link); > continue; > @@ -123,8 +126,8 @@ static int xvip_graph_build_one(struct > xvip_composite_device *xdev, > > /* Skip DMA engines, they will be processed separately. */ > if (link.remote_node == of_fwnode_handle(xdev->dev->of_node)) { > - dev_dbg(xdev->dev, "skipping DMA port %pOF:%u\n", > - to_of_node(link.local_node), > + dev_dbg(xdev->dev, "skipping DMA port %s:%u\n", > + REMOTE_NAME(link), > link.local_port); > v4l2_fwnode_put_link(&link); > continue; > @@ -134,8 +137,8 @@ static int xvip_graph_build_one(struct > xvip_composite_device *xdev, ent = xvip_graph_find_entity(xdev, > to_of_node(link.remote_node)); > if (ent == NULL) { > - dev_err(xdev->dev, "no entity found for %pOF\n", > - to_of_node(link.remote_node)); > + dev_err(xdev->dev, "no entity found for %s\n", > + REMOTE_NAME(link)); > v4l2_fwnode_put_link(&link); > ret = -ENODEV; > break; > @@ -144,8 +147,8 @@ static int xvip_graph_build_one(struct > xvip_composite_device *xdev, remote = ent->entity; > > if (link.remote_port >= remote->num_pads) { > - dev_err(xdev->dev, "invalid port number %u on %pOF\n", > - link.remote_port, to_of_node(link.remote_node)); > + dev_err(xdev->dev, "invalid port number %u on %s\n", > + link.remote_port, REMOTE_NAME(link)); > v4l2_fwnode_put_link(&link); > ret = -EINVAL; > break; > @@ -241,17 +244,17 @@ static int xvip_graph_build_dma(struct > xvip_composite_device *xdev) ent = xvip_graph_find_entity(xdev, > to_of_node(link.remote_node)); > if (ent == NULL) { > - dev_err(xdev->dev, "no entity found for %pOF\n", > - to_of_node(link.remote_node)); > + dev_err(xdev->dev, "no entity found for %s\n", > + REMOTE_NAME(link)); > v4l2_fwnode_put_link(&link); > ret = -ENODEV; > break; > } > > if (link.remote_port >= ent->entity->num_pads) { > - dev_err(xdev->dev, "invalid port number %u on %pOF\n", > + dev_err(xdev->dev, "invalid port number %u on %s\n", > link.remote_port, > - to_of_node(link.remote_node)); > + REMOTE_NAME(link)); > v4l2_fwnode_put_link(&link); > ret = -EINVAL; > break; -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2 14/26] media: xilinx: fix a debug printk 2017-11-02 2:43 ` Laurent Pinchart @ 2017-11-02 9:20 ` Sakari Ailus 2017-11-02 9:57 ` [PATCH 1/1] of: Make return types of to_of_node and of_fwnode_handle macros consistent Sakari Ailus 2017-11-02 9:59 ` [RESEND PATCH " Sakari Ailus 2 siblings, 0 replies; 7+ messages in thread From: Sakari Ailus @ 2017-11-02 9:20 UTC (permalink / raw) To: linux-arm-kernel Hi Laurent and Mauro, On Thu, Nov 02, 2017 at 04:43:51AM +0200, Laurent Pinchart wrote: > Hi Mauro, > > (CC'ing Rob and Sakari) > > Thank you for the patch. > > On Wednesday, 1 November 2017 23:05:51 EET Mauro Carvalho Chehab wrote: > > Two orthogonal changesets caused a breakage at several printk > > messages inside xilinx. Changeset 859969b38e2e > > ("[media] v4l: Switch from V4L2 OF not V4L2 fwnode API") > > made davinci to use struct fwnode_handle instead of > > struct device_node. Changeset 68d9c47b1679 > > ("media: Convert to using %pOF instead of full_name") > > changed the printk to not use ->full_name, but, instead, > > to rely on %pOF. > > > > With both patches applied, the Kernel will do the wrong > > thing, as warned by smatch: > > drivers/media/platform/xilinx/xilinx-vipp.c:108 xvip_graph_build_one() > > error: '%pOF' expects argument of type 'struct device_node*', argument 4 > > has type 'void*' > > drivers/media/platform/xilinx/xilinx-vipp.c:117 xvip_graph_build_one() > > error: '%pOF' expects argument of type 'struct device_node*', argument 4 has > > type 'void*' > > drivers/media/platform/xilinx/xilinx-vipp.c:126 xvip_graph_build_one() > > error: '%pOF' expects argument of type 'struct device_node*', argument 4 > > has type 'void*' > > drivers/media/platform/xilinx/xilinx-vipp.c:138 xvip_graph_build_one() > > error: '%pOF' expects argument of type 'struct device_node*', argument 3 has > > type 'void*' > > drivers/media/platform/xilinx/xilinx-vipp.c:148 xvip_graph_build_one() > > error: '%pOF' expects argument of type 'struct device_node*', argument 4 > > has type 'void*' > > drivers/media/platform/xilinx/xilinx-vipp.c:245 xvip_graph_build_dma() > > error: '%pOF' expects argument of type 'struct device_node*', argument 3 has > > type 'void*' > > drivers/media/platform/xilinx/xilinx-vipp.c:254 xvip_graph_build_dma() > > error: '%pOF' expects argument of type 'struct device_node*', argument 4 > > has type 'void*' > > > > So, change the logic to actually print the device name > > that was obtained before the print logic. > > This doesn't seem like a good idea to me. The main point of commit > 68d9c47b1679 is to avoid accessing full_name directly to prepare for removal > of that field. > > to_of_node() is defined as > > #define to_of_node(__fwnode) \ > ({ \ > typeof(__fwnode) __to_of_node_fwnode = (__fwnode); \ > \ > is_of_node(__to_of_node_fwnode) ? \ > container_of(__to_of_node_fwnode, \ > struct device_node, fwnode) : \ > NULL; \ > }) > > when CONFIG_OF is set, and as > > static inline struct device_node *to_of_node(const struct fwnode_handle > *fwnode) > { > return NULL; > } > > otherwise. I wonder why smatch believes the value is a void *, but in any case > that should be fixed either in smatch (if it's a smatch bug) or in the > definition of the to_of_node() macro. Probably because NULL maybe returned by the function. I can write a patch for that. I presume the same issue would be present in a lot of places. -- Sakari Ailus sakari.ailus at linux.intel.com ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/1] of: Make return types of to_of_node and of_fwnode_handle macros consistent 2017-11-02 2:43 ` Laurent Pinchart 2017-11-02 9:20 ` Sakari Ailus @ 2017-11-02 9:57 ` Sakari Ailus 2017-11-02 9:59 ` [RESEND PATCH " Sakari Ailus 2 siblings, 0 replies; 7+ messages in thread From: Sakari Ailus @ 2017-11-02 9:57 UTC (permalink / raw) To: linux-arm-kernel to_of_node() macro checks whether the fwnode_handle passed to it is not an OF node, and if so, returns NULL in order to be NULL-safe. Otherwise it returns the pointer to the OF node which the fwnode_handle contains. The problem with returning NULL is that its type was void *, which sometimes matters. Explicitly return struct device_node * instead. Make a similar change to of_fwnode_handle() as well. Fixes: d20dc1493db4 ("of: Support const and non-const use for to_of_node()") Fixes: debd3a3b27c7 ("of: Make of_fwnode_handle() safer") Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> --- Hi Mauro, Could you check whether this addresses the smatch issue with the xilinx driver? Thanks. include/linux/of.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/include/linux/of.h b/include/linux/of.h index b240ed69dc96..0651231c115e 100644 --- a/include/linux/of.h +++ b/include/linux/of.h @@ -161,7 +161,7 @@ static inline bool is_of_node(const struct fwnode_handle *fwnode) is_of_node(__to_of_node_fwnode) ? \ container_of(__to_of_node_fwnode, \ struct device_node, fwnode) : \ - NULL; \ + (struct device_node *)NULL; \ }) #define of_fwnode_handle(node) \ @@ -169,7 +169,8 @@ static inline bool is_of_node(const struct fwnode_handle *fwnode) typeof(node) __of_fwnode_handle_node = (node); \ \ __of_fwnode_handle_node ? \ - &__of_fwnode_handle_node->fwnode : NULL; \ + &__of_fwnode_handle_node->fwnode : \ + (struct fwnode_handle *)NULL; \ }) static inline bool of_have_populated_dt(void) -- 2.11.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [RESEND PATCH 1/1] of: Make return types of to_of_node and of_fwnode_handle macros consistent 2017-11-02 2:43 ` Laurent Pinchart 2017-11-02 9:20 ` Sakari Ailus 2017-11-02 9:57 ` [PATCH 1/1] of: Make return types of to_of_node and of_fwnode_handle macros consistent Sakari Ailus @ 2017-11-02 9:59 ` Sakari Ailus 2017-11-02 17:37 ` Laurent Pinchart 2017-11-06 21:45 ` Rob Herring 2 siblings, 2 replies; 7+ messages in thread From: Sakari Ailus @ 2017-11-02 9:59 UTC (permalink / raw) To: linux-arm-kernel (Fixed Mauro's e-mail.) to_of_node() macro checks whether the fwnode_handle passed to it is not an OF node, and if so, returns NULL in order to be NULL-safe. Otherwise it returns the pointer to the OF node which the fwnode_handle contains. The problem with returning NULL is that its type was void *, which sometimes matters. Explicitly return struct device_node * instead. Make a similar change to of_fwnode_handle() as well. Fixes: d20dc1493db4 ("of: Support const and non-const use for to_of_node()") Fixes: debd3a3b27c7 ("of: Make of_fwnode_handle() safer") Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> --- Hi Mauro, Could you check whether this addresses the smatch issue with the xilinx driver? Thanks. include/linux/of.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/include/linux/of.h b/include/linux/of.h index b240ed69dc96..0651231c115e 100644 --- a/include/linux/of.h +++ b/include/linux/of.h @@ -161,7 +161,7 @@ static inline bool is_of_node(const struct fwnode_handle *fwnode) is_of_node(__to_of_node_fwnode) ? \ container_of(__to_of_node_fwnode, \ struct device_node, fwnode) : \ - NULL; \ + (struct device_node *)NULL; \ }) #define of_fwnode_handle(node) \ @@ -169,7 +169,8 @@ static inline bool is_of_node(const struct fwnode_handle *fwnode) typeof(node) __of_fwnode_handle_node = (node); \ \ __of_fwnode_handle_node ? \ - &__of_fwnode_handle_node->fwnode : NULL; \ + &__of_fwnode_handle_node->fwnode : \ + (struct fwnode_handle *)NULL; \ }) static inline bool of_have_populated_dt(void) -- 2.11.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [RESEND PATCH 1/1] of: Make return types of to_of_node and of_fwnode_handle macros consistent 2017-11-02 9:59 ` [RESEND PATCH " Sakari Ailus @ 2017-11-02 17:37 ` Laurent Pinchart 2017-11-06 21:45 ` Rob Herring 1 sibling, 0 replies; 7+ messages in thread From: Laurent Pinchart @ 2017-11-02 17:37 UTC (permalink / raw) To: linux-arm-kernel Hi Sakari, Thank you for the patch. On Thursday, 2 November 2017 11:59:18 EET Sakari Ailus wrote: > (Fixed Mauro's e-mail.) > > to_of_node() macro checks whether the fwnode_handle passed to it is not an > OF node, and if so, returns NULL in order to be NULL-safe. Otherwise it > returns the pointer to the OF node which the fwnode_handle contains. > > The problem with returning NULL is that its type was void *, which > sometimes matters. Explicitly return struct device_node * instead. > > Make a similar change to of_fwnode_handle() as well. > > Fixes: d20dc1493db4 ("of: Support const and non-const use for to_of_node()") > Fixes: debd3a3b27c7 ("of: Make of_fwnode_handle() safer") > Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > --- > Hi Mauro, > > Could you check whether this addresses the smatch issue with the xilinx > driver? > > Thanks. > > include/linux/of.h | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/include/linux/of.h b/include/linux/of.h > index b240ed69dc96..0651231c115e 100644 > --- a/include/linux/of.h > +++ b/include/linux/of.h > @@ -161,7 +161,7 @@ static inline bool is_of_node(const struct fwnode_handle > *fwnode) is_of_node(__to_of_node_fwnode) ? \ > container_of(__to_of_node_fwnode, \ > struct device_node, fwnode) : \ > - NULL; \ > + (struct device_node *)NULL; \ > }) > > #define of_fwnode_handle(node) \ > @@ -169,7 +169,8 @@ static inline bool is_of_node(const struct fwnode_handle > *fwnode) typeof(node) __of_fwnode_handle_node = (node); \ > \ > __of_fwnode_handle_node ? \ > - &__of_fwnode_handle_node->fwnode : NULL; \ > + &__of_fwnode_handle_node->fwnode : \ > + (struct fwnode_handle *)NULL; \ > }) > > static inline bool of_have_populated_dt(void) -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 7+ messages in thread
* [RESEND PATCH 1/1] of: Make return types of to_of_node and of_fwnode_handle macros consistent 2017-11-02 9:59 ` [RESEND PATCH " Sakari Ailus 2017-11-02 17:37 ` Laurent Pinchart @ 2017-11-06 21:45 ` Rob Herring 1 sibling, 0 replies; 7+ messages in thread From: Rob Herring @ 2017-11-06 21:45 UTC (permalink / raw) To: linux-arm-kernel On Thu, Nov 02, 2017 at 11:59:18AM +0200, Sakari Ailus wrote: > (Fixed Mauro's e-mail.) > > to_of_node() macro checks whether the fwnode_handle passed to it is not an > OF node, and if so, returns NULL in order to be NULL-safe. Otherwise it > returns the pointer to the OF node which the fwnode_handle contains. > > The problem with returning NULL is that its type was void *, which > sometimes matters. Explicitly return struct device_node * instead. > > Make a similar change to of_fwnode_handle() as well. > > Fixes: d20dc1493db4 ("of: Support const and non-const use for to_of_node()") > Fixes: debd3a3b27c7 ("of: Make of_fwnode_handle() safer") > Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> > --- > Hi Mauro, > > Could you check whether this addresses the smatch issue with the xilinx > driver? > > Thanks. > > include/linux/of.h | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) Acked-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-11-06 21:45 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <c4389ab1c02bb08c1a55012fdb859c8b10bdc47e.1509569763.git.mchehab@s-opensource.com>
2017-11-01 21:05 ` [PATCH v2 14/26] media: xilinx: fix a debug printk Mauro Carvalho Chehab
2017-11-02 2:43 ` Laurent Pinchart
2017-11-02 9:20 ` Sakari Ailus
2017-11-02 9:57 ` [PATCH 1/1] of: Make return types of to_of_node and of_fwnode_handle macros consistent Sakari Ailus
2017-11-02 9:59 ` [RESEND PATCH " Sakari Ailus
2017-11-02 17:37 ` Laurent Pinchart
2017-11-06 21:45 ` Rob Herring
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox