diff mbox series

hwmon: (stts751) Remove an unused field in struct stts751_priv

Message ID 94ccf9caaa6b0101351bf381f09f4428c5e0835c.1714511322.git.christophe.jaillet@wanadoo.fr (mailing list archive)
State Accepted
Headers show
Series hwmon: (stts751) Remove an unused field in struct stts751_priv | expand

Commit Message

Christophe JAILLET April 30, 2024, 9:09 p.m. UTC
In "struct stts751_priv", the 'smbus_timeout' field is unused.
Remove it.

Found with cppcheck, unusedStructMember.

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
---
Compile tested only.

This field was added in the initial commit 7f07ec0fa17a ("hwmon: new driver
for ST stts751 thermal sensor") but was never used.
---
 drivers/hwmon/stts751.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Guenter Roeck May 1, 2024, 2:45 p.m. UTC | #1
On Tue, Apr 30, 2024 at 11:09:07PM +0200, Christophe JAILLET wrote:
> In "struct stts751_priv", the 'smbus_timeout' field is unused.
> Remove it.
> 
> Found with cppcheck, unusedStructMember.
> 
> Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>

Applied.

Thanks,
Guenter
diff mbox series

Patch

diff --git a/drivers/hwmon/stts751.c b/drivers/hwmon/stts751.c
index 847c99376930..867410afa4f8 100644
--- a/drivers/hwmon/stts751.c
+++ b/drivers/hwmon/stts751.c
@@ -91,7 +91,6 @@  struct stts751_priv {
 	int event_max, event_min;
 	int therm;
 	int hyst;
-	bool smbus_timeout;
 	int temp;
 	unsigned long last_update, last_alert_update;
 	u8 config;