On 01/03/2012 10:21 AM, Tomi Valkeinen wrote:
> On Tue, 2011-12-20 at 10:54 +0100, Daniel Mack wrote:
>> This patch adds support for Microtip Technologies' UMSH-8173MD 800x480
>> RGB TFT panel. Tested with an OMAP3 board.
>>
>> Signed-off-by: Daniel Mack <[email protected]>
>> Cc: Tomi Valkeinen <[email protected]>
>> Cc: Florian Tobias Schandinat <[email protected]>
>> ---
>>  drivers/video/omap2/displays/panel-generic-dpi.c |   25 
>> ++++++++++++++++++++++
>>  1 files changed, 25 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/video/omap2/displays/panel-generic-dpi.c 
>> b/drivers/video/omap2/displays/panel-generic-dpi.c
>> index 519c47d..cbcf164 100644
>> --- a/drivers/video/omap2/displays/panel-generic-dpi.c
>> +++ b/drivers/video/omap2/displays/panel-generic-dpi.c
>> @@ -297,6 +297,31 @@ static struct panel_config generic_dpi_panels[] = {
>>  
>>              .name                   = "apollon",
>>      },
>> +
>> +    /* Microtips Technologies - UMSH-8173MD */
>> +    {
>> +            {
>> +                    .x_res          = 800,
>> +                    .y_res          = 480,
>> +
>> +                    .pixel_clock    = 34560,
>> +
>> +                    .hsw            = 13,
>> +                    .hfp            = 101,
>> +                    .hbp            = 101,
>> +
>> +                    .vsw            = 23,
>> +                    .vfp            = 1,
>> +                    .vbp            = 1,
>> +            },
>> +            .acbi                   = 0x0,
>> +            .acb                    = 0x0,
>> +            .config                 = OMAP_DSS_LCD_TFT | OMAP_DSS_LCD_IVS |
>> +                                      OMAP_DSS_LCD_IHS | OMAP_DSS_LCD_IPC,
>> +            .power_on_delay         = 0,
>> +            .power_off_delay        = 0,
>> +            .name                   = "UMSH-8173MD",
> 
> Looks fine, except the panel name is not in line with the other panel
> names. I suggest "microtips_umsh_8173md". I can apply the patch and
> make the change if you are ok with the name.

Yes, please do! :)


Thanks,
Daniel
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to