power: supply: qcom: Remove useless NULL checks against thresh array in msm_bcl_enable

Clang warns:

../drivers/power/supply/qcom/msm_bcl.c:230:42: warning: address of array
'bcl[BCL_PARAM_VOLTAGE]->thresh' will always evaluate to 'true'
[-Wpointer-bool-conversion]
        if (!bcl[i] || !bcl[BCL_PARAM_VOLTAGE]->thresh
                       ~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~
../drivers/power/supply/qcom/msm_bcl.c:231:31: warning: address of array
'bcl[BCL_PARAM_CURRENT]->thresh' will always evaluate to 'true'
[-Wpointer-bool-conversion]
                || !bcl[BCL_PARAM_CURRENT]->thresh) {
                   ~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~
2 warnings generated.

Fixes: 77dd35597191 ("qpnp: Add snapshot of some qpnp, regulator and charger drivers")
Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>

diff --git a/drivers/power/supply/qcom/msm_bcl.c b/drivers/power/supply/qcom/msm_bcl.c
index aea3f46..b4347ee 100644 (file)
--- a/drivers/power/supply/qcom/msm_bcl.c
+++ b/drivers/power/supply/qcom/msm_bcl.c
@@ -227,8 +227,7 @@ int msm_bcl_enable(void)
        int ret = 0, i = 0;
        struct bcl_param_data *param_data = NULL;
 
-       if (!bcl[i] || !bcl[BCL_PARAM_VOLTAGE]->thresh
-               || !bcl[BCL_PARAM_CURRENT]->thresh) {
+       if (!bcl[i]) {
                pr_err("BCL not initialized\n");
                return -EINVAL;
        }