Re: [Linux-zigbee-devel] [PATCH 2/3] at86rf230: add irq type configuration option

2013-03-12 Thread Alan Ott
On 03/12/2013 07:36 PM, Alexander Aring wrote: > I don't know if this patch is necessary. I think Alan told about this > some days ago. In my case I setting my irq_type with a device_tree and I > know other boards, which are not using a device_tree, do the same in the > boardspecific code file in a

Re: [Linux-zigbee-devel] [PATCH 2/3] at86rf230: add irq type configuration option

2013-03-12 Thread Alexander Aring
Hi, I don't know if this patch is necessary. I think Alan told about this some days ago. In my case I setting my irq_type with a device_tree and I know other boards, which are not using a device_tree, do the same in the boardspecific code file in arch/arm/**/*_board.c. I would only add this, if t

Re: [Linux-zigbee-devel] [PATCH 3/3] at86rf230: changed irq handling to not disable irq for, edge type interrupt

2013-03-12 Thread Alexander Aring
Hi, On Wed, Mar 13, 2013 at 12:07:40AM +0100, Sascha Herrmann wrote: > >> Can anyone else here test this? Any other at86rf230 users? Alex, do you > > >> have any time to give this a go? > > That will be some difficult... I use a at86rf230 which is controlled by > > a microcontoller which has a sp

Re: [Linux-zigbee-devel] [PATCH 3/3] at86rf230: changed irq handling to not disable irq for, edge type interrupt

2013-03-12 Thread Sascha Herrmann
>> Can anyone else here test this? Any other at86rf230 users? Alex, do you >> have any time to give this a go? > That will be some difficult... I use a at86rf230 which is controlled by > a microcontoller which has a spi<=>usb bridge. I modified this driver > myself to get this working with spi<=>u

Re: [Linux-zigbee-devel] [PATCH 2/3] at86rf230: add irq type configuration option

2013-03-12 Thread Alan Ott
On 03/12/2013 06:36 PM, Sascha Herrmann wrote: >>> @@ -881,6 +882,12 @@ static int at86rf230_probe(struct spi_device *spi) >>> if (rc) >>> goto err_gpio_dir; >>> + if (lp->irq_type) { >> Stray space before the + in the above line. Are you using git >> format-patch and git send-ema

Re: [Linux-zigbee-devel] [PATCH 2/3] at86rf230: add irq type configuration option

2013-03-12 Thread Sascha Herrmann
>> @@ -881,6 +882,12 @@ static int at86rf230_probe(struct spi_device *spi) >> if (rc) >> goto err_gpio_dir; >> + if (lp->irq_type) { > Stray space before the + in the above line. Are you using git > format-patch and git send-email? Argh..., missed this :/ I used git format-pa

Re: [Linux-zigbee-devel] [PATCH 3/3] at86rf230: changed irq handling to not disable irq for, edge type interrupt

2013-03-12 Thread Alexander Aring
On Tue, Mar 12, 2013 at 06:24:48PM -0400, Alan Ott wrote: > On 03/12/2013 06:14 PM, Alexander Aring wrote: > > Hi, > > > > On Tue, Mar 12, 2013 at 05:19:31PM -0400, Alan Ott wrote: > >> On 02/23/2013 11:12 AM, Sascha Herrmann wrote: > >>> If interrupt is configured to edge type, do not disable irq

Re: [Linux-zigbee-devel] [PATCH 3/3] at86rf230: changed irq handling to not disable irq for, edge type interrupt

2013-03-12 Thread Alan Ott
On 03/12/2013 06:14 PM, Alexander Aring wrote: > Hi, > > On Tue, Mar 12, 2013 at 05:19:31PM -0400, Alan Ott wrote: >> On 02/23/2013 11:12 AM, Sascha Herrmann wrote: >>> If interrupt is configured to edge type, do not disable irq in the isr. >>> The at86rf230 resets the irq line only after the irq s

Re: [Linux-zigbee-devel] [PATCH 3/3] at86rf230: changed irq handling to not disable irq for, edge type interrupt

2013-03-12 Thread Alexander Aring
Hi, On Tue, Mar 12, 2013 at 05:19:31PM -0400, Alan Ott wrote: > On 02/23/2013 11:12 AM, Sascha Herrmann wrote: > > If interrupt is configured to edge type, do not disable irq in the isr. > > The at86rf230 resets the irq line only after the irq status register is > > read. Disabling the irq can loc

Re: [Linux-zigbee-devel] [PATCH 2/2] addrdb: fix include dependency and remove redefined

2013-03-12 Thread Alexander Aring
Hi Alan, On Tue, Mar 12, 2013 at 05:44:33PM -0400, Alan Ott wrote: > On 03/12/2013 05:41 PM, Alexander Aring wrote: > > Hi Alan, > > > > On Tue, Mar 12, 2013 at 05:37:15PM -0400, Alan Ott wrote: > >> On 03/12/2013 05:37 PM, Alexander Aring wrote: > >>> On Tue, Mar 12, 2013 at 05:05:22PM -0400, Ala

Re: [Linux-zigbee-devel] [PATCH 2/2] addrdb: fix include dependency and remove redefined

2013-03-12 Thread Sascha Herrmann
>> The two I added to CC :) > That's nice to know, but I don't see this people on this mailinglist or > netdev. So my question, which I hope it isn't true... are they alive? luckely, at least Alexander gave a live sing here some days ago :) Sascha ---

Re: [Linux-zigbee-devel] [PATCH 2/2] addrdb: fix include dependency and remove redefined

2013-03-12 Thread Alan Ott
On 03/12/2013 05:41 PM, Alexander Aring wrote: > Hi Alan, > > On Tue, Mar 12, 2013 at 05:37:15PM -0400, Alan Ott wrote: >> On 03/12/2013 05:37 PM, Alexander Aring wrote: >>> On Tue, Mar 12, 2013 at 05:05:22PM -0400, Alan Ott wrote: Would someone with access please push this to the lowpan-tools

Re: [Linux-zigbee-devel] [PATCH 2/2] addrdb: fix include dependency and remove redefined

2013-03-12 Thread Alexander Aring
Hi Alan, On Tue, Mar 12, 2013 at 05:37:15PM -0400, Alan Ott wrote: > On 03/12/2013 05:37 PM, Alexander Aring wrote: > > On Tue, Mar 12, 2013 at 05:05:22PM -0400, Alan Ott wrote: > >> Would someone with access please push this to the lowpan-tools repo, or > >> comment as to why it shouldn't be push

Re: [Linux-zigbee-devel] [PATCH 2/2] addrdb: fix include dependency and remove redefined

2013-03-12 Thread Alan Ott
On 03/12/2013 05:37 PM, Alexander Aring wrote: > On Tue, Mar 12, 2013 at 05:05:22PM -0400, Alan Ott wrote: >> Would someone with access please push this to the lowpan-tools repo, or >> comment as to why it shouldn't be pushed? >> > This is a good point! Who has access to the lowpan-tools? The two

Re: [Linux-zigbee-devel] [PATCH 2/2] addrdb: fix include dependency and remove redefined

2013-03-12 Thread Alexander Aring
Hi Alan, On Tue, Mar 12, 2013 at 05:05:22PM -0400, Alan Ott wrote: > Would someone with access please push this to the lowpan-tools repo, or > comment as to why it shouldn't be pushed? > This is a good point! Who has access to the lowpan-tools? Alex -

Re: [Linux-zigbee-devel] [PATCH] net/ieee802154/6lowpan: Fix initialization for fragment offset

2013-03-12 Thread Alan Ott
On 03/12/2013 04:35 PM, Alan Ott wrote: > This seems awfully familiar, but does look necessary. Maybe we've just > been getting lucky with offset happening to initialize to zero, as > fragmentation and reassembly (with Tony's patches) does seem to work > without this. > > Acked-by: Alan Ott err..

Re: [Linux-zigbee-devel] [PATCH 3/3] at86rf230: changed irq handling to not disable irq for, edge type interrupt

2013-03-12 Thread Alan Ott
On 02/23/2013 11:12 AM, Sascha Herrmann wrote: > If interrupt is configured to edge type, do not disable irq in the isr. > The at86rf230 resets the irq line only after the irq status register is > read. Disabling the irq can lock the driver in situations where a irq is > set by the radio while the

Re: [Linux-zigbee-devel] [PATCH 2/3] at86rf230: add irq type configuration option

2013-03-12 Thread Alan Ott
On 02/23/2013 11:12 AM, Sascha Herrmann wrote: > Add option to at86rf230 platform data to configure the type of the > interrupt used by the driver. > > Signed-off-by: Sascha Herrmann > --- > drivers/net/ieee802154/at86rf230.c |9 - > include/linux/spi/at86rf230.h |5 + >

Re: [Linux-zigbee-devel] [PATCH 2/2] addrdb: fix include dependency and remove redefined

2013-03-12 Thread Alan Ott
Would someone with access please push this to the lowpan-tools repo, or comment as to why it shouldn't be pushed? On 12/21/2012 12:37 AM, Alexander Aring wrote: > Currently I get this compile error: > > coord-config-parse.h:106:14: error: unknown type name 'yyscan_t' > coord-config-parse.y:38:0: e

Re: [Linux-zigbee-devel] 6lowpan: strange addressing?

2013-03-12 Thread Alan Ott
On 03/05/2013 10:48 AM, Wolf-Bastian Pöttner wrote: > finally I have 6lowpan running. However, acknowledgement of frames is not > working. When I enable address filtering in the at86rf230, I only receive > broadcast frames and no unicast frames anymore. > > The reason seems to be the addressing.

Re: [Linux-zigbee-devel] [PATCH] net/ieee802154/6lowpan: Fix initialization for fragment offset

2013-03-12 Thread Alan Ott
On 03/11/2013 03:00 AM, Wolf-Bastian Pöttner wrote: > Am 08.03.2013 um 17:37 schrieb Alexander Aring : > >> This isn't based on current mainline or? I don't know which have access >> to the linux-zigbee-devel kernel repository. > > exactly, the patch I am referring too is not mainline yet as far as

Re: [Linux-zigbee-devel] Where to pull latest source and patches.

2013-03-12 Thread Alan Ott
On 03/10/2013 07:53 PM, Randy Graham wrote: > I am new to 6lowpan and the list, so please forgive the rookie questions. > > Where do I download the latest source and patches ? As Alexander said, just grab the net-next kernel. Beyond that, grab Tony Chennau's patches which are linked to on my webs

Re: [Linux-zigbee-devel] [PATCH] net/ieee802154/6lowpan: Fix initialization for fragment offset

2013-03-12 Thread Alan Ott
This seems awfully familiar, but does look necessary. Maybe we've just been getting lucky with offset happening to initialize to zero, as fragmentation and reassembly (with Tony's patches) does seem to work without this. Acked-by: Alan Ott Please send this patch to the net-next mailing list. On

Re: [Linux-zigbee-devel] izcoordinator not working

2013-03-12 Thread Alan Ott
On 03/05/2013 08:35 AM, Christophe Aeschlimann wrote: > Le 05.03.2013 13:15, Wolf-Bastian Pöttner a écrit : > >> $ izcoordinator -i wpan0 -d 1 -s 1 -p 777 -c 23 -l lease > [...] > >> GDB tells me, that the coordinator sits here: > That's why Alan in his setup script [1] calls the coordinator with t

Re: [Linux-zigbee-devel] linux-zigbee and 6lowpan.c

2013-03-12 Thread Alan Ott
On 03/11/2013 02:03 PM, Tony Cheneau wrote: > Hi, > > Sorry for the delay, I wanted to check the code first. See my answer > inline. > > Le 2013-03-05 21:20, Alan Ott a écrit : >> On 03/05/2013 02:38 PM, Ralph Droms (rdroms) wrote: >>> On Mar 5, 2013, at 10:59 AM 3/5/13, Wolf-Bastian Pöttner >>> w