[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <bdad8391-53f2-4968-95f0-d6c69de88903@linaro.org>
Date: Fri, 4 Aug 2023 15:06:56 +0200
From: Neil Armstrong <neil.armstrong@...aro.org>
To: Michael Riesch <michael.riesch@...fvision.net>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Conor Dooley <conor+dt@...nel.org>,
Sam Ravnborg <sam@...nborg.org>,
David Airlie <airlied@...il.com>,
Daniel Vetter <daniel@...ll.ch>,
Maxime Ripard <mripard@...nel.org>,
Miquel Raynal <miquel.raynal@...tlin.com>,
Sebastian Reichel <sre@...nel.org>,
Gerald Loacker <gerald.loacker@...fvision.net>
Cc: devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
dri-devel@...ts.freedesktop.org
Subject: Re: [PATCH v2 4/4] drm/panel: sitronix-st7789v: add jasonic
jt240mhqs-hwt-ek-e3 support
On 04/08/2023 15:02, Michael Riesch wrote:
> The Jasonic JT240MHQS-HWT-EK-E3 is a custom panel using the Sitronix
> ST7789V controller. While the controller features a resolution of
> 320x240, only an area of 280x240 is visible by design.
>
> Signed-off-by: Michael Riesch <michael.riesch@...fvision.net>
> ---
> drivers/gpu/drm/panel/panel-sitronix-st7789v.c | 29 ++++++++++++++++++++++++++
> 1 file changed, 29 insertions(+)
>
> diff --git a/drivers/gpu/drm/panel/panel-sitronix-st7789v.c b/drivers/gpu/drm/panel/panel-sitronix-st7789v.c
> index ebc9a3bd6db3..88e80fe98112 100644
> --- a/drivers/gpu/drm/panel/panel-sitronix-st7789v.c
> +++ b/drivers/gpu/drm/panel/panel-sitronix-st7789v.c
> @@ -279,6 +279,21 @@ static const struct drm_display_mode et028013dma_mode = {
> .flags = DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC,
> };
>
> +static const struct drm_display_mode jt240mhqs_hwt_ek_e3_mode = {
> + .clock = 6000,
> + .hdisplay = 240,
> + .hsync_start = 240 + 28,
> + .hsync_end = 240 + 28 + 10,
> + .htotal = 240 + 28 + 10 + 10,
> + .vdisplay = 280,
> + .vsync_start = 280 + 8,
> + .vsync_end = 280 + 8 + 4,
> + .vtotal = 280 + 8 + 4 + 4,
> + .width_mm = 43,
> + .height_mm = 37,
> + .flags = DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC,
> +};
> +
> static const struct st7789_panel_info default_panel = {
> .mode = &default_mode,
> .invert_mode = true,
> @@ -303,6 +318,17 @@ static const struct st7789_panel_info et028013dma_panel = {
> DRM_BUS_FLAG_PIXDATA_SAMPLE_POSEDGE,
> };
>
> +static const struct st7789_panel_info jt240mhqs_hwt_ek_e3_panel = {
> + .mode = &jt240mhqs_hwt_ek_e3_mode,
> + .invert_mode = true,
> + .bus_format = MEDIA_BUS_FMT_RGB666_1X18,
> + .bus_flags = DRM_BUS_FLAG_DE_HIGH |
> + DRM_BUS_FLAG_PIXDATA_SAMPLE_NEGEDGE,
> + .partial_mode = true,
> + .partial_start = 38,
> + .partial_end = 318,
> +};
> +
> static int st7789v_get_modes(struct drm_panel *panel,
> struct drm_connector *connector)
> {
> @@ -635,6 +661,7 @@ static const struct spi_device_id st7789v_spi_id[] = {
> { "st7789v", (unsigned long) &default_panel },
> { "t28cp45tn89-v17", (unsigned long) &t28cp45tn89_panel },
> { "et028013dma", (unsigned long) &et028013dma_panel },
> + { "jt240mhqs-hwt-ek-e3", (unsigned long) &jt240mhqs_hwt_ek_e3_panel },
> { }
> };
> MODULE_DEVICE_TABLE(spi, st7789v_spi_id);
> @@ -643,6 +670,8 @@ static const struct of_device_id st7789v_of_match[] = {
> { .compatible = "sitronix,st7789v", .data = &default_panel },
> { .compatible = "inanbo,t28cp45tn89-v17", .data = &t28cp45tn89_panel },
> { .compatible = "edt,et028013dma", .data = &et028013dma_panel },
> + { .compatible = "jasonic,jt240mhqs-hwt-ek-e3",
> + .data = &jt240mhqs_hwt_ek_e3_panel },
> { }
> };
> MODULE_DEVICE_TABLE(of, st7789v_of_match);
>
Reviewed-by: Neil Armstrong <neil.armstrong@...aro.org>
Powered by blists - more mailing lists