From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Boyd Subject: Re: [PATCH] mmc: sdhci-msm: Make tuning block table endian agnostic Date: Wed, 3 Sep 2014 06:37:42 -0700 Message-ID: <20140903133742.GB28256@codeaurora.org> References: <1409702308-3023-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.11.231]:43737 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932411AbaICNho (ORCPT ); Wed, 3 Sep 2014 09:37:44 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Ulf Hansson Cc: Chris Ball , "linux-kernel@vger.kernel.org" , "linux-arm-msm@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Asutosh Das , Venkat Gopalakrishnan , Georgi Djakov On 09/03, Ulf Hansson wrote: > On 3 September 2014 01:58, Stephen Boyd wrote: > > If we're tuning on a big-endian CPU we'll never determine we properly > > tuned the device because we compare the data we received from the > > controller with a table that assumes the CPU is little-endian. > > Change the table to be an array of bytes instead of 32-bit words > > so we can use memcmp() without needing to byte-swap every word > > depending on the endianess of the CPU. > > > > Cc: Asutosh Das > > Cc: Venkat Gopalakrishnan > > Cc: Georgi Djakov > > Fixes: 415b5a75da43 "mmc: sdhci-msm: Add platform_execute_tuning implementation" > > Signed-off-by: Stephen Boyd > > Hi Stephen, > > If you want me to pick this up, please repost to linux-mmc as well. > Otherwise you have my ack for it! > Ok I'll repost it now. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation