You've already forked linux-apfs
mirror of
https://github.com/linux-apfs/linux-apfs.git
synced 2026-05-01 15:00:59 -07:00
long vs. unsigned long - low-hanging fruits in drivers
deal with signedness of the stuff passed to set_bit() et.al. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
@@ -1,7 +1,7 @@
|
||||
u32 scx200_gpio_configure(unsigned index, u32 set, u32 clear);
|
||||
|
||||
extern unsigned scx200_gpio_base;
|
||||
extern long scx200_gpio_shadow[2];
|
||||
extern unsigned long scx200_gpio_shadow[2];
|
||||
extern struct nsc_gpio_ops scx200_gpio_ops;
|
||||
|
||||
#define scx200_gpio_present() (scx200_gpio_base!=0)
|
||||
@@ -9,7 +9,7 @@ extern struct nsc_gpio_ops scx200_gpio_ops;
|
||||
/* Definitions to make sure I do the same thing in all functions */
|
||||
#define __SCx200_GPIO_BANK unsigned bank = index>>5
|
||||
#define __SCx200_GPIO_IOADDR unsigned short ioaddr = scx200_gpio_base+0x10*bank
|
||||
#define __SCx200_GPIO_SHADOW long *shadow = scx200_gpio_shadow+bank
|
||||
#define __SCx200_GPIO_SHADOW unsigned long *shadow = scx200_gpio_shadow+bank
|
||||
#define __SCx200_GPIO_INDEX index &= 31
|
||||
|
||||
#define __SCx200_GPIO_OUT __asm__ __volatile__("outsl":"=mS" (shadow):"d" (ioaddr), "0" (shadow))
|
||||
@@ -42,7 +42,7 @@ static inline void scx200_gpio_set_high(unsigned index) {
|
||||
__SCx200_GPIO_IOADDR;
|
||||
__SCx200_GPIO_SHADOW;
|
||||
__SCx200_GPIO_INDEX;
|
||||
set_bit(index, shadow);
|
||||
set_bit(index, shadow); /* __set_bit()? */
|
||||
__SCx200_GPIO_OUT;
|
||||
}
|
||||
|
||||
@@ -53,7 +53,7 @@ static inline void scx200_gpio_set_low(unsigned index) {
|
||||
__SCx200_GPIO_IOADDR;
|
||||
__SCx200_GPIO_SHADOW;
|
||||
__SCx200_GPIO_INDEX;
|
||||
clear_bit(index, shadow);
|
||||
clear_bit(index, shadow); /* __clear_bit()? */
|
||||
__SCx200_GPIO_OUT;
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user