On Sat, May 01, 2010 at 03:46:16PM +0200, Sven Eckelmann wrote:
From: Andrea Gelmini andrea.gelmini@gelma.net
drivers/staging/batman-adv/send.c:137: CHECK: multiple assignments should be avoided
Signed-off-by: Andrea Gelmini andrea.gelmini@gelma.net Signed-off-by: Greg Kroah-Hartman gregkh@suse.de
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