From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751344AbdJED7I (ORCPT ); Wed, 4 Oct 2017 23:59:08 -0400 Received: from mail-pf0-f170.google.com ([209.85.192.170]:53214 "EHLO mail-pf0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751260AbdJED7F (ORCPT ); Wed, 4 Oct 2017 23:59:05 -0400 X-Google-Smtp-Source: AOwi7QBqfozII8RukP58Saf/0VATyd7DxGB+qFMdDlE+yVWxZW28DJZxS8dOkntL2axan1OVvXlSmQ== Date: Wed, 4 Oct 2017 20:59:02 -0700 From: Bjorn Andersson To: Chris Lew Cc: Stephen Boyd , 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 Subject: Re: [PATCH v2 1/5] soc: qcom: smem: Use le32_to_cpu for partition size comparison Message-ID: <20171005035902.GW1165@minitux> References: <1505424302-29147-1-git-send-email-clew@codeaurora.org> <1505424302-29147-2-git-send-email-clew@codeaurora.org> <20170915183907.GB3349@codeaurora.org> <204750f6-c1eb-e971-63f1-577c3880878f@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <204750f6-c1eb-e971-63f1-577c3880878f@codeaurora.org> User-Agent: Mutt/1.8.3 (2017-05-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri 15 Sep 13:53 PDT 2017, Chris Lew wrote: > > > On 9/15/2017 11:39 AM, Stephen Boyd wrote: > > 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? > > > > Hey Bjorn, should we remove this patch? You had flagged this comparison in > the first version of the global partition changes. > As Stephen says; it's (technically) unnecessary. But I think we should take this patch anyways, for the sake of saving future readers of having to think about why there's one case in the entire file where we don't do this translation. Regards, Bjorn