Re: [PATCH V2] video: implement a simple framebuffer driver

On Tuesday 30 April 2013, Tomi Valkeinen wrote:

quoted

+     framebuffer {
+             compatible = "simple-framebuffer";
+             reg = <0x1d385000 (1600 * 1200 * 2)>;
+             width = <1600>;
+             height = <1200>;
+             stride = <(1600 * 2)>;
+             format = "r5g6b5";
+     };

I'm not an expert on DT, but I think the point of DT is to describe the
hardware. This doesn't describe the hardware at all.

That's ok.

It's not uncommon to have settings in the device tree that describe how
hardware is set up. Other similar properties would be the line rate
of a serial port, or a keymap describing what each button is labeled.
They are not physical properties, but they are necessary platform specific
pieces of information that are not available otherwise.

	Arnd