Hi 

Is there any was we can get the latest s2io driver changes accepted by
kernel community. I tried to get the latest source using 

#git pull
rsync://rsync.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git

The s2io.c which I got doesn't contain modification to UFO and the s2io
warning fix mentioned in the following mail.

Can any one let us know how we can get the s2io.c file which contain UFO and
"s2io warning fix" patch applied

We are planning for one more patch submission, to avoid any conflicts while
applying to 2.6 kernel we want the latest file 


Thanks
Ananda 

-----Original Message-----
From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] 
Sent: Wednesday, December 14, 2005 7:25 PM
To: [EMAIL PROTECTED]
Cc: [EMAIL PROTECTED]; [EMAIL PROTECTED]
Subject: [patch 1/1] git-netdev-all: s2io warning fix


From: Andrew Morton <[EMAIL PROTECTED]>

drivers/net/s2io.c: In function `s2io_txdl_getskb':
drivers/net/s2io.c:2023: warning: cast from pointer to integer of different
size
drivers/net/s2io.c: In function `s2io_open':
drivers/net/s2io.c:3325: warning: long long unsigned int format, u64 arg
(arg 3)
drivers/net/s2io.c:3333: warning: long long unsigned int format, u64 arg
(arg 3)
drivers/net/s2io.c: In function `s2io_eeprom_test':
drivers/net/s2io.c:4749: warning: long long unsigned int format, long
unsigned int arg (arg 3)
drivers/net/s2io.c:4749: warning: long long unsigned int format, u64 arg
(arg 4)
drivers/net/s2io.c:4768: warning: long long unsigned int format, long
unsigned int arg (arg 3)
drivers/net/s2io.c:4768: warning: long long unsigned int format, u64 arg
(arg 4)

I had to update this patch because more warnings have just appeared.

You cannot print a u64 with %l or %ll.  You do not know what type the
architecture is using.  It must be cast to a type which matches the printf
control string - unsigned long long.

The patch also fixes some overly-long strings.  Please try to keep the code
looking neat in an 80-col window.

Cc: Jeff Garzik <[EMAIL PROTECTED]>
Cc: Ananda Raju <[EMAIL PROTECTED]>
Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
---

 drivers/net/s2io.c |   16 +++++++++++-----
 1 files changed, 11 insertions(+), 5 deletions(-)

diff -puN drivers/net/s2io.c~git-netdev-all-s2io-warning-fix
drivers/net/s2io.c
--- 25-alpha/drivers/net/s2io.c~git-netdev-all-s2io-warning-fix 2005-12-14
19:25:05.000000000 -0800
+++ 25-alpha-akpm/drivers/net/s2io.c    2005-12-14 19:25:05.000000000 -0800
@@ -2020,7 +2020,7 @@ static struct sk_buff *s2io_txdl_getskb(
        u16 j, frg_cnt;
 
        txds = txdlp;
-       if (txds->Host_Control == (u64) nic->ufo_in_band_v) {
+       if (txds->Host_Control == (u64)(long)nic->ufo_in_band_v) {
                pci_unmap_single(nic->pdev, (dma_addr_t)
                        txds->Buffer_Pointer, sizeof(u64),
                        PCI_DMA_TODEVICE);
@@ -3323,7 +3323,7 @@ failed\n", dev->name);
                                          s2io_msix_fifo_handle, 0,
sp->desc1,
                                          sp->s2io_entries[i].arg);
                                DBG_PRINT(ERR_DBG, "%s @ 0x%llx\n",
sp->desc1, 
-
sp->msix_info[i].addr);
+                                   (unsigned long
long)sp->msix_info[i].addr);
                        } else {
                                sprintf(sp->desc2, "%s:MSI-X-%d-RX",
                                        dev->name, i);
@@ -3331,7 +3331,7 @@ failed\n", dev->name);
                                          s2io_msix_ring_handle, 0,
sp->desc2,
                                          sp->s2io_entries[i].arg);
                                DBG_PRINT(ERR_DBG, "%s @ 0x%llx\n",
sp->desc2, 
-
sp->msix_info[i].addr);
+                                    (unsigned long
long)sp->msix_info[i].addr);
                        }
                        if (err) {
                                DBG_PRINT(ERR_DBG, "%s: MSI-X-%d
registration \
@@ -4746,7 +4746,10 @@ static int s2io_eeprom_test(nic_t * sp, 
                fail = 1;
 
        if (ret_data != 0x012345) {
-               DBG_PRINT(ERR_DBG, "%s: eeprom test error at offset 0x4F0.
Data written %llx Data read %llx\n", dev->name, (u64)0x12345, ret_data); 
+               DBG_PRINT(ERR_DBG, "%s: eeprom test error at offset 0x4F0. "
+                       "Data written %llx Data read %llx\n",
+                       dev->name, (unsigned long long)0x12345,
+                       (unsigned long long)ret_data);
                fail = 1;
        }
 
@@ -4765,7 +4768,10 @@ static int s2io_eeprom_test(nic_t * sp, 
                fail = 1;
 
        if (ret_data != 0x012345) {
-               DBG_PRINT(ERR_DBG, "%s: eeprom test error at offset 0x7F0.
Data written %llx Data read %llx\n", dev->name, (u64)0x12345, ret_data); 
+               DBG_PRINT(ERR_DBG, "%s: eeprom test error at offset 0x7F0. "
+                       "Data written %llx Data read %llx\n",
+                       dev->name, (unsigned long long)0x12345,
+                       (unsigned long long)ret_data);
                fail = 1;
        }
 
_

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to