On 30/07/13 21:26, Darren Etheridge wrote: > From: Afzal Mohammed <afzal@xxxxxx> > > Driver is provided a means to have the probe triggered by DT. > > Signed-off-by: Afzal Mohammed <afzal@xxxxxx> > Signed-off-by: Darren Etheridge <detheridge@xxxxxx> > --- > .../devicetree/bindings/video/fb-da8xx.txt | 16 ++++++++++++++++ > drivers/video/da8xx-fb.c | 7 +++++++ > 2 files changed, 23 insertions(+), 0 deletions(-) > create mode 100644 Documentation/devicetree/bindings/video/fb-da8xx.txt > > diff --git a/Documentation/devicetree/bindings/video/fb-da8xx.txt b/Documentation/devicetree/bindings/video/fb-da8xx.txt > new file mode 100644 > index 0000000..581e014 > --- /dev/null > +++ b/Documentation/devicetree/bindings/video/fb-da8xx.txt > @@ -0,0 +1,16 @@ > +TI LCD Controller on DA830/DA850/AM335x SoC's > + > +Required properties: > +- compatible: > + DA830 - "ti,da830-lcdc" > + AM335x SoC's - "ti,am3352-lcdc", "ti,da830-lcdc" I'm not totally sure about this, but how I understand the compatible property, the above reads as: "this device is ti,am3352-lcdc and it's fully compatible with ti,da830-lcdc". I.e. if the kernel has a driver for da830-lcdc, it should work with AM335x also (without any of the new features in AM335x, obviously). Which I believe is not the case, as the point of this series is to add the AM335x support. Or should the current da830-lcdc work with AM335x also, but it just didn't because there were bugs in da830-lcdc? Tomi
Attachment:
signature.asc
Description: OpenPGP digital signature
