[2.4 patch] fix a compile warning in amd76x_pm.c

From: Adrian Bunk (bunk_at_fs.tum.de)
Date: 08/03/03

  • Next message: Stefan Jones: "Re: [2.6.0-test1] yenta_socket.c:yenta_get_status returns bad value compared to 2.4" 
    Date:	Sun, 3 Aug 2003 12:58:08 +0200
To: Marcelo Tosatti <marcelo@conectiva.com.br>, Johnathan Hicks <thetech@folkwolf.net>

    I got the following compile warning in 2.4.22-pre10:

    <-- snip -->

    ...
    gcc -D__KERNEL__
    -I/home/bunk/linux/kernel-2.4/linux-2.4.22-pre10-full/include -
    Wall -Wstrict-prototypes -Wno-trigraphs -O2 -fno-strict-aliasing
    -fno-common -pipe -mpreferred-stack-boundary=2 -march=k6 -nostdinc -iwithprefix
    include -DKBUILD_BASENAME=amd76x_pm -c -o amd76x_pm.o amd76x_pm.c
    amd76x_pm.c:483: warning: `activate_amd76x_SLP' defined but not used
    ...

    <-- snip -->

    The following patch puts this function under the same #ifdef as it's
    only caller:

    --- linux-2.4.22-pre10-full/drivers/char/amd76x_pm.c.old 2003-08-02 23:02:38.000000000 +0200
    +++ linux-2.4.22-pre10-full/drivers/char/amd76x_pm.c 2003-08-02 23:03:11.000000000 +0200
    @@ -474,7 +474,7 @@
     }
     #endif
     
    -
    +#ifdef AMD76X_POS
     /*
      * Activate sleep state via its ACPI register (PM1_CNT).
      */
    @@ -489,8 +489,6 @@
             outw(regshort, amd76x_pm_cfg.slp_reg);
     }
     
    -
    -#ifdef AMD76X_POS
     /*
      * Wrapper function to activate POS sleep state.
      */

    I've tested the compilation with 2.4.22-pre10.

    Please apply
    Adrian 

    -- 
       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed
-
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: Stefan Jones: "Re: [2.6.0-test1] yenta_socket.c:yenta_get_status returns bad value compared to 2.4"

    Relevant Pages