Linux IEEE 802.15.4 and 6LoWPAN development
 help / color / mirror / Atom feed
From: Alexander Aring <alex.aring@gmail.com>
To: Varka Bhadram <varkabhadram@gmail.com>
Cc: linux-wpan@vger.kernel.org, kernel@pengutronix.de
Subject: Re: [PATCH bluetooth-next 07/17] mac802154: remove might_sleep from driver layer
Date: Wed, 29 Oct 2014 08:23:34 +0100	[thread overview]
Message-ID: <20141029072333.GF4709@omega> (raw)
In-Reply-To: <20141029061306.GE4709@omega>

Hi Varka,

it's too early in the morning, apologize my english here.

On Wed, Oct 29, 2014 at 07:13:06AM +0100, Alexander Aring wrote:
> Hi Varka,
> 
> On Wed, Oct 29, 2014 at 11:27:53AM +0530, Varka Bhadram wrote:
> > On 10/29/2014 11:23 AM, Alexander Aring wrote:
> > >Hi Varka,
> > >
> > >On Wed, Oct 29, 2014 at 09:03:49AM +0530, Varka Bhadram wrote:
> > >>On 10/28/2014 10:51 PM, Alexander Aring wrote:
> > >>>This patch removes all might_sleep calls from driver layer. This
> > >>>handling is already done by mac802154 layer.
> > >>>
> > >>>Signed-off-by: Alexander Aring <alex.aring@gmail.com>
> > >>>---
> > >>>  drivers/net/ieee802154/at86rf230.c | 3 ---
> > >>>  drivers/net/ieee802154/cc2520.c    | 1 -
> > >>>  drivers/net/ieee802154/fakelb.c    | 4 ----
> > >>>  3 files changed, 8 deletions(-)
> > >>>
> > >>For CC2520 driver
> > >>
> > >I would Cc you here if you stand in the MAINTAINERS file for cc2520
> > >driver. I asked you some months ago if you like to be there. I didn't
> > >saw any patches for that. I thought it's out of interest by you that you
> > >want to be maintainer for this driver.  When these changes are very trivial
> > >like this one, it's okay for you when Marcel simple apply these patches
> > >without any ack? I already told him that we don't need to wait of Alan's
> > >ack for the mrf24j40 driver.
> > >
> > >Nevertheless if you want to add really a Acked here and we need to wait
> > >then please insert you in the MAINTAINERS file. When I run
> > >get_maintainers it's currently me who is maintainer for the cc2520 driver.
> > >
> > >If you want to take care of this driver, please see my improvements and
> > >fixes for the at86rf230. I am sure you could also take some code for the
> > >cc2520 from it. Also I would be very happy for async xmit handling or
> > >regmap support. Most of the cc2520 code is lowlevel spi api calls.
> > >
> > >- Alex
> > 
> > I think i clarified you that i want to be the CC2520 driver maintainer.
> > 
> > I asked you to submit the patche for MAINTAINER file, you told that its
> > not required.
> > 
> 
> I looked up the conversion:
> 

s/conversion/conversation/

> You:
> 
> "You mean to say that i need to add an entry in MAINTAINER file or ..?"
> 
> Me:
> 
> "not need, only if you like. I have no idea about the cc2520 driver."
> 
> need != required. For me this sentence is more... you really don't need
> that do be maintainer of this if you don't like that. But if you like to

s/do/to


well, I think I cherry-pick some commits now to generate the next
series.

- Alex

  reply	other threads:[~2014-10-29  7:23 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-28 17:21 [PATCH bluetooth-next 00/17] mac802154: iface and driver-ops cleanup Alexander Aring
2014-10-28 17:21 ` [PATCH bluetooth-next 01/17] mac802154: monitor: merge into iface implementation Alexander Aring
2014-10-29  3:39   ` Varka Bhadram
2014-10-29  3:49     ` Varka Bhadram
2014-10-29  5:39       ` Alexander Aring
2014-10-28 17:21 ` [PATCH bluetooth-next 02/17] mac802154: main: move open and close into iface Alexander Aring
2014-10-28 17:21 ` [PATCH bluetooth-next 03/17] mac802154: declare struct ieee802154_ops as const Alexander Aring
2014-10-29  3:31   ` Varka Bhadram
2014-10-28 17:21 ` [PATCH bluetooth-next 04/17] mac802154: ops: declare channel and page as u8 Alexander Aring
2014-10-29  3:32   ` Varka Bhadram
2014-10-28 17:21 ` [PATCH bluetooth-next 05/17] mac802154: introduce driver-ops header Alexander Aring
2014-10-28 17:21 ` [PATCH bluetooth-next 06/17] mac802154: use driver-ops function wrappers Alexander Aring
2014-10-28 17:21 ` [PATCH bluetooth-next 07/17] mac802154: remove might_sleep from driver layer Alexander Aring
2014-10-29  3:33   ` Varka Bhadram
2014-10-29  5:53     ` Alexander Aring
2014-10-29  5:57       ` Varka Bhadram
2014-10-29  6:13         ` Alexander Aring
2014-10-29  7:23           ` Alexander Aring [this message]
2014-10-29  8:10           ` Varka Bhadram
2014-10-29  8:12             ` Varka Bhadram
2014-10-28 17:21 ` [PATCH bluetooth-next 08/17] mac802154: remove driver ops in wpan-phy Alexander Aring
2014-10-28 17:21 ` [PATCH bluetooth-next 09/17] mac802154: rework sdata state change to running Alexander Aring
2014-10-28 17:21 ` [PATCH bluetooth-next 10/17] mac802154: rename running to started Alexander Aring
2014-10-28 17:21 ` [PATCH bluetooth-next 11/17] mac802154: move local started handling Alexander Aring
2014-10-28 17:21 ` [PATCH bluetooth-next 12/17] mac802154: add synchronization handling Alexander Aring
2014-10-28 17:21 ` [PATCH bluetooth-next 13/17] mac802154: iface: remove assign to zero Alexander Aring
2014-10-28 17:21 ` [PATCH bluetooth-next 14/17] mac802154: remove channel attributes from sdata Alexander Aring
2014-10-29  3:52   ` Varka Bhadram
2014-10-29  5:22     ` Alexander Aring
2014-10-29  5:35       ` Varka Bhadram
2014-10-29  6:07         ` Alexander Aring
2014-10-28 17:21 ` [PATCH bluetooth-next 15/17] mac802154: move mac_params functions into mac_cmd Alexander Aring
2014-10-28 17:21 ` [PATCH bluetooth-next 16/17] mac802154: cleanup open count handling Alexander Aring
2014-10-28 17:21 ` [PATCH bluetooth-next 17/17] ieee802154: introduce sysfs file Alexander Aring
2014-10-28 22:24 ` [PATCH bluetooth-next 00/17] mac802154: iface and driver-ops cleanup Marcel Holtmann

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=20141029072333.GF4709@omega \
    --to=alex.aring@gmail.com \
    --cc=kernel@pengutronix.de \
    --cc=linux-wpan@vger.kernel.org \
    --cc=varkabhadram@gmail.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