linux-kernel - Re: [PATCH v2] drm/panel: add lg4573 driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <55CD8825.80901@denx.de>
Date:	Fri, 14 Aug 2015 08:18:13 +0200
From:	Heiko Schocher <hs@...x.de>
To:	Thierry Reding <thierry.reding@...il.com>
CC:	linux-kernel@...r.kernel.org, David Airlie <airlied@...ux.ie>,
	dri-devel@...ts.freedesktop.org
Subject: Re: [PATCH v2] drm/panel: add lg4573 driver

Hello Thierry,

Am 13.08.2015 um 14:59 schrieb Thierry Reding:
> On Tue, Jun 09, 2015 at 07:51:22AM +0200, Heiko Schocher wrote:
>> Add support for LG LG4573 480x800 4,3" panel. the LG4573
>> is used on the LG LCD LB043WV2-SD01, an industrial 4.3" TFT
>> panel with SPI control interface.
>>
>> Signed-off-by: Heiko Schocher <hs@...x.de>
>>
>> ---
>>
>> Changes in v2:
>> - add comments from Thierry Reding:
>>    - fix some spelling issues
>>    - remove "power-on-delay"
>>    - remove display timings usage from DT, instead specify the
>>      timing in the driver.
>>    - change config symbol to DRM_PANEL_LG_LG4573
>>    - change filename to panel-lg-lg4573.c
>>    - get rid of power_on_delay
>>    - use cpu_to_be16()
>>    - rework lg4573_spi_write_u16_array()
>>    - introduce lg4573_spi_write_dcs()
>>    - reworked error reporting
>>
>>   .../devicetree/bindings/panel/lg,lg4573.txt        |  19 +
>>   drivers/gpu/drm/panel/Kconfig                      |   8 +
>>   drivers/gpu/drm/panel/Makefile                     |   1 +
>>   drivers/gpu/drm/panel/panel-lg-lg4573.c            | 382 +++++++++++++++++++++
>>   4 files changed, 410 insertions(+)
>>   create mode 100644 Documentation/devicetree/bindings/panel/lg,lg4573.txt
>>   create mode 100644 drivers/gpu/drm/panel/panel-lg-lg4573.c
>
> There were a couple of review comments that you missed, but I've

Hups ... It seems I missed them :-(

> addressed them while applying. Might be good to give it[0] a spin just
> to make sure I didn't accidentally break anything.

Uh, thank you! Just tested it on the aristainetos2 board, works fine.

bye,
Heiko
>
> Thierry
>
> [0]: git://anongit.freedesktop.org/tegra/linux#drm/panel/for-next
>

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/