[PATCH] Fix compiler warning noise with FORE200E driver - Kernel

This is a discussion on [PATCH] Fix compiler warning noise with FORE200E driver - Kernel ; gcc throws these warnings with: CONFIG_ATM_FORE200E=m # CONFIG_ATM_FORE200E_PCA is not set drivers/atm/fore200e.c:2695: warning: 'fore200e_pca_detect' defined but not used drivers/atm/fore200e.c:2748: warning: 'fore200e_pca_remove_one' defined but not used By moving the #ifdef CONFIG_ATM_FORE200E_PCA around those two functions, the compiler warnings are silenced. (btw, ...

+ Reply to Thread
Results 1 to 2 of 2

Thread: [PATCH] Fix compiler warning noise with FORE200E driver

  1. [PATCH] Fix compiler warning noise with FORE200E driver

    gcc throws these warnings with:

    CONFIG_ATM_FORE200E=m
    # CONFIG_ATM_FORE200E_PCA is not set

    drivers/atm/fore200e.c:2695: warning: 'fore200e_pca_detect' defined but
    not used
    drivers/atm/fore200e.c:2748: warning: 'fore200e_pca_remove_one' defined
    but not used

    By moving the #ifdef CONFIG_ATM_FORE200E_PCA around those two functions,
    the compiler warnings are silenced.

    (btw, I'm not subscribed to lkml, so please CC me on any responses,
    although this is a wholly trivial patch, so I'm not expecting any.)

    Signed-off-by: Tom "spot" Callaway



  2. Re: [PATCH] Fix compiler warning noise with FORE200E driver

    From: "Tom \"spot\" Callaway"
    Date: Thu, 13 Dec 2007 11:21:40 -0500

    > gcc throws these warnings with:
    >
    > CONFIG_ATM_FORE200E=m
    > # CONFIG_ATM_FORE200E_PCA is not set
    >
    > drivers/atm/fore200e.c:2695: warning: 'fore200e_pca_detect' defined but
    > not used
    > drivers/atm/fore200e.c:2748: warning: 'fore200e_pca_remove_one' defined
    > but not used
    >
    > By moving the #ifdef CONFIG_ATM_FORE200E_PCA around those two functions,
    > the compiler warnings are silenced.
    >
    > (btw, I'm not subscribed to lkml, so please CC me on any responses,
    > although this is a wholly trivial patch, so I'm not expecting any.)
    >
    > Signed-off-by: Tom "spot" Callaway


    Applied, thanks Tom.

    --
    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/

+ Reply to Thread