On Fri, May 28, 2010 at 11:09:12PM -0400, Chris Metcalf wrote: > +config ZONE_DMA > + def_bool y > + Do you really want ZONE_DMA? Looking through the code it seems like you are just using this in place of ZONE_NORMAL instead of for dealing with any specific DMA limitations. > +config CC_OPTIMIZE_FOR_SIZE > + def_bool y > + This is a duplicate of the init/Kconfig entry. If you always want this enabled you can select it. > +config CLOCKSOURCE_WATCHDOG > + def_bool y > + Are you also sure that you want this? It doesn't seem like you have any of the clocksource stability issues that x86 does, so it's not obvious why you are enabling this. > +config ARCH_DISCONTIGMEM_ENABLE > + def_bool y > + > +config ARCH_DISCONTIGMEM_DEFAULT > + def_bool y > + Have you considered sparsemem instead? > +# SMP is required for Tilera Linux. > +config SMP > + def_bool y > + Forcing on SMP is fairly unusual, you do not support booting UP kernels at all? > +config SERIAL_CONSOLE > + def_bool y > + This seems unused and looks like it was just copied over from some other architecture? > +config HVC_TILE > + select HVC_DRIVER > + def_bool y > + > +config TILE > + def_bool y > + select GENERIC_FIND_FIRST_BIT > + select GENERIC_FIND_NEXT_BIT > + select RESOURCES_64BIT > + select USE_GENERIC_SMP_HELPERS > + RESOURCES_64BIT is more legacy stuff, you don't need this anymore by virtue of the 64-bit phys_addr_t that you're already forcing on. > +menu "Bus options" > + > +config NO_IOMEM > + bool > + def_bool !PCI > + Have you inverted the logic here? Judging from your I/O routines it's the PIO stuff you want disabled, not MMIO. As such, it's NO_IOPORT that you want. Some of the PCI drivers will still use inb/outb and friends for PCI IO space so disabling it for the !PCI case is fine. -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html