Re: [PATCH v2 2/4] drm/ssd130x: Add RGB565 support to SSD133X family
From: Javier Martinez Canillas <javierm@redhat.com>
Date: 2026-06-29 10:13:31
Also in:
dri-devel, linux-devicetree, linux-staging, lkml
Amit Barzilai [off-list ref] writes: Hello Amit, Thanks for doing a re-spin and adding the SSD1351 support to the ssd130x driver. This is much closer to what I had in mind than v1. [...]
quoted hunk ↗ jump to hunk
const struct ssd130x_deviceinfo ssd130x_variants[] = {@@ -206,6 +212,7 @@ const struct ssd130x_deviceinfo ssd130x_variants[] = { [SSD1331_ID] = { .default_width = 96, .default_height = 64, + .format_rgb565 = 1, .family_id = SSD133X_FAMILY, }
[...]
iosys_map_set_vaddr(&dst, data_array); - drm_fb_xrgb8888_to_rgb332(&dst, &dst_pitch, vmap, fb, rect, fmtcnv_state); + if (ssd130x->device_info->format_rgb565) + drm_fb_xrgb8888_to_rgb565be(&dst, &dst_pitch, vmap, fb, rect, fmtcnv_state); + else + drm_fb_xrgb8888_to_rgb332(&dst, &dst_pitch, vmap, fb, rect, fmtcnv_state);
Since you are changing the SSD1331_ID to use .format_rgb565, what's the value of keeping the RGB323 support ? It's not used by any other Solomon family supported by the driver. Ideally, we would like the driver to expose native formats besides the virtualized XRGB8888. Geert posted some patches for SSD1301 some time ago: https://lists.freedesktop.org/archives/dri-devel/2023-August/419937.html but that never landed. If that was the case, then SSD130X could expose DRM_FORMAT_XRGB8888 and DRM_FORMAT_R1; SSD132X would expose DRM_FORMAT_XRGB8888 and DRM_FORMAT_R4 and SSD133X expose DRM_FORMAT_XRGB8888, DRM_FORMAT_RGB332 and DRM_FORMAT_RGB565. Given that we don't have that, I suppose that we could just get rid of the DRM_FORMAT_RGB332 support for SSD133X and just default to DRM_FORMAT_RGB565. -- Best regards, Javier Martinez Canillas Core Platforms Red Hat