[2.4 patch] fix two IDE warnings

From: Adrian Bunk (bunk_at_fs.tum.de)
Date: 01/26/04

  • Next message: John Stoffel: "Re: [patch] Re: Kernels > 2.6.1-mm3 do not boot. - SOLVED"
    Date:	Mon, 26 Jan 2004 04:19:21 +0100
    To: andre@linux-ide.org, Marcelo Tosatti <marcelo.tosatti@cyclades.com>
    
    

    I got the following warnings in 2.4.25-pre7:

    <-- snip -->

    ...
    gcc-2.95 -D__KERNEL__
    -I/home/bunk/linux/kernel-2.4/linux-2.4.25-pre7-full/include -Wall
    -Wstrict-prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common
    -fomit-frame-pointer -pipe -mpreferred-stack-boundary=2 -march=k6 -I../
    -nostdinc -iwithprefix include -DKBUILD_BASENAME=siimage -c -o
    siimage.o siimage.c
    siimage.c: In function `pdev_is_sata':
    siimage.c:65: warning: control reaches end of non-void function
    ...
    gcc-2.95 -D__KERNEL__
    -I/home/bunk/linux/kernel-2.4/linux-2.4.25-pre7-full/include -Wall
    -Wstrict-prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common
    -fomit-frame-pointer -pipe -mpreferred-stack-boundary=2 -march=k6 -I../
    -nostdinc -iwithprefix include -DKBUILD_BASENAME=generic -c -o
    generic.o generic.c
    generic.h:151: warning: `unknown_chipset' defined but not used
    ...

    <-- snip -->

    The patch below (completely stolen from 2.6) fixes these two warnings.

    Please apply
    Adrian

    --- linux-2.4.25-pre7-full/drivers/ide/pci/siimage.c.old 2004-01-26 04:02:58.000000000 +0100
    +++ linux-2.4.25-pre7-full/drivers/ide/pci/siimage.c 2004-01-26 04:03:16.000000000 +0100
    @@ -62,6 +62,7 @@
                             return 0;
             }
             BUG();
    + return 0;
     }
      
     /**
    --- linux-2.4.25-pre7-full/drivers/ide/pci/generic.h.old 2004-01-26 04:05:31.000000000 +0100
    +++ linux-2.4.25-pre7-full/drivers/ide/pci/generic.h 2004-01-26 04:05:46.000000000 +0100
    @@ -148,6 +148,7 @@
             }
     };
     
    +#if 0
     static ide_pci_device_t unknown_chipset[] __devinitdata = {
             { /* 0 */
                     .vendor = 0,
    @@ -170,5 +171,6 @@
             }
     
     };
    +#endif
     
     #endif /* IDE_GENERIC_H */
    -
    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: John Stoffel: "Re: [patch] Re: Kernels > 2.6.1-mm3 do not boot. - SOLVED"

    Relevant Pages

    • [-mm patch] kernel/signal.c: fix compile warning
      ... warnings... ... kernel/signal.c:638: warning: control may reach end of non-void function 'sigisemptyset' being inlined ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: K&R Ex 1-3
      ... so any ANSI C conforming compiler is not required to issue ... Compilers talk in terms of warnings and errors. ... "control reaches the end of a non-void function" ... then the compiler needs to prove that there exists a function ...
      (comp.lang.c)
    • Re: non-void function
      ... > Compiling it with gcc, I get the warnings: ... > Seems like standard requires non-void function to return something, ...
      (comp.lang.c)
    • Character Repeat Broken, ZA events
      ... Event Source: Service Control Manager ... Event Type: Warning ... The log number is different for each of the warnings. ...
      (microsoft.public.windowsxp.general)
    • Character Repeat Broken, ZA events
      ... Event Source: Service Control Manager ... Event Type: Warning ... The log number is different for each of the warnings. ...
      (comp.security.firewalls)