On 03/24/2013 09:40 AM, Stefan Schmidt wrote:
> As promised here is the reworked patch to add the hardware address filter
> callback for at86rf230. I added two more patches that might be good to go in 
> as
> well.
>
> I did not add DaveM yet as I wanted to get some review on these first. Just
> let me know what you folks think about them and I will resend with DaveM in CC
> to pick them up. Patches are already against his net-next tree.

And of course send also to the netdev mailing list when ready.

> [PATCH 1/3] ieee802154/dgram: Pass source address in dgram_recvmsg
>
> Patch from Stephen to fix the ieee802154 stack to actually put in the source
> address in datagrams. This allows using recvfrom() from userspace. This is a
> long standing bug. Actually I think it never worked. :) Might be a good
> candidate for -rc as well. Put I leave that up to you. Can also wait for the
> next merge window.
>
> [PATCH 2/3] ieee802154/at86rf230: Implement hardware address filter

> Reworked this patch to not include any AACK code. Also removed the at86rf230
> start and stop calls.

You did?

>  I was reading through the datasheet again and can't find
> anything that would need them. Maybe I brought that over from another driver.

In mrf24j40.c, I start and stop the IRQ generation from the start/stop
functions. I got this from the cc2420 driver (old linux-zigbee tree, not
in mainline).

I'm not sure what's "right," but I think it makes sense.

>
> Biggest problem on this on this that I don't have the setup ready to test it.
> Whcih mean I would really appreciate if other at86rf230 users could give this 
> a
> test. Tests would be to see if frames to other addresses still show up when 
> the
> address filter are set. (They should not). You can earn a Tested-by for this. 
> ;)
>
> [PATCH 3/3] ieee802154/at86rf230: Fix register names for RX_AACK_ON
>
> Not needed right now but still a good idea to get in imho. Register names have
> always been wrong but never used.

No reason to leave it out, now that you've done the work.

Thanks Stefan!

Alan.


------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_mar
_______________________________________________
Linux-zigbee-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linux-zigbee-devel

Reply via email to