Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp1142001pxu; Wed, 2 Dec 2020 12:07:08 -0800 (PST) X-Google-Smtp-Source: ABdhPJyViXEo70PPOUZBbEu6z1GzM4C0EUs/wYuxPojvAmZZ6J7gVH7AJV51jf8laKywqaPUKwhz X-Received: by 2002:a17:906:3187:: with SMTP id 7mr1399805ejy.225.1606939628243; Wed, 02 Dec 2020 12:07:08 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1606939628; cv=none; d=google.com; s=arc-20160816; b=NXw0YSACG9U2aNCyqbbF/WUD58OyinStk4Wn1rmHY5ziIEfPVTq7SW9Xm+K37jseX0 biU2pM3iwWuNAr7laZqlgYlivGjhOnODpQCsH7b3W2y0xOgVkPbzy69IQKImgmIRNFGH 9SLpEmHitWD7vId9wiwvSgUGOyVgq5Trn1MfG8r0aux81WD3CoAoS069/Vx5M8Q0JZ8f OaYdnTJQXAQLcbKtQ8acpKV8Uzl8uBD6dYaN5KtbNc3gFUGDqNAu09nxRfKKYtPIfQq+ T5QKxTyL5Z3l9IXTUnoO0HZ3Nff+4znNHXspkdc1BjuMguC9JikOVsBSAK0MLYLkDAbS edsg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:subject:cc:to:from:dkim-signature :date; bh=OH0QFSrO8XjsLvkQ3bqMXEdgKu0LROGsEpNLOkTU/iA=; b=coZkvP+40QuDPVP2cYdRISCBcdSd/R7KTC3DUpaYTdHegMB69+keqzZw9r5v5Q3mDn A0zZbyL+KPVMTzUj4wr2dBZbhK6asveS+BEfEPOepNGcXunDYmBXopM5jfADd5Pxmxjv W2v20CkLL6nFuXmn9MP4YwukneIsxYv5IV6ls6WcTT/n/RNfa2iaJFLr/RiAy7pYHEqB 7t0ksQ51jnf7anst2E11/414tcdzPqDaTxstE14akh+8tuazJRDreFvC/LtMTTam3zrk w4q3LFPuE2O0IKT9QrJncdMwNVlUiietzoCQCDMpLAcER0CS/qhw65/l4G+CS9el5xG1 dEWg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=VEmjDoMl; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id e18si651289ejq.261.2020.12.02.12.06.45; Wed, 02 Dec 2020 12:07:08 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=VEmjDoMl; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731194AbgLBUCL (ORCPT + 99 others); Wed, 2 Dec 2020 15:02:11 -0500 Received: from mail.kernel.org ([198.145.29.99]:58300 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729165AbgLBUCL (ORCPT ); Wed, 2 Dec 2020 15:02:11 -0500 Date: Wed, 2 Dec 2020 12:01:27 -0800 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1606939290; bh=CR+EL2ZIg5zPuyiVUwgwAYmi7H5wK9yp7oqtDbYE/Ec=; h=From:To:Cc:Subject:In-Reply-To:References:From; b=VEmjDoMlah11tBUxdG2tdsnTW/vE4/1L6aQhlJWUs5p8rplWcR6Th09MESX/fRokT inv2VRfXp6qeubM9mvoY2CZ8K3nXr9dBTnhtEamR5EqirUqrJ0brUgYN62gpbXiRQn BAdn7V3cH6seTQOaVbDWUomINbZ1gnh/fqNc0YEbJlfiWsCQZP/q+qaYssu2PbuyrF wL5z2Rvk4bci1BIGKUDpJBUVSsHFDoQ3gbHPz57NQD66Udy25v4MpjnbOxeC9ig+wV wTUYjbQ5bqgfHznWsAip+kqtnwU8cSi6P0p+Q4LR91elwLHFBlHEdUSqhvix1cVPoO laVSQxvmAdKbQ== From: Jakub Kicinski To: Sunil Kovvuri Cc: Dan Carpenter , Sunil Goutham , Linu Cherian , Geetha sowjanya , Jerin Jacob , "David S. Miller" , Linux Netdev List , LKML , kernel-janitors@vger.kernel.org Subject: Re: [PATCH net-next] octeontx2-af: debugfs: delete dead code Message-ID: <20201202120127.7cc7453f@kicinski-fedora-pc1c0hjn.DHCP.thefacebook.com> In-Reply-To: References: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2 Dec 2020 12:33:00 +0530 Sunil Kovvuri wrote: > On Wed, Dec 2, 2020 at 12:28 PM Dan Carpenter wrote: > > These debugfs never return NULL so all this code will never be run. > > > > In the normal case, (and in this case particularly), the debugfs > > functions are not supposed to be checked for errors so all this error > > checking code can be safely deleted. > > > > Signed-off-by: Dan Carpenter > > Thanks for the changes. Applied, thanks!