Received: by 2002:a05:6a10:22f:0:0:0:0 with SMTP id 15csp3456704pxk; Mon, 7 Sep 2020 13:41:02 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxsHLgbWUZkk2Wdk0tvmQVdWl/gG3NxGEEU6qh4FHyDbPGRZCmJ9PjW14VwSM2xTmY89F8X X-Received: by 2002:a17:906:8c8:: with SMTP id o8mr22416747eje.91.1599511262117; Mon, 07 Sep 2020 13:41:02 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1599511262; cv=none; d=google.com; s=arc-20160816; b=BVGuG7z+09IbL/oAEJvBtmiP2hxmtCyYFPhJ/oVJU2L80XAxw2RlJsJsRs6AcTswNP XVgyXUq/KqeC1TNZm8wukZWUOwra0j4h1IpLQscthquOKjNwg0uxwe7Bd2tLTcvlB7lO b1bVojl42jnM7kYvjzby11aHOs1KIXrsXiuMcuSTgf1w8NLoqeviJ5OsbhYoCHD7YGRT UtytDOBm+W0mlfxPcdzDBingDiHzMZTTeA9LUwMLd+WurFd7T/sHbOfIMPL6g+Ld4Og4 EiKXcCsqf0RrjjJc3SZ/l0rb3g1NwSK2gpkeYv8odq5xojAkT5rHo2ubeJmrVrOHKPjH AgCg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :references:in-reply-to:message-id:subject:cc:to:from:date :dkim-signature; bh=KyOKmGGM9IcHrUMV+d88d0JSkDbPAnDgNwem68nl4QA=; b=tFDKXe090p/nlLeBke+IFckAj+QQf2u5qxWUECMOhEfRh4BAJLfdPUiyhsmlUF0ewi NRyr1+N1Dgt/FanCT8Qzhbo1iJHCwqmPBcoJb8J18TZSMO96nMYG6hk2vo8BmakG8vQ0 bYQGrs2SVvf7l6JAGQs+gaoPW73nmCf5TLpXtE+SoLZnovmq5yBEX3Nk70OdAS0t0Y2F 9Rw3zbaHgFTGHR4SrKX0aIUVoiMwhSZVVFAyJ7AXXOuAsjhto1vmmicZuH5FHlRjTGmK 4gYz6oLfwiXW1T22iY+w+WRS4eMwOZSO7u7wSHKHbz7n9YQ0jaN6bHX8NamRce23UKl3 6oKQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=AD98mbgs; 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 bh4si11090533ejb.629.2020.09.07.13.40.40; Mon, 07 Sep 2020 13:41:02 -0700 (PDT) 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=default header.b=AD98mbgs; 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 S1727792AbgIGUjX (ORCPT + 99 others); Mon, 7 Sep 2020 16:39:23 -0400 Received: from mail.kernel.org ([198.145.29.99]:44268 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726446AbgIGUjT (ORCPT ); Mon, 7 Sep 2020 16:39:19 -0400 Received: from kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com (unknown [163.114.132.7]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id C2DF621556; Mon, 7 Sep 2020 20:39:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1599511159; bh=MJCDlbZVjFXDweVbafndYKQCuv0J0EE/gY2hOM4Q2pQ=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=AD98mbgsUcRpyiMVAmsAWCu/HD6K385Q7YaxxowbIujMUfcKmgk0Yq78Np7z2kZ6T 8NsIjcXW6m1gP2Qt14Bk74VVbtvDw6xh7l5XGvz+8jdMuZFyoEZz3mwCUNv4aBq/lR YdxEnyvi80z/PXhRPkruPFhw9TivA1vIig6uxrtA= Date: Mon, 7 Sep 2020 13:39:17 -0700 From: Jakub Kicinski To: Radhey Shyam Pandey Cc: Zhang Changzhong , "davem@davemloft.net" , Michal Simek , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH net-next] net: xilinx: remove redundant null check before clk_disable_unprepare() Message-ID: <20200907133917.0dd18fc3@kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com> In-Reply-To: References: <1599483723-43704-1-git-send-email-zhangchangzhong@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 7 Sep 2020 13:14:05 +0000 Radhey Shyam Pandey wrote: > > From: Zhang Changzhong > > Sent: Monday, September 7, 2020 6:32 PM > > To: Radhey Shyam Pandey ; davem@davemloft.net; > > kuba@kernel.org; Michal Simek > > Cc: netdev@vger.kernel.org; linux-kernel@vger.kernel.org > > Subject: [PATCH net-next] net: xilinx: remove redundant null check before > > clk_disable_unprepare() > > > > Because clk_prepare_enable() and clk_disable_unprepare() already checked > > NULL clock parameter, so the additional checks are unnecessary, just > > remove them. > > > > Reported-by: Hulk Robot > > Signed-off-by: Zhang Changzhong > > Reviewed-by: Radhey Shyam Pandey Applied, thanks!