hwmon: (f71882fg) Secure chip property definition arrays
authorJean Delvare <khali@linux-fr.org>
Sat, 26 Mar 2011 09:45:01 +0000 (10:45 +0100)
committerJean Delvare <khali@endymion.delvare>
Sat, 26 Mar 2011 09:45:01 +0000 (10:45 +0100)
Using C99-style array initialization will ensure definitions won't
drift if the chips enum gets new values added.

Signed-off-by: Jean Delvare <khali@linux-fr.org>
Acked-by: Hans de Goede <hdegoede@redhat.com>
Acked-by: Guenter Roeck <guenter.roeck@ericsson.com>
drivers/hwmon/f71882fg.c

index a4d430ee7e20e8696f75700664f572f211ffe5b3..0f60b058ff9d53980554bd4120d9babbe6711704 100644 (file)
@@ -119,37 +119,37 @@ static const char *f71882fg_names[] = {
        "f8000",
 };
 
-static const char f71882fg_has_in[8][F71882FG_MAX_INS] = {
-       { 1, 1, 1, 1, 1, 1, 0, 1, 1 }, /* f71808e */
-       { 1, 1, 1, 0, 0, 0, 0, 0, 0 }, /* f71858fg */
-       { 1, 1, 1, 1, 1, 1, 1, 1, 1 }, /* f71862fg */
-       { 1, 1, 1, 1, 1, 1, 1, 1, 1 }, /* f71869 */
-       { 1, 1, 1, 1, 1, 1, 1, 1, 1 }, /* f71882fg */
-       { 1, 1, 1, 1, 1, 1, 1, 1, 1 }, /* f71889fg */
-       { 1, 1, 1, 1, 1, 1, 1, 1, 1 }, /* f71889ed */
-       { 1, 1, 1, 0, 0, 0, 0, 0, 0 }, /* f8000 */
+static const char f71882fg_has_in[][F71882FG_MAX_INS] = {
+       [f71808e]       = { 1, 1, 1, 1, 1, 1, 0, 1, 1 },
+       [f71858fg]      = { 1, 1, 1, 0, 0, 0, 0, 0, 0 },
+       [f71862fg]      = { 1, 1, 1, 1, 1, 1, 1, 1, 1 },
+       [f71869]        = { 1, 1, 1, 1, 1, 1, 1, 1, 1 },
+       [f71882fg]      = { 1, 1, 1, 1, 1, 1, 1, 1, 1 },
+       [f71889fg]      = { 1, 1, 1, 1, 1, 1, 1, 1, 1 },
+       [f71889ed]      = { 1, 1, 1, 1, 1, 1, 1, 1, 1 },
+       [f8000]         = { 1, 1, 1, 0, 0, 0, 0, 0, 0 },
 };
 
-static const char f71882fg_has_in1_alarm[8] = {
-       0, /* f71808e */
-       0, /* f71858fg */
-       0, /* f71862fg */
-       0, /* f71869 */
-       1, /* f71882fg */
-       1, /* f71889fg */
-       1, /* f71889ed */
-       0, /* f8000 */
+static const char f71882fg_has_in1_alarm[] = {
+       [f71808e]       = 0,
+       [f71858fg]      = 0,
+       [f71862fg]      = 0,
+       [f71869]        = 0,
+       [f71882fg]      = 1,
+       [f71889fg]      = 1,
+       [f71889ed]      = 1,
+       [f8000]         = 0,
 };
 
-static const char f71882fg_has_beep[8] = {
-       0, /* f71808e */
-       0, /* f71858fg */
-       1, /* f71862fg */
-       1, /* f71869 */
-       1, /* f71882fg */
-       1, /* f71889fg */
-       1, /* f71889ed */
-       0, /* f8000 */
+static const char f71882fg_has_beep[] = {
+       [f71808e]       = 0,
+       [f71858fg]      = 0,
+       [f71862fg]      = 1,
+       [f71869]        = 1,
+       [f71882fg]      = 1,
+       [f71889fg]      = 1,
+       [f71889ed]      = 1,
+       [f8000]         = 0,
 };
 
 static struct platform_device *f71882fg_pdev;