Re: [PATCH 1/2 v2] drm/pl111: Support the Versatile Express

2018-04-18 Thread Liviu Dudau
On Wed, Apr 18, 2018 at 12:50:10PM +0100, Robin Murphy wrote: > On 17/04/18 20:31, Linus Walleij wrote: > > On Tue, Apr 17, 2018 at 3:12 PM, Robin Murphy wrote: > > > On 17/04/18 13:32, Linus Walleij wrote: > > > [...] > > > > > > > > Unfortunately there is just one single vexpress core tile in t

Re: [PATCH 1/2 v2] drm/pl111: Support the Versatile Express

2018-04-18 Thread Robin Murphy
On 17/04/18 20:31, Linus Walleij wrote: On Tue, Apr 17, 2018 at 3:12 PM, Robin Murphy wrote: On 17/04/18 13:32, Linus Walleij wrote: [...] Unfortunately there is just one single vexpress core tile in the upstream kernel that define a CLCD controller, the CA9 (4xA9) that I am using. All the ot

Re: [PATCH 1/2 v2] drm/pl111: Support the Versatile Express

2018-04-17 Thread Linus Walleij
On Tue, Apr 17, 2018 at 3:12 PM, Robin Murphy wrote: > On 17/04/18 13:32, Linus Walleij wrote: > [...] >> >> Unfortunately there is just one single vexpress core tile in the >> upstream kernel that define a CLCD controller, the CA9 (4xA9) >> that I am using. All the others just use the MB CLCD. >>

Re: [PATCH 1/2 v2] drm/pl111: Support the Versatile Express

2018-04-17 Thread Robin Murphy
On 17/04/18 13:32, Linus Walleij wrote: [...] Unfortunately there is just one single vexpress core tile in the upstream kernel that define a CLCD controller, the CA9 (4xA9) that I am using. All the others just use the MB CLCD. I am thinking there is some never finished DTS upstreaming here that

Re: [PATCH 1/2 v2] drm/pl111: Support the Versatile Express

2018-04-17 Thread Linus Walleij
On Tue, Apr 10, 2018 at 11:57 AM, Liviu Dudau wrote: > I need to do a bit of archeology to get my Versatile Express back > to working order in order to test your patches, which might not > happen this week or so, but I have some comments on your approach > which I would like clarification on. OK

Re: [PATCH 1/2 v2] drm/pl111: Support the Versatile Express

2018-04-10 Thread Liviu Dudau
Hi Linus, I need to do a bit of archeology to get my Versatile Express back to working order in order to test your patches, which might not happen this week or so, but I have some comments on your approach which I would like clarification on. On Fri, Apr 06, 2018 at 04:19:34PM +0200, Linus Wallei

Re: [PATCH 1/2 v2] drm/pl111: Support the Versatile Express

2018-04-08 Thread Linus Walleij
On Sun, Apr 8, 2018 at 3:16 AM, Eric Anholt wrote: > Linus Walleij writes: >> - void *regs; >> + void __iomem *clcd_memory; > > This doesn't seem to be used anywhere. Ah development artifact, I'll get rid of it. >> +int pl111_vexpress_clcd_init(struct device *dev, >> +

Re: [PATCH 1/2 v2] drm/pl111: Support the Versatile Express

2018-04-07 Thread Eric Anholt
Linus Walleij writes: > The Versatile Express uses a special configuration controller > deeply embedded in the system motherboard FPGA to multiplex the > two to three (!) CLCD instances out to the single SiI9022 > bridge. > > Set up an extra file with the logic to probe to the FPGA mux > register

[PATCH 1/2 v2] drm/pl111: Support the Versatile Express

2018-04-06 Thread Linus Walleij
The Versatile Express uses a special configuration controller deeply embedded in the system motherboard FPGA to multiplex the two to three (!) CLCD instances out to the single SiI9022 bridge. Set up an extra file with the logic to probe to the FPGA mux register on the system controller bus, then p