diff mbox

[V2,1/2] ARM: bcm2835: add simple framebuffer to device tree

Message ID 1365043257-29296-1-git-send-email-swarren@wwwdotorg.org (mailing list archive)
State New, archived
Headers show

Commit Message

Stephen Warren April 4, 2013, 2:40 a.m. UTC
Place a template simple-framebuffer device into the RPi device tree. The
bootloader is expected to fill in all the parameters at boot time.
Hence, disable the device by default, so if the bootloader doesn't do
this, the device isn't activated with invalid parameters.

Signed-off-by: Stephen Warren <swarren@wwwdotorg.org>
---
v2: s/dumb/simple/ throughout.
---
 arch/arm/boot/dts/bcm2835-rpi-b.dts |   11 +++++++++++
 1 file changed, 11 insertions(+)

Comments

Stephen Warren May 30, 2013, 4:10 a.m. UTC | #1
On 04/03/2013 08:40 PM, Stephen Warren wrote:
> Place a template simple-framebuffer device into the RPi device tree. The
> bootloader is expected to fill in all the parameters at boot time.
> Hence, disable the device by default, so if the bootloader doesn't do
> this, the device isn't activated with invalid parameters.

In the end, I dropped this patch, and decided to just have U-Boot create
this DT node itself from scratch, just like Olof did for the Samsung ARM
Chromebook.

I have however applied patch 2/2, to enable the simplefb driver in
bcm2835_defconfig.
diff mbox

Patch

diff --git a/arch/arm/boot/dts/bcm2835-rpi-b.dts b/arch/arm/boot/dts/bcm2835-rpi-b.dts
index aafda17..09e0d40 100644
--- a/arch/arm/boot/dts/bcm2835-rpi-b.dts
+++ b/arch/arm/boot/dts/bcm2835-rpi-b.dts
@@ -8,6 +8,17 @@ 
 	memory {
 		reg = <0 0x10000000>;
 	};
+
+	framebuffer {
+		compatible = "simple-framebuffer";
+		status = "disabled";
+		/* The bootloader is expected to fill these in */
+		reg = <0 0>;
+		width = <0>;
+		height = <0>;
+		stride = <0>;
+		format = "";
+	};
 };
 
 &gpio {