Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1313892 > unrolled thread

[PATCH] net/irda: bfin_sir: remove duplicate defines

Started bySudip Mukherjee <sudipm.mukherjee@gmail.com>
First post2016-01-21 06:20 +0100
Last post2016-01-21 19:50 +0100
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] net/irda: bfin_sir: remove duplicate defines Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2016-01-21 06:20 +0100
    Re: [PATCH] net/irda: bfin_sir: remove duplicate defines David Miller <davem@davemloft.net> - 2016-01-21 19:50 +0100

#1313892 — [PATCH] net/irda: bfin_sir: remove duplicate defines

FromSudip Mukherjee <sudipm.mukherjee@gmail.com>
Date2016-01-21 06:20 +0100
Subject[PATCH] net/irda: bfin_sir: remove duplicate defines
Message-ID<qTfP5-x3-29@gated-at.bofh.it>
The defconfig build of blackfin is failing with the error:

arch/blackfin/include/asm/bfin_serial.h:269:0: warning: "port_membase" redefined
drivers/net/irda/bfin_sir.h:85:0: note: this is the location of the previous definition
arch/blackfin/include/asm/bfin_serial.h:382:0: warning: "get_lsr_cache" redefined
drivers/net/irda/bfin_sir.h:86:0: note: this is the location of the previous definition
arch/blackfin/include/asm/bfin_serial.h:383:0: warning: "put_lsr_cache" redefined
drivers/net/irda/bfin_sir.h:87:0: note: this is the location of the previous definition

port_membase, get_lsr_cache, put_lsr_cache are already defined in the
architecture files, no need to define them again in the driver.

Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
---
 drivers/net/irda/bfin_sir.h | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/net/irda/bfin_sir.h b/drivers/net/irda/bfin_sir.h
index 29cbde8..d47cf14 100644
--- a/drivers/net/irda/bfin_sir.h
+++ b/drivers/net/irda/bfin_sir.h
@@ -82,9 +82,6 @@ struct bfin_sir_self {
 
 #define DRIVER_NAME "bfin_sir"
 
-#define port_membase(port)     (((struct bfin_sir_port *)(port))->membase)
-#define get_lsr_cache(port)    (((struct bfin_sir_port *)(port))->lsr)
-#define put_lsr_cache(port, v) (((struct bfin_sir_port *)(port))->lsr = (v))
 #include <asm/bfin_serial.h>
 
 static const unsigned short per[][4] = {
-- 
1.9.1

[toc] | [next] | [standalone]


#1314412

FromDavid Miller <davem@davemloft.net>
Date2016-01-21 19:50 +0100
Message-ID<qTssW-M0-13@gated-at.bofh.it>
In reply to#1313892
From: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
Date: Thu, 21 Jan 2016 10:42:31 +0530

> The defconfig build of blackfin is failing with the error:
> 
> arch/blackfin/include/asm/bfin_serial.h:269:0: warning: "port_membase" redefined
> drivers/net/irda/bfin_sir.h:85:0: note: this is the location of the previous definition
> arch/blackfin/include/asm/bfin_serial.h:382:0: warning: "get_lsr_cache" redefined
> drivers/net/irda/bfin_sir.h:86:0: note: this is the location of the previous definition
> arch/blackfin/include/asm/bfin_serial.h:383:0: warning: "put_lsr_cache" redefined
> drivers/net/irda/bfin_sir.h:87:0: note: this is the location of the previous definition
> 
> port_membase, get_lsr_cache, put_lsr_cache are already defined in the
> architecture files, no need to define them again in the driver.
> 
> Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>

Applied, thank you.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web