Re: 2.6.0-test5: ISDN kcapi.c no longer compiles

From: Karsten Keil (kkeil_at_suse.de)
Date: 09/15/03

  • Next message: Amir Hermelin: "Developing Kernel Code newbie"
    Date:	Mon, 15 Sep 2003 08:57:34 +0200
    To: Adrian Bunk <bunk@fs.tum.de>
    
    

    On Wed, Sep 10, 2003 at 06:57:42PM +0200, Adrian Bunk wrote:
    > On Mon, Sep 08, 2003 at 01:32:05PM -0700, Linus Torvalds wrote:
    > >...
    > > Summary of changes from v2.6.0-test4 to v2.6.0-test5
    > > ============================================
    > >...
    > > Karsten Keil:
    > >...
    > > o next fixes
    > >...
    >
    > It seems this change broke the compilation of kcapi.c:
    >

    Ah, with your .config now it's clear what was broken: none MODULE compile

    diff -ur -x '.built-in*' -x '.*cmd' linux-2.6.0-test5/drivers/isdn/capi/kcapi.c linux-2.6.0-test5-bk3/drivers/isdn/capi/kcapi.c
    --- linux-2.6.0-test5/drivers/isdn/capi/kcapi.c 2003-09-14 17:43:45.000000000 +0200
    +++ linux-2.6.0-test5-bk3/drivers/isdn/capi/kcapi.c 2003-09-14 22:39:28.000000000 +0200
    @@ -77,17 +77,21 @@
     static inline struct capi_ctr *
     capi_ctr_get(struct capi_ctr *card)
     {
    +#ifdef MODULE
             if (!try_module_get(card->owner))
                     return NULL;
             DBG("Reserve module: %s", card->owner->name);
    +#endif
             return card;
     }
     
     static inline void
     capi_ctr_put(struct capi_ctr *card)
     {
    +#ifdef MODULE
             module_put(card->owner);
             DBG("Release module: %s", card->owner->name);
    +#endif
     }
     
     /* ------------------------------------------------------------- */

    This should fix it.

    -- 
    Karsten Keil
    SuSE Labs
    ISDN development
    -
    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: Amir Hermelin: "Developing Kernel Code newbie"

    Relevant Pages