hollalinux
/
linux-sg2042
mirror of https://github.com/sophgo/linux-riscv.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

isdn: eicon: fix old-style declarations 

Modern C standards expect the '__inline__' keyword to come before the return
type in a declaration, and we get many warnings for this with "make W=1"
because the eicon driver has this in a header file:

eicon/divasmain.c:448:1: error: '__inline__' is not at beginning of declaration [-Werror=old-style-declaration]
eicon/divasmain.c:453:1: error: '__inline__' is not at beginning of declaration [-Werror=old-style-declaration]
eicon/divasmain.c:458:1: error: '__inline__' is not at beginning of declaration [-Werror=old-style-declaration]
eicon/divasmain.c:463:1: error: '__inline__' is not at beginning of declaration [-Werror=old-style-declaration]
eicon/divasmain.c:468:1: error: '__inline__' is not at beginning of declaration [-Werror=old-style-declaration]
eicon/divasmain.c:473:1: error: '__inline__' is not at beginning of declaration [-Werror=old-style-declaration]
eicon/platform.h:274:1: error: '__inline__' is not at beginning of declaration [-Werror=old-style-declaration]
eicon/platform.h:280:1: error: '__inline__' is not at beginning of declaration [-Werror=old-style-declaration]

A similar warning gets printed for the diva_os_register_io_port()
declaration, because 'register' is interpreted as a keyword instead
of a variable name:

In file included from eicon/diva_didd.c:21:0:
eicon/platform.h:206:1: error: 'register' is not at beginning of declaration [-Werror=old-style-declaration]

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Arnd Bergmann 2016-06-16 15:52:12 +02:00 committed by David S. Miller
parent 60a25d00db
commit ac565065a6
2 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
drivers/isdn/hardware/eicon/divasmain.c
View File

@ -445,32 +445,32 @@ void divasa_unmap_pci_bar(void __iomem *bar)
/********************************************************* /*********************************************************
** I/O port access ** I/O port access
*********************************************************/ *********************************************************/
byte __inline__ inpp(void __iomem *addr) inline byte inpp(void __iomem *addr)
{ {
return (inb((unsigned long) addr)); return (inb((unsigned long) addr));
} }
word __inline__ inppw(void __iomem *addr) inline word inppw(void __iomem *addr)
{ {
return (inw((unsigned long) addr)); return (inw((unsigned long) addr));
} }
void __inline__ inppw_buffer(void __iomem *addr, void *P, int length) inline void inppw_buffer(void __iomem *addr, void *P, int length)
{ {
insw((unsigned long) addr, (word *) P, length >> 1); insw((unsigned long) addr, (word *) P, length >> 1);
} }
void __inline__ outppw_buffer(void __iomem *addr, void *P, int length) inline void outppw_buffer(void __iomem *addr, void *P, int length)
{ {
outsw((unsigned long) addr, (word *) P, length >> 1); outsw((unsigned long) addr, (word *) P, length >> 1);
} }
void __inline__ outppw(void __iomem *addr, word w) inline void outppw(void __iomem *addr, word w)
{ {
outw(w, (unsigned long) addr); outw(w, (unsigned long) addr);
} }
void __inline__ outpp(void __iomem *addr, word p) inline void outpp(void __iomem *addr, word p)
{ {
outb(p, (unsigned long) addr); outb(p, (unsigned long) addr);
} }

6
drivers/isdn/hardware/eicon/platform.h
View File

@ -203,7 +203,7 @@ void PCIread(byte bus, byte func, int offset, void *data, int length, void *pci_
/* /*
** I/O Port utilities ** I/O Port utilities
*/ */
int diva_os_register_io_port(void *adapter, int register, unsigned long port, int diva_os_register_io_port(void *adapter, int reg, unsigned long port,
unsigned long length, const char *name, int id); unsigned long length, const char *name, int id);
/* /*
** I/O port access abstraction ** I/O port access abstraction
@ -271,13 +271,13 @@ void diva_os_get_time(dword *sec, dword *usec);
** atomic operation, fake because we use threads ** atomic operation, fake because we use threads
*/ */
typedef int diva_os_atomic_t; typedef int diva_os_atomic_t;
static diva_os_atomic_t __inline__ static inline diva_os_atomic_t
diva_os_atomic_increment(diva_os_atomic_t *pv) diva_os_atomic_increment(diva_os_atomic_t *pv)
{ {
*pv += 1; *pv += 1;
return (*pv); return (*pv);
} }
static diva_os_atomic_t __inline__ static inline diva_os_atomic_t
diva_os_atomic_decrement(diva_os_atomic_t *pv) diva_os_atomic_decrement(diva_os_atomic_t *pv)
{ {
*pv -= 1; *pv -= 1;