Re: IA32 (2.6.9-rc3 - 2004-09-29.21.30) - 10 New warnings (gcc 3.2.2)

From: Alan Cox (alan_at_lxorguk.ukuu.org.uk)
Date: 09/30/04

  • Next message: Takashi Iwai: "Re: [PATCH] 2.6.9-rc3 fix warnings in sound/drivers/opl3/opl3_lib.c"
    To: John Cherry <cherry@osdl.org>
    Date:	Thu, 30 Sep 2004 16:01:52 +0100
    
    
    

    On Iau, 2004-09-30 at 15:32, John Cherry wrote:
    > drivers/net/wan/pc300_tty.c:704: warning: passing arg 1 of `tty_ldisc_ref' from incompatible pointer type

    I've got a patch for this and the other 2 64bit warnings in that file.
    - Fix the ldisc ref bug
    - Fix the 64bit pointer cast warnings
    - Fix the coding style from "Alan bracketing" to "Linus bracketing"

    
    

    --- ../linux.vanilla-2.6.9rc3/drivers/net/wan/pc300_tty.c 2004-09-30 16:13:09.075305760 +0100
    +++ drivers/net/wan/pc300_tty.c 2004-09-30 17:02:18.400940112 +0100
    @@ -192,13 +192,14 @@
      */
     void cpc_tty_init(pc300dev_t *pc300dev)
     {
    - int port, aux;
    + unsigned long port;
    + int aux;
             st_cpc_tty_area * cpc_tty;
     
             /* hdlcX - X=interface number */
             port = pc300dev->dev->name[4] - '0';
             if (port >= CPC_TTY_NPORTS) {
    - printk("%s-tty: invalid interface selected (0-%i): %i",
    + printk("%s-tty: invalid interface selected (0-%i): %li",
                             pc300dev->dev->name,
                             CPC_TTY_NPORTS-1,port);
                     return;
    @@ -682,7 +683,8 @@
      */
     static void cpc_tty_rx_work(void * data)
     {
    - int port, i, j;
    + unsigned long port;
    + int i, j;
             st_cpc_tty_area *cpc_tty;
             volatile st_cpc_rx_buf * buf;
             char flags=0,flg_rx=1;
    @@ -693,18 +695,15 @@
             
             for (i=0; (i < 4) && flg_rx ; i++) {
                     flg_rx = 0;
    - port = (int) data;
    + port = (unsigned long) data;
                     for (j=0; j < CPC_TTY_NPORTS; j++) {
                             cpc_tty = &cpc_tty_area[port];
                     
                             if ((buf=cpc_tty->buf_rx.first) != 0) {
    -
    - if(cpc_tty->tty)
    - {
    - ld = tty_ldisc_ref(cpc_tty);
    - if(ld)
    - {
    - if (ld->receive_buf)) {
    + if(cpc_tty->tty) {
    + ld = tty_ldisc_ref(cpc_tty->tty);
    + if(ld) {
    + if (ld->receive_buf) {
                                                             CPC_TTY_DBG("%s: call line disc. receive_buf\n",cpc_tty->name);
                                                             ld->receive_buf(cpc_tty->tty, (char *)(buf->data), &flags, buf->size);
                                                     }
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/


  • Next message: Takashi Iwai: "Re: [PATCH] 2.6.9-rc3 fix warnings in sound/drivers/opl3/opl3_lib.c"

    Relevant Pages

    • Re: 2.6.0-test8-mm1
      ... Do you consider this a "real fix" for the various ... Sort of the same question, were the warnings really bogus, or were ... | A slab debugging tool for finding memory leaks: ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • [git pull for 2.6.22-rc4 - resend] kbuild: fix sh64 section mismatch problems
      ... Please apply following 2 liners fix. ... It will fix a lot of false section mismatch warnings on sh64 and ... Subject: kbuild: fix sh64 section mismatch problems ... Silence the warnings by allowing references from .init.text to .cranges. ...
      (Linux-Kernel)
    • [PATCH for -rc] kbuild: fix sh64 section mismatch problems
      ... Please apply following 2 liners fix. ... It will fix a lot of false section mismatch warnings on sh64 and ... Subject: kbuild: fix sh64 section mismatch problems ... Silence the warnings by allowing references from .init.text to .cranges. ...
      (Linux-Kernel)
    • 2.6.12-rc3-mm2: fs/proc/task_mmu.c warnings
      ... proc-pid-smaps.patch caused the following warnings with ... I do not understand how to fix this properly. ... There had been need of rain for many days. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • [PATCH] fix warnings in drivers/mtd/devices/doc200?.c
      ... What goes on in both files is identical, and my attempted fix is also ... expected argument types and thus kill the warnings. ... I've only compile tested these patches. ... To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ ...
      (Linux-Kernel)