From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Gross Subject: Re: [PATCH] regulator: qcom_spmi: Add missing braces for aligned code Date: Tue, 16 Jun 2015 15:31:28 -0500 Message-ID: <20150616203128.GA8507@qualcomm.com> References: <1434478282-13552-1-git-send-email-sboyd@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:52864 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756579AbbFPUbb (ORCPT ); Tue, 16 Jun 2015 16:31:31 -0400 Content-Disposition: inline In-Reply-To: <1434478282-13552-1-git-send-email-sboyd@codeaurora.org> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Stephen Boyd Cc: Mark Brown , linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Julia Lawall On Tue, Jun 16, 2015 at 11:11:22AM -0700, Stephen Boyd wrote: > drivers/regulator/qcom_spmi-regulator.c:751:3-50: code aligned > with following code on line 753 > drivers/regulator/qcom_spmi-regulator.c:584:3-41: code aligned > with following code on line 587 > > These lines where missing braces causing the break to always > be executed even when it shouldn't be. Fix it. > > Reported-by: kbuild test robot > Cc: Julia Lawall > Signed-off-by: Stephen Boyd Reviewed-by: Andy Gross -- Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project