watchdog: rt2880-wdt: prefer unsigned int over unsigned

paraka · Wim Van Sebroeck · commit af3ac8e41786 · 2023-04-22T12:53:57.000+02:00
Instead of declare 'reg' variable in read and write operations as a bare 'unsigned' type prefer to declate it as 'unsigned int'. Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com> Reviewed-by: Guenter Roeck <linux@roeck-us.net> Link: https://lore.kernel.org/r/20230301065510.2818425-2-sergio.paracuellos@gmail.com Signed-off-by: Guenter Roeck <linux@roeck-us.net> Signed-off-by: Wim Van Sebroeck <wim@linux-watchdog.org>
diff --git a/drivers/watchdog/rt2880_wdt.c b/drivers/watchdog/rt2880_wdt.c
@@ -54,12 +54,12 @@ MODULE_PARM_DESC(nowayout,
 		"Watchdog cannot be stopped once started (default="
 		__MODULE_STRING(WATCHDOG_NOWAYOUT) ")");
 
-static inline void rt_wdt_w32(void __iomem *base, unsigned reg, u32 val)
+static inline void rt_wdt_w32(void __iomem *base, unsigned int reg, u32 val)
 {
 	iowrite32(val, base + reg);
 }
 
-static inline u32 rt_wdt_r32(void __iomem *base, unsigned reg)
+static inline u32 rt_wdt_r32(void __iomem *base, unsigned int reg)
 {
 	return ioread32(base + reg);
 }

Original file line number	Diff line number	Diff line change
`@@ -54,12 +54,12 @@ MODULE_PARM_DESC(nowayout,`
`54`	`54`	`"Watchdog cannot be stopped once started (default="`
`55`	`55`	`__MODULE_STRING(WATCHDOG_NOWAYOUT) ")");`
`56`	`56`
`57`		`-static inline void rt_wdt_w32(void __iomem *base, unsigned reg, u32 val)`
	`57`	`+static inline void rt_wdt_w32(void __iomem *base, unsigned int reg, u32 val)`
`58`	`58`	`{`
`59`	`59`	`iowrite32(val, base + reg);`
`60`	`60`	`}`
`61`	`61`
`62`		`-static inline u32 rt_wdt_r32(void __iomem *base, unsigned reg)`
	`62`	`+static inline u32 rt_wdt_r32(void __iomem *base, unsigned int reg)`
`63`	`63`	`{`
`64`	`64`	`return ioread32(base + reg);`
`65`	`65`	`}`