All of lore.kernel.org
 help / color / mirror / Atom feed
From: Simon Vincent <simon.vincent@xsilon.com>
To: Alexander Aring <alex.aring@gmail.com>, Alan Ott <alan@signal11.us>
Cc: Walter Mack <wmack@componentsw.com>, linux-wpan@vger.kernel.org
Subject: Re: [PATCH wpan-next] ieee802154: mrf24j40: support for external rx/tx
Date: Tue, 23 Sep 2014 08:58:06 +0100	[thread overview]
Message-ID: <5421280E.2060203@xsilon.com> (raw)
In-Reply-To: <20140923071932.GA11396@omega>

I was under the impression the patch had been signed off. Maybe I missed 
a mail...
I can confirm the mrf24j40 works just fine using a DT.

Simon

On 23/09/14 08:19, Alexander Aring wrote:
> Hi,
>
> On Mon, Sep 22, 2014 at 11:51:43PM -0400, Alan Ott wrote:
>> On 09/16/2014 04:20 PM, Alexander Aring wrote:
>>> Hi Walter,
>>>
>>> On Tue, Sep 16, 2014 at 01:13:12PM -0700, Walter Mack wrote:
>>>> Alex,
>>>>
>>>> this is not a question of chip revisions. Chips with the same revision can
>>>> be used in designs with or without the external transmitter/receiver. Its
>>>> just that the chip has to be set up to be compatible with the design.
>>>>
>>> ah, ok. Dunno, I will wait on alan's opinion. If I don't heard anything in one
>>> week I will apply it.
>> Did Simon's patch get applied? I couldn't see it in any of the obvious
>> places when I looked the other day.
>>
> no, I finally wait for your Acked-by: .... otherwise I will not apply.
> That's indicate for me that this patch is okay for you and came into
> mainline. There are also pending patches from Varka, if you don't reply
> on it, I will not apply it. Should I always simple apply the patches
> if you don't response? It's better for me that I see a "Acked-by" from you.
>
> Simon's patch state:
> You had some reviewing notes at Simon's patch. I think Simon doesn't
> response to this yet. I didn't see any new versions of this.
>
>
>
> For DT example you can take a look into at86rf230 driver or cc2520. I
> mean examples to add some DT properties. Also if you want to make this
> driver dt enabled then you need a documentation in:
>
> Documentation/devicetree/bindings/net/ieee802154/$DRIVERNAME.txt
>
>
> - Alex


  reply	other threads:[~2014-09-23  7:58 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-15  5:35 [PATCH wpan-next] ieee802154: mrf24j40: support for external rx/tx Walter Mack
2014-09-16  7:01 ` Alexander Aring
2014-09-16 20:13   ` Walter Mack
2014-09-16 20:20     ` Alexander Aring
2014-09-23  3:51       ` Alan Ott
2014-09-23  7:19         ` Alexander Aring
2014-09-23  7:58           ` Simon Vincent [this message]
2014-09-23  8:06             ` Alexander Aring
2014-09-24  6:11         ` Walter Mack
2014-09-24  6:35           ` 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=5421280E.2060203@xsilon.com \
    --to=simon.vincent@xsilon.com \
    --cc=alan@signal11.us \
    --cc=alex.aring@gmail.com \
    --cc=linux-wpan@vger.kernel.org \
    --cc=wmack@componentsw.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 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.