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 v2] tty/hvc_opal: simplify if-if to if-else
Date:   Tue, 26 Apr 2022 15:10:41 +0800
Message-Id: <20220426071041.168282-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?VLRtUaZcJrGLTm8Ap0R3tOk7NUwPHzirXi/o/RrSra2sQncIF78PQd+I1Vdx?=
 =?us-ascii?Q?ibFa1wDCN5oMyLAcJOt5qP33MHZrY4zv7RggpicSCe3SI5a5VY6OzjJoWV1K?=
 =?us-ascii?Q?D/a6jh3yo9PrpIppVMshYwP8JtH9/8DN0itais4nqeksp2yKDAmipvuuSg31?=
 =?us-ascii?Q?kmgrg3Fho3vHmxq3f5rbWH7sWletNcfVGSV7c/sVSa2HJ9qzYQwpyN2tucS2?=
 =?us-ascii?Q?ool5aeGTWwRkPyaFAPeesBxESZ/ICOpxxlbz/LZzWlyDiW6nHK8QqBcGSV6A?=
 =?us-ascii?Q?YAjyy4NysyrP/ANi11H0RsMLQd5MKhDIlGjdYHnnwqRsKQPQOY1VN2z1nEqf?=
 =?us-ascii?Q?oFCOLFf11csQnxLOhSXwRRD1auWWWN31kg8/P4VpX0zX15BPwHtJaShNcn3x?=
 =?us-ascii?Q?1kiIuKWkPTxxg9HnfC7BGypOKG1mzr8LJJYVNSiA85pVQAyBcSjZTDvmTPkT?=
 =?us-ascii?Q?bGKi/MHrFnilxZmNkKtQRFWbm4cs+WyiKQGAX6QkyYJfO6ZsM33xbbDSF4ao?=
 =?us-ascii?Q?CH26bUSQH2ivLFOMJoKBcis09hYwhVvYMLqI9JRxqEP0mWDE54z8NYNg+8w9?=
 =?us-ascii?Q?7hsTwXDR1Xa51JKrxnmV4je36X7SvE6yhJtMlcJhWFRv77k43AqGYT3P8SXW?=
 =?us-ascii?Q?W1jJ8hRmkMDaW6pEuhY2fk+OKJJgKT+8HVpMzf6BhJR2GQI9L/OZt7qhGRLN?=
 =?us-ascii?Q?Bkfcl1jb/lEQdlpzKyn6I4by97ek3LncJtXqPBt1svs0g1O3JH8Z0BiNSyFH?=
 =?us-ascii?Q?otoYi+XFdwPSAKuEhhOAiSNjLdL/EdCH421+tL+FncTS+Fw0HyJWLlfiNMpN?=
 =?us-ascii?Q?Enf+1+N/hZOahcChO6xOgF6lpYH1LDtHijmbsFsH+B8UKkML49jiTcPDPd/Z?=
 =?us-ascii?Q?6dl0BdJsGwxY1d1npHH/MjB17yhmTGeQl/bCQUuNCThNnmdYVAPhf2+EIGtJ?=
 =?us-ascii?Q?GT12NExCdtb3oKkBe7AmqAA5GnyUL9tN3+chG9yqXFBACvRzd9WvlQcE9FTN?=
 =?us-ascii?Q?61S8BGjxxZAoRezUTvdy1lAZPfOQNngSSYbSEOscTaUMuJyStlf916eIuDWP?=
 =?us-ascii?Q?qHFGL9mFM8IVbNYdXZERqjHdgyBVBa3zpYzzzYJNF9rudk6231/A7JfspOIa?=
 =?us-ascii?Q?pnglhw3uXIcX31Asz2pheWIiB6HQLqxnZqv7sYII1ZFzg6WAdbjXxTyESDdd?=
 =?us-ascii?Q?Ul2rd7hmQprnrwEstxtbAlacSgCROh2FeHkjeG4o97BcpfRYqtYvNY6oxmr6?=
 =?us-ascii?Q?yCzF9fqbbgREudolbAPyE6tEm1Rs0Bj+mYs3WSHpExMefir/DRPkpDf/bxAz?=
 =?us-ascii?Q?orBWTEyZ7Ezdtde2tnpl+VNr9bS2hTes4qhTTQyvA25UyY/ULdU+mwiF60Ot?=
 =?us-ascii?Q?NezeMj3h76OTnpsfZZpY65hnvni04v8j9bpxTGtNr3LBVPvPyuRYaVTe7sTV?=
 =?us-ascii?Q?9blWNfIQe2vFvR3uFpsoGJ0pZPej38ZQPmw8XIF2stsP3QXZR8YJ7rXGk9Nq?=
 =?us-ascii?Q?clhC2JwLn1TwzisbN7O89ax1Dj9MssP8JPDVXecFjw1pWmafBZ5J2QZ69BYK?=
 =?us-ascii?Q?LPTNCPG2DPflUfkqILg9LpUKobNhFSSz1Zdt9F1r1vd1kzSuIZDyFhraNuvq?=
 =?us-ascii?Q?s+EY+PA77RovpHUt9NYrmAf1Ow7tpHt6ZcbEzQ3LopmS7paoHu7ira7gIgQp?=
 =?us-ascii?Q?j0m/xHc/p7iAfpSIJ1coWWz9ke1/7nZw6EEX98gUSnORrZThR1QaTVGMpXwW?=
 =?us-ascii?Q?M7Yp0+3f0g=3D=3D?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 84e7db3e-70d2-4021-a09b-08da2753e5cf
X-MS-Exchange-CrossTenant-AuthSource: SG2PR06MB3367.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Apr 2022 07:10:52.0048
 (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: V3UOW26yKt4xKIE0RZU1aRUbIc/u9Ffe2X5lH/EiRKmPjuqdBmyBq+MljTKJY569NjBwHIk3E8mdDToEdOi6VA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PS2PR06MB3477
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>
---
Change log:
v2:
- add braces to the if block.
---
 drivers/tty/hvc/hvc_opal.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/tty/hvc/hvc_opal.c b/drivers/tty/hvc/hvc_opal.c
index 84776bc641e6..794c7b18aa06 100644
--- a/drivers/tty/hvc/hvc_opal.c
+++ b/drivers/tty/hvc/hvc_opal.c
@@ -342,9 +342,9 @@ void __init hvc_opal_init_early(void)
 		 * path, so we hard wire it
 		 */
 		opal = of_find_node_by_path("/ibm,opal/consoles");
-		if (opal)
+		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: "
-- 
2.35.1