Messages in this thread | | | Subject | Re: [PATCH v1] slimbus: core: Fix mismatch in of_node_get/put | From | Srinivas Kandagatla <> | Date | Fri, 17 Apr 2020 10:35:41 +0100 |
| |
On 26/03/2020 17:34, Saravana Kannan wrote: > Also, remove some unnecessary NULL checks. The functions in question > already do NULL checks. > > Signed-off-by: Saravana Kannan <saravanak@google.com>
Applied Thanks, --srini
> --- > drivers/slimbus/core.c | 8 +++----- > 1 file changed, 3 insertions(+), 5 deletions(-) > > diff --git a/drivers/slimbus/core.c b/drivers/slimbus/core.c > index 44228a5b246d..ae1e248a8fb8 100644 > --- a/drivers/slimbus/core.c > +++ b/drivers/slimbus/core.c > @@ -162,11 +162,8 @@ static int slim_add_device(struct slim_controller *ctrl, > sbdev->ctrl = ctrl; > INIT_LIST_HEAD(&sbdev->stream_list); > spin_lock_init(&sbdev->stream_list_lock); > - > - if (node) { > - sbdev->dev.of_node = of_node_get(node); > - sbdev->dev.fwnode = of_fwnode_handle(node); > - } > + sbdev->dev.of_node = of_node_get(node); > + sbdev->dev.fwnode = of_fwnode_handle(node); > > dev_set_name(&sbdev->dev, "%x:%x:%x:%x", > sbdev->e_addr.manf_id, > @@ -285,6 +282,7 @@ EXPORT_SYMBOL_GPL(slim_register_controller); > /* slim_remove_device: Remove the effect of slim_add_device() */ > static void slim_remove_device(struct slim_device *sbdev) > { > + of_node_put(sbdev->dev.of_node); > device_unregister(&sbdev->dev); > } > >
| |