From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Date: Tue, 26 Aug 2014 09:27:43 +0000 Subject: Re: [PATCH v2 6/6] devicetree: bindings: Document Renesas JPEG Processing Unit. Message-Id: List-Id: References: <1408452653-14067-7-git-send-email-mikhail.ulyanov@cogentembedded.com> <1408970132-6690-1-git-send-email-mikhail.ulyanov@cogentembedded.com> <20140825235720.GB7217@verge.net.au> <20140826090126.GC29667@verge.net.au> In-Reply-To: <20140826090126.GC29667@verge.net.au> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Simon Horman Cc: Mikhail Ulyanov , Magnus Damm , Mauro Carvalho Chehab , Rob Herring , Grant Likely , Mark Rutland , Ian Campbell , Laurent Pinchart , Hans Verkuil , Linux-sh list , Linux Media Mailing List , "devicetree@vger.kernel.org" On Tue, Aug 26, 2014 at 11:01 AM, Simon Horman wrote: > On Tue, Aug 26, 2014 at 10:03:34AM +0200, Geert Uytterhoeven wrote: >> On Tue, Aug 26, 2014 at 1:57 AM, Simon Horman wrote: >> > On Mon, Aug 25, 2014 at 02:59:46PM +0200, Geert Uytterhoeven wrote: >> >> On Mon, Aug 25, 2014 at 2:35 PM, Mikhail Ulyanov >> >> wrote: >> >> > + - compatible: should containg one of the following: >> >> > + - "renesas,jpu-r8a7790" for R-Car H2 >> >> > + - "renesas,jpu-r8a7791" for R-Car M2 >> >> > + - "renesas,jpu-gen2" for R-Car second generation >> >> >> >> Isn't "renesas,jpu-gen2" meant as a fallback? >> >> >> >> I.e. the DTS should have one of '7790 and '7791, AND the gen2 fallback, >> >> so we can make the driver match against '7790 and '7791 is we find >> >> out about an incompatibility. >> > >> > Is there a document that clearly states that there is such a thing >> > as jpu-gen2 in hardware? If not I would prefer not to add a binding for it. >> >> We do have a document that describes the "JPEG Processing Unit (JPU)", >> as found in the following members of the "Second Generation R-Car Series >> Products": "R-Car H2", "R-Car M2-W", "R-Car M2-N", and "R-Car V2H". > > Oh, that is nice :) > > From my point of view that ticks a lot of boxes. > But I wonder if we can come up with a better name than jpu,-gen2. "jpu-rcar-gen2"? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Subject: Re: [PATCH v2 6/6] devicetree: bindings: Document Renesas JPEG Processing Unit. Date: Tue, 26 Aug 2014 11:27:43 +0200 Message-ID: References: <1408452653-14067-7-git-send-email-mikhail.ulyanov@cogentembedded.com> <1408970132-6690-1-git-send-email-mikhail.ulyanov@cogentembedded.com> <20140825235720.GB7217@verge.net.au> <20140826090126.GC29667@verge.net.au> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <20140826090126.GC29667@verge.net.au> Sender: linux-sh-owner@vger.kernel.org To: Simon Horman Cc: Mikhail Ulyanov , Magnus Damm , Mauro Carvalho Chehab , Rob Herring , Grant Likely , Mark Rutland , Ian Campbell , Laurent Pinchart , Hans Verkuil , Linux-sh list , Linux Media Mailing List , "devicetree@vger.kernel.org" List-Id: devicetree@vger.kernel.org On Tue, Aug 26, 2014 at 11:01 AM, Simon Horman wrote: > On Tue, Aug 26, 2014 at 10:03:34AM +0200, Geert Uytterhoeven wrote: >> On Tue, Aug 26, 2014 at 1:57 AM, Simon Horman wrote: >> > On Mon, Aug 25, 2014 at 02:59:46PM +0200, Geert Uytterhoeven wrote: >> >> On Mon, Aug 25, 2014 at 2:35 PM, Mikhail Ulyanov >> >> wrote: >> >> > + - compatible: should containg one of the following: >> >> > + - "renesas,jpu-r8a7790" for R-Car H2 >> >> > + - "renesas,jpu-r8a7791" for R-Car M2 >> >> > + - "renesas,jpu-gen2" for R-Car second generation >> >> >> >> Isn't "renesas,jpu-gen2" meant as a fallback? >> >> >> >> I.e. the DTS should have one of '7790 and '7791, AND the gen2 fallback, >> >> so we can make the driver match against '7790 and '7791 is we find >> >> out about an incompatibility. >> > >> > Is there a document that clearly states that there is such a thing >> > as jpu-gen2 in hardware? If not I would prefer not to add a binding for it. >> >> We do have a document that describes the "JPEG Processing Unit (JPU)", >> as found in the following members of the "Second Generation R-Car Series >> Products": "R-Car H2", "R-Car M2-W", "R-Car M2-N", and "R-Car V2H". > > Oh, that is nice :) > > From my point of view that ticks a lot of boxes. > But I wonder if we can come up with a better name than jpu,-gen2. "jpu-rcar-gen2"? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds