Re: [PATCH bluetooth-next 2/3] ieee802154: mrf24j40: remove return statement

[Alan Ott <alan@signal11.us>]

Make the first line of the commit message:
	mrf24j40: remove unnecessary return statement

On 09/16/2014 12:48 AM, Varka Bhadram wrote:
> This patch remove the return statement for void function.
> void function return statements are not generally useful.
>

Take out "this patch" and the second line. Make the comment:

	Remove the return statement in the void function.


> Signed-off-by: Varka Bhadram <varkab@cdac.in>
> ---
>   drivers/net/ieee802154/mrf24j40.c |    2 --
>   1 file changed, 2 deletions(-)
>
> diff --git a/drivers/net/ieee802154/mrf24j40.c b/drivers/net/ieee802154/mrf24j40.c
> index 466da57..2c617e3 100644
> --- a/drivers/net/ieee802154/mrf24j40.c
> +++ b/drivers/net/ieee802154/mrf24j40.c
> @@ -420,8 +420,6 @@ static void mrf24j40_stop(struct ieee802154_dev *dev)
>   		return;
>   	val |= 0x1|0x8; /* Set TXNIE and RXIE. Disable Interrupts */
>   	write_short_reg(devrec, REG_INTCON, val);
> -
> -	return;
>   }
>
>   static int mrf24j40_set_channel(struct ieee802154_dev *dev,
>