Hi Johannes, On Mon, 2009-07-27 at 15:29 -0700, Johannes Berg wrote: > On Fri, 2009-07-24 at 11:13 -0700, Reinette Chatre wrote: > > > /** > > + * struct iwl_tt_restriction - Thermal Throttling restriction table used > > + * by advance thermal throttling management > > + * based on the current thermal throttling state, determine > > + * number of tx/rx streams; and the status of HT operation > > + * @tx_stream: number of tx stream allowed > > + * @is_ht: ht enable/disable > > + * @rx_stream: number of rx stream allowed > > + */ > > This kernel-doc, and some others, are really botched btw -- the bit > after 'struct foo - ' really needs to be a single line only, you can put > longer descriptions after the parameters. Sorry for missing this. We are trying to be better with documentation and need to do more work in this area. This patch is currently in wireless-testing. I am not sure how to proceed here. If I send a cleanup patch on top of current wireless-testing it will only make sense to address other kernel-doc mishaps. As our driver has not been tested (afaik) to provide meaningful documentation when kernel-doc is used I am hesitant to take on this right now ... but it is something we have to address at some point. Reinette -- 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
