From d02337d29d33da6f1f83bebd7d3997e78df1df9e Mon Sep 17 00:00:00 2001 From: Greg Kroah-Hartman Date: Wed, 5 Dec 2018 10:53:42 +0100 Subject: [PATCH] Revert "serial/sunsu: add missing of_node_put()" This reverts commit 20d8e8611eb0596047fd4389be7a7203a883b9bf. As David Miller points out, it's wrong. Reported-by: David Miller Cc: Yangtao Li Signed-off-by: Greg Kroah-Hartman --- drivers/tty/serial/sunsu.c | 20 +++++--------------- 1 file changed, 5 insertions(+), 15 deletions(-) diff --git a/drivers/tty/serial/sunsu.c b/drivers/tty/serial/sunsu.c index 4a27c0114d50..6cf3e9b0728f 100644 --- a/drivers/tty/serial/sunsu.c +++ b/drivers/tty/serial/sunsu.c @@ -1394,32 +1394,22 @@ static inline struct console *SUNSU_CONSOLE(void) static enum su_type su_get_type(struct device_node *dp) { struct device_node *ap = of_find_node_by_path("/aliases"); - enum su_type rc = SU_PORT_PORT; if (ap) { - struct device_node *tmp; const char *keyb = of_get_property(ap, "keyboard", NULL); const char *ms = of_get_property(ap, "mouse", NULL); if (keyb) { - tmp = of_find_node_by_path(keyb); - if (tmp && dp == tmp){ - rc = SU_PORT_KBD; - goto out; - } + if (dp == of_find_node_by_path(keyb)) + return SU_PORT_KBD; } if (ms) { - tmp = of_find_node_by_path(ms); - if (tmp && dp == tmp){ - rc = SU_PORT_MS; - goto out; - } + if (dp == of_find_node_by_path(ms)) + return SU_PORT_MS; } } -out: - of_node_put(ap); - return rc; + return SU_PORT_PORT; } static int su_probe(struct platform_device *op) -- GitLab