From: Sean Paul <[email protected]>
For the warning
../drivers/gpu/drm/drm_gem_ttm_helper.c:26:20: warning: duplicate ‘const’
declaration specifier [-Wduplicate-decl-specifier]
Reading between the lines, I think the double const is to preserve both
the values and the pointers in the array (which makes total sense). This
patch moves the second const to achieve this (and fix the warning).
Fixes: ff540b76f14a ("drm/ttm: add drm gem ttm helpers, starting with
drm_gem_ttm_print_info()")
Cc: Gerd Hoffmann <[email protected]>
Cc: Thomas Zimmermann <[email protected]>
Cc: Daniel Vetter <[email protected]>
Cc: David Airlie <[email protected]>
Cc: Daniel Vetter <[email protected]>
Cc: Maarten Lankhorst <[email protected]>
Cc: Maxime Ripard <[email protected]>
Cc: Sean Paul <[email protected]>
Cc: [email protected]
Signed-off-by: Sean Paul <[email protected]>
---
drivers/gpu/drm/drm_gem_ttm_helper.c | 2 +-
drivers/gpu/drm/drm_print.c | 2 +-
include/drm/drm_print.h | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/drm_gem_ttm_helper.c
b/drivers/gpu/drm/drm_gem_ttm_helper.c
index 9a4bafcf20df..ae7ba0052959 100644
--- a/drivers/gpu/drm/drm_gem_ttm_helper.c
+++ b/drivers/gpu/drm/drm_gem_ttm_helper.c
@@ -23,7 +23,7 @@
void drm_gem_ttm_print_info(struct drm_printer *p, unsigned int indent,
const struct drm_gem_object *gem)
{
- static const char const *plname[] = {
+ static const char * const plname[] = {
[ TTM_PL_SYSTEM ] = "system",
[ TTM_PL_TT ] = "tt",
[ TTM_PL_VRAM ] = "vram",
diff --git a/drivers/gpu/drm/drm_print.c b/drivers/gpu/drm/drm_print.c
index dfa27367ebb8..20c49c85b7a2 100644
--- a/drivers/gpu/drm/drm_print.c
+++ b/drivers/gpu/drm/drm_print.c
@@ -198,7 +198,7 @@ EXPORT_SYMBOL(drm_printf);
* @to: end of bit range to print (exclusive).
*/
void drm_print_bits(struct drm_printer *p,
- unsigned long value, const char *bits[],
+ unsigned long value, const char * const bits[],
unsigned int from, unsigned int to)
{
bool first = true;
diff --git a/include/drm/drm_print.h b/include/drm/drm_print.h
index 12d4916254b4..1e6a4ee8a6b2 100644
--- a/include/drm/drm_print.h
+++ b/include/drm/drm_print.h
@@ -90,7 +90,7 @@ void drm_printf(struct drm_printer *p, const char *f, ...);
void drm_puts(struct drm_printer *p, const char *str);
void drm_print_regset32(struct drm_printer *p, struct debugfs_regset32
*regset);
void drm_print_bits(struct drm_printer *p,
- unsigned long value, const char *bits[],
+ unsigned long value, const char * const bits[],
unsigned int from, unsigned int to);
__printf(2, 0)
--
Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel
