From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Boyd Subject: Re: [PATCH V1 4/4] spmi: pmic-arb: Remove checking opc value not less than 0 Date: Wed, 5 Jul 2017 23:53:58 -0700 Message-ID: <20170706065358.GF22780@codeaurora.org> References: <1499086338-19498-1-git-send-email-kgunda@codeaurora.org> <1499086338-19498-5-git-send-email-kgunda@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:44236 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753130AbdGFGx7 (ORCPT ); Thu, 6 Jul 2017 02:53:59 -0400 Content-Disposition: inline In-Reply-To: <1499086338-19498-5-git-send-email-kgunda@codeaurora.org> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Kiran Gunda Cc: Abhijeet Dharmapurikar , David Collins , linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, Fenglin Wu On 07/03, Kiran Gunda wrote: > From: Fenglin Wu > > The opc parameter in pmic_arb_write_cmd() function is defined with type > u8 and it's always greater than or equal to 0. Checking that it's not > less than 0 is redundant and it can cause a forbidden warning during > compilation. Remove the check. > > Signed-off-by: Fenglin Wu > --- You forgot your sign-off here. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project