All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dan Carpenter <dan.carpenter@oracle.com>
To: linux@roeck-us.net
Cc: linux-hwmon@vger.kernel.org
Subject: [bug report] hwmon: (lm83) Convert to use with_info API
Date: Thu, 27 Jan 2022 11:52:18 +0300	[thread overview]
Message-ID: <20220127085218.GE25644@kili> (raw)

Hello Guenter Roeck,

The patch 1c8467e8e8b2: "hwmon: (lm83) Convert to use with_info API"
from Dec 22, 2021, leads to the following Smatch static checker
warning:

	drivers/hwmon/lm83.c:337 lm83_is_visible()
	warn: signedness bug returning '(-19)'

drivers/hwmon/lm83.c
    300 static umode_t lm83_is_visible(const void *_data, enum hwmon_sensor_types type,
    301                                u32 attr, int channel)
    302 {
    303         const struct lm83_data *data = _data;
    304 
    305         /*
    306          * LM82 only supports a single external channel, modeled as channel 2.
    307          */
    308         if (data->type == lm82 && (channel == 1 || channel == 3))
    309                 return 0;
    310 
    311         switch (type) {
    312         case hwmon_chip:
    313                 if (attr == hwmon_chip_alarms)
    314                         return 0444;
    315                 break;
    316         case hwmon_temp:
    317                 switch (attr) {
    318                 case hwmon_temp_input:
    319                 case hwmon_temp_max_alarm:
    320                 case hwmon_temp_crit_alarm:
    321                         return 0444;
    322                 case hwmon_temp_fault:
    323                         if (channel)
    324                                 return 0444;
    325                         break;
    326                 case hwmon_temp_max:
    327                         return 0644;
    328                 case hwmon_temp_crit:
    329                         if (channel == 2)
    330                                 return 0644;
    331                         return 0444;
    332                 default:
    333                         break;
    334                 }
    335                 break;
    336         default:
--> 337                 return -ENODEV;

return 0;?

    338         }
    339         return 0;
    340 }

regards,
dan carpenter

             reply	other threads:[~2022-01-27  8:52 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-27  8:52 Dan Carpenter [this message]
2022-01-27 14:10 ` [bug report] hwmon: (lm83) Convert to use with_info API Guenter Roeck

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220127085218.GE25644@kili \
    --to=dan.carpenter@oracle.com \
    --cc=linux-hwmon@vger.kernel.org \
    --cc=linux@roeck-us.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.