Re: [PATCH V5 3/4] mac80211: add multiple bssid/EMA support to beacon handling

[Johannes Berg <johannes@sipsolutions.net>]

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