Re: [PATCH v2] video: fbdev: add Marvell PXA framebuffer binding

2015-10-08 Thread Rob Herring
On Thu, Oct 8, 2015 at 2:15 PM, Robert Jarzmik wrote: > Rob Herring writes: >>> And then, when a board maintainer will create a devicetree description, he >>> will >>> write something like : >>> compatible = "toshiba,ltm0305a776"; >>> compatible = "marvell,pxa2xx-panel"; >> >> Drop

Re: [PATCH v2] video: fbdev: add Marvell PXA framebuffer binding

2015-10-08 Thread Robert Jarzmik
Rob Herring writes: >> The pxafb driver acts today on the subset of registers which are the same >> across >> all pxaXXX variants. This is what made me think only one compatible property >> was >> required. >> >> If I'm wrong, I could add "marvell,pxa3xx-lcdc", is that what you think I >> shou

Re: [PATCH v2] video: fbdev: add Marvell PXA framebuffer binding

2015-10-08 Thread Rob Herring
On Tue, Oct 6, 2015 at 2:32 PM, Robert Jarzmik wrote: > Rob Herring writes: > >> On Sun, Oct 4, 2015 at 5:31 AM, Robert Jarzmik >> wrote: >>> Add documentation for the PXA frambuffer devicetree binding. >> >> Strictly speaking this is a binding for PXA display controller, not a >> Linux FB driv

Re: [PATCH v2] video: fbdev: add Marvell PXA framebuffer binding

2015-10-07 Thread Philipp Zabel
On Tue, Oct 6, 2015 at 9:32 PM, Robert Jarzmik wrote: > Rob Herring writes: >>> + "marvell,pxa2xx-lcdc", >> >> No differences in h/w for any of the chips? > All pxa25x, pxa27x and pxa3xx are compatible. > AFAIK, pxa3xx has an IP with additional registers. But : > - these (this) register(s)

Re: [PATCH v2] video: fbdev: add Marvell PXA framebuffer binding

2015-10-06 Thread Robert Jarzmik
Rob Herring writes: > On Sun, Oct 4, 2015 at 5:31 AM, Robert Jarzmik wrote: >> Add documentation for the PXA frambuffer devicetree binding. > > Strictly speaking this is a binding for PXA display controller, not a > Linux FB driver. There are lots of "framebuffer" and "DRM" bindings > which I'm

Re: [PATCH v2] video: fbdev: add Marvell PXA framebuffer binding

2015-10-06 Thread Robert Jarzmik
Philipp Zabel writes: > On Sun, Oct 4, 2015 at 12:31 PM, Robert Jarzmik > wrote: >> + lcd-controller@4050 { >> + compatible = "marvell,pxa2xx-lcdc"; >> + reg = <0x4400 0x1>; >> + interrupts = <17>; >> + clocks = <&clks CL

Re: [PATCH v2] video: fbdev: add Marvell PXA framebuffer binding

2015-10-06 Thread Rob Herring
On Sun, Oct 4, 2015 at 5:31 AM, Robert Jarzmik wrote: > Add documentation for the PXA frambuffer devicetree binding. Strictly speaking this is a binding for PXA display controller, not a Linux FB driver. There are lots of "framebuffer" and "DRM" bindings which I'm trying to curb. > Signed-off-by

Re: [PATCH v2] video: fbdev: add Marvell PXA framebuffer binding

2015-10-05 Thread Philipp Zabel
On Sun, Oct 4, 2015 at 12:31 PM, Robert Jarzmik wrote: > Add documentation for the PXA frambuffer devicetree binding. > > Signed-off-by: Robert Jarzmik > --- > Since v1: Philipp's review on the whole binding > --- > .../devicetree/bindings/video/marvell,pxafb.txt| 80 > +

[PATCH v2] video: fbdev: add Marvell PXA framebuffer binding

2015-10-04 Thread Robert Jarzmik
Add documentation for the PXA frambuffer devicetree binding. Signed-off-by: Robert Jarzmik --- Since v1: Philipp's review on the whole binding --- .../devicetree/bindings/video/marvell,pxafb.txt| 80 ++ 1 file changed, 80 insertions(+) create mode 100644 Documentation/de