| Message ID | 1370590290-13467-1-git-send-email-acourbot@nvidia.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On 06/07/2013 01:31 AM, Alexandre Courbot wrote: > A framebuffer of this format is set up by SHIELD's bootloader. > > Signed-off-by: Alexandre Courbot <acourbot@nvidia.com> > Acked-by: Olof Johansson <olof@lixom.net> Reviewed-by: Stephen Warren <swarren@nvidia.com> -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Sat, Jun 8, 2013 at 1:24 AM, Stephen Warren <swarren@wwwdotorg.org> wrote: > On 06/07/2013 01:31 AM, Alexandre Courbot wrote: >> A framebuffer of this format is set up by SHIELD's bootloader. >> >> Signed-off-by: Alexandre Courbot <acourbot@nvidia.com> >> Acked-by: Olof Johansson <olof@lixom.net> > > Reviewed-by: Stephen Warren <swarren@nvidia.com> Jean-Christophe, Tomi, do you plan to merge this? Alex. -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 07/06/13 10:31, Alexandre Courbot wrote: > A framebuffer of this format is set up by SHIELD's bootloader. > > Signed-off-by: Alexandre Courbot <acourbot@nvidia.com> > Acked-by: Olof Johansson <olof@lixom.net> > --- > Changes from v2: > - Fixed typo in format (Thanks Alexander van Heukelum!) > > Changes from v1: > - Added description to motivate the change > - Added ack > > Documentation/devicetree/bindings/video/simple-framebuffer.txt | 1 + > drivers/video/simplefb.c | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/Documentation/devicetree/bindings/video/simple-framebuffer.txt b/Documentation/devicetree/bindings/video/simple-framebuffer.txt > index 3ea4605..70c26f3 100644 > --- a/Documentation/devicetree/bindings/video/simple-framebuffer.txt > +++ b/Documentation/devicetree/bindings/video/simple-framebuffer.txt > @@ -12,6 +12,7 @@ Required properties: > - stride: The number of bytes in each line of the framebuffer. > - format: The format of the framebuffer surface. Valid values are: > - r5g6b5 (16-bit pixels, d[15:11]=r, d[10:5]=g, d[4:0]=b). > + - a8b8g8r8 (32-bit pixels, d[31:24]=a, d[23:16]=b, d[15:8]=g, d[7:0]=r). > > Example: > > diff --git a/drivers/video/simplefb.c b/drivers/video/simplefb.c > index e2e9e3e..f015482 100644 > --- a/drivers/video/simplefb.c > +++ b/drivers/video/simplefb.c > @@ -84,6 +84,7 @@ struct simplefb_format { > > static struct simplefb_format simplefb_formats[] = { > { "r5g6b5", 16, {11, 5}, {5, 6}, {0, 5}, {0, 0} }, > + { "a8b8g8r8", 32, {0, 8}, {8, 8}, {16, 8}, {24, 8} }, > }; > > struct simplefb_params { Thanks, I've applied this into my 3.12/fbdev branch. Tomi
diff --git a/Documentation/devicetree/bindings/video/simple-framebuffer.txt b/Documentation/devicetree/bindings/video/simple-framebuffer.txt index 3ea4605..70c26f3 100644 --- a/Documentation/devicetree/bindings/video/simple-framebuffer.txt +++ b/Documentation/devicetree/bindings/video/simple-framebuffer.txt @@ -12,6 +12,7 @@ Required properties: - stride: The number of bytes in each line of the framebuffer. - format: The format of the framebuffer surface. Valid values are: - r5g6b5 (16-bit pixels, d[15:11]=r, d[10:5]=g, d[4:0]=b). + - a8b8g8r8 (32-bit pixels, d[31:24]=a, d[23:16]=b, d[15:8]=g, d[7:0]=r). Example: diff --git a/drivers/video/simplefb.c b/drivers/video/simplefb.c index e2e9e3e..f015482 100644 --- a/drivers/video/simplefb.c +++ b/drivers/video/simplefb.c @@ -84,6 +84,7 @@ struct simplefb_format { static struct simplefb_format simplefb_formats[] = { { "r5g6b5", 16, {11, 5}, {5, 6}, {0, 5}, {0, 0} }, + { "a8b8g8r8", 32, {0, 8}, {8, 8}, {16, 8}, {24, 8} }, }; struct simplefb_params {