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 9F6E6C7EE22 for ; Wed, 17 May 2023 20:11:27 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229533AbjEQUL0 (ORCPT ); Wed, 17 May 2023 16:11:26 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45430 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229445AbjEQULY (ORCPT ); Wed, 17 May 2023 16:11:24 -0400 Received: from pandora.armlinux.org.uk (pandora.armlinux.org.uk [IPv6:2001:4d48:ad52:32c8:5054:ff:fe00:142]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E8010E7C; Wed, 17 May 2023 13:11:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=Ncx1qH8n9sOF9FCZslMLbmv0w/78dVsCubBVxxWQ9gE=; b=nSy+KBG+i376ssgh15Q3xm7H/w 3XNzbdDbyv1xjXNJE9FqV1X823Zjg4sm5tlDoULoBWhxToyEPKex3bV+xciamtVV9ORTyzikMYpps /3yzuoNysIx1dzYZqZ7lBAcWpq7FAj2O1McNtBR59TKUEC6UutvH3haEp5gBXb0AhqPE3crcl8lws SCMQe5UbPEKFvYxHgtZWonSPhO4WFmhwCcZxwjaOrMomeKjkfQYH5kuaY/0kuMnw95LGkpwdQY/Cy IqJICTQ7OrF+7Lx9+UjSL4Rjz93GF4KrVp/c5nklaEu8l5bPjgBFG1YTWd58IVJDDBplB2oBy3yqN vJ0QN9xg==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:53498) by pandora.armlinux.org.uk with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1pzNUP-0000B4-8p; Wed, 17 May 2023 21:11:13 +0100 Received: from linux by shell.armlinux.org.uk with local (Exim 4.94.2) (envelope-from ) id 1pzNUM-0002Lp-1y; Wed, 17 May 2023 21:11:10 +0100 Date: Wed, 17 May 2023 21:11:10 +0100 From: "Russell King (Oracle)" To: Yeqi Fu Cc: mw@semihalf.com, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Ivan Orlov Subject: Re: [PATCH] net: mvpp2: Fix error checking Message-ID: References: <20230517190811.367461-1-asuk4.q@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230517190811.367461-1-asuk4.q@gmail.com> Sender: Russell King (Oracle) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, May 18, 2023 at 03:08:11AM +0800, Yeqi Fu wrote: > The function debugfs_create_dir returns ERR_PTR if an error occurs, > and the appropriate way to verify for errors is to use the inline > function IS_ERR. The patch will substitute the null-comparison with > IS_ERR. Exactly as I said to a very similar patch received a few days ago from SikkiLadho: "The modern wisdom for debugfs is not to check for any errors, so if we're going to touch this, that's the route that any patch should be taking. Thanks." Your patch seems to have the same Suggested-by: which suggests to me that you probably know SikkiLadho and are working together with the person who suggested the change, so it would be good that when a patch from one of you is commented upon, those comments are taken into account rather than someone else sending an identical patch to the first. Thanks. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!