power: supply: surface-charger: Fix type of integer variable

qzed · sre · commit 601423bc0c06 · 2021-06-03T20:53:54.000+02:00
The ac->state field is __le32, not u32. So change the variable we're temporarily storing it in to __le32 as well. Reported-by: kernel test robot <lkp@intel.com> Fixes: e61ffb3 ("power: supply: Add AC driver for Surface Aggregator Module") Signed-off-by: Maximilian Luz <luzmaximilian@gmail.com> Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
diff --git a/drivers/power/supply/surface_charger.c b/drivers/power/supply/surface_charger.c
@@ -66,7 +66,7 @@ struct spwr_ac_device {
 
 static int spwr_ac_update_unlocked(struct spwr_ac_device *ac)
 {
-	u32 old = ac->state;
+	__le32 old = ac->state;
 	int status;
 
 	lockdep_assert_held(&ac->lock);

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ struct spwr_ac_device {`
`66`	`66`
`67`	`67`	`static int spwr_ac_update_unlocked(struct spwr_ac_device *ac)`
`68`	`68`	`{`
`69`		`- u32 old = ac->state;`
	`69`	`+ __le32 old = ac->state;`
`70`	`70`	`int status;`
`71`	`71`
`72`	`72`	`lockdep_assert_held(&ac->lock);`