Re: [PATCH] drm: move allocation out of drm_get_format_name()
From: Christian KÃnig
Date: Sat Nov 05 2016 - 09:44:47 EST
Am 05.11.2016 um 02:33 schrieb Eric Engestrom:
+typedef char drm_format_name_buf[32];
Please don't use a typedef for this, just define the maximum size of
characters the function might write somewhere.
See the kernel coding style as well:
In general, a pointer, or a struct that has elements that can reasonably
be directly accessed should **never** be a typedef.
Regards,
Christian.