b.a.t.m.a.n.lists.open-mesh.org archive mirror
 help / color / mirror / Atom feed
From: Sven Eckelmann <sven.eckelmann@gmx.de>
To: greg@kroah.com
Cc: b.a.t.m.a.n@lists.open-mesh.net
Subject: [B.A.T.M.A.N.] Staging: batman-adv for 2.6.36 (5)
Date: Wed, 21 Jul 2010 00:25:19 +0200	[thread overview]
Message-ID: <1279664722-27500-1-git-send-email-sven.eckelmann@gmx.de> (raw)

Hi,

first thing: thanks for your help to get some review for batman-adv from davem
and the netdev ml. I've added some new points to the TODO list. They are partly
resolved or work is done to solve them, but will postpone them and get them
hopefully ready and tested for 2.6.37.

Nevertheless there are some smaller changes which want to resolve with 2.6.36.
One is to allow to build it together with dt3155 directly into the kernel and
the other one should fix that we try to change stats of a different device. The
latter one will clash with an unlucky try by the netdev guys to change the
behaviour of dev_get_stats and the batman-adv source code.

This merge conflict is quite easy to resolve. Just delete the dev_get_stats
line in hard-interface.c in batman_skb_recv and the new variable temp added by
them.

I can also provide a patch on top of linux-next instead of staging-next-2.6 if
you would prefer it in this situation. I will call it
"[PATCH-next 1/3] Staging: batman-adv: Don't increment stats of foreign device"

I hope that it doesn't create more confusion as necessary. :)

thanks,
	Sven

Randy Dunlap (1):
      Staging: batman-adv: don't use default init_module/cleanup_module function names

Sven Eckelmann (2):
      Staging: batman-adv: Don't increment stats of foreign device
      Staging: batman-adv: Update TODO with new points from review

 drivers/staging/batman-adv/TODO             |    9 ++++++---
 drivers/staging/batman-adv/hard-interface.c |    7 -------
 drivers/staging/batman-adv/main.c           |    7 +++++--
 3 files changed, 11 insertions(+), 12 deletions(-)


             reply	other threads:[~2010-07-20 22:25 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-20 22:25 Sven Eckelmann [this message]
2010-07-20 22:25 ` [B.A.T.M.A.N.] [PATCH 1/3] Staging: batman-adv: Don't increment stats of foreign device Sven Eckelmann
2010-07-20 22:25 ` [B.A.T.M.A.N.] [PATCH 2/3] Staging: batman-adv: don't use default init_module/cleanup_module function names Sven Eckelmann
2010-07-20 22:25 ` [B.A.T.M.A.N.] [PATCH 3/3] Staging: batman-adv: Update TODO with new points from review Sven Eckelmann
2010-07-22 21:30 ` [B.A.T.M.A.N.] Staging: batman-adv for 2.6.36 (5) Greg KH
2010-07-22 21:46   ` Sven Eckelmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1279664722-27500-1-git-send-email-sven.eckelmann@gmx.de \
    --to=sven.eckelmann@gmx.de \
    --cc=b.a.t.m.a.n@lists.open-mesh.net \
    --cc=b.a.t.m.a.n@lists.open-mesh.org \
    --cc=greg@kroah.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).