On Mon, Sep 24, 2007 at 01:51:17AM -0700, Jonathan Campbell wrote:

 > +#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.

        Dave

-- 
http://www.codemonkey.org.uk
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to