From nobody Fri Nov 29 18:32:19 2024 Received: from mail-wr1-f54.google.com (mail-wr1-f54.google.com [209.85.221.54]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6A6DE157E9F; Mon, 16 Sep 2024 12:03:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.54 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1726488211; cv=none; b=FAh/zfnA20IPCS9DcyHuB0oWGu6FFxe3oisUYWUluOi79eH3cSk5th+E8cTJDIW6jk1LmkqkxIEZ/vGe/nVImLoHq1YHa5JG+bji2G3ymbmZaT/o6QAda/re6a1XsnyKxjyePyf0SOkZByTmxFKPrtsLqnKW0+v9n1CFp2zjdAE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1726488211; c=relaxed/simple; bh=aHadeGUkjHAVTilEnzf8JCrKBU9lqGeBd1x1b5xFxC8=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:To:Cc; b=eRO2ztJdFWKRRX1cKed8eQ77enK9EC9nQCrWF5BBmERlgwWXVne9TegNlFsf4oZLvNkDA/7HoiMpeQ7NLtSaOY9fkpod43bLXlkPbauaHcV+yct6kLBhSEJkdINJinE+8iRPGt36n9iSzvSDPuwOhf93JK5RdFwC6Y2Xa+Vy9lo= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=OWOnDfwE; arc=none smtp.client-ip=209.85.221.54 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="OWOnDfwE" Received: by mail-wr1-f54.google.com with SMTP id ffacd0b85a97d-374bd059b12so1931651f8f.1; Mon, 16 Sep 2024 05:03:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1726488208; x=1727093008; darn=vger.kernel.org; h=cc:to:message-id:content-transfer-encoding:mime-version:subject :date:from:from:to:cc:subject:date:message-id:reply-to; bh=j7rhiN0cwffHbbzhzNEaKmIKxR7usCqkEQIorTMTnxI=; b=OWOnDfwE+UvJu6jWIgTra0hLZSdg9la1Tpq7rK8yjYOA95kF4aoITUekwVQ3Wdc4dn X+tFB+O2QX1L+mFv5D3YSZxPzxkVDpla0MPoAPLr1nbGI2z1zAPHnzHwn3Rs5hibH+B9 +4becn88PfrxM4i+rVAwNL6Nz5W4dpsCan9tDQSVcCLX8AIHaA1uwEbfXebVLXjTsd6A fGPsZWwp+jLxSHdtuwqO9xsAsJ+NcJYMWocIzAAR+P83n0jLNvzbmitOthoRZ1BezLhO CUMqj8wykNbXd7jAawMvgiL/+4nt5r+tnByJjh/OpGYcSrzE3KP39THZoV5+3sCshd/Q VkSg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1726488208; x=1727093008; h=cc:to:message-id:content-transfer-encoding:mime-version:subject :date:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=j7rhiN0cwffHbbzhzNEaKmIKxR7usCqkEQIorTMTnxI=; b=WV7YOKqNmOfhLKi29vAUGnwqN1eODK/g8BWTrqVy3/jShDEmabZw21RVi6M9sSCA0I brdUG6l8f4NxlxlpfPWN7+SqTGt3A0dhaZ1GxzQhxLZ3fPi6fblsSmYK6dauk/QiaFV5 CId0LwGGsIhIiOj1WMv1YQ1R8p5FpgM0M7QEub6v0oG3wQbgte4oOrw/lsFXi+f/8JPh 5k1PXS542jQZNLfmMujPFDUVN5tFGPFAU+98WOBEd2Yt+v1jdTl8CJ+jetEnpdHLHA5K Q6n0AHzy2nkZoGgojfa1bIKM2HqXFBwh0meseqCgsmDrmekclQd8XowNwd05Vq4afrl6 /LhQ== X-Forwarded-Encrypted: i=1; AJvYcCUFn66lLfOjqa598Zu8ToXBO/VPS5+d1KtfEO0q0xXaUTJtzPjD8Vd2aiNz+TCcEEHiawqT9WuH3/1wlU8e@vger.kernel.org, AJvYcCV0wnr1ObEOwBOcWhna1PTjCPl0a73FnvTgRCU4UZ8ceiLufoVZCsWtZUsj38E3SmJt6OptSgdHmsY=@vger.kernel.org X-Gm-Message-State: AOJu0YxLKQZTC/c0q/+YGUtXqIk7BYuPVtW/0uBQePJBKI+SAznlhFGD vz0EQQW9UarAAN54Rc1vuE0rEcxzc1Ip4CWouUPNqdXGuBKurfs2 X-Google-Smtp-Source: AGHT+IElm0k84RKTk7MnXnflcV7uakeyrdC7fcFKyQ5BT8hUrQaeNzkZCNd4OygZK2uzW9kFMTCI4A== X-Received: by 2002:adf:a302:0:b0:376:274c:c8b3 with SMTP id ffacd0b85a97d-378d61d4d81mr6536181f8f.5.1726488207226; Mon, 16 Sep 2024 05:03:27 -0700 (PDT) Received: from localhost (host-80-104-176-42.retail.telecomitalia.it. [80.104.176.42]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-378e73f9a17sm7066603f8f.64.2024.09.16.05.03.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 16 Sep 2024 05:03:26 -0700 (PDT) From: Matteo Martelli Date: Mon, 16 Sep 2024 14:00:05 +0200 Subject: [PATCH] iio: pac1921: remove unnecessary explicit casts Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20240916-iio-pac1921-nocast-v1-1-a0f96d321eee@gmail.com> X-B4-Tracking: v=1; b=H4sIAMQd6GYC/x3MMQqAMAxA0auUzAaaomC9ijiUGDVLK62IULy7x fEN/1coklUKTKZClluLpthAnQE+QtwFdW0GZ11vPRGqJjwDk3eEMXEoFw7sR15F2A4eWnhm2fT 5p/Pyvh+GbNPXZAAAAA== To: Jonathan Cameron , Lars-Peter Clausen Cc: Jonathan Cameron , linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, Matteo Martelli X-Mailer: b4 0.14.1 Many explicit casts were introduced to address Wconversion and Wsign-compare warnings. Remove them to improve readability. Fixes: 371f778b83cd ("iio: adc: add support for pac1921") Signed-off-by: Matteo Martelli --- Link: https://lore.kernel.org/linux-iio/1fa4ab12-0939-477d-bc92-306fd32e4fd= 9@stanley.mountain/ --- drivers/iio/adc/pac1921.c | 43 +++++++++++++++++++++---------------------- 1 file changed, 21 insertions(+), 22 deletions(-) diff --git a/drivers/iio/adc/pac1921.c b/drivers/iio/adc/pac1921.c index 4c2a1c07bc39..de69a1619a9e 100644 --- a/drivers/iio/adc/pac1921.c +++ b/drivers/iio/adc/pac1921.c @@ -240,8 +240,8 @@ static inline void pac1921_calc_scale(int dividend, int= divisor, int *val, { s64 tmp; =20 - tmp =3D div_s64(dividend * (s64)NANO, divisor); - *val =3D (int)div_s64_rem(tmp, NANO, val2); + tmp =3D div_s64(dividend * NANO, divisor); + *val =3D div_s64_rem(tmp, NANO, val2); } =20 /* @@ -260,7 +260,7 @@ static void pac1921_calc_current_scales(struct pac1921_= priv *priv) int max =3D (PAC1921_MAX_VSENSE_MV * MICRO) >> i; int vsense_lsb =3D DIV_ROUND_CLOSEST(max, PAC1921_RES_RESOLUTION); =20 - pac1921_calc_scale(vsense_lsb, (int)priv->rshunt_uohm, + pac1921_calc_scale(vsense_lsb, priv->rshunt_uohm, &priv->current_scales[i][0], &priv->current_scales[i][1]); } @@ -314,7 +314,7 @@ static int pac1921_check_push_overflow(struct iio_dev *= indio_dev, s64 timestamp) timestamp); } =20 - priv->prev_ovf_flags =3D (u8)flags; + priv->prev_ovf_flags =3D flags; =20 return 0; } @@ -329,8 +329,7 @@ static int pac1921_check_push_overflow(struct iio_dev *= indio_dev, s64 timestamp) static int pac1921_read_res(struct pac1921_priv *priv, unsigned long reg, u16 *val) { - int ret =3D regmap_bulk_read(priv->regmap, (unsigned int)reg, val, - sizeof(*val)); + int ret =3D regmap_bulk_read(priv->regmap, reg, val, sizeof(*val)); if (ret) return ret; =20 @@ -366,7 +365,7 @@ static int pac1921_read_raw(struct iio_dev *indio_dev, if (ret) return ret; =20 - *val =3D (int)res_val; + *val =3D res_val; =20 return IIO_VAL_INT; } @@ -397,13 +396,13 @@ static int pac1921_read_raw(struct iio_dev *indio_dev, int *curr_scale =3D priv->current_scales[priv->di_gain]; =20 /* Convert current_scale from INT_PLUS_NANO to INT */ - s64 tmp =3D curr_scale[0] * (s64)NANO + curr_scale[1]; + s64 tmp =3D curr_scale[0] * NANO + curr_scale[1]; =20 /* Multiply by max_vbus (V) / dv_gain */ - tmp *=3D PAC1921_MAX_VBUS_V >> (int)priv->dv_gain; + tmp *=3D PAC1921_MAX_VBUS_V >> priv->dv_gain; =20 /* Convert back to INT_PLUS_NANO */ - *val =3D (int)div_s64_rem(tmp, NANO, val2); + *val =3D div_s64_rem(tmp, NANO, val2); =20 return IIO_VAL_INT_PLUS_NANO; } @@ -426,7 +425,7 @@ static int pac1921_read_raw(struct iio_dev *indio_dev, * 1/(integr_period_usecs/MICRO) =3D MICRO/integr_period_usecs */ *val =3D MICRO; - *val2 =3D (int)priv->integr_period_usecs; + *val2 =3D priv->integr_period_usecs; return IIO_VAL_FRACTIONAL; =20 default: @@ -503,7 +502,7 @@ static int pac1921_lookup_scale(const int (*const scale= s_tbl)[2], size_t size, for (unsigned int i =3D 0; i < size; i++) if (scales_tbl[i][0] =3D=3D scale_val && scales_tbl[i][1] =3D=3D scale_val2) - return (int)i; + return i; =20 return -EINVAL; } @@ -553,7 +552,7 @@ static int pac1921_update_gain_from_scale(struct pac192= 1_priv *priv, if (ret < 0) return ret; =20 - return pac1921_update_gain(priv, &priv->dv_gain, (u8)ret, + return pac1921_update_gain(priv, &priv->dv_gain, ret, PAC1921_GAIN_DV_GAIN_MASK); case PAC1921_CHAN_VSENSE: ret =3D pac1921_lookup_scale(pac1921_vsense_scales, @@ -562,7 +561,7 @@ static int pac1921_update_gain_from_scale(struct pac192= 1_priv *priv, if (ret < 0) return ret; =20 - return pac1921_update_gain(priv, &priv->di_gain, (u8)ret, + return pac1921_update_gain(priv, &priv->di_gain, ret, PAC1921_GAIN_DI_GAIN_MASK); case PAC1921_CHAN_CURRENT: ret =3D pac1921_lookup_scale(priv->current_scales, @@ -571,7 +570,7 @@ static int pac1921_update_gain_from_scale(struct pac192= 1_priv *priv, if (ret < 0) return ret; =20 - return pac1921_update_gain(priv, &priv->di_gain, (u8)ret, + return pac1921_update_gain(priv, &priv->di_gain, ret, PAC1921_GAIN_DI_GAIN_MASK); default: return -EINVAL; @@ -586,7 +585,7 @@ static int pac1921_lookup_int_num_samples(int num_sampl= es) { for (unsigned int i =3D 0; i < ARRAY_SIZE(pac1921_int_num_samples); i++) if (pac1921_int_num_samples[i] =3D=3D num_samples) - return (int)i; + return i; =20 return -EINVAL; } @@ -607,7 +606,7 @@ static int pac1921_update_int_num_samples(struct pac192= 1_priv *priv, if (ret < 0) return ret; =20 - n_samples =3D (u8)ret; + n_samples =3D ret; =20 if (priv->n_samples =3D=3D n_samples) return 0; @@ -770,7 +769,7 @@ static ssize_t pac1921_read_shunt_resistor(struct iio_d= ev *indio_dev, =20 guard(mutex)(&priv->lock); =20 - vals[0] =3D (int)priv->rshunt_uohm; + vals[0] =3D priv->rshunt_uohm; vals[1] =3D MICRO; =20 return iio_format_value(buf, IIO_VAL_FRACTIONAL, 1, vals); @@ -793,13 +792,13 @@ static ssize_t pac1921_write_shunt_resistor(struct ii= o_dev *indio_dev, if (ret) return ret; =20 - rshunt_uohm =3D (u32)val * MICRO + (u32)val_fract; + rshunt_uohm =3D val * MICRO + val_fract; if (rshunt_uohm =3D=3D 0 || rshunt_uohm > INT_MAX) return -EINVAL; =20 guard(mutex)(&priv->lock); =20 - priv->rshunt_uohm =3D (u32)rshunt_uohm; + priv->rshunt_uohm =3D rshunt_uohm; =20 pac1921_calc_current_scales(priv); =20 @@ -1168,7 +1167,7 @@ static int pac1921_probe(struct i2c_client *client) =20 priv->regmap =3D devm_regmap_init_i2c(client, &pac1921_regmap_config); if (IS_ERR(priv->regmap)) - return dev_err_probe(dev, (int)PTR_ERR(priv->regmap), + return dev_err_probe(dev, PTR_ERR(priv->regmap), "Cannot initialize register map\n"); =20 devm_mutex_init(dev, &priv->lock); @@ -1191,7 +1190,7 @@ static int pac1921_probe(struct i2c_client *client) =20 priv->vdd =3D devm_regulator_get(dev, "vdd"); if (IS_ERR(priv->vdd)) - return dev_err_probe(dev, (int)PTR_ERR(priv->vdd), + return dev_err_probe(dev, PTR_ERR(priv->vdd), "Cannot get vdd regulator\n"); =20 ret =3D regulator_enable(priv->vdd); --- base-commit: fec496684388685647652ab4213454fbabdab099 change-id: 20240911-iio-pac1921-nocast-5c98cdeec059 Best regards, --=20 Matteo Martelli