Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Wan Jiabing <wanjiabing@vivo.com>
To:     Michael Ellerman <mpe@ellerman.id.au>,
        Benjamin Herrenschmidt <benh@kernel.crashing.org>,
        Paul Mackerras <paulus@samba.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Jiri Slaby <jirislaby@kernel.org>,
        linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org
Cc:     kael_w@yeah.net, Wan Jiabing <wanjiabing@vivo.com>
Subject: [PATCH] tty/hvc_opal: simplify if-if to if-else
Date:   Sun, 24 Apr 2022 17:25:11 +0800
Message-Id: <20220424092511.100309-1-wanjiabing@vivo.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?LJURF6oGKcGyNIBd/QwA7BbNUj4y7PrRrfFgKGg1rERj1bIm1VKjPbx6g5Jx?=
 =?us-ascii?Q?xqvSf/9Pjeuml8nuOh8f2U4EbgkbJrby0SiivcvIj8z6HwLnfXSZQH6k4NM4?=
 =?us-ascii?Q?RLDUvXvwE0qFOjyiNJ6rhr4WxK2VZNmCJXHdFCXtfKEG8UtDb+onV4b0oTur?=
 =?us-ascii?Q?idRngf5A0M6bQqnxptnpYB2XK8bQ5Vu31LawUcK2vp0q++6VfK0vv2bCscAG?=
 =?us-ascii?Q?s7wCuvhMHoKzqkWtpKNhra809+3CbKrwgtYZRkNexCMI+9Ikb5pCxOcIb2Rs?=
 =?us-ascii?Q?cZphvVou1frj5hiPbSpR53I4wBXH0ypZpfdXolrwBPx433SbqdJGZBFj21iA?=
 =?us-ascii?Q?Rv2XmqE0OTIfvM5DeaPJ6rX8JkHvyaRMUllpbZsdRZW/avBCJF1vQXlS3h1C?=
 =?us-ascii?Q?WxR3waU51pOj+0xaxieuV2tRb3P6EneJlhDTbYXeKpELCwlgwu9UkvnFC4fr?=
 =?us-ascii?Q?J5cSDoZZCD7eohss2kStnGUkQl8rPBcMkQyVLDUtL3XTWQdm/OP9JdvyNCxo?=
 =?us-ascii?Q?o1P6Dt/Xwax/GdH4tgGq/m67MxRCYYPdeiCsqC+8h4I7ADTAPfU0MHr7BZtC?=
 =?us-ascii?Q?QEMlFJRAIyNsURLsx6WEcJyEXI6X18cVvYrxRUYZmt1SHjbWYbeA2Bz6C763?=
 =?us-ascii?Q?cTZC9ZV2zt0cI33W4p3XmNn3ZlM3pBT6A+T+qdsD0UtDB3x+hhzCZ0lkKVKH?=
 =?us-ascii?Q?UOaH5vMF3g27YrryPGtxtCWgPpWZ/vSvboR1U35MY0ZRFu77P7AHBYOcfas5?=
 =?us-ascii?Q?+AqMBbbGLpcyMpYEk62IGzk96bfSBaij8OycycBqcEe6RZII2aIXadIp7YTV?=
 =?us-ascii?Q?m3mB9hRbjlaDZl350wRBIUn0Mj2n3uStYGD37RSJW7EA5HyFGO/wm08O5U5f?=
 =?us-ascii?Q?MF0XNOU0F8H00CnpG5GgQgB5lqpdnwn9TCJm3iRTGv8aomXDkihy84/GDLeY?=
 =?us-ascii?Q?jjPF5iwzcmrOARf8xTyQes3lzPHSYs1iXhTWd5ajbcN7uyasLTGIRJdJylX5?=
 =?us-ascii?Q?9Ad8j0lr2OvkX1+kK1t9LELUN0oQrSuLJVvXa5C2Yggr0y8Oi7lNKKnd+5Ut?=
 =?us-ascii?Q?7BwBIo6Osnhw4SH6C9eaZSfjFTi5Vt66AC/yWHJXpIA9C+MyKSLFiYM+p+iK?=
 =?us-ascii?Q?p6KuVamIRBJrOyW5rkSes/TjfnR0/VNfj7cTAh/2snIa0zowntvvNOIGpU0Z?=
 =?us-ascii?Q?3f1t+dmNlKt8abn3ubE+bKyQ9sklHZ3rqe5LQTLvTYgwHmDD4rDqdecUIt9I?=
 =?us-ascii?Q?FRsxYb9b9cVBHh5D9aikQVDtycaTkb+mIyqbKbpb8TaCKIOui6XUfqz8wqpy?=
 =?us-ascii?Q?DslqdPpj4RJ8eQk7gH/EC6vsz3iQPq37kODGg05e7ICMo8UM1EenMnlM1WZ2?=
 =?us-ascii?Q?vw5v9Pfes83Wqm17klOQlHvHDysEYlHoz74j1YdUUfNQMTL9VzmjTpQyq1PF?=
 =?us-ascii?Q?4dNSEjDL+PQdQsS1OWuqZHTZGMJS+cf4xMYa/iiNptC5cSknF4SrR8s5YtLp?=
 =?us-ascii?Q?99k0HpikglCu8+xW++uu9gtjQwCY6cY0LDD59JOXqv06VSlSM1AmZb0rPzmz?=
 =?us-ascii?Q?RLLV/noYV5Oi9NykmNew1yr7AHvcse0ieo73RcovI8Cb8z2QKwkqWwXkrbox?=
 =?us-ascii?Q?lXqRs4GonQqJVisaElGcHhJutRWARqFwAiLE9dGxXMoKlED0qSTSsBWp4t16?=
 =?us-ascii?Q?ryFssWNvdVyz0vYQhPSDm5FggdnQ4qwQfQSrteAiFxOiDLReGFsSg00FA4wi?=
 =?us-ascii?Q?YNtfP6/Dlw=3D=3D?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ab21901b-caaa-48d7-babe-08da25d45be6
X-MS-Exchange-CrossTenant-AuthSource: SG2PR06MB3367.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Apr 2022 09:25:23.3832
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: Ci+v+N9GnV293YxITS1+bO6ZsQpsjmgNQtCG0CMFpDJ7XJBEM+y6Rlwuwfe0ZyqfNtTxpTUgVSmOCOz2E4Pzyw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SL2PR06MB3241
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Use if and else instead of if(A) and if (!A).

Signed-off-by: Wan Jiabing <wanjiabing@vivo.com>
---
 drivers/tty/hvc/hvc_opal.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/drivers/tty/hvc/hvc_opal.c b/drivers/tty/hvc/hvc_opal.c
index 84776bc641e6..2dafa0964c2a 100644
--- a/drivers/tty/hvc/hvc_opal.c
+++ b/drivers/tty/hvc/hvc_opal.c
@@ -344,14 +344,15 @@ void __init hvc_opal_init_early(void)
 		opal = of_find_node_by_path("/ibm,opal/consoles");
 		if (opal)
 			pr_devel("hvc_opal: Found consoles in new location\n");
-		if (!opal) {
+		else {
 			opal = of_find_node_by_path("/ibm,opal");
 			if (opal)
 				pr_devel("hvc_opal: "
 					 "Found consoles in old location\n");
+			else
+				return;
 		}
-		if (!opal)
-			return;
+
 		for_each_child_of_node(opal, np) {
 			if (of_node_name_eq(np, "serial")) {
 				stdout_node = np;
-- 
2.35.1