From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sascha Hauer Date: Mon, 04 Mar 2013 07:48:51 +0000 Subject: Re: [PATCH 2/2] video: imxfb: Add DT support Message-Id: <20130304074851.GZ1906@pengutronix.de> List-Id: References: <1362152467-27998-1-git-send-email-mpa@pengutronix.de> <1362152467-27998-3-git-send-email-mpa@pengutronix.de> In-Reply-To: <1362152467-27998-3-git-send-email-mpa@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org On Fri, Mar 01, 2013 at 04:41:07PM +0100, Markus Pargmann wrote: > Add devicetree support for imx framebuffer driver. It uses the generic > display bindings and helper functions. > > Signed-off-by: Markus Pargmann > --- > .../devicetree/bindings/video/fsl,imx-fb.txt | 30 ++++ > drivers/video/imxfb.c | 176 +++++++++++++++++---- > 2 files changed, 173 insertions(+), 33 deletions(-) > create mode 100644 Documentation/devicetree/bindings/video/fsl,imx-fb.txt > > diff --git a/Documentation/devicetree/bindings/video/fsl,imx-fb.txt b/Documentation/devicetree/bindings/video/fsl,imx-fb.txt > new file mode 100644 > index 0000000..fd88d26 > --- /dev/null > +++ b/Documentation/devicetree/bindings/video/fsl,imx-fb.txt > @@ -0,0 +1,30 @@ > +Freescale imx21 Framebuffer > + > +This framebuffer driver supports chips imx1 and imx21. > + > +Required properties: > +- compatible : "fsl,-fb" > +- reg : Should contain 1 register ranges(address and length) > +- interrupts : One interrupt of the fb dev > + > +Required nodes: > +- display: a display node is required to initialize the lcd panel > + This should be in the board dts. See definition in > + Documentation/devicetree/bindings/video/via,vt8500-fb.txt > +- default-mode: a videomode node as specified in > + Documentation/devicetree/bindings/video/via,vt8500-fb.txt Since the of_videomode helpers didn't add a binding in itself, they don't have a binding description in Documentation/devicetree/. It's forseeable that drivers will use this binding in the near future, so we should probably add a separate Documentation file for it instead of referencing some driver which already implements the binding. > + > +Optional properties: > +- pwmr: Address of pwmr register This describes the imxfb internal PWM controller. Please drop this for now or add a proper PWM driver for it. > +- lscr1: Address of lscr1 register > +- dmacr: Address of dmacr register I think we shouldn't expose these to the devicetree. With platform_data this hasn't been nice, but ok. With devicetree this becomes an API, so we should think of something better. Can't we make up sensible values during runtime? Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |