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); }