From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Date: Thu, 27 Apr 2017 09:29:49 +0000 Subject: [PATCH 2/4] hwmon-nct6683: Adjust five checks for null pointers Message-Id: <339214ae-9c04-9e8c-e97e-368aa83a17b5@users.sourceforge.net> List-Id: References: <2f58a509-4aba-5c44-bc79-568b260ab692@users.sourceforge.net> In-Reply-To: <2f58a509-4aba-5c44-bc79-568b260ab692@users.sourceforge.net> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: linux-hwmon@vger.kernel.org, =?UTF-8?B?R8O8bnRlciBSw7Zjaw==?= , Jean Delvare Cc: LKML , kernel-janitors@vger.kernel.org From: Markus Elfring Date: Thu, 27 Apr 2017 10:22:39 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written … Thus fix the affected source code places. Signed-off-by: Markus Elfring --- drivers/hwmon/nct6683.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/hwmon/nct6683.c b/drivers/hwmon/nct6683.c index dd14a98cc6dd..4deb46ce8834 100644 --- a/drivers/hwmon/nct6683.c +++ b/drivers/hwmon/nct6683.c @@ -423,16 +423,16 @@ nct6683_create_attr_group(struct device *dev, return ERR_PTR(-EINVAL); group = devm_kzalloc(dev, sizeof(*group), GFP_KERNEL); - if (group = NULL) + if (!group) return ERR_PTR(-ENOMEM); attrs = devm_kcalloc(dev, repeat * count + 1, sizeof(*attrs), GFP_KERNEL); - if (attrs = NULL) + if (!attrs) return ERR_PTR(-ENOMEM); su = devm_kcalloc(dev, repeat * count, sizeof(*su), GFP_KERNEL); - if (su = NULL) + if (!su) return ERR_PTR(-ENOMEM); group->attrs = attrs; @@ -440,7 +440,7 @@ nct6683_create_attr_group(struct device *dev, for (i = 0; i < repeat; i++) { t = tg->templates; - for (j = 0; *t != NULL; j++) { + for (j = 0; *t; j++) { snprintf(su->name, sizeof(su->name), (*t)->dev_attr.attr.name, tg->base + i); if ((*t)->s2) { @@ -1179,7 +1179,7 @@ static void nct6683_setup_sensors(struct nct6683_data *data) if (reg >= NUM_MON_LABELS) continue; /* Skip if disabled or reserved */ - if (nct6683_mon_label[reg] = NULL) + if (!nct6683_mon_label[reg]) continue; if (reg < MON_VOLTAGE_START) { data->temp_index[data->temp_num] = i; -- 2.12.2