On Sat, May 01, 2010 at 03:46:16PM +0200, Sven Eckelmann wrote:
> From: Andrea Gelmini <[email protected]>
>
> drivers/staging/batman-adv/send.c:137: CHECK: multiple assignments should be
> avoided
>
> Signed-off-by: Andrea Gelmini <[email protected]>
> Signed-off-by: Greg Kroah-Hartman <[email protected]>
> ---
> drivers/staging/batman-adv/send.c | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/staging/batman-adv/send.c
> b/drivers/staging/batman-adv/send.c
> index 2a9fac8..ff7b1f1 100644
> --- a/drivers/staging/batman-adv/send.c
> +++ b/drivers/staging/batman-adv/send.c
> @@ -134,7 +134,8 @@ static void send_packet_to_if(struct forw_packet
> *forw_packet,
> if (batman_if->if_active != IF_ACTIVE)
> return;
>
> - packet_num = buff_pos = 0;
> + packet_num = 0;
> + buff_pos = 0;
> batman_packet = (struct batman_packet *)
> (forw_packet->packet_buff);
>
> --
I was too lazy to go searching upstream. We can just import this patch
and use your three checkpatch patches.
Andrew