hwmon: (scmi) Remove redundant pointer check
authorNathan Chancellor <natechancellor@gmail.com>
Sun, 16 Sep 2018 00:05:07 +0000 (17:05 -0700)
committerGuenter Roeck <linux@roeck-us.net>
Thu, 11 Oct 2018 03:37:13 +0000 (20:37 -0700)
commita31796c30e423f58d266df30a9bbf321fc071b30
treeebbaea3d57f44f504a5c49279454c8c324227e38
parent2738b767be548d6397a74141c18182fddd8abfc3
hwmon: (scmi) Remove redundant pointer check

Clang warns when the address of a pointer is used in a boolean context
as it will always return true.

drivers/hwmon/scmi-hwmon.c:59:24: warning: address of array
'sensor->name' will always evaluate to 'true'
[-Wpointer-bool-conversion]
        if (sensor && sensor->name)
                   ~~ ~~~~~~~~^~~~
1 warning generated.

Remove the check as it isn't doing anything currently; if validation
of the contents of the data structure was intended by the original
author (since this line has been present from the first version of
this driver), it can be added in a follow-up patch.

Reported-by: Nick Desaulniers <ndesaulniers@google.com>
Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/scmi-hwmon.c