[PATCH] drm/plane: Fix a couple of checkpatch warnings

Matt Roper matthew.d.roper at intel.com
Tue May 13 08:32:31 PDT 2014


On Tue, May 13, 2014 at 04:58:35PM +0200, Thierry Reding wrote:
> From: Thierry Reding <treding at nvidia.com>
> 
> Code should be indented using tabs rather than spaces (see CodingStyle)
> and the canonical form to declare a constant static variable is using
> "static const" rather than "const static". Fixes the following warnings
> from checkpatch:
> 
> 	$ scripts/checkpatch.pl -f drivers/gpu/drm/drm_plane_helper.c
> 	WARNING: storage class should be at the beginning of the declaration
> 	#40: FILE: drivers/gpu/drm/drm_plane_helper.c:40:
> 	+const static uint32_t safe_modeset_formats[] = {
> 
> 	WARNING: please, no spaces at the start of a line
> 	#41: FILE: drivers/gpu/drm/drm_plane_helper.c:41:
> 	+       DRM_FORMAT_XRGB8888,$
> 
> 	WARNING: please, no spaces at the start of a line
> 	#42: FILE: drivers/gpu/drm/drm_plane_helper.c:42:
> 	+       DRM_FORMAT_ARGB8888,$
> 
> Signed-off-by: Thierry Reding <treding at nvidia.com>

Reviewed-by: Matt Roper <matthew.d.roper at intel.com>

> ---
>  drivers/gpu/drm/drm_plane_helper.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/drm_plane_helper.c b/drivers/gpu/drm/drm_plane_helper.c
> index 0b58dbc86707..97e5518ca15b 100644
> --- a/drivers/gpu/drm/drm_plane_helper.c
> +++ b/drivers/gpu/drm/drm_plane_helper.c
> @@ -37,9 +37,9 @@
>   * creating the primary plane.  However drivers that still call
>   * drm_plane_init() will use this minimal format list as the default.
>   */
> -const static uint32_t safe_modeset_formats[] = {
> -       DRM_FORMAT_XRGB8888,
> -       DRM_FORMAT_ARGB8888,
> +static const uint32_t safe_modeset_formats[] = {
> +	DRM_FORMAT_XRGB8888,
> +	DRM_FORMAT_ARGB8888,
>  };
>  
>  /*
> -- 
> 1.9.2
> 

-- 
Matt Roper
Graphics Software Engineer
IoTG Platform Enabling & Development
Intel Corporation
(916) 356-2795


More information about the dri-devel mailing list