All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexander Aring <alex.aring@gmail.com>
To: linux-wpan@vger.kernel.org
Cc: Alexander Aring <alex.aring@gmail.com>
Subject: [PATCH wpan-next 00/11] ieee802154: mac802154: wireless transformation
Date: Tue, 12 Aug 2014 15:14:04 +0200	[thread overview]
Message-ID: <1407849255-11500-1-git-send-email-alex.aring@gmail.com> (raw)

Hi,

this is the first part to make the mac802154/ieee802154 implementation like
wireless. There is much things do to but at around 10 patches I will send
a complete patch series of my work to make the review easier.

This patch renames more variables/structs like it's also available in wireless.
I think this was done also in the current implementation but all variables
was renamed and now I get confuse about priv/ipriv (I call it priv variable hell)
and netdev/ieee802154_dev.

We rename the ieee802154_dev to ieee802154_hw, like a ieee80211_hw.

The mac802154_priv is now ieee802154_local, like a ieee80211_local.
The mac802154_sub_if_data is now ieee802154_sub_if_data.

Also change the variables name to this so there should never be a variable priv
and you don't know if it's a mac802154_sub_if_data or mac802154_priv. That's
confusing me so much sometimes. Now it's also clear to see mutliple dereferencing.
Please send patches for this, if you like.

The next step is the frame parsing like wireless and interface registration like
wireless.


The last one replace the receiving workqueue with a tasklet, which is also used
in wireless. This have a less overhead and I removed a kmalloc call in this hot
path.

- Alex

Alexander Aring (11):
  ieee802154: rename ieee802154_dev to ieee802154_hw
  mac802154: rename ieee802154_dev.c to main.c
  mac802154: remove not functional monitor device
  ieee802154: add new interface types
  nl802154: add missing endif comment
  mac802154: rename mac802154_priv to ieee802154_local
  mac802154: rename mac802154_sub_if_data to ieee802154_sub_if_data
  mac802154: rename mac802154.h to ieee802154_i.h
  mac802154: rename hw subif_data variable to local
  mac802154: use hw_to_local
  mac802154: rx: use tasklet instead workqueue

 drivers/net/ieee802154/at86rf230.c |  90 ++++----
 drivers/net/ieee802154/cc2520.c    |  50 ++---
 drivers/net/ieee802154/mrf24j40.c  |  48 ++--
 include/linux/nl802154.h           |  15 +-
 include/net/mac802154.h            |  46 ++--
 net/ieee802154/nl-phy.c            |   4 +-
 net/mac802154/Makefile             |   4 +-
 net/mac802154/ieee802154_dev.c     | 415 -----------------------------------
 net/mac802154/ieee802154_i.h       | 178 +++++++++++++++
 net/mac802154/llsec.c              |   2 +-
 net/mac802154/mac802154.h          | 172 ---------------
 net/mac802154/mac_cmd.c            |   6 +-
 net/mac802154/main.c               | 435 +++++++++++++++++++++++++++++++++++++
 net/mac802154/mib.c                | 207 +++++++++---------
 net/mac802154/monitor.c            | 117 ----------
 net/mac802154/rx.c                 |  60 ++---
 net/mac802154/tx.c                 |  42 ++--
 net/mac802154/wpan.c               | 145 +++++++------
 18 files changed, 958 insertions(+), 1078 deletions(-)
 delete mode 100644 net/mac802154/ieee802154_dev.c
 create mode 100644 net/mac802154/ieee802154_i.h
 delete mode 100644 net/mac802154/mac802154.h
 create mode 100644 net/mac802154/main.c
 delete mode 100644 net/mac802154/monitor.c

-- 
2.0.3


             reply	other threads:[~2014-08-12 13:14 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-12 13:14 Alexander Aring [this message]
2014-08-12 13:14 ` [PATCH wpan-next 01/11] ieee802154: rename ieee802154_dev to ieee802154_hw Alexander Aring
2014-08-12 13:14 ` [PATCH wpan-next 02/11] mac802154: rename ieee802154_dev.c to main.c Alexander Aring
2014-08-12 13:14 ` [PATCH wpan-next 03/11] mac802154: remove not functional monitor device Alexander Aring
2014-08-12 13:14 ` [PATCH wpan-next 04/11] ieee802154: add new interface types Alexander Aring
2014-08-12 13:14 ` [PATCH wpan-next 05/11] nl802154: add missing endif comment Alexander Aring
2014-08-12 13:14 ` [PATCH wpan-next 06/11] mac802154: rename mac802154_priv to ieee802154_local Alexander Aring
2014-08-12 13:14 ` [PATCH wpan-next 07/11] mac802154: rename mac802154_sub_if_data to ieee802154_sub_if_data Alexander Aring
2014-08-12 13:14 ` [PATCH wpan-next 08/11] mac802154: rename mac802154.h to ieee802154_i.h Alexander Aring
2014-08-12 13:14 ` [PATCH wpan-next 09/11] mac802154: rename hw subif_data variable to local Alexander Aring
2014-08-12 13:14 ` [PATCH wpan-next 10/11] mac802154: use hw_to_local Alexander Aring
2014-08-12 13:14 ` [PATCH wpan-next 11/11] mac802154: rx: use tasklet instead workqueue Alexander Aring
2014-08-14  7:37 ` [PATCH wpan-next 00/11] ieee802154: mac802154: wireless transformation Martin Townsend
2014-08-14  7:59   ` Alexander Aring
2014-08-14  8:09     ` Martin Townsend
2014-08-14  8:24       ` Alexander Aring
2014-08-14  8:13     ` Alexander Aring

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=1407849255-11500-1-git-send-email-alex.aring@gmail.com \
    --to=alex.aring@gmail.com \
    --cc=linux-wpan@vger.kernel.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.