[PATCH] fbdev/simplefb: Add the N64's mode

[PATCH] fbdev/simplefb: Add the N64's mode

[Lauri Kasanen]

Signed-off-by: Lauri Kasanen <cand@gmx.com>
---
 include/linux/platform_data/simplefb.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/include/linux/platform_data/simplefb.h b/include/linux/platform_data/simplefb.h
index ca83376..27ea99a 100644
--- a/include/linux/platform_data/simplefb.h
+++ b/include/linux/platform_data/simplefb.h
@@ -16,6 +16,7 @@
 #define SIMPLEFB_FORMATS \
 { \
 	{ "r5g6b5", 16, {11, 5}, {5, 6}, {0, 5}, {0, 0}, DRM_FORMAT_RGB565 }, \
+	{ "r5g5b5a1", 16, {11, 5}, {6, 5}, {1, 5}, {0, 1}, DRM_FORMAT_RGBA5551 }, \
 	{ "x1r5g5b5", 16, {10, 5}, {5, 5}, {0, 5}, {0, 0}, DRM_FORMAT_XRGB1555 }, \
 	{ "a1r5g5b5", 16, {10, 5}, {5, 5}, {0, 5}, {15, 1}, DRM_FORMAT_ARGB1555 }, \
 	{ "r8g8b8", 24, {16, 8}, {8, 8}, {0, 8}, {0, 0}, DRM_FORMAT_RGB888 }, \
--
2.6.2

Re: [PATCH] fbdev/simplefb: Add the N64's mode

[Hans de Goede]

Hi,

On 1/15/21 3:01 PM, Lauri Kasanen wrote:
> Signed-off-by: Lauri Kasanen <cand@gmx.com>

Thanks, patch looks good to me:

Reviewed-by: Hans de Goede <hdegoede@redhat.com>

What is the plan for merging this patch? I've another simplefb patch which
I need/plan to push to drm-misc-nexy soon, shall I also push this one then?

Or is there some plan to merge the entire series this used to be part of
in one go?

Regards,

Hans

> ---
>  include/linux/platform_data/simplefb.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/include/linux/platform_data/simplefb.h b/include/linux/platform_data/simplefb.h
> index ca83376..27ea99a 100644
> --- a/include/linux/platform_data/simplefb.h
> +++ b/include/linux/platform_data/simplefb.h
> @@ -16,6 +16,7 @@
>  #define SIMPLEFB_FORMATS \
>  { \
>  	{ "r5g6b5", 16, {11, 5}, {5, 6}, {0, 5}, {0, 0}, DRM_FORMAT_RGB565 }, \
> +	{ "r5g5b5a1", 16, {11, 5}, {6, 5}, {1, 5}, {0, 1}, DRM_FORMAT_RGBA5551 }, \
>  	{ "x1r5g5b5", 16, {10, 5}, {5, 5}, {0, 5}, {0, 0}, DRM_FORMAT_XRGB1555 }, \
>  	{ "a1r5g5b5", 16, {10, 5}, {5, 5}, {0, 5}, {15, 1}, DRM_FORMAT_ARGB1555 }, \
>  	{ "r8g8b8", 24, {16, 8}, {8, 8}, {0, 8}, {0, 0}, DRM_FORMAT_RGB888 }, \
> --
> 2.6.2
> 

Re: [PATCH] fbdev/simplefb: Add the N64's mode

[Thomas Bogendoerfer]

On Fri, Jan 15, 2021 at 03:06:13PM +0100, Hans de Goede wrote:
> Hi,
> 
> On 1/15/21 3:01 PM, Lauri Kasanen wrote:
> > Signed-off-by: Lauri Kasanen <cand@gmx.com>
> 
> Thanks, patch looks good to me:
> 
> Reviewed-by: Hans de Goede <hdegoede@redhat.com>
> 
> What is the plan for merging this patch? I've another simplefb patch which
> I need/plan to push to drm-misc-nexy soon, shall I also push this one then?
> 
> Or is there some plan to merge the entire series this used to be part of
> in one go?

this looks independant enough to me, so I'd appreciate, if you take it.

Thomas.

-- 
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea.                                                [ RFC1925, 2.3 ]

Re: [PATCH] fbdev/simplefb: Add the N64's mode

[Hans de Goede]

Hi,

On 1/15/21 3:37 PM, Thomas Bogendoerfer wrote:
> On Fri, Jan 15, 2021 at 03:06:13PM +0100, Hans de Goede wrote:
>> Hi,
>>
>> On 1/15/21 3:01 PM, Lauri Kasanen wrote:
>>> Signed-off-by: Lauri Kasanen <cand@gmx.com>
>>
>> Thanks, patch looks good to me:
>>
>> Reviewed-by: Hans de Goede <hdegoede@redhat.com>
>>
>> What is the plan for merging this patch? I've another simplefb patch which
>> I need/plan to push to drm-misc-nexy soon, shall I also push this one then?
>>
>> Or is there some plan to merge the entire series this used to be part of
>> in one go?
> 
> this looks independant enough to me, so I'd appreciate, if you take it.

Ok, I've just pushed this patch to drm-misc-next.

Regards,

Hans