From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932273AbcA0M0A (ORCPT ); Wed, 27 Jan 2016 07:26:00 -0500 Received: from mga03.intel.com ([134.134.136.65]:54174 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932086AbcA0MZx (ORCPT ); Wed, 27 Jan 2016 07:25:53 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,354,1449561600"; d="scan'208";a="890193380" Message-ID: <1453897583.2521.246.camel@linux.intel.com> Subject: Re: [PATCH 07/15] dmaengine: dw: revisit data_width property From: Andy Shevchenko To: Rob Herring , Mans Rullgard Cc: Viresh Kumar , Vinod Koul , linux-kernel@vger.kernel.org, dmaengine@vger.kernel.org, Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Vineet Gupta , Russell King , Dan Williams , devicetree@vger.kernel.org, linux-snps-arc@lists.infradead.org, linux-arm-kernel@lists.infradead.org Date: Wed, 27 Jan 2016 14:26:23 +0200 In-Reply-To: <20160126210726.GA4096@rob-hp-laptop> References: <1453663322-14474-1-git-send-email-mans@mansr.com> <1453663322-14474-8-git-send-email-mans@mansr.com> <20160126210726.GA4096@rob-hp-laptop> Organization: Intel Finland Oy Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.18.3-1 Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 2016-01-26 at 15:07 -0600, Rob Herring wrote: > On Sun, Jan 24, 2016 at 07:21:54PM +0000, Mans Rullgard wrote: > > This patch changes the DT binding, so it should probably be amended > > for > > compatibility with old device trees.  I've included it as is since > > I think > > the change as such is good. > > Just because you update the dts files, it doesn't make the change > okay.  > I'm fine with the DT change, but the driver would have to support > both  > old and new property names. We will fix this. > Doesn't really seem worth doing to me. The big issue with DT, you know, is hanging around names. This moves name to be de facto standard for similar in the other drivers. -- Andy Shevchenko Intel Finland Oy From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Subject: Re: [PATCH 07/15] dmaengine: dw: revisit data_width property Date: Wed, 27 Jan 2016 14:26:23 +0200 Message-ID: <1453897583.2521.246.camel@linux.intel.com> References: <1453663322-14474-1-git-send-email-mans@mansr.com> <1453663322-14474-8-git-send-email-mans@mansr.com> <20160126210726.GA4096@rob-hp-laptop> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <20160126210726.GA4096@rob-hp-laptop> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Rob Herring , Mans Rullgard Cc: Viresh Kumar , Vinod Koul , linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, dmaengine-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Vineet Gupta , Russell King , Dan Williams , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-snps-arc-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: devicetree@vger.kernel.org On Tue, 2016-01-26 at 15:07 -0600, Rob Herring wrote: > On Sun, Jan 24, 2016 at 07:21:54PM +0000, Mans Rullgard wrote: > > This patch changes the DT binding, so it should probably be amended > > for > > compatibility with old device trees.=C2=A0=C2=A0I've included it as= is since > > I think > > the change as such is good. >=20 > Just because you update the dts files, it doesn't make the change > okay.=C2=A0 > I'm fine with the DT change, but the driver would have to support > both=C2=A0 > old and new property names. We will fix this. > Doesn't really seem worth doing to me. The big issue with DT, you know, is hanging around names. This moves name to be de facto standard for similar in the other drivers. --=20 Andy Shevchenko Intel Finland Oy -- To unsubscribe from this list: send the line "unsubscribe devicetree" i= n the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: andriy.shevchenko@linux.intel.com (Andy Shevchenko) Date: Wed, 27 Jan 2016 14:26:23 +0200 Subject: [PATCH 07/15] dmaengine: dw: revisit data_width property In-Reply-To: <20160126210726.GA4096@rob-hp-laptop> References: <1453663322-14474-1-git-send-email-mans@mansr.com> <1453663322-14474-8-git-send-email-mans@mansr.com> <20160126210726.GA4096@rob-hp-laptop> List-ID: Message-ID: <1453897583.2521.246.camel@linux.intel.com> To: linux-snps-arc@lists.infradead.org On Tue, 2016-01-26@15:07 -0600, Rob Herring wrote: > On Sun, Jan 24, 2016@07:21:54PM +0000, Mans Rullgard wrote: > > This patch changes the DT binding, so it should probably be amended > > for > > compatibility with old device trees.??I've included it as is since > > I think > > the change as such is good. > > Just because you update the dts files, it doesn't make the change > okay.? > I'm fine with the DT change, but the driver would have to support > both? > old and new property names. We will fix this. > Doesn't really seem worth doing to me. The big issue with DT, you know, is hanging around names. This moves name to be de facto standard for similar in the other drivers. -- Andy Shevchenko Intel Finland Oy From mboxrd@z Thu Jan 1 00:00:00 1970 From: andriy.shevchenko@linux.intel.com (Andy Shevchenko) Date: Wed, 27 Jan 2016 14:26:23 +0200 Subject: [PATCH 07/15] dmaengine: dw: revisit data_width property In-Reply-To: <20160126210726.GA4096@rob-hp-laptop> References: <1453663322-14474-1-git-send-email-mans@mansr.com> <1453663322-14474-8-git-send-email-mans@mansr.com> <20160126210726.GA4096@rob-hp-laptop> Message-ID: <1453897583.2521.246.camel@linux.intel.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, 2016-01-26 at 15:07 -0600, Rob Herring wrote: > On Sun, Jan 24, 2016 at 07:21:54PM +0000, Mans Rullgard wrote: > > This patch changes the DT binding, so it should probably be amended > > for > > compatibility with old device trees.??I've included it as is since > > I think > > the change as such is good. > > Just because you update the dts files, it doesn't make the change > okay.? > I'm fine with the DT change, but the driver would have to support > both? > old and new property names. We will fix this. > Doesn't really seem worth doing to me. The big issue with DT, you know, is hanging around names. This moves name to be de facto standard for similar in the other drivers. -- Andy Shevchenko Intel Finland Oy