Re: [PATCH} SMBIOS support

Dave Jones (davej@suse.de)
Sat, 27 Apr 2002 17:01:01 +0200


On Sat, Apr 27, 2002 at 09:45:10AM -0500, Corey Minyard wrote:
> >Any reason for initialising it there instead of using a subsys_initcall
> >from smbios.c ?
> Can you control other orders within the subsystem?

Use different types of _initcall. See include/linux/init.h

> Other things that come later in this subsystem might need it
> Actually, it might be needed before this, I've moved it to before the PCI
> initialization,
> since this contains interrupt information and possible PCI information.

Ok, I overlooked the relationship of PCI to SMBIOS.
As pci is initialised with subsys_initcall, you could initialise
earlier by using core_initcall.

> > > + char str[80];
> >Worth adding a if (len>80) return here in case of crap biosen?
> >Or am I overly paranoid?
> It's printing 16 hex characters every time, so it doesn't matter. And
> it's only debug code, anyway.

Ok.

Dave.

-- 
| Dave Jones.        http://www.codemonkey.org.uk
| SuSE Labs
-
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/