Re: [PATCH next v6 2/5] bonding: Implement port churn-machine (AD standard 43.4.17).

[David Miller <davem@davemloft.net>]

From: Mahesh Bandewar <maheshb@google.com>
Date: Wed, 18 Feb 2015 22:50:19 -0800

> @@ -1131,6 +1137,44 @@ static void ad_rx_machine(struct lacpdu *lacpdu, struct port *port)
>  	}
>  }
>  
> +/* ad_churn_machine - handle port churn's state machine
> + * @port: the port we're looking at
> + *
> + */

Please don't invent new styles for the function comments, do it like it is
done in the rest of this file, for example, the very next funtion does:

>  /**
>   * ad_tx_machine - handle a port's tx state machine
>   * @port: the port we're looking at
> +	static const char *const churn_description[] =
> +		{ "monitoring",
> +		  "churned",
> +		  "none",
> +		  "unknown"
> +		};
> +	int max_size = sizeof(churn_description) / sizeof(churn_description[0]);

Please format this as:

> +	static const char *const churn_description[] = {
> +		"monitoring",
> +		"churned",
> +		"none",
> +		"unknown"
> +	};