On Sep 24 2007 15:09, Dave Jones wrote:
quoted text
>
> > +#if defined(__i386__) && defined(CONFIG_DMI)
> >  	dmi_check_system(acpi_dmi_table);
> >  #endif
> >  
> > +#ifdef CONFIG_DMI
> >  	dmi_scan_machine();
> > +#endif
> >  
> > +#ifdef CONFIG_DMI
> >  	/* Check and install the TSC clocksource */
> >  	dmi_check_system(bad_tsc_dmi_table);
> > +#endif
> >  
> > +#ifdef CONFIG_DMI
> >  	dmi_check_system(acpi_osl_dmi_table);
> > +#endif
> 
>Instead of adding all these ifdefs, we could just define
>add something along the lines of..
>
>#ifndef CONFIG_DMI
>#define dmi_check_system do {} while (0)
>#endif
>
>in some header, which hides the uglies away from the code
>whilst having the same net effect.

Need #define dmi_check_system(x) otherwise you get

	/* Check and install the TSC clocksource */
	do {} while (0) (bad_tsc_dmi_table);
-
unsubscribe notice
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/