diff mbox

[v5,3/4] drm/panel: add Innolux TV123WAM panel driver support

Message ID 1525235522-8954-4-git-send-email-spanda@codeaurora.org (mailing list archive)
State New, archived
Headers show

Commit Message

Sandeep Panda May 2, 2018, 4:32 a.m. UTC
Add support for Innolux TV123WAM, which is a 12.3" eDP
display panel with 2160x1440 resolution.

Signed-off-by: Sandeep Panda <spanda@codeaurora.org>
---
 drivers/gpu/drm/panel/panel-simple.c | 27 +++++++++++++++++++++++++++
 1 file changed, 27 insertions(+)

Comments

Sean Paul May 2, 2018, 7:05 p.m. UTC | #1
On Wed, May 02, 2018 at 10:02:01AM +0530, Sandeep Panda wrote:
> Add support for Innolux TV123WAM, which is a 12.3" eDP
> display panel with 2160x1440 resolution.
> 
> Signed-off-by: Sandeep Panda <spanda@codeaurora.org>
> ---
>  drivers/gpu/drm/panel/panel-simple.c | 27 +++++++++++++++++++++++++++
>  1 file changed, 27 insertions(+)
> 
> diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
> index 234af81..52bbcfb 100644
> --- a/drivers/gpu/drm/panel/panel-simple.c
> +++ b/drivers/gpu/drm/panel/panel-simple.c
> @@ -1939,6 +1939,30 @@ static void panel_simple_shutdown(struct device *dev)
>  	.bus_format = MEDIA_BUS_FMT_RGB888_1X24,
>  };
>  
> +static const struct drm_display_mode innolux_tv123wam_mode = {
> +	.clock = 206016,
> +	.hdisplay = 2160,
> +	.hsync_start = 2160 + 48,
> +	.hsync_end = 2160 + 48 + 32,
> +	.htotal = 2160 + 48 + 32 + 80,
> +	.vdisplay = 1440,
> +	.vsync_start = 1440 + 3,
> +	.vsync_end = 1440 + 3 + 10,
> +	.vtotal = 1440 + 3 + 10 + 27,
> +	.vrefresh = 60,
> +	.flags = DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC,
> +};
> +
> +static const struct panel_desc innolux_tv123wam = {
> +	.modes = &innolux_tv123wam_mode,
> +	.num_modes = 1,
> +	.bpc = 8,
> +	.size = {
> +		.width = 259,
> +		.height = 173,
> +	},
> +};
> +
>  static const struct of_device_id platform_of_match[] = {
>  	{
>  		.compatible = "ampire,am-480272h3tmqw-t01h",
> @@ -2142,6 +2166,9 @@ static void panel_simple_shutdown(struct device *dev)
>  		.compatible = "winstar,wf35ltiacd",
>  		.data = &winstar_wf35ltiacd,
>  	}, {
> +		.compatible = "innolux,tv123wam",
> +		.data = &innolux_tv123wam,
> +	}, {

These are alphabetically ordered, as are the drm_display_mode/panel_desc
structs.

Sean

>  		/* sentinel */
>  	}
>  };
> -- 
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project
>
Sandeep Panda May 15, 2018, 5:50 a.m. UTC | #2
On 2018-05-03 00:35, Sean Paul wrote:
> On Wed, May 02, 2018 at 10:02:01AM +0530, Sandeep Panda wrote:
>> Add support for Innolux TV123WAM, which is a 12.3" eDP
>> display panel with 2160x1440 resolution.
>> 
>> Signed-off-by: Sandeep Panda <spanda@codeaurora.org>
>> ---
>>  drivers/gpu/drm/panel/panel-simple.c | 27 +++++++++++++++++++++++++++
>>  1 file changed, 27 insertions(+)
>> 
>> diff --git a/drivers/gpu/drm/panel/panel-simple.c 
>> b/drivers/gpu/drm/panel/panel-simple.c
>> index 234af81..52bbcfb 100644
>> --- a/drivers/gpu/drm/panel/panel-simple.c
>> +++ b/drivers/gpu/drm/panel/panel-simple.c
>> @@ -1939,6 +1939,30 @@ static void panel_simple_shutdown(struct device 
>> *dev)
>>  	.bus_format = MEDIA_BUS_FMT_RGB888_1X24,
>>  };
>> 
>> +static const struct drm_display_mode innolux_tv123wam_mode = {
>> +	.clock = 206016,
>> +	.hdisplay = 2160,
>> +	.hsync_start = 2160 + 48,
>> +	.hsync_end = 2160 + 48 + 32,
>> +	.htotal = 2160 + 48 + 32 + 80,
>> +	.vdisplay = 1440,
>> +	.vsync_start = 1440 + 3,
>> +	.vsync_end = 1440 + 3 + 10,
>> +	.vtotal = 1440 + 3 + 10 + 27,
>> +	.vrefresh = 60,
>> +	.flags = DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC,
>> +};
>> +
>> +static const struct panel_desc innolux_tv123wam = {
>> +	.modes = &innolux_tv123wam_mode,
>> +	.num_modes = 1,
>> +	.bpc = 8,
>> +	.size = {
>> +		.width = 259,
>> +		.height = 173,
>> +	},
>> +};
>> +
>>  static const struct of_device_id platform_of_match[] = {
>>  	{
>>  		.compatible = "ampire,am-480272h3tmqw-t01h",
>> @@ -2142,6 +2166,9 @@ static void panel_simple_shutdown(struct device 
>> *dev)
>>  		.compatible = "winstar,wf35ltiacd",
>>  		.data = &winstar_wf35ltiacd,
>>  	}, {
>> +		.compatible = "innolux,tv123wam",
>> +		.data = &innolux_tv123wam,
>> +	}, {
> 
> These are alphabetically ordered, as are the 
> drm_display_mode/panel_desc
> structs.
> 
> Sean
> 
Done.
>>  		/* sentinel */
>>  	}
>>  };
>> --
>> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora 
>> Forum,
>> a Linux Foundation Collaborative Project
>>
diff mbox

Patch

diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
index 234af81..52bbcfb 100644
--- a/drivers/gpu/drm/panel/panel-simple.c
+++ b/drivers/gpu/drm/panel/panel-simple.c
@@ -1939,6 +1939,30 @@  static void panel_simple_shutdown(struct device *dev)
 	.bus_format = MEDIA_BUS_FMT_RGB888_1X24,
 };
 
+static const struct drm_display_mode innolux_tv123wam_mode = {
+	.clock = 206016,
+	.hdisplay = 2160,
+	.hsync_start = 2160 + 48,
+	.hsync_end = 2160 + 48 + 32,
+	.htotal = 2160 + 48 + 32 + 80,
+	.vdisplay = 1440,
+	.vsync_start = 1440 + 3,
+	.vsync_end = 1440 + 3 + 10,
+	.vtotal = 1440 + 3 + 10 + 27,
+	.vrefresh = 60,
+	.flags = DRM_MODE_FLAG_PHSYNC | DRM_MODE_FLAG_PVSYNC,
+};
+
+static const struct panel_desc innolux_tv123wam = {
+	.modes = &innolux_tv123wam_mode,
+	.num_modes = 1,
+	.bpc = 8,
+	.size = {
+		.width = 259,
+		.height = 173,
+	},
+};
+
 static const struct of_device_id platform_of_match[] = {
 	{
 		.compatible = "ampire,am-480272h3tmqw-t01h",
@@ -2142,6 +2166,9 @@  static void panel_simple_shutdown(struct device *dev)
 		.compatible = "winstar,wf35ltiacd",
 		.data = &winstar_wf35ltiacd,
 	}, {
+		.compatible = "innolux,tv123wam",
+		.data = &innolux_tv123wam,
+	}, {
 		/* sentinel */
 	}
 };