2017-12-01 17:55:53

by Emil Velikov

[permalink] [raw]
Subject: Re: [PATCH] drm/panel: support Innolux P097PFG panel

On 30 November 2017 at 06:13, Lin Huang <[email protected]> wrote:
> Support Innolux P097PFG 9.7" 1536x2048 TFT LCD panel,
> it refactor Innolux P079ZCA panel driver, let it support
> multi panel, and add support P097PFG panel in this driver.
>
Couple of drive-by suggestions:

Split the refactor vs new hw?

> MODULE_DESCRIPTION("Innolux P079ZCA panel driver");
Update this reference alongside the ones in Kconfig?

-Emil


2017-12-04 01:06:08

by Lin Huang

[permalink] [raw]
Subject: Re: [PATCH] drm/panel: support Innolux P097PFG panel

Hi Emil,


On Saturday, December 02, 2017 01:55 AM, Emil Velikov wrote:
> On 30 November 2017 at 06:13, Lin Huang <[email protected]> wrote:
>> Support Innolux P097PFG 9.7" 1536x2048 TFT LCD panel,
>> it refactor Innolux P079ZCA panel driver, let it support
>> multi panel, and add support P097PFG panel in this driver.
>>
> Couple of drive-by suggestions:
>
> Split the refactor vs new hw?
Sorry,  i don't get what you mean.
>
>> MODULE_DESCRIPTION("Innolux P079ZCA panel driver");
> Update this reference alongside the ones in Kconfig?
Okay, will fix it next version.
>
> -Emil
>
>
>