On Tue, 2020-11-03 at 10:17 +0100, John Crispin wrote:
> +/**
> + * enum ieee80211_bcn_tmpl_ema - EMA beacon generation type
> + * @IEEE80211_BCN_EMA_NONE: don't generate an EMA beacon.
> + * @IEEE80211_BCN_EMA_NEXT: generate the next periodicity beacon.
> + * @IEEE80211_BCN_EMA_INDEX: generate beacon by periodicity index
> + * if the value is >= this enum value.
> + */
> +enum ieee80211_bcn_tmpl_ema {
> + IEEE80211_BCN_EMA_NONE = -2,
> + IEEE80211_BCN_EMA_NEXT = -1,
> + IEEE80211_BCN_EMA_INDEX = 0,
> +};
I don't see this used in the new API below?
> +/** struct ieee80211_ema_bcn_list - list entry of an EMA beacon
missing newline
> + * Return: The nuber of entries in the list or 0 on error.
> + */
> +
> +int
> +ieee80211_beacon_get_template_ema_list(struct ieee80211_hw *hw,
could remove that newline and put the int on the same line, I guess?
> + struct ieee80211_vif *vif,
> + struct list_head *head);
> +
> +/**
> + * ieee80211_beacon_free_ema_list - free an EMA beacon template list
> + * @head: linked list head containing &struct ieee80211_ema_bcn_list pointers.
> + *
> + * This function will free a list previously acquired by calling
> + * ieee80211_beacon_get_template_ema_list()
> + */
> +
> +void
> +ieee80211_beacon_free_ema_list(struct list_head *head);
certainly here void fits on the line.
johannes
