From c09ce4b8e53ad6a122d64f8729995aa584825c76 Mon Sep 17 00:00:00 2001 From: Joel Porquet Date: Mon, 26 May 2014 18:07:38 +0200 Subject: [PATCH] staging: silicom/bypasslib/bp_ioctl.h: Fix erroneous global variable definitions Originally, this header was defining two new typedefs, CMND_TYPE and CMND_TYPE_SD, following this type of declaration: typedef enum { ... } CMND_TYPE; A previous commit (8dc3eb66177af5c442a505f60c63f5cad37caa55) tried to fix warnings that were pointed out by checkpatch.pl, concerning not adding new typedefs. But this commit only removed the 'typedef' keyword, thus transforming both the typedefs into two the definition of global variables. For example: enum { ... } CMND_TYPE; As noticed by the Sparse tool, this patch removes those erroneous global variable definitions, and just leaves anonymous enum type definitions: enum { ... }; Signed-off-by: Joel Porquet Signed-off-by: Greg Kroah-Hartman --- drivers/staging/silicom/bypasslib/bp_ioctl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/silicom/bypasslib/bp_ioctl.h b/drivers/staging/silicom/bypasslib/bp_ioctl.h index bf47f786866ba..a13932af733b9 100644 --- a/drivers/staging/silicom/bypasslib/bp_ioctl.h +++ b/drivers/staging/silicom/bypasslib/bp_ioctl.h @@ -101,7 +101,7 @@ enum { SET_BP_WAIT_AT_PWUP, GET_BP_HW_RESET, SET_BP_HW_RESET, -} CMND_TYPE; +}; enum { IF_SCAN_SD, @@ -154,7 +154,7 @@ enum { GET_BP_HW_RESET_SD, SET_BP_HW_RESET_SD, -} CMND_TYPE_SD; +}; #define SIOCGIFBYPASS (SIOCDEVPRIVATE+10) -- 2.39.5