On Tue, Aug 28, 2007 at 12:01:30PM -0400, Jiri Slaby wrote: > +config ATH5K_AR5210 > + bool "Support AR5210" > + depends on ATH5K > + default y > + > +config ATH5K_AR5211 > + bool "Support AR5211" > + depends on ATH5K > + default y > + > +config ATH5K_AR5212 > + bool "Support AR5212" > + depends on ATH5K > + default y Please don't add more default statements. Also this whole patch seems rather pointless. It saves only very little and turns the driver into a complete ifdef maze. - To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html