Re: [PATCH 0/3] OMAP: DSS2: introduce dummy panel driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Nov 2, 2010 at 12:53 PM, Anand Gadiyar <gadiyar@xxxxxx> wrote:
> On 11/2/2010 9:43 PM, Bryan Wu wrote:
>>
>> There are 4 duplicated panel driver sin DSS2 display driver. They are
>> similar
>> and dummy panel driver can support all them with specific panel
>> configuration.
>> And new panel driver can be easily supported by adding panel
>> configurations
>> into dummy panel driver.
>>
>> This patchset introduces dummy panel driver, remove 3 panel drivers and
>> enable
>> dummy panel driver in board files. And it is based on 2.6.37-rc1.
>>
>> Building with omap2plus_defconfig is successful.
>>
>> Keep sharp_ls_panel, since it contains blacklight control code which will
>> be
>> moved out later.
>>
>> Bryan Wu (3):
>>   OMAP: DSS2: Add dummy panel display driver
>>   OMAP: DSS2: remove dummy panel driver duplicated panel drivers
>>   OMAP: use dummy panel driver in board files
>>
>>  arch/arm/mach-omap2/board-3430sdp.c                |    3 +-
>>  arch/arm/mach-omap2/board-am3517evm.c              |    6 +-
>>  arch/arm/mach-omap2/board-cm-t35.c                 |    6 +-
>>  arch/arm/mach-omap2/board-devkit8000.c             |    6 +-
>>  arch/arm/mach-omap2/board-igep0020.c               |    3 +-
>>  arch/arm/mach-omap2/board-omap3beagle.c            |    3 +-
>>  arch/arm/mach-omap2/board-omap3evm.c               |    3 +-
>>  arch/arm/mach-omap2/board-omap3stalker.c           |    6 +-
>>  arch/arm/plat-omap/include/plat/display.h          |   28 +-
>>  drivers/video/omap2/displays/Kconfig               |   22 +-
>>  drivers/video/omap2/displays/Makefile              |    4 +-
>>  drivers/video/omap2/displays/panel-dummy.c         |  195 +++++++
>>  drivers/video/omap2/displays/panel-dummy.h         |  119 +++++
>>  drivers/video/omap2/displays/panel-generic.c       |  174 -------
>>  .../video/omap2/displays/panel-sharp-lq043t1dg01.c |  165 ------
>>  .../video/omap2/displays/panel-tpo-td043mtea1.c    |  535
>> --------------------
>>  16 files changed, 364 insertions(+), 914 deletions(-)
>>  create mode 100644 drivers/video/omap2/displays/panel-dummy.c
>>  create mode 100644 drivers/video/omap2/displays/panel-dummy.h
>>  delete mode 100644 drivers/video/omap2/displays/panel-generic.c
>>  delete mode 100644 drivers/video/omap2/displays/panel-sharp-lq043t1dg01.c
>>  delete mode 100644 drivers/video/omap2/displays/panel-tpo-td043mtea1.c
>>
>
> Why can't panel-generic.c be reused instead of a new panel-dummy.c?
>
> Sounds like what you're trying to do is to get a generic panel driver; so
> shouldn't it at least be called generic instead of dummy.
>

I assume panel-generic is for some DVI output connection, so I picked
up dummy. But yeah, I do prefer to panel-generic driver and maybe
change the name "generic-panel" to "dvi".

-Bryan
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux