From: Simon Horman <horms@verge.net.au> To: Geert Uytterhoeven <geert@linux-m68k.org> Cc: Mikhail Ulyanov <mikhail.ulyanov@cogentembedded.com>, Magnus Damm <magnus.damm@gmail.com>, Mauro Carvalho Chehab <m.chehab@samsung.com>, Rob Herring <robh+dt@kernel.org>, Grant Likely <grant.likely@linaro.org>, Mark Rutland <mark.rutland@arm.com>, Ian Campbell <ijc+devicetree@hellion.org.uk>, Laurent Pinchart <laurent.pinchart@ideasonboard.com>, Hans Verkuil <hans.verkuil@cisco.com>, Linux-sh list <linux-sh@vger.kernel.org>, Linux Media Mailing List <linux-media@vger.kernel.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org> Subject: Re: [PATCH v2 6/6] devicetree: bindings: Document Renesas JPEG Processing Unit. Date: Tue, 26 Aug 2014 09:01:26 +0000 [thread overview] Message-ID: <20140826090126.GC29667@verge.net.au> (raw) In-Reply-To: <CAMuHMdW0r-xGJZtc4AFvxMnTu1rdEROf2DvywScC-XhSELFMtQ@mail.gmail.com> On Tue, Aug 26, 2014 at 10:03:34AM +0200, Geert Uytterhoeven wrote: > On Tue, Aug 26, 2014 at 1:57 AM, Simon Horman <horms@verge.net.au> wrote: > > On Mon, Aug 25, 2014 at 02:59:46PM +0200, Geert Uytterhoeven wrote: > >> Hi Mikhail, > >> > >> On Mon, Aug 25, 2014 at 2:35 PM, Mikhail Ulyanov > >> <mikhail.ulyanov@cogentembedded.com> 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. > As for other SoCs, M1, H1, and A1 also seem to contain a JPU, but just > like Mikhail, I don't have enough information to say anything about those. > > 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 >
WARNING: multiple messages have this Message-ID (diff)
From: Simon Horman <horms@verge.net.au> To: Geert Uytterhoeven <geert@linux-m68k.org> Cc: Mikhail Ulyanov <mikhail.ulyanov@cogentembedded.com>, Magnus Damm <magnus.damm@gmail.com>, Mauro Carvalho Chehab <m.chehab@samsung.com>, Rob Herring <robh+dt@kernel.org>, Grant Likely <grant.likely@linaro.org>, Mark Rutland <mark.rutland@arm.com>, Ian Campbell <ijc+devicetree@hellion.org.uk>, Laurent Pinchart <laurent.pinchart@ideasonboard.com>, Hans Verkuil <hans.verkuil@cisco.com>, Linux-sh list <linux-sh@vger.kernel.org>, Linux Media Mailing List <linux-media@vger.kernel.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org> Subject: Re: [PATCH v2 6/6] devicetree: bindings: Document Renesas JPEG Processing Unit. Date: Tue, 26 Aug 2014 18:01:26 +0900 [thread overview] Message-ID: <20140826090126.GC29667@verge.net.au> (raw) In-Reply-To: <CAMuHMdW0r-xGJZtc4AFvxMnTu1rdEROf2DvywScC-XhSELFMtQ@mail.gmail.com> On Tue, Aug 26, 2014 at 10:03:34AM +0200, Geert Uytterhoeven wrote: > On Tue, Aug 26, 2014 at 1:57 AM, Simon Horman <horms@verge.net.au> wrote: > > On Mon, Aug 25, 2014 at 02:59:46PM +0200, Geert Uytterhoeven wrote: > >> Hi Mikhail, > >> > >> On Mon, Aug 25, 2014 at 2:35 PM, Mikhail Ulyanov > >> <mikhail.ulyanov@cogentembedded.com> 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. > As for other SoCs, M1, H1, and A1 also seem to contain a JPU, but just > like Mikhail, I don't have enough information to say anything about those. > > 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 >
next prev parent reply other threads:[~2014-08-26 9:01 UTC|newest] Thread overview: 79+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-08-19 12:50 [PATCH 0/6] R-Car JPEG Processing Unit Mikhail Ulyanov 2014-08-19 12:50 ` Mikhail Ulyanov 2014-08-19 12:50 ` Mikhail Ulyanov 2014-08-19 12:50 ` [PATCH 1/6] V4L2: Add Renesas R-Car JPEG codec driver Mikhail Ulyanov 2014-08-19 12:50 ` Mikhail Ulyanov 2014-08-19 13:37 ` Hans Verkuil 2014-08-19 13:37 ` Hans Verkuil 2014-08-22 13:39 ` Mikhail Ulianov 2014-08-22 13:39 ` Mikhail Ulianov 2014-08-22 17:03 ` Hans Verkuil 2014-08-25 12:29 ` [PATCH v2 " Mikhail Ulyanov 2014-08-25 12:29 ` Mikhail Ulyanov 2014-08-25 12:39 ` Hans Verkuil 2014-08-25 12:39 ` Hans Verkuil 2014-08-25 12:49 ` Hans Verkuil 2014-08-25 12:49 ` Hans Verkuil 2014-09-21 17:56 ` Sergei Shtylyov 2014-09-21 17:56 ` Sergei Shtylyov 2014-09-21 17:56 ` Sergei Shtylyov 2014-09-21 17:56 ` Sergei Shtylyov 2014-09-23 13:31 ` Kamil Debski 2014-09-23 13:31 ` Kamil Debski 2014-09-25 12:49 ` Sergei Shtylyov 2014-09-25 12:49 ` Sergei Shtylyov 2014-09-26 14:01 ` Laurent Pinchart 2014-09-26 14:01 ` Laurent Pinchart 2014-09-23 14:02 ` [PATCH " Geert Uytterhoeven 2014-09-23 14:02 ` Geert Uytterhoeven 2014-08-19 12:50 ` [PATCH 2/6] ARM: shmobile: r8a7790: Add JPU clock dt and CPG define Mikhail Ulyanov 2014-08-19 12:50 ` Mikhail Ulyanov 2014-08-20 23:07 ` Laurent Pinchart 2014-08-20 23:07 ` Laurent Pinchart 2014-08-22 1:57 ` Simon Horman 2014-08-22 1:57 ` Simon Horman 2014-08-19 12:50 ` [PATCH 3/6] ARM: shmobile: r8a7790: Add JPU device node Mikhail Ulyanov 2014-08-19 12:50 ` Mikhail Ulyanov 2014-08-22 1:54 ` [3/6] " Simon Horman 2014-08-22 1:54 ` Simon Horman 2014-08-19 12:50 ` [PATCH 4/6] ARM: shmobile: r8a7791: Add JPU clock dt and CPG define Mikhail Ulyanov 2014-08-19 12:50 ` Mikhail Ulyanov 2014-08-20 23:07 ` Laurent Pinchart 2014-08-20 23:07 ` Laurent Pinchart 2014-08-22 1:56 ` Simon Horman 2014-08-22 1:56 ` Simon Horman 2014-08-19 12:50 ` [PATCH 5/6] ARM: shmobile: r8a7791: Add JPU device node Mikhail Ulyanov 2014-08-19 12:50 ` Mikhail Ulyanov 2014-08-19 12:50 ` [PATCH 6/6] devicetree: bindings: Document Renesas JPEG Processing Unit Mikhail Ulyanov 2014-08-19 12:50 ` Mikhail Ulyanov 2014-08-20 23:01 ` Laurent Pinchart 2014-08-20 23:01 ` Laurent Pinchart 2014-08-22 14:20 ` Mikhail Ulianov 2014-08-22 14:20 ` Mikhail Ulianov 2014-08-25 9:49 ` Laurent Pinchart 2014-08-25 9:49 ` Laurent Pinchart 2014-08-25 12:35 ` [PATCH v2 " Mikhail Ulyanov 2014-08-25 12:35 ` Mikhail Ulyanov 2014-08-25 12:59 ` Geert Uytterhoeven 2014-08-25 12:59 ` Geert Uytterhoeven 2014-08-25 23:57 ` Simon Horman 2014-08-25 23:57 ` Simon Horman 2014-08-26 0:02 ` Laurent Pinchart 2014-08-26 0:02 ` Laurent Pinchart 2014-08-26 2:42 ` Simon Horman 2014-08-26 2:42 ` Simon Horman 2014-09-24 13:38 ` Mikhail Ulianov 2014-09-24 13:38 ` Mikhail Ulianov 2014-08-26 8:03 ` Geert Uytterhoeven 2014-08-26 8:03 ` Geert Uytterhoeven 2014-08-26 9:01 ` Simon Horman [this message] 2014-08-26 9:01 ` Simon Horman 2014-08-26 9:27 ` Geert Uytterhoeven 2014-08-26 9:27 ` Geert Uytterhoeven 2014-08-27 5:15 ` Simon Horman 2014-08-27 5:15 ` Simon Horman 2014-08-27 6:06 ` Laurent Pinchart 2014-08-27 6:06 ` Laurent Pinchart 2014-08-27 6:16 ` Simon Horman 2014-08-27 6:16 ` Simon Horman 2014-08-26 0:17 Ray
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20140826090126.GC29667@verge.net.au \ --to=horms@verge.net.au \ --cc=devicetree@vger.kernel.org \ --cc=geert@linux-m68k.org \ --cc=grant.likely@linaro.org \ --cc=hans.verkuil@cisco.com \ --cc=ijc+devicetree@hellion.org.uk \ --cc=laurent.pinchart@ideasonboard.com \ --cc=linux-media@vger.kernel.org \ --cc=linux-sh@vger.kernel.org \ --cc=m.chehab@samsung.com \ --cc=magnus.damm@gmail.com \ --cc=mark.rutland@arm.com \ --cc=mikhail.ulyanov@cogentembedded.com \ --cc=robh+dt@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.