lkml.org 
[lkml]   [2004]   [Dec]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [2.6.9+] PnPBIOS: Missing SMALL_TAG_ENDDEP tag
Rene Herman wrote:
> Hi Adam.
>
> Between 2.6.8 and 2.6.9, the following patch to rsparser.c was merged:
>
> http://linus.bkbits.net:8080/linux-2.5/cset@414703f7MEe33PTYY-aFQaM3CLKjZw?nav=index.html|src/|src/drivers|src/drivers/pnp|src/drivers/pnp/pnpbios|related/drivers/pnp/pnpbios/rsparser.c
>
>
> The added warning triggers on my machine:
>
> Linux Plug and Play Support v0.97 (c) Adam Belay
> PnPBIOS: Scanning system for PnP BIOS support...
> PnPBIOS: Found PnP BIOS installation structure at 0xc00f7740
> PnPBIOS: PnP BIOS version 1.0, entry 0xf0000:0x6634, dseg 0xf0000
> PnPBIOS: Missing SMALL_TAG_ENDDEP tag
> PnPBIOS: Missing SMALL_TAG_ENDDEP tag
> PnPBIOS: Missing SMALL_TAG_ENDDEP tag
> PnPBIOS: Missing SMALL_TAG_ENDDEP tag
> PnPBIOS: 13 nodes reported by PnP BIOS; 13 recorded by driver
>
> I don't believe those warnings should be printed, what with "broken" the
> expected state of anything coming from the BIOS. The attached patch
> removes them again. Works for me...
>
> Rene.
>
>
>
>
>
>
> ------------------------------------------------------------------------
>
> --- linux-2.6.10-rc3.orig/drivers/pnp/pnpbios/rsparser.c 2004-12-04 03:10:03.000000000 +0100
> +++ linux-2.6.10-rc3/drivers/pnp/pnpbios/rsparser.c 2004-12-06 01:12:50.000000000 +0100
> @@ -433,14 +433,10 @@
> case SMALL_TAG_ENDDEP:
> if (len != 0)
> goto len_err;
> - if (option_independent == option)
> - printk(KERN_WARNING "PnPBIOS: Missing SMALL_TAG_STARTDEP tag\n");
this one shouldn't be removed
> option = option_independent;
> break;
>
> case SMALL_TAG_END:
> - if (option_independent != option)
> - printk(KERN_WARNING "PnPBIOS: Missing SMALL_TAG_ENDDEP tag\n");
ok for this one, may be change it to pnp_dbg(...)
> p = p + 2;
> return (unsigned char *)p;
> break;

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

\
 
 \ /
  Last update: 2005-03-22 14:08    [W:0.037 / U:0.392 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site