Re: [PATCH 0/4] ARM: Renesas: R-Car3: Add product register support

2016-06-29 Thread Dirk Behme
Hi Magnus, On 01.06.2016 07:19, Magnus Damm wrote: Hi Dirk, On Fri, May 27, 2016 at 4:56 PM, Dirk Behme wrote: On 27.05.2016 05:13, Magnus Damm wrote: I don't think anyone disagrees that it makes sense to be able to determine ES version during runtime. The questions

Re: [PATCH 0/4] ARM: Renesas: R-Car3: Add product register support

2016-06-01 Thread Dirk Behme
Hi Geert, On 01.06.2016 10:26, Geert Uytterhoeven wrote: Hi Dirk, On Wed, Jun 1, 2016 at 9:38 AM, Dirk Behme wrote: Sorry, if that was unclear. I took the example and transferred it to R-Car3 where we have ES1 - ES3. So, taking this example, on R-Car3 we might end

Re: [PATCH 0/4] ARM: Renesas: R-Car3: Add product register support

2016-06-01 Thread Dirk Behme
On 01.06.2016 10:40, Geert Uytterhoeven wrote: Hi Dirk, On Wed, Jun 1, 2016 at 10:36 AM, Dirk Behme wrote: On 01.06.2016 10:26, Geert Uytterhoeven wrote: On Wed, Jun 1, 2016 at 9:38 AM, Dirk Behme wrote: Sorry, if that was unclear. I took

Re: [PATCH 0/4] ARM: Renesas: R-Car3: Add product register support

2016-06-01 Thread Geert Uytterhoeven
Hi Dirk, On Wed, Jun 1, 2016 at 10:36 AM, Dirk Behme wrote: > On 01.06.2016 10:26, Geert Uytterhoeven wrote: >> On Wed, Jun 1, 2016 at 9:38 AM, Dirk Behme >> wrote: >>> Sorry, if that was unclear. I took the example and transferred it to >>>

Re: [PATCH 0/4] ARM: Renesas: R-Car3: Add product register support

2016-06-01 Thread Geert Uytterhoeven
Hi Dirk, On Wed, Jun 1, 2016 at 9:38 AM, Dirk Behme wrote: > Sorry, if that was unclear. I took the example and transferred it to R-Car3 > where we have ES1 - ES3. > > So, taking this example, on R-Car3 we might end up with > > { > .compatible =

Re: [PATCH 0/4] ARM: Renesas: R-Car3: Add product register support

2016-06-01 Thread Dirk Behme
Hi Geert, On 01.06.2016 09:27, Geert Uytterhoeven wrote: Hi Dirk, On Wed, Jun 1, 2016 at 9:19 AM, Dirk Behme wrote: On 01.06.2016 07:19, Magnus Damm wrote: On Fri, May 27, 2016 at 4:56 PM, Dirk Behme wrote: On 27.05.2016 05:13, Magnus Damm

Re: [PATCH 0/4] ARM: Renesas: R-Car3: Add product register support

2016-05-27 Thread Geert Uytterhoeven
On Fri, May 27, 2016 at 10:44 AM, Dirk Behme wrote: > Btw, what's about the MODEMR patch series? As it's from you and there have > been no further comments with v3, I'm somehow hoping to get it integrated. I'll be working on it soon. Gr{oetje,eeting}s,

Re: [PATCH 0/4] ARM: Renesas: R-Car3: Add product register support

2016-05-27 Thread Dirk Behme
On 27.05.2016 10:39, Geert Uytterhoeven wrote: On Fri, May 27, 2016 at 9:56 AM, Dirk Behme wrote: Given the use of PRR you saw is in a patch that's definitely not ready for upstream, I think adding a full-fledged PRR driver is a bit premature. I don't think anyone

Re: [PATCH 0/4] ARM: Renesas: R-Car3: Add product register support

2016-05-26 Thread Magnus Damm
Hi Dirk, On Thu, May 26, 2016 at 4:48 PM, Dirk Behme wrote: > Hi Geert, > > On 26.05.2016 09:14, Geert Uytterhoeven wrote: >> >> Hi Dirk, >> >> On Wed, May 25, 2016 at 10:58 AM, Dirk Behme >> wrote: >>> >>> Instead of hard coding the product

Re: [PATCH 0/4] ARM: Renesas: R-Car3: Add product register support

2016-05-26 Thread Dirk Behme
Hi Geert, On 26.05.2016 09:14, Geert Uytterhoeven wrote: Hi Dirk, On Wed, May 25, 2016 at 10:58 AM, Dirk Behme wrote: Instead of hard coding the product register in rcar_du_crtc.c, read it based on the device tree. This patch series is based on

Re: [PATCH 0/4] ARM: Renesas: R-Car3: Add product register support

2016-05-26 Thread Geert Uytterhoeven
Hi Dirk, On Wed, May 25, 2016 at 10:58 AM, Dirk Behme wrote: > Instead of hard coding the product register in rcar_du_crtc.c, > read it based on the device tree. > > This patch series is based on > >