From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754181AbaH2R0V (ORCPT ); Fri, 29 Aug 2014 13:26:21 -0400 Received: from cam-admin0.cambridge.arm.com ([217.140.96.50]:56824 "EHLO cam-admin0.cambridge.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753693AbaH2R0T (ORCPT ); Fri, 29 Aug 2014 13:26:19 -0400 Date: Fri, 29 Aug 2014 18:26:02 +0100 From: Mark Rutland To: Nicolin Chen Cc: Mark Brown , Xiubo Li , "alsa-devel@alsa-project.org" , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" Subject: Re: [PATCHv2] ASoC: fsl-sai: using 'lsb-first' property instead of 'big-endian-data'. Message-ID: <20140829172601.GC18555@leverpostej> References: <20140829110601.GB21473@leverpostej> <20140829115943.GV29327@sirena.org.uk> <20140829164010.GA5520@Asurada> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140829164010.GA5520@Asurada> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Aug 29, 2014 at 05:40:11PM +0100, Nicolin Chen wrote: > On Fri, Aug 29, 2014 at 12:59:43PM +0100, Mark Brown wrote: > > On Fri, Aug 29, 2014 at 12:06:01PM +0100, Mark Rutland wrote: > > > On Fri, Aug 29, 2014 at 08:12:12AM +0100, Xiubo Li wrote: > > > > > > The 'big-endian-data' property is originally used to indicate whether the > > > > LSB firstly or MSB firstly will be transmitted to the CODEC or received > > > > from the CODEC, and there has nothing relation to the memory data. > > > > > Both Nicolin [1] and Mark [2] asked for the old property to be kept > > > around for compatibility reasons. Those requests seem to have been > > > ignored entirely; the commit message doesn't even describe why you > > > believe removing support for the old property is safe. > > > > The response was that the relevant chips aren't out in the wild so it's > > vanishingly unlikely that anyone would have a real DT with this option > > enabled. I'll defer to Nicolin on if that's the case or not. > > Yea, it sounds like so. I thought Vf610 also used this property. But if > only LS1 using it, I think we may consent this rename right? If you two are happy, then fine by me. My only concern was that the rationale for this being safe wasn't in the commit message. Mark.