On Mon, Jun 18, 2012 at 5:38 PM, Stanislaw Gruszka <sgruszka@xxxxxxxxxx> wrote: > On Mon, Jun 18, 2012 at 11:01:54AM +0200, Gertjan van Wingerde wrote: >> Hi Xose, >> >> On Sat, Jun 16, 2012 at 11:36 PM, Xose Vazquez Perez >> <xose.vazquez@xxxxxxxxx> wrote: >> > removed, no support for: >> > >> > RF2853 2.4G/5G 3T3R >> > RF3322 2.4G 2T2R(RT3352/RT3371/RT3372/RT3391/RT3392) >> > RF3053 2.4G/5G 3T3R(RT3883/RT3563/RT3573/RT3593/RT3662) >> > >> > RT3593 >> > RT3883 >> > >> > Signed-off-by: Xose Vazquez Perez <xose.vazquez@xxxxxxxxx> >> >> I don't mind removing the RT chipset definitions, but I would like to >> retain the RF chipset definitions. Even >> though these definitions are not used in the code, the do represent >> values we could see from devices, so >> I consider the list to be documentation as well. > > Perhaps we can comment them out, as compromise :-) Personally I don't see the point. In my opinion the defines can all stay as they are. They are all defines which are clearly related to the chipsets themselves, so they are useful, and the fact that they are currently unused shouldn't mean they should be deleted. Ivo -- 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