Re: [PATCH 2/3] drm/panel: Add Ilitek ILI9322 driver

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

 



On Fri, Dec 01, 2017 at 05:16:58PM +0100, Linus Walleij wrote:
> This adds support for the Ilitek ILI9322 QVGA (320x240)
> TFT panel driver.
> 
> This panel driver supports serial or parallel RGB or
> YUV input and also ITU-T BT.656 input streams.
> 
> The controller is combined with a physical panel and
> configured through the device tree.
> 
> Cc: David Lechner <david@xxxxxxxxxxxxxx>
> Cc: Stefano Babic <sbabic@xxxxxxx>
> Cc: Ben Dooks <ben.dooks@xxxxxxxxxxxxxxx>
> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
> ---
> ChangeLog v1->v2:
> - Dropped all DT parsing code in favor of open-coding the
>   display config on a per-system basis based on system-specific
>   compatible strings, after feedback from the DT maintainers.
> - Define a set of configs for the D-Link DIR-685 router.
> - Tested on the D-Link DIR-685.
> ---
>  drivers/gpu/drm/panel/Kconfig                |   8 +
>  drivers/gpu/drm/panel/Makefile               |   1 +
>  drivers/gpu/drm/panel/panel-ilitek-ili9322.c | 962 +++++++++++++++++++++++++++
>  3 files changed, 971 insertions(+)
>  create mode 100644 drivers/gpu/drm/panel/panel-ilitek-ili9322.c

checkpatch.pl gives me these:

	-:30: WARNING: please write a paragraph that describes the config symbol fully
	#30: FILE: drivers/gpu/drm/panel/Kconfig:31:
	+config DRM_PANEL_ILITEK_IL9322

	-:54: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
	#54:
	new file mode 100644

	-:130: CHECK: Prefer using the BIT macro
	#130: FILE: drivers/gpu/drm/panel/panel-ilitek-ili9322.c:72:
	+#define ILI9322_ENTRY_PAL              (1 << 2)

	-:134: CHECK: Prefer using the BIT macro
	#134: FILE: drivers/gpu/drm/panel/panel-ilitek-ili9322.c:76:
	+#define ILI9322_ENTRY_SERIAL_RGB_ALIGNED (1 << 4)

	-:196: CHECK: spaces preferred around that '|' (ctx:VxV)
	#196: FILE: drivers/gpu/drm/panel/panel-ilitek-ili9322.c:138:
	+#define ILI9322_IF_CTRL_SYNC_DISABLED  (BIT(2)|BIT(3))
						       ^

	-:551: WARNING: msleep < 20ms can sleep for up to 20ms; see Documentation/timers/timers-howto.txt
	#551: FILE: drivers/gpu/drm/panel/panel-ilitek-ili9322.c:493:
	+       msleep(10);

	total: 0 errors, 3 warnings, 3 checks, 983 lines checked

I'd like to see at least the warnings fixed. You can probably skip the
entry in MAINTAINERS, though. I don't mind fixing these up while
applying, but I don't know what to put in Kconfig. If you can send out
a short paragraph for me to include I'll add it when I apply.

Thierry

Attachment: signature.asc
Description: PGP signature

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux