From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH v4 2/7] media: s5p-mfc: use generic reserved memory bindings Date: Wed, 25 May 2016 12:36:14 -0500 Message-ID: <20160525173614.GA8309@rob-hp-laptop> References: <1464096690-23605-1-git-send-email-m.szyprowski@samsung.com> <1464096690-23605-3-git-send-email-m.szyprowski@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-media-owner@vger.kernel.org To: Javier Martinez Canillas Cc: Marek Szyprowski , linux-media@vger.kernel.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, Sylwester Nawrocki , Kamil Debski , Kukjin Kim , Krzysztof Kozlowski , Uli Middelberg , Bartlomiej Zolnierkiewicz List-Id: devicetree@vger.kernel.org On Wed, May 25, 2016 at 11:18:59AM -0400, Javier Martinez Canillas wrote: > Hello Marek, > > On 05/24/2016 09:31 AM, Marek Szyprowski wrote: > > Use generic reserved memory bindings and mark old, custom properties > > as obsoleted. > > > > Signed-off-by: Marek Szyprowski > > --- > > .../devicetree/bindings/media/s5p-mfc.txt | 39 +++++++++++++++++----- > > 1 file changed, 31 insertions(+), 8 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/media/s5p-mfc.txt b/Documentation/devicetree/bindings/media/s5p-mfc.txt > > index 2d5787e..92c94f5 100644 > > --- a/Documentation/devicetree/bindings/media/s5p-mfc.txt > > +++ b/Documentation/devicetree/bindings/media/s5p-mfc.txt > > @@ -21,15 +21,18 @@ Required properties: > > - clock-names : from common clock binding: must contain "mfc", > > corresponding to entry in the clocks property. > > > > - - samsung,mfc-r : Base address of the first memory bank used by MFC > > - for DMA contiguous memory allocation and its size. > > - > > - - samsung,mfc-l : Base address of the second memory bank used by MFC > > - for DMA contiguous memory allocation and its size. > > - > > Optional properties: > > - power-domains : power-domain property defined with a phandle > > to respective power domain. > > + - memory-region : from reserved memory binding: phandles to two reserved > > + memory regions, first is for "left" mfc memory bus interfaces, > > + second if for the "right" mfc memory bus, used when no SYSMMU > > + support is available > > + > > +Obsolete properties: > > + - samsung,mfc-r, samsung,mfc-l : support removed, please use memory-region > > + property instead > > + > > > > I wonder if we should maintain backward compatibility for this driver > since s5p-mfc memory allocation won't work with an old FDT if support > for the old properties are removed. Well, minimally the commit log should indicate that compatibility is being broken. Rob