Hi Sam,
Thank you for the patch.
On Fri, Jul 03, 2020 at 09:23:57PM +0200, Sam Ravnborg wrote:
The boe,hv070wsa-100 panel is a LVDS panel. Fix connector type to reflect this.
With this change users of this panel do not have to specify the connector type.
v3:
- Drop PIXDATA bus_flag, not relevant
v2:
- Add .bus_format (Laurent)
- Add .bus_flags
Signed-off-by: Sam Ravnborg sam@ravnborg.org Cc: Laurent Pinchart laurent.pinchart@ideasonboard.com Cc: Thierry Reding thierry.reding@gmail.com Cc: Sam Ravnborg sam@ravnborg.org
drivers/gpu/drm/panel/panel-simple.c | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c index 4f0ec5e5b0aa..7b5d212215e0 100644 --- a/drivers/gpu/drm/panel/panel-simple.c +++ b/drivers/gpu/drm/panel/panel-simple.c @@ -1188,6 +1188,9 @@ static const struct panel_desc boe_hv070wsa = { .width = 154, .height = 90, },
- .bus_format = MEDIA_BUS_FMT_RGB888_1X7X4_SPWG,
- .bus_flags = DRM_BUS_FLAG_DE_HIGH,
- .connector_type = DRM_MODE_CONNECTOR_LVDS,
While at it, should you set .bpc to 8 ?
Reviewed-by: Laurent Pinchart laurent.pinchart@ideasonboard.com
};
static const struct drm_display_mode boe_nv101wxmn51_modes[] = {