hwmon: cleanup non-bool "valid" data fields
authorPaul Fertser <fercerpav@gmail.com>
Fri, 24 Sep 2021 19:52:02 +0000 (22:52 +0300)
committerGuenter Roeck <linux@roeck-us.net>
Tue, 12 Oct 2021 14:22:41 +0000 (07:22 -0700)
We have bool so use it consistently in all the drivers.

The following Coccinelle script was used:

@@
identifier T;
type t = { char, int };
@@
struct T {
...
- t valid;
+ bool valid;
...
}

@@
identifier v;
@@
(
- v->valid = 0
+ v->valid = false
|
- v->valid = 1
+ v->valid = true
)

followed by sed to fixup the comments:
sed '/bool valid;/{s/!=0/true/;s/zero/false/}'

Few whitespace changes were fixed manually. All modified drivers were
compile-tested.

Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Link: https://lore.kernel.org/r/20210924195202.27917-1-fercerpav@gmail.com
[groeck: Fixed up 'u8 valid' to 'boool valid' in atxp1.c]
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
69 files changed:
drivers/hwmon/abituguru3.c
drivers/hwmon/ad7414.c
drivers/hwmon/ad7418.c
drivers/hwmon/adm1021.c
drivers/hwmon/adm1025.c
drivers/hwmon/adm1026.c
drivers/hwmon/adm1029.c
drivers/hwmon/adm1031.c
drivers/hwmon/amc6821.c
drivers/hwmon/applesmc.c
drivers/hwmon/asb100.c
drivers/hwmon/asc7621.c
drivers/hwmon/atxp1.c
drivers/hwmon/coretemp.c
drivers/hwmon/dme1737.c
drivers/hwmon/ds1621.c
drivers/hwmon/ds620.c
drivers/hwmon/emc6w201.c
drivers/hwmon/f71805f.c
drivers/hwmon/f71882fg.c
drivers/hwmon/f75375s.c
drivers/hwmon/fschmd.c
drivers/hwmon/g760a.c
drivers/hwmon/gl518sm.c
drivers/hwmon/gl520sm.c
drivers/hwmon/ibmaem.c
drivers/hwmon/ibmpex.c
drivers/hwmon/it87.c
drivers/hwmon/lineage-pem.c
drivers/hwmon/lm63.c
drivers/hwmon/lm77.c
drivers/hwmon/lm78.c
drivers/hwmon/lm80.c
drivers/hwmon/lm83.c
drivers/hwmon/lm85.c
drivers/hwmon/lm87.c
drivers/hwmon/lm92.c
drivers/hwmon/lm93.c
drivers/hwmon/lm95241.c
drivers/hwmon/ltc4151.c
drivers/hwmon/ltc4215.c
drivers/hwmon/ltc4261.c
drivers/hwmon/max16065.c
drivers/hwmon/max1619.c
drivers/hwmon/max1668.c
drivers/hwmon/max6639.c
drivers/hwmon/max6642.c
drivers/hwmon/pc87360.c
drivers/hwmon/sch5636.c
drivers/hwmon/sht21.c
drivers/hwmon/sis5595.c
drivers/hwmon/smm665.c
drivers/hwmon/smsc47b397.c
drivers/hwmon/smsc47m192.c
drivers/hwmon/thmc50.c
drivers/hwmon/tmp401.c
drivers/hwmon/tmp421.c
drivers/hwmon/via686a.c
drivers/hwmon/vt1211.c
drivers/hwmon/vt8231.c
drivers/hwmon/w83627ehf.c
drivers/hwmon/w83627hf.c
drivers/hwmon/w83781d.c
drivers/hwmon/w83791d.c
drivers/hwmon/w83792d.c
drivers/hwmon/w83793.c
drivers/hwmon/w83795.c
drivers/hwmon/w83l785ts.c
drivers/hwmon/w83l786ng.c

index 112dd0d9377c764f940d2f2377d35835c2401520..8229ad30c9090d8babf3aabdfa8219b7f682aa0e 100644 (file)
@@ -145,7 +145,7 @@ struct abituguru3_data {
        struct device *hwmon_dev;       /* hwmon registered device */
        struct mutex update_lock;       /* protect access to data and uGuru */
        unsigned short addr;            /* uguru base address */
-       char valid;                     /* !=0 if following fields are valid */
+       bool valid;                     /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        /*
@@ -1083,7 +1083,7 @@ static struct abituguru3_data *abituguru3_update_device(struct device *dev)
        mutex_lock(&data->update_lock);
        if (!data->valid || time_after(jiffies, data->last_updated + HZ)) {
                /* Clear data->valid while updating */
-               data->valid = 0;
+               data->valid = false;
                /* Read alarms */
                if (abituguru3_read_increment_offset(data,
                                ABIT_UGURU3_SETTINGS_BANK,
@@ -1117,7 +1117,7 @@ static struct abituguru3_data *abituguru3_update_device(struct device *dev)
                                goto LEAVE_UPDATE;
                }
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 LEAVE_UPDATE:
        mutex_unlock(&data->update_lock);
index 6a765755d0616510af9ff9285a8eefb47d5151f4..0afb89c4629db8f52a55a3e59f8cb0a80d2b960b 100644 (file)
@@ -37,7 +37,7 @@ static u8 AD7414_REG_LIMIT[] = { AD7414_REG_T_HIGH, AD7414_REG_T_LOW };
 struct ad7414_data {
        struct i2c_client       *client;
        struct mutex            lock;   /* atomic read data updates */
-       char                    valid;  /* !=0 if following fields are valid */
+       bool                    valid;  /* true if following fields are valid */
        unsigned long           next_update;    /* In jiffies */
        s16                     temp_input;     /* Register values */
        s8                      temps[ARRAY_SIZE(AD7414_REG_LIMIT)];
@@ -95,7 +95,7 @@ static struct ad7414_data *ad7414_update_device(struct device *dev)
                }
 
                data->next_update = jiffies + HZ + HZ / 2;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->lock);
index d618f6b2f3820b1fd57ba2855c5d800b181b09a7..22bdb7e5b9e0344091c345ed4b458a3de9727c18 100644 (file)
@@ -46,7 +46,7 @@ struct ad7418_data {
        enum chips              type;
        struct mutex            lock;
        int                     adc_max;        /* number of ADC channels */
-       char                    valid;
+       bool                    valid;
        unsigned long           last_updated;   /* In jiffies */
        s16                     temp[3];        /* Register values */
        u16                     in[4];
@@ -111,14 +111,14 @@ static int ad7418_update_device(struct device *dev)
                        goto abort;
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->lock);
        return 0;
 
 abort:
-       data->valid = 0;
+       data->valid = false;
        mutex_unlock(&data->lock);
        return val;
 }
index 71deb2cd20f558678acb850fc021d80cf831d6d6..38b447c6e8cdb1210c99f36cd9feefb4bf4a7c41 100644 (file)
@@ -72,7 +72,7 @@ struct adm1021_data {
        const struct attribute_group *groups[3];
 
        struct mutex update_lock;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        char low_power;         /* !=0 if device in low power mode */
        unsigned long last_updated;     /* In jiffies */
 
@@ -135,7 +135,7 @@ static struct adm1021_data *adm1021_update_device(struct device *dev)
                                                ADM1023_REG_REM_OFFSET_PREC);
                }
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index de51e01c061bae74744b2cccc2227320c2c849c2..4352f6a884e8f3eec93dd324336a50c4c021dd84 100644 (file)
@@ -97,7 +97,7 @@ struct adm1025_data {
        struct i2c_client *client;
        const struct attribute_group *groups[3];
        struct mutex update_lock;
-       char valid; /* zero until following fields are valid */
+       bool valid; /* false until following fields are valid */
        unsigned long last_updated; /* in jiffies */
 
        u8 in[6];               /* register value */
@@ -148,7 +148,7 @@ static struct adm1025_data *adm1025_update_device(struct device *dev)
                              ADM1025_REG_VID4) & 0x01) << 4);
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 49cefbadb156a1dcf8984459372089ba8eecfc27..69b3ec752944744aa0e7deacad484b9eacaf65a7 100644 (file)
@@ -259,7 +259,7 @@ struct adm1026_data {
        const struct attribute_group *groups[3];
 
        struct mutex update_lock;
-       int valid;              /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_reading;     /* In jiffies */
        unsigned long last_config;      /* In jiffies */
 
@@ -459,7 +459,7 @@ static struct adm1026_data *adm1026_update_device(struct device *dev)
                data->last_config = jiffies;
        }       /* last_config */
 
-       data->valid = 1;
+       data->valid = true;
        mutex_unlock(&data->update_lock);
        return data;
 }
index 50b1df7b008c2db22a1a0c1c4db4f0c68286a719..3e1999413f3252128d3caeecd251ba396fececf6 100644 (file)
@@ -99,7 +99,7 @@ static const u8 ADM1029_REG_FAN_DIV[] = {
 struct adm1029_data {
        struct i2c_client *client;
        struct mutex update_lock; /* protect register access */
-       char valid;             /* zero until following fields are valid */
+       bool valid;             /* false until following fields are valid */
        unsigned long last_updated;     /* in jiffies */
 
        /* registers values, signed for temperature, unsigned for other stuff */
@@ -143,7 +143,7 @@ static struct adm1029_data *adm1029_update_device(struct device *dev)
                }
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index b538ace2d2924da6e77d51484fbbded84af16666..257ec53ae72302292b5f7eabad138d0bcdcc299a 100644 (file)
@@ -65,7 +65,7 @@ struct adm1031_data {
        const struct attribute_group *groups[3];
        struct mutex update_lock;
        int chip_type;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
        unsigned int update_interval;   /* In milliseconds */
        /*
@@ -187,7 +187,7 @@ static struct adm1031_data *adm1031_update_device(struct device *dev)
                                        ADM1031_REG_PWM) >> (4 * chan)) & 0x0f;
                }
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
@@ -650,7 +650,7 @@ static ssize_t fan_div_store(struct device *dev,
                            data->fan_min[nr]);
 
        /* Invalidate the cache: fan speed is no longer valid */
-       data->valid = 0;
+       data->valid = false;
        mutex_unlock(&data->update_lock);
        return count;
 }
index 6b1ce2242c6187619e49c4cff77c8f5779c88a83..0c16face3fd3a686283de57167da5d834e509b63 100644 (file)
@@ -141,7 +141,7 @@ static const u8 fan_reg_hi[] = {AMC6821_REG_TDATA_HI,
 struct amc6821_data {
        struct i2c_client *client;
        struct mutex update_lock;
-       char valid; /* zero until following fields are valid */
+       bool valid; /* false until following fields are valid */
        unsigned long last_updated; /* in jiffies */
 
        /* register values */
@@ -258,7 +258,7 @@ static struct amc6821_data *amc6821_update_device(struct device *dev)
                }
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
        mutex_unlock(&data->update_lock);
        return data;
@@ -511,7 +511,7 @@ static ssize_t temp_auto_point_temp_store(struct device *dev,
        }
 
        mutex_lock(&data->update_lock);
-       data->valid = 0;
+       data->valid = false;
 
        switch (ix) {
        case 0:
@@ -584,7 +584,7 @@ static ssize_t pwm1_auto_point_pwm_store(struct device *dev,
        }
 
 EXIT:
-       data->valid = 0;
+       data->valid = false;
        mutex_unlock(&data->update_lock);
        return count;
 }
index c31759794a29ad22afa5aaf65290292a0f30b8bf..fc6d6a9053cee84e4cf9d4fe3e1f3c8c08bf2b0f 100644 (file)
@@ -391,7 +391,7 @@ static const struct applesmc_entry *applesmc_get_entry_by_index(int index)
        cache->len = info[0];
        memcpy(cache->type, &info[1], 4);
        cache->flags = info[5];
-       cache->valid = 1;
+       cache->valid = true;
 
 out:
        mutex_unlock(&smcreg.mutex);
index ba9fcf6f9264c949b0bbb07c01385a1f23a61cb9..8cf0bcb85eb4e31f55f8b5ff527024e15678588e 100644 (file)
@@ -186,7 +186,7 @@ struct asb100_data {
        /* array of 2 pointers to subclients */
        struct i2c_client *lm75[2];
 
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        u8 in[7];               /* Register value */
        u8 in_max[7];           /* Register value */
        u8 in_min[7];           /* Register value */
@@ -993,7 +993,7 @@ static struct asb100_data *asb100_update_device(struct device *dev)
                        (asb100_read_value(client, ASB100_REG_ALARM2) << 8);
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
 
                dev_dbg(&client->dev, "... device update complete\n");
        }
index 600ffc7e1900c04ea1525ac523f11998b48e641c..e835605a7456991a12dd4fff1fbf8ee8e870ff1c 100644 (file)
@@ -77,7 +77,7 @@ struct asc7621_data {
        struct i2c_client client;
        struct device *class_dev;
        struct mutex update_lock;
-       int valid;              /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_high_reading;        /* In jiffies */
        unsigned long last_low_reading;         /* In jiffies */
        /*
@@ -1032,7 +1032,7 @@ static struct asc7621_data *asc7621_update_device(struct device *dev)
                data->last_low_reading = jiffies;
        }                       /* last_reading */
 
-       data->valid = 1;
+       data->valid = true;
 
        mutex_unlock(&data->update_lock);
 
index 1e08a5431f128183b29f4ae5eec3fd8302602468..4fd8de8022bc7237dce179d317623d5c6acc8f7b 100644 (file)
@@ -37,7 +37,7 @@ struct atxp1_data {
        struct i2c_client *client;
        struct mutex update_lock;
        unsigned long last_updated;
-       u8 valid;
+       bool valid;
        struct {
                u8 vid;         /* VID output register */
                u8 cpu_vid; /* VID input from CPU */
@@ -63,7 +63,7 @@ static struct atxp1_data *atxp1_update_device(struct device *dev)
                data->reg.gpio1 = i2c_smbus_read_byte_data(client, ATXP1_GPIO1);
                data->reg.gpio2 = i2c_smbus_read_byte_data(client, ATXP1_GPIO2);
 
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
@@ -136,7 +136,7 @@ static ssize_t cpu0_vid_store(struct device *dev,
                                                ATXP1_VID, cvid | ATXP1_VIDENA);
        }
 
-       data->valid = 0;
+       data->valid = false;
 
        return count;
 }
@@ -180,7 +180,7 @@ static ssize_t gpio1_store(struct device *dev, struct device_attribute *attr,
 
                i2c_smbus_write_byte_data(client, ATXP1_GPIO1, value);
 
-               data->valid = 0;
+               data->valid = false;
        }
 
        return count;
@@ -224,7 +224,7 @@ static ssize_t gpio2_store(struct device *dev, struct device_attribute *attr,
 
                i2c_smbus_write_byte_data(client, ATXP1_GPIO2, value);
 
-               data->valid = 0;
+               data->valid = false;
        }
 
        return count;
index bb9211215a688f8eb06ae11b94a58ae68e368cc9..ccf0af5b988a7c37fa91e8ed2361ac83689c67b3 100644 (file)
@@ -167,7 +167,7 @@ static ssize_t show_temp(struct device *dev,
                 * really help at all.
                 */
                tdata->temp = tdata->tjmax - ((eax >> 16) & 0x7f) * 1000;
-               tdata->valid = 1;
+               tdata->valid = true;
                tdata->last_updated = jiffies;
        }
 
index c1e4cfb40c3def02d7e464661c6fb4ee4cffc6b4..e3ad4c2d0038c8225ef7c54551995e496390d637 100644 (file)
@@ -203,7 +203,7 @@ struct dme1737_data {
        unsigned int addr;              /* for ISA devices only */
 
        struct mutex update_lock;
-       int valid;                      /* !=0 if following fields are valid */
+       bool valid;                     /* true if following fields are valid */
        unsigned long last_update;      /* in jiffies */
        unsigned long last_vbat;        /* in jiffies */
        enum chips type;
@@ -778,7 +778,7 @@ static struct dme1737_data *dme1737_update_device(struct device *dev)
                }
 
                data->last_update = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index bf1c4b7ecb404a8223a5aff4085614be34eb3cc1..0886abf6ebab1459dfa4a58b16ad95515016f7bc 100644 (file)
@@ -109,7 +109,7 @@ static const u8 DS1621_REG_TEMP[3] = {
 struct ds1621_data {
        struct i2c_client *client;
        struct mutex update_lock;
-       char valid;                     /* !=0 if following fields are valid */
+       bool valid;                     /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
        enum chips kind;                /* device type */
 
@@ -213,7 +213,7 @@ static struct ds1621_data *ds1621_update_client(struct device *dev)
                                                  new_conf);
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 9ec722798c4a8504eb0189b76f04a398a286919c..82d7c3d58f49bd7020557c9ece7e96005370c8c9 100644 (file)
@@ -56,7 +56,7 @@ static const u8 DS620_REG_TEMP[3] = {
 struct ds620_data {
        struct i2c_client *client;
        struct mutex update_lock;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        s16 temp[3];            /* Register values, word */
@@ -118,7 +118,7 @@ static struct ds620_data *ds620_update_client(struct device *dev)
                }
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 abort:
        mutex_unlock(&data->update_lock);
index ec5c98702bf547dab43229e5a7e89c12e6408ee6..29082c8463f4984fe31ef3a77c055c78a3557566 100644 (file)
@@ -45,7 +45,7 @@ enum subfeature { input, min, max };
 struct emc6w201_data {
        struct i2c_client *client;
        struct mutex update_lock;
-       char valid; /* zero until following fields are valid */
+       bool valid; /* false until following fields are valid */
        unsigned long last_updated; /* in jiffies */
 
        /* registers values */
@@ -162,7 +162,7 @@ static struct emc6w201_data *emc6w201_update_device(struct device *dev)
                }
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 67b47de8263afb930cb30407c1fff3ad2af9d2fd..7f20edb0677ccc102840eb72c90675f57ade210e 100644 (file)
@@ -165,7 +165,7 @@ struct f71805f_data {
        struct device *hwmon_dev;
 
        struct mutex update_lock;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
        unsigned long last_limits;      /* In jiffies */
 
@@ -404,7 +404,7 @@ static struct f71805f_data *f71805f_update_device(struct device *dev)
                        + (f71805f_read8(data, F71805F_REG_STATUS(2)) << 16);
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 4dec793fd07d576caa8c9ae5b4ff1b38ad9a6f1b..4673d403759a526b33f30a2d233bdb2de4393778 100644 (file)
@@ -253,7 +253,7 @@ struct f71882fg_data {
 
        struct mutex update_lock;
        int temp_start;                 /* temp numbering start (0 or 1) */
-       char valid;                     /* !=0 if following fields are valid */
+       bool valid;                     /* true if following fields are valid */
        char auto_point_temp_signed;
        unsigned long last_updated;     /* In jiffies */
        unsigned long last_limits;      /* In jiffies */
@@ -1359,7 +1359,7 @@ static struct f71882fg_data *f71882fg_update_device(struct device *dev)
                                                        F71882FG_REG_IN(nr));
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 3e567be60fb158e9d2763468235ae2410337cbc3..57c8a473698d20968fc9f104e4be7645aa272d80 100644 (file)
@@ -85,7 +85,7 @@ struct f75375_data {
        const char *name;
        int kind;
        struct mutex update_lock; /* protect register access */
-       char valid;
+       bool valid;
        unsigned long last_updated;     /* In jiffies */
        unsigned long last_limits;      /* In jiffies */
 
@@ -228,7 +228,7 @@ static struct f75375_data *f75375_update_device(struct device *dev)
                                f75375_read8(client, F75375_REG_VOLT(nr));
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 5191cd85a8d13cacd1f26c5c593810acbe2f5c71..c26195e3aad78076c5c471427183de70ab7e953d 100644 (file)
@@ -264,7 +264,7 @@ struct fschmd_data {
        unsigned long watchdog_is_open;
        char watchdog_expect_close;
        char watchdog_name[10]; /* must be unique to avoid sysfs conflict */
-       char valid; /* zero until following fields are valid */
+       bool valid; /* false until following fields are valid */
        unsigned long last_updated; /* in jiffies */
 
        /* register values */
@@ -1356,7 +1356,7 @@ static struct fschmd_data *fschmd_update_device(struct device *dev)
                                               FSCHMD_REG_VOLT[data->kind][i]);
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index a692f7b2f6f7dfbcf15f318c18020cad462231e0..36717b524dbd5f1447dcadebd24c30954a3aa43b 100644 (file)
@@ -95,7 +95,7 @@ static struct g760a_data *g760a_update_client(struct device *dev)
                data->fan_sta = g760a_read_value(client, G760A_REG_FAN_STA);
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 7aaee5a48243baa93e025615a5dc415936bf65ab..dd683b0a648f9a6d8526d1866e037f109f33b6c3 100644 (file)
@@ -107,7 +107,7 @@ struct gl518_data {
        enum chips type;
 
        struct mutex update_lock;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        u8 voltage_in[4];       /* Register values; [0] = VDD */
@@ -211,7 +211,7 @@ static struct gl518_data *gl518_update_device(struct device *dev)
                    gl518_read_value(client, GL518_REG_VIN3);
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 4ae1295cc3ea7a2b354d081182af4d1bc3be018e..096ba97972115477dce57c5f149aa2b1dd007025 100644 (file)
@@ -64,7 +64,7 @@ struct gl520_data {
        struct i2c_client *client;
        const struct attribute_group *groups[3];
        struct mutex update_lock;
-       char valid;             /* zero until the following fields are valid */
+       bool valid;             /* false until the following fields are valid */
        unsigned long last_updated;     /* in jiffies */
 
        u8 vid;
@@ -174,7 +174,7 @@ static struct gl520_data *gl520_update_device(struct device *dev)
                }
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index a4ec85207782d87dc8c97e51cec8b15ceafd4a8c..de6baf6ca3d1e007debad7768cc1df82885b5f45 100644 (file)
@@ -127,7 +127,7 @@ struct aem_data {
        struct device           *hwmon_dev;
        struct platform_device  *pdev;
        struct mutex            lock;
-       char                    valid;
+       bool                    valid;
        unsigned long           last_updated;   /* In jiffies */
        u8                      ver_major;
        u8                      ver_minor;
index b2ab83c9fd9a8c54f1aa36c4f7d264673f92aad5..f6ec165c0fa8b6bdbe8986bbcb7023b8342361b3 100644 (file)
@@ -66,7 +66,7 @@ struct ibmpex_bmc_data {
        struct device           *hwmon_dev;
        struct device           *bmc_device;
        struct mutex            lock;
-       char                    valid;
+       bool                    valid;
        unsigned long           last_updated;   /* In jiffies */
 
        struct ipmi_addr        address;
@@ -239,7 +239,7 @@ static void ibmpex_update_device(struct ibmpex_bmc_data *data)
        }
 
        data->last_updated = jiffies;
-       data->valid = 1;
+       data->valid = true;
 
 out:
        mutex_unlock(&data->lock);
index 1f93134afcb9f90351ea97324d82bde0bf150251..0e543dbe0a6b3ece7c81c690a405432740350cd5 100644 (file)
@@ -519,7 +519,7 @@ struct it87_data {
        unsigned short addr;
        const char *name;
        struct mutex update_lock;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        u16 in_scaled;          /* Internal voltage sensors are scaled */
@@ -844,7 +844,7 @@ static struct it87_data *it87_update_device(struct device *dev)
                        data->vid &= 0x3f;
                }
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
@@ -980,7 +980,7 @@ static ssize_t set_temp(struct device *dev, struct device_attribute *attr,
                        regval |= 0x80;
                        it87_write_value(data, IT87_REG_BEEP_ENABLE, regval);
                }
-               data->valid = 0;
+               data->valid = false;
                reg = IT87_REG_TEMP_OFFSET[nr];
                break;
        }
@@ -1079,7 +1079,7 @@ static ssize_t set_temp_type(struct device *dev, struct device_attribute *attr,
        it87_write_value(data, IT87_REG_TEMP_ENABLE, data->sensor);
        if (has_temp_old_peci(data, nr))
                it87_write_value(data, IT87_REG_TEMP_EXTRA, data->extra);
-       data->valid = 0;        /* Force cache refresh */
+       data->valid = false;    /* Force cache refresh */
        mutex_unlock(&data->update_lock);
        return count;
 }
@@ -1834,7 +1834,7 @@ static ssize_t clear_intrusion(struct device *dev,
                config |= BIT(5);
                it87_write_value(data, IT87_REG_CONFIG, config);
                /* Invalidate cache to force re-read */
-               data->valid = 0;
+               data->valid = false;
        }
        mutex_unlock(&data->update_lock);
 
@@ -3229,7 +3229,7 @@ static int __maybe_unused it87_resume(struct device *dev)
        it87_start_monitoring(data);
 
        /* force update */
-       data->valid = 0;
+       data->valid = false;
 
        mutex_unlock(&data->update_lock);
 
index 1109fffa76fb08954ed654004123c2ddfc442049..ef5a49cd91498aa49e84bd19821cdf803808b99f 100644 (file)
@@ -191,7 +191,7 @@ static struct pem_data *pem_update_device(struct device *dev)
                i2c_smbus_write_byte(client, PEM_CLEAR_INFO_FLAGS);
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 abort:
        mutex_unlock(&data->update_lock);
index c8f93c5d1ccc1ce8eda7368b87d881ff02107473..339a145afc093c06247818ce0277b217f6b6fe16 100644 (file)
@@ -139,7 +139,7 @@ struct lm63_data {
        struct i2c_client *client;
        struct mutex update_lock;
        const struct attribute_group *groups[5];
-       char valid; /* zero until following fields are valid */
+       bool valid; /* false until following fields are valid */
        char lut_valid; /* zero until lut fields are valid */
        unsigned long last_updated; /* in jiffies */
        unsigned long lut_last_updated; /* in jiffies */
@@ -289,7 +289,7 @@ static struct lm63_data *lm63_update_device(struct device *dev)
                               LM63_REG_ALERT_STATUS) & 0x7F;
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        lm63_update_lut(data);
@@ -714,7 +714,7 @@ static ssize_t temp2_type_store(struct device *dev,
        reg = i2c_smbus_read_byte_data(client, LM96163_REG_TRUTHERM) & ~0x02;
        i2c_smbus_write_byte_data(client, LM96163_REG_TRUTHERM,
                                  reg | (data->trutherm ? 0x02 : 0x00));
-       data->valid = 0;
+       data->valid = false;
        mutex_unlock(&data->update_lock);
 
        return count;
index 7570c9d50ddcaeb2fb8677b170a654015814aac1..df6af85e170aa838380ad49d9e5caacea0db662c 100644 (file)
@@ -55,7 +55,7 @@ static const u8 temp_regs[t_num_temp] = {
 struct lm77_data {
        struct i2c_client       *client;
        struct mutex            update_lock;
-       char                    valid;
+       bool                    valid;
        unsigned long           last_updated;   /* In jiffies */
        int                     temp[t_num_temp]; /* index using temp_index */
        u8                      alarms;
@@ -118,7 +118,7 @@ static struct lm77_data *lm77_update_device(struct device *dev)
                data->alarms =
                        lm77_read_value(client, LM77_REG_TEMP) & 0x0007;
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 1aa35ca0c6fed7340578820bdeb0691a2f2b63e5..5e129cbec1cbbd90948c08ce87891a8ad0525054 100644 (file)
@@ -117,7 +117,7 @@ struct lm78_data {
        int isa_addr;
 
        struct mutex update_lock;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        u8 in[7];               /* Register value */
@@ -772,7 +772,7 @@ static struct lm78_data *lm78_update_device(struct device *dev)
                data->alarms = lm78_read_value(data, LM78_REG_ALARM1) +
                    (lm78_read_value(data, LM78_REG_ALARM2) << 8);
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
 
                data->fan_div[2] = 1;
        }
index 97ab491d2922cf0c81611a50ac3f09b221f0f944..e85e062bbf327dbca2d4e41225cd1515e99b17bd 100644 (file)
@@ -117,7 +117,7 @@ struct lm80_data {
        struct i2c_client *client;
        struct mutex update_lock;
        char error;             /* !=0 if error occurred during last update */
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        u8 in[i_num_in][7];     /* Register value, 1st index is enum in_index */
@@ -236,14 +236,14 @@ static struct lm80_data *lm80_update_device(struct device *dev)
                data->alarms = prev_rv + (rv << 8);
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
                data->error = 0;
        }
        goto done;
 
 abort:
        ret = ERR_PTR(rv);
-       data->valid = 0;
+       data->valid = false;
        data->error = 1;
 
 done:
index 2ff5ecce608e3ea9f4e269b6989b28d06a327a01..74fd7aa373a3eea3ec642b0472637b9da1f37679 100644 (file)
@@ -105,7 +105,7 @@ struct lm83_data {
        struct i2c_client *client;
        const struct attribute_group *groups[3];
        struct mutex update_lock;
-       char valid; /* zero until following fields are valid */
+       bool valid; /* false until following fields are valid */
        unsigned long last_updated; /* in jiffies */
 
        /* registers values */
@@ -137,7 +137,7 @@ static struct lm83_data *lm83_update_device(struct device *dev)
                    << 8);
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index c7bf5de7b70f05c4e27190aaf75dbada38d75152..88cf2012d34b415911f673b14d3dc97b31f05cb1 100644 (file)
@@ -294,7 +294,7 @@ struct lm85_data {
        bool has_vid5;  /* true if VID5 is configured for ADT7463 or ADT7468 */
 
        struct mutex update_lock;
-       int valid;              /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_reading;     /* In jiffies */
        unsigned long last_config;      /* In jiffies */
 
@@ -541,7 +541,7 @@ static struct lm85_data *lm85_update_device(struct device *dev)
                data->last_config = jiffies;
        }  /* last_config */
 
-       data->valid = 1;
+       data->valid = true;
 
        mutex_unlock(&data->update_lock);
 
index b2d820125bb6704d07733a37baeb6c4d3115fd9b..1750bc588856c22011012cb07c325d05d4b77e11 100644 (file)
@@ -141,7 +141,7 @@ static u8 LM87_REG_TEMP_LOW[3] = { 0x3A, 0x38, 0x2C };
 
 struct lm87_data {
        struct mutex update_lock;
-       char valid; /* zero until following fields are valid */
+       bool valid; /* false until following fields are valid */
        unsigned long last_updated; /* In jiffies */
 
        u8 channel;             /* register value */
@@ -251,7 +251,7 @@ static struct lm87_data *lm87_update_device(struct device *dev)
                data->aout = lm87_read_value(client, LM87_REG_AOUT);
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 9bf278cf0bd03b705d1236c11a33fa2167891673..5bae6eedcaf1f8c0f426c4e2d4c3bbac3248a331 100644 (file)
@@ -99,7 +99,7 @@ static const u8 regs[t_num_regs] = {
 struct lm92_data {
        struct i2c_client *client;
        struct mutex update_lock;
-       char valid; /* zero until following fields are valid */
+       bool valid; /* false until following fields are valid */
        unsigned long last_updated; /* in jiffies */
 
        /* registers values */
@@ -126,7 +126,7 @@ static struct lm92_data *lm92_update_device(struct device *dev)
                                i2c_smbus_read_word_swapped(client, regs[i]);
                }
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 78d6dfaf145b0cc5ad3ccc1a8774e4bb7bed6fdf..dc67bf954b2164573b920374ed497703a1a481b8 100644 (file)
@@ -202,7 +202,7 @@ struct lm93_data {
        /* client update function */
        void (*update)(struct lm93_data *, struct i2c_client *);
 
-       char valid; /* !=0 if following fields are valid */
+       bool valid; /* true if following fields are valid */
 
        /* register values, arranged by block read groups */
        struct block1_t block1;
@@ -917,7 +917,7 @@ static struct lm93_data *lm93_update_device(struct device *dev)
 
                data->update(data, client);
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 00dbc170c8c6b7b3b1db428a14e486088bcb2136..8ea46ff20be5fac7f28fde75da56b412af667560 100644 (file)
@@ -78,7 +78,7 @@ struct lm95241_data {
        struct mutex update_lock;
        unsigned long last_updated;     /* in jiffies */
        unsigned long interval;         /* in milli-seconds */
-       char valid;             /* zero until following fields are valid */
+       bool valid;             /* false until following fields are valid */
        /* registers values */
        u8 temp[ARRAY_SIZE(lm95241_reg_address)];
        u8 status, config, model, trutherm;
@@ -118,7 +118,7 @@ static struct lm95241_data *lm95241_update_device(struct device *dev)
                data->status = i2c_smbus_read_byte_data(client,
                                                        LM95241_REG_R_STATUS);
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
@@ -257,7 +257,7 @@ static int lm95241_write_temp(struct device *dev, u32 attr, int channel,
                        else
                                data->config &= ~R2DF_MASK;
                }
-               data->valid = 0;
+               data->valid = false;
                ret = i2c_smbus_write_byte_data(client, LM95241_REG_RW_CONFIG,
                                                data->config);
                break;
@@ -273,7 +273,7 @@ static int lm95241_write_temp(struct device *dev, u32 attr, int channel,
                        else
                                data->config &= ~R2DF_MASK;
                }
-               data->valid = 0;
+               data->valid = false;
                ret = i2c_smbus_write_byte_data(client, LM95241_REG_RW_CONFIG,
                                                data->config);
                break;
index 13b85367a21f44b3b685894a74e556450f6f8713..e3ac004c1ed12a10e5d822fefb344dba4a5447d2 100644 (file)
@@ -77,7 +77,7 @@ static struct ltc4151_data *ltc4151_update_device(struct device *dev)
                        data->regs[i] = val;
                }
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 abort:
        mutex_unlock(&data->update_lock);
index 1d18c212054fc2451fd966a66ee937d1fd20ff75..fa43d26ddd4fd17b1fb3b5341a9f326929bd4dd9 100644 (file)
@@ -64,7 +64,7 @@ static struct ltc4215_data *ltc4215_update_device(struct device *dev)
                }
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index b81e9c3d297bc27c99dce9ff2671560117dabb3c..b91cc4fe84e538850921690f4a7bffd1afe07d5c 100644 (file)
@@ -73,13 +73,13 @@ static struct ltc4261_data *ltc4261_update_device(struct device *dev)
                                        "Failed to read ADC value: error %d\n",
                                        val);
                                ret = ERR_PTR(val);
-                               data->valid = 0;
+                               data->valid = false;
                                goto abort;
                        }
                        data->regs[i] = val;
                }
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 abort:
        mutex_unlock(&data->update_lock);
index ae3a6a7bdaa23d9a7fecaf9663353613688dd643..daa5d8af1e69d81965da1b436ebace34287cbb99 100644 (file)
@@ -166,7 +166,7 @@ static struct max16065_data *max16065_update_device(struct device *dev)
                          = i2c_smbus_read_byte_data(client, MAX16065_FAULT(i));
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
        mutex_unlock(&data->update_lock);
        return data;
index 8bd941cae4d14650e7725ac8037787de9baf27a2..eae9e68027bcbe7085a1cef803a746c79e8405b2 100644 (file)
@@ -79,7 +79,7 @@ enum temp_index {
 struct max1619_data {
        struct i2c_client *client;
        struct mutex update_lock;
-       char valid; /* zero until following fields are valid */
+       bool valid; /* false until following fields are valid */
        unsigned long last_updated; /* in jiffies */
 
        /* registers values */
@@ -124,7 +124,7 @@ static struct max1619_data *max1619_update_device(struct device *dev)
                        data->alarms ^= 0x02;
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 5c41c78f04583516a769023c944f64d3472804e7..78688e6cb87d99be3094df1933c52b675e421706 100644 (file)
@@ -58,7 +58,7 @@ struct max1668_data {
        enum chips type;
 
        struct mutex update_lock;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        /* 1x local and 4x remote */
@@ -120,7 +120,7 @@ static struct max1668_data *max1668_update_device(struct device *dev)
        data->alarms |= val;
 
        data->last_updated = jiffies;
-       data->valid = 1;
+       data->valid = true;
 abort:
        mutex_unlock(&data->update_lock);
 
index b71899c641fa7a4185f5b8367b6e8ead6ce355bc..ccc0f047bd446ee5fc70c61125c05e9cdcc2396e 100644 (file)
@@ -69,7 +69,7 @@ static const int rpm_ranges[] = { 2000, 4000, 8000, 16000 };
 struct max6639_data {
        struct i2c_client *client;
        struct mutex update_lock;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        /* Register values sampled regularly */
@@ -141,7 +141,7 @@ static struct max6639_data *max6639_update_device(struct device *dev)
                }
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 abort:
        mutex_unlock(&data->update_lock);
index 23d93142b0b3154900464cb30ec4fba819bd7dda..699d265aae2e718d1fc40b50cad33cb645872124 100644 (file)
@@ -181,7 +181,7 @@ static struct max6642_data *max6642_update_device(struct device *dev)
                                        MAX6642_REG_R_STATUS);
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 6a9ba23cd302cb91fd223cba961380db95dddc16..0828436a1f6cfee2f15b4b58aea2654df2934a01 100644 (file)
@@ -178,7 +178,7 @@ struct pc87360_data {
        struct device *hwmon_dev;
        struct mutex lock;
        struct mutex update_lock;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        int address[3];
@@ -1673,7 +1673,7 @@ static struct pc87360_data *pc87360_update_device(struct device *dev)
                }
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index a5cd4de36575e4a801fa9ef0c8b050fac2e975b5..39ff1c9b1df54334ded498f5a0b54c925dcf5712 100644 (file)
@@ -56,7 +56,7 @@ struct sch5636_data {
        struct device *hwmon_dev;
 
        struct mutex update_lock;
-       char valid;                     /* !=0 if following fields are valid */
+       bool valid;                     /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
        u8 in[SCH5636_NO_INS];
        u8 temp_val[SCH5636_NO_TEMPS];
@@ -140,7 +140,7 @@ static struct sch5636_data *sch5636_update_device(struct device *dev)
        }
 
        data->last_updated = jiffies;
-       data->valid = 1;
+       data->valid = true;
 abort:
        mutex_unlock(&data->update_lock);
        return ret;
index 7d18ce5d383973ed9e68f352bec11f286e5db715..e23dbf287233a851657ecb6579c64d16a489d063 100644 (file)
@@ -41,7 +41,7 @@ struct sht21 {
        unsigned long last_update;
        int temperature;
        int humidity;
-       char valid;
+       bool valid;
        char eic[18];
 };
 
@@ -105,7 +105,7 @@ static int sht21_update_measurements(struct device *dev)
                        goto out;
                sht21->humidity = sht21_rh_ticks_to_per_cent_mille(ret);
                sht21->last_update = jiffies;
-               sht21->valid = 1;
+               sht21->valid = true;
        }
 out:
        mutex_unlock(&sht21->lock);
index 0c6741f949f5fd29a407fa972413f0c851634b58..018cb5a7651fba649257a51d7f0e8ca9df1ba61c 100644 (file)
@@ -172,7 +172,7 @@ struct sis5595_data {
        struct mutex lock;
 
        struct mutex update_lock;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
        char maxins;            /* == 3 if temp enabled, otherwise == 4 */
        u8 revision;            /* Reg. value */
@@ -728,7 +728,7 @@ static struct sis5595_data *sis5595_update_device(struct device *dev)
                    sis5595_read_value(data, SIS5595_REG_ALARM1) |
                    (sis5595_read_value(data, SIS5595_REG_ALARM2) << 8);
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 62906d9c4b86c560639d56e72c0bc6cb1d292fb5..8c4ed72e5d684571bdfc78eed2528269088c30ff 100644 (file)
@@ -265,7 +265,7 @@ static struct smm665_data *smm665_update_device(struct device *dev)
                        data->adc[i] = val;
                }
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 abort:
        mutex_unlock(&data->update_lock);
index f928b8d4ff4892a761a526737148561f8a5dcfdf..c26d6eae0e4e4ac12af23908d7a217a5b8350591 100644 (file)
@@ -96,7 +96,7 @@ struct smsc47b397_data {
 
        struct mutex update_lock;
        unsigned long last_updated; /* in jiffies */
-       int valid;
+       bool valid;
 
        /* register values */
        u16 fan[4];
@@ -137,7 +137,7 @@ static struct smsc47b397_data *smsc47b397_update_device(struct device *dev)
                }
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
 
                dev_dbg(dev, "... device update complete\n");
        }
index 03a87aa2017aed2d2c1c7f25f8cd1956e536cc27..a5db15c087ae1b8d0da37778f90a8cf4b059d97b 100644 (file)
@@ -86,7 +86,7 @@ struct smsc47m192_data {
        struct i2c_client *client;
        const struct attribute_group *groups[3];
        struct mutex update_lock;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        u8 in[8];               /* Register value */
@@ -157,7 +157,7 @@ static struct smsc47m192_data *smsc47m192_update_device(struct device *dev)
                                                SMSC47M192_REG_ALARM2) << 8);
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index fde5e2d0825a4ead0686f600362e8e274a5b25af..6a804f5036f4774fb28105ca6cf300df69edf9c6 100644 (file)
@@ -62,7 +62,7 @@ struct thmc50_data {
        enum chips type;
        unsigned long last_updated;     /* In jiffies */
        char has_temp3;         /* !=0 if it is ADM1022 in temp3 mode */
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
 
        /* Register values */
        s8 temp_input[3];
@@ -107,7 +107,7 @@ static struct thmc50_data *thmc50_update_device(struct device *dev)
                data->alarms =
                    i2c_smbus_read_byte_data(client, THMC50_REG_INTR);
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 9dc210b55e69b19f05dc2cf0f0666e8a0cc1a384..88007416c210d2d1498b8a1353439bf8d06a471a 100644 (file)
@@ -136,7 +136,7 @@ struct tmp401_data {
        struct i2c_client *client;
        const struct attribute_group *groups[3];
        struct mutex update_lock;
-       char valid; /* zero until following fields are valid */
+       bool valid; /* false until following fields are valid */
        unsigned long last_updated; /* in jiffies */
        enum chips kind;
 
@@ -267,7 +267,7 @@ static struct tmp401_data *tmp401_update_device(struct device *dev)
                data->temp_crit_hyst = val;
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
 abort:
@@ -413,7 +413,7 @@ static ssize_t reset_temp_history_store(struct device *dev,
        }
        mutex_lock(&data->update_lock);
        i2c_smbus_write_byte_data(client, TMP401_TEMP_MSB_WRITE[5][0], val);
-       data->valid = 0;
+       data->valid = false;
        mutex_unlock(&data->update_lock);
 
        return count;
index a4ac6e16d5920471b2f086d5e0189cb4ccbec6c3..356c96c3588bd8d8fea053102b23abeef76bc772 100644 (file)
@@ -94,7 +94,7 @@ struct tmp421_data {
        struct hwmon_channel_info temp_info;
        const struct hwmon_channel_info *info[2];
        struct hwmon_chip_info chip;
-       char valid;
+       bool valid;
        unsigned long last_updated;
        unsigned long channels;
        u8 config;
@@ -141,14 +141,14 @@ static int tmp421_update_device(struct tmp421_data *data)
                        data->temp[i] |= ret;
                }
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
 exit:
        mutex_unlock(&data->update_lock);
 
        if (ret < 0) {
-               data->valid = 0;
+               data->valid = false;
                return ret;
        }
 
index a2eddd2c2538ac3ac01af79c4cb041dbf6280310..55634110c2f962cc3fe04e81f78b09d8b60a24de 100644 (file)
@@ -304,7 +304,7 @@ struct via686a_data {
        const char *name;
        struct device *hwmon_dev;
        struct mutex update_lock;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        u8 in[5];               /* Register value */
@@ -800,7 +800,7 @@ static struct via686a_data *via686a_update_device(struct device *dev)
                                       VIA686A_REG_ALARM1) |
                    (via686a_read_value(data, VIA686A_REG_ALARM2) << 8);
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 2fbdc532aed4a743f4498177e414bee5418bdaa3..4a5e911d26ebf953558c0c9fe033413e6cf667fb 100644 (file)
@@ -105,7 +105,7 @@ struct vt1211_data {
        struct device *hwmon_dev;
 
        struct mutex update_lock;
-       char valid;                     /* !=0 if following fields are valid */
+       bool valid;                     /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        /* Register values */
@@ -319,7 +319,7 @@ static struct vt1211_data *vt1211_update_device(struct device *dev)
                                vt1211_read8(data, VT1211_REG_ALARM1);
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 6603727e15a02b8e92f93261f02c61599159c5c7..03275ac8ba72c9cd4baf271a74c0b31ef3ca4aec 100644 (file)
@@ -145,7 +145,7 @@ struct vt8231_data {
 
        struct mutex update_lock;
        struct device *hwmon_dev;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        u8 in[6];               /* Register value */
@@ -929,7 +929,7 @@ static struct vt8231_data *vt8231_update_device(struct device *dev)
                        data->alarms &= ~0x80;
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 705a59663d42c5be078ba67e89e323bf4062dad5..af89b32a93a56b9504fab6040b0110f4fa60bb1e 100644 (file)
@@ -320,7 +320,7 @@ struct w83627ehf_data {
        const u16 *scale_in;
 
        struct mutex update_lock;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        /* Register values */
@@ -688,7 +688,7 @@ static struct w83627ehf_data *w83627ehf_update_device(struct device *dev)
                                                W83627EHF_REG_CASEOPEN_DET);
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
@@ -1099,7 +1099,7 @@ clear_caseopen(struct device *dev, struct w83627ehf_data *data, int channel,
        reg = w83627ehf_read_value(data, W83627EHF_REG_CASEOPEN_CLR);
        w83627ehf_write_value(data, W83627EHF_REG_CASEOPEN_CLR, reg | mask);
        w83627ehf_write_value(data, W83627EHF_REG_CASEOPEN_CLR, reg & ~mask);
-       data->valid = 0;        /* Force cache refresh */
+       data->valid = false;    /* Force cache refresh */
        mutex_unlock(&data->update_lock);
 
        return 0;
@@ -2004,7 +2004,7 @@ static int __maybe_unused w83627ehf_resume(struct device *dev)
        w83627ehf_write_value(data, W83627EHF_REG_VBAT, data->vbat);
 
        /* Force re-reading all values */
-       data->valid = 0;
+       data->valid = false;
        mutex_unlock(&data->update_lock);
 
        return 0;
index a07b97400cba3571691cc6637a0721a18197daed..9be277156ed272fdb2abd8b85da30f7d6368c6a6 100644 (file)
@@ -355,7 +355,7 @@ struct w83627hf_data {
        enum chips type;
 
        struct mutex update_lock;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        u8 in[9];               /* Register value */
@@ -448,7 +448,7 @@ static int w83627hf_resume(struct device *dev)
        w83627hf_write_value(data, W83781D_REG_SCFG2, data->scfg2);
 
        /* Force re-reading all values */
-       data->valid = 0;
+       data->valid = false;
        mutex_unlock(&data->update_lock);
 
        return 0;
@@ -1905,7 +1905,7 @@ static struct w83627hf_data *w83627hf_update_device(struct device *dev)
                    w83627hf_read_value(data, W83781D_REG_BEEP_INTS1) |
                    w83627hf_read_value(data, W83781D_REG_BEEP_INTS3) << 16;
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index ce8e2c10e854f106f23fb7e98c43f38f2bbd9cff..b3579721265f7529d8823ac74a98396600db3d7d 100644 (file)
@@ -203,7 +203,7 @@ struct w83781d_data {
        int isa_addr;
 
        struct mutex update_lock;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        struct i2c_client *lm75[2];     /* for secondary I2C addresses */
@@ -1554,7 +1554,7 @@ static struct w83781d_data *w83781d_update_device(struct device *dev)
                                               W83781D_REG_BEEP_INTS3) << 16;
                }
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 3c1be2c11fdf0252daf6bc66fc751874227229b9..80a9a78d7ce98a52a1791d4b95c67578b30af235 100644 (file)
@@ -270,7 +270,7 @@ struct w83791d_data {
        struct device *hwmon_dev;
        struct mutex update_lock;
 
-       char valid;                     /* !=0 if following fields are valid */
+       bool valid;                     /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        /* volts */
@@ -1596,7 +1596,7 @@ static struct w83791d_data *w83791d_update_device(struct device *dev)
                                << 4;
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 1f175f3813506ddd074650a09eb5970b32ddfbc7..31a1cdc308779ac1288813b4266a6fbb9c22bfd4 100644 (file)
@@ -261,7 +261,7 @@ struct w83792d_data {
        struct device *hwmon_dev;
 
        struct mutex update_lock;
-       char valid;             /* !=0 if following fields are valid */
+       bool valid;             /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
 
        u8 in[9];               /* Register value */
@@ -740,7 +740,7 @@ intrusion0_alarm_store(struct device *dev, struct device_attribute *attr,
        mutex_lock(&data->update_lock);
        reg = w83792d_read_value(client, W83792D_REG_CHASSIS_CLR);
        w83792d_write_value(client, W83792D_REG_CHASSIS_CLR, reg | 0x80);
-       data->valid = 0;                /* Force cache refresh */
+       data->valid = false;            /* Force cache refresh */
        mutex_unlock(&data->update_lock);
 
        return count;
@@ -1589,7 +1589,7 @@ static struct w83792d_data *w83792d_update_device(struct device *dev)
                }
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 1d2854de1cfc961b705516fc3f50b90865eac64e..0a65d164c8f094847f3042da81a479e5e20bc2c2 100644 (file)
@@ -204,7 +204,7 @@ static inline s8 TEMP_TO_REG(long val, s8 min, s8 max)
 struct w83793_data {
        struct device *hwmon_dev;
        struct mutex update_lock;
-       char valid;                     /* !=0 if following fields are valid */
+       bool valid;                     /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
        unsigned long last_nonvolatile; /* In jiffies, last time we update the
                                         * nonvolatile registers
@@ -452,7 +452,7 @@ store_chassis_clear(struct device *dev,
        mutex_lock(&data->update_lock);
        reg = w83793_read_value(client, W83793_REG_CLR_CHASSIS);
        w83793_write_value(client, W83793_REG_CLR_CHASSIS, reg | 0x80);
-       data->valid = 0;                /* Force cache refresh */
+       data->valid = false;            /* Force cache refresh */
        mutex_unlock(&data->update_lock);
        return count;
 }
@@ -2077,7 +2077,7 @@ static struct w83793_data *w83793_update_device(struct device *dev)
                data->vid[1] = w83793_read_value(client, W83793_REG_VID_INB);
        w83793_update_nonvolatile(dev);
        data->last_updated = jiffies;
-       data->valid = 1;
+       data->valid = true;
 
 END:
        mutex_unlock(&data->update_lock);
index 621b05afa8373d0fead4348ef16efa9ab3682a77..45b12c4287dfafdf8b788d8d3a47271d45207f14 100644 (file)
@@ -379,7 +379,7 @@ struct w83795_data {
        u8 enable_beep;
        u8 beeps[6];            /* Register value */
 
-       char valid;
+       bool valid;
        char valid_limits;
        char valid_pwm_config;
 };
@@ -684,7 +684,7 @@ static struct w83795_data *w83795_update_device(struct device *dev)
                             tmp & ~ALARM_CTRL_RTSACS);
 
        data->last_updated = jiffies;
-       data->valid = 1;
+       data->valid = true;
 
 END:
        mutex_unlock(&data->update_lock);
@@ -764,7 +764,7 @@ store_chassis_clear(struct device *dev,
 
        /* Clear status and force cache refresh */
        w83795_read(client, W83795_REG_ALARM(5));
-       data->valid = 0;
+       data->valid = false;
        mutex_unlock(&data->update_lock);
        return count;
 }
index 656a77102ca6f422e903c2d43a5ba9729d64cfa3..a41f989d66e2702ffac5b86315b89991f66e9f3d 100644 (file)
@@ -98,7 +98,7 @@ static struct i2c_driver w83l785ts_driver = {
 struct w83l785ts_data {
        struct device *hwmon_dev;
        struct mutex update_lock;
-       char valid; /* zero until following fields are valid */
+       bool valid; /* false until following fields are valid */
        unsigned long last_updated; /* in jiffies */
 
        /* registers values */
@@ -270,7 +270,7 @@ static struct w83l785ts_data *w83l785ts_update_device(struct device *dev)
                                W83L785TS_REG_TEMP_OVER, data->temp[1]);
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
        }
 
        mutex_unlock(&data->update_lock);
index 542afff1423beb6288cc7e0e1685004f74270489..11ba23c1af855cbf87da0ac125e1b3de1b48e576 100644 (file)
@@ -113,7 +113,7 @@ DIV_TO_REG(long val)
 struct w83l786ng_data {
        struct i2c_client *client;
        struct mutex update_lock;
-       char valid;                     /* !=0 if following fields are valid */
+       bool valid;                     /* true if following fields are valid */
        unsigned long last_updated;     /* In jiffies */
        unsigned long last_nonvolatile; /* In jiffies, last time we update the
                                         * nonvolatile registers */
@@ -209,7 +209,7 @@ static struct w83l786ng_data *w83l786ng_update_device(struct device *dev)
                data->tolerance[1] = (reg_tmp >> 4) & 0x0f;
 
                data->last_updated = jiffies;
-               data->valid = 1;
+               data->valid = true;
 
        }