Re: [2.6 patch] net/hamradio/dmascc: remove inlines

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Adrian Bunk wrote:
Trying to compile drivers/scsi/ips.c with gcc 3.4 and
  # define inline         __inline__ __attribute__((always_inline))
results in compile errors starting with the following:

<--  snip  -->

...
CC drivers/net/hamradio/dmascc.o
drivers/net/hamradio/dmascc.c: In function `scc_isr':
drivers/net/hamradio/dmascc.c:250: sorry, unimplemented: inlining failed in call to 'z8530_isr': function body not available
drivers/net/hamradio/dmascc.c:969: sorry, unimplemented: called from here
drivers/net/hamradio/dmascc.c:250: sorry, unimplemented: inlining failed in call to 'z8530_isr': function body not available
drivers/net/hamradio/dmascc.c:978: sorry, unimplemented: called from here
make[3]: *** [drivers/net/hamradio/dmascc.o] Error 1


<--  snip  -->


The patch below removes all inlines from dmascc.c .


diffstat output: drivers/net/hamradio/dmascc.c | 20 ++++++++++---------- 1 files changed, 10 insertions(+), 10 deletions(-)


Signed-off-by: Adrian Bunk <bunk@fs.tum.de>

--- linux-2.6.7-mm7-full-gcc3.4/drivers/net/hamradio/dmascc.c.old 2004-07-10 02:51:07.000000000 +0200
+++ linux-2.6.7-mm7-full-gcc3.4/drivers/net/hamradio/dmascc.c 2004-07-10 03:00:43.000000000 +0200
@@ -247,7 +247,7 @@
static int scc_set_mac_address(struct net_device *dev, void *sa);
static irqreturn_t scc_isr(int irq, void *dev_id, struct pt_regs * regs);
-static inline void z8530_isr(struct scc_info *info);
+static void z8530_isr(struct scc_info *info);
static void rx_isr(struct scc_priv *priv);
static void special_condition(struct scc_priv *priv, int rc);
static void rx_bh(void *arg);
@@ -255,11 +255,11 @@
static void es_isr(struct scc_priv *priv);
static void tm_isr(struct scc_priv *priv);
-static inline void tx_on(struct scc_priv *priv);
-static inline void rx_on(struct scc_priv *priv);
-static inline void rx_off(struct scc_priv *priv);
+static void tx_on(struct scc_priv *priv);
+static void rx_on(struct scc_priv *priv);
+static void rx_off(struct scc_priv *priv);
static void start_timer(struct scc_priv *priv, int t, int r15);
-static inline unsigned char random(void);
+static unsigned char random(void);
/* Initialization variables */
@@ -981,7 +981,7 @@
}
-static inline void z8530_isr(struct scc_info *info) {
+static void z8530_isr(struct scc_info *info) {
int is, i = 100;
while ((is = read_scc(&info->priv[0], R3)) && i--) {
@@ -1294,7 +1294,7 @@
}
-static inline void tx_on(struct scc_priv *priv) {
+static void tx_on(struct scc_priv *priv) {
int i, n;
unsigned long flags;
@@ -1330,7 +1330,7 @@
}
-static inline void rx_on(struct scc_priv *priv) {
+static void rx_on(struct scc_priv *priv) {
unsigned long flags;
/* Clear RX FIFO */
@@ -1364,7 +1364,7 @@
}
-static inline void rx_off(struct scc_priv *priv) {
+static void rx_off(struct scc_priv *priv) {
/* Disable receiver */
write_scc(priv, R3, Rx8);
/* Disable DREQ / RX interrupt */
@@ -1397,7 +1397,7 @@
}
-static inline unsigned char random(void) {
+static unsigned char random(void) {
/* See "Numerical Recipes in C", second edition, p. 284 */
rand = rand * 1664525L + 1013904223L;
return (unsigned char) (rand >> 24);




It seems like you are going directly against the intent of the author, with this patch.

	Jeff


- : send the line "unsubscribe linux-net" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux 802.1Q VLAN]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Git]     [Bugtraq]     [Yosemite News and Information]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux PCI]     [Linux Admin]     [Samba]

  Powered by Linux