[PATCH 4/4] cleanup firmware loading printks

From: Johannes Stezenbach (js_at_linuxtv.org)
Date: 01/25/05

  • Next message: john stultz: "[RFC][PATCH] new timeofday arch specific timesources (v. A2)"
    Date:	Tue, 25 Jan 2005 01:31:41 +0100
    To: Linus Torvalds <torvalds@osdl.org>
    
    

    - [DVB] frontends: sp887x: improve confusing firmware loading messages

    Signed-off-by: Johannes Stezenbach <js@linuxtv.org>

    diff -rupN linux-2.6.11-rc2-bk2/drivers/media/dvb/frontends/sp8870.c linux-2.6.11-rc2-bk2-dvb/drivers/media/dvb/frontends/sp8870.c
    --- linux-2.6.11-rc2-bk2/drivers/media/dvb/frontends/sp8870.c 2004-12-24 22:35:40.000000000 +0100
    +++ linux-2.6.11-rc2-bk2-dvb/drivers/media/dvb/frontends/sp8870.c 2005-01-25 00:40:10.000000000 +0100
    @@ -313,7 +313,7 @@ static int sp8870_init (struct dvb_front
     
     
             /* request the firmware, this will block until someone uploads it */
    - printk("sp8870: waiting for firmware upload...\n");
    + printk("sp8870: waiting for firmware upload (%s)...\n", SP8870_DEFAULT_FIRMWARE);
             if (state->config->request_firmware(fe, &fw, SP8870_DEFAULT_FIRMWARE)) {
                     printk("sp8870: no firmware upload (timeout or file not found?)\n");
                     release_firmware(fw);
    @@ -325,6 +325,7 @@ static int sp8870_init (struct dvb_front
                     release_firmware(fw);
                     return -EIO;
             }
    + printk("sp8870: firmware upload complete\n");
     
             /* enable TS output and interface pins */
             sp8870_writereg(state, 0xc18, 0x00d);
    diff -rupN linux-2.6.11-rc2-bk2/drivers/media/dvb/frontends/sp887x.c linux-2.6.11-rc2-bk2-dvb/drivers/media/dvb/frontends/sp887x.c
    --- linux-2.6.11-rc2-bk2/drivers/media/dvb/frontends/sp887x.c 2004-12-24 22:35:24.000000000 +0100
    +++ linux-2.6.11-rc2-bk2-dvb/drivers/media/dvb/frontends/sp887x.c 2005-01-25 00:41:31.000000000 +0100
    @@ -518,7 +518,7 @@ static int sp887x_init(struct dvb_fronte
     
             if (!state->initialised) {
             /* request the firmware, this will block until someone uploads it */
    - printk("sp887x: waiting for firmware upload...\n");
    + printk("sp887x: waiting for firmware upload (%s)...\n", SP887X_DEFAULT_FIRMWARE);
                     ret = state->config->request_firmware(fe, &fw, SP887X_DEFAULT_FIRMWARE);
             if (ret) {
                     printk("sp887x: no firmware upload (timeout or file not found?)\n");
    @@ -531,6 +531,7 @@ static int sp887x_init(struct dvb_fronte
                             release_firmware(fw);
                             return ret;
             }
    + printk("sp887x: firmware upload complete\n");
                     state->initialised = 1;
             }
     

    -
    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 stultz: "[RFC][PATCH] new timeofday arch specific timesources (v. A2)"

    Relevant Pages

    • 2.6.10 - problem with cx88-cards.c
      ... Relevant configs: ... # Supported DVB Frontends ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [linux-dvb-maintainer] PATCH: DVB bt8xx in 2.6.10
      ... > functions and returning NULL instead for unconfigured frontends. ... > To keep this patch small, no attempt is made to #ifdef away other static functions or data ... unless all four frontends relevant to bt8xx based cards are selected. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)