From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B713AC433F5 for ; Fri, 6 May 2022 22:29:32 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1444746AbiEFWdO (ORCPT ); Fri, 6 May 2022 18:33:14 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40880 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229912AbiEFWdL (ORCPT ); Fri, 6 May 2022 18:33:11 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 52F0F5E153; Fri, 6 May 2022 15:29:26 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 1A187B839E6; Fri, 6 May 2022 22:29:25 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 3A067C385A8; Fri, 6 May 2022 22:29:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1651876163; bh=y0FngmsSSvJom30mMBhLoZvRrNbqeZl4qnTn2t+QXjI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=vLWpiL6W1i7NALe3Gr1u0yqXziOB2D1Y79FbmrcBm6ez0KEs8X2L2Xg6dGtG+UuM5 kQG4w38vN1g4pYSLCWI7rgZc283xkPUrFja+fD+v1YFscqyKdq/cvgsAaYPsLei/nf BWuoKQiXm34Gs1xfReP3QoxC2ZhjrdGHkUnI0KC6vSOiHli5y/nlmeIFkBgbpu+KnO TIjLVVvdcJx2f/OvbcDVoZ2EIg3g9++OfPd7eqsiE0SaqFx4UKA2aLoPcP9ieWe7yu OQc+PnxI8onIBUc3Az541pmd3zndBZ+rwn+NVpaLrPL5ThN8OO3HdoYAnkhhB5nbea kwbVGmHVd0Zmg== Date: Sat, 7 May 2022 01:30:48 +0300 From: Jarkko Sakkinen To: Dave Hansen Cc: cgel.zte@gmail.com, dave.hansen@linux.intel.com, tglx@linutronix.de, linux-sgx@vger.kernel.org, linux-kernel@vger.kernel.org, Minghao Chi , Zeal Robot Subject: Re: [PATCH] x86/sgx: simplify the return expression of sgx_drv_init() Message-ID: References: <20220505021659.54210-1-chi.minghao@zte.com.cn> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, May 05, 2022 at 10:31:41AM -0700, Dave Hansen wrote: > On 5/4/22 19:16, cgel.zte@gmail.com wrote: > > From: Minghao Chi > > > > Simplify the return expression. > > > > Reported-by: Zeal Robot > > Signed-off-by: Minghao Chi > > Folks, I'd really encourage you to spend your time elsewhere. These > "cleanup" or "simplify" patches as a whole have high rates of bugs. I > don't trust them. Plus, they don't really make the code easier to > understand. I agree. I get this kind of clean up in the context of doing something to the functionality (in the same path) but it does not live by its own. Plus, these type of patches add to the effort backporting fixes. BR, Jarkko