From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Boyd Subject: Re: [PATCH v2 1/5] soc: qcom: smem: Use le32_to_cpu for partition size comparison Date: Fri, 15 Sep 2017 11:39:07 -0700 Message-ID: <20170915183907.GB3349@codeaurora.org> References: <1505424302-29147-1-git-send-email-clew@codeaurora.org> <1505424302-29147-2-git-send-email-clew@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:44312 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751454AbdIOSjK (ORCPT ); Fri, 15 Sep 2017 14:39:10 -0400 Content-Disposition: inline In-Reply-To: <1505424302-29147-2-git-send-email-clew@codeaurora.org> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Chris Lew Cc: bjorn.andersson@linaro.org, andy.gross@linaro.org, david.brown@linaro.org, aneela@codeaurora.org, linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, linux-kernel@vger.kernel.org On 09/14, Chris Lew wrote: > Endianness can vary in the system, add le32_to_cpu when comparing > size values from smem. > > Signed-off-by: Chris Lew > --- > > Changes since v1: > - New change > > drivers/soc/qcom/smem.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/soc/qcom/smem.c b/drivers/soc/qcom/smem.c > index c28275be0038..db04c45d4132 100644 > --- a/drivers/soc/qcom/smem.c > +++ b/drivers/soc/qcom/smem.c > @@ -698,7 +698,7 @@ static int qcom_smem_enumerate_partitions(struct qcom_smem *smem, > return -EINVAL; > } > > - if (header->size != entry->size) { > + if (le32_to_cpu(header->size) != le32_to_cpu(entry->size)) { Also, it doesn't really matter. We're comparing two numbers with the same endianness, so comparing them for equality before or after swapping makes no difference. Sparse also (correctly) doesn't complain here, because adding the conversion is not necessary. Drop this patch? -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project