Re: [PATCH v2 6/6] devicetree: bindings: Document Renesas JPEG Processing Unit.

Simon Horman
2014-08-26T02:43:01Z

On Tue, Aug 26, 2014 at 02:02:00AM +0200, Laurent Pinchart wrote:
> Hi Simon,
>
> On Tuesday 26 August 2014 08:57:20 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.
>
> How about going the other way around and requesting that document ?

Good idea, I will make a request.