Hello.
On 10/01/17 16:41, Harry Morris wrote:
Add driver source and config for softMAC implementation of Cascoda's CA8210
IEEE 802.15.4 transceiver device. The driver mimics a common PHY-only
implementation despite the CA8210 being a hardMAC device which exposes a SAP
interface to the fully integrated MAC.
The chip is a modem-only device with an integrated processor which runs the
802.15.4 MAC. The chip communicates via full-duplex SPI with additional pins
for NIRQ and NRESET. The chip can also output its 16MHz clock to a GPIO with a
configurable divider.
The driver can be configured to implement a debugfs node that provides access
to the SAP-based API to drive mechanisms not currently supported by the
standard kernel interface.
Signed-off-by: Harry Morris <h.morris@xxxxxxxxxxx>
---
drivers/net/ieee802154/Kconfig | 21 +
drivers/net/ieee802154/Makefile | 1 +
drivers/net/ieee802154/ca8210.c | 3273 +++++++++++++++++++++++++++++++++++++++
Great to that your latest rework got rid of around 500 lines of code.
Reduced by this the mail does made its way to the list finally. :)
As I expressed before I'm ok with this driver even if it has some short
comings. Something we should work on over this year. We on adding better
hardMAC transceiver support to the stack and you adapting the driver and
telling us whats missing :)
I did give it another run and make C=1 pointed the following out to me:
drivers/net/ieee802154/ca8210.c:1840:27: warning: incorrect type in
assignment (different base types)
drivers/net/ieee802154/ca8210.c:1840:27: expected restricted __le16
[assigned] [usertype] pan_id
drivers/net/ieee802154/ca8210.c:1840:27: got unsigned short
[unsigned] [short] [usertype] <noident>
drivers/net/ieee802154/ca8210.c:1845:25: warning: incorrect type in
assignment (different base types)
drivers/net/ieee802154/ca8210.c:1845:25: expected restricted __le16
[assigned] [usertype] pan_id
drivers/net/ieee802154/ca8210.c:1845:25: got unsigned short
[unsigned] [short] [usertype] <noident>
drivers/net/ieee802154/ca8210.c:1951:42: warning: missing braces around
initializer
drivers/net/ieee802154/ca8210.c:1974:28: warning: incorrect type in
argument 3 (different base types)
drivers/net/ieee802154/ca8210.c:1974:28: expected unsigned short
[unsigned] [usertype] dst_pan_id
drivers/net/ieee802154/ca8210.c:1974:28: got restricted __le16
[addressable] [usertype] pan_id
Maybe the code checker just got confused and these are false positives
but please have a look at them.
I'm offline for the next two weeks. Will let this version sit here and
if nothing problematic comes up we can get this applied when I'm back.
regards
Stefan Schmidt
--
To unsubscribe from this list: send the line "unsubscribe linux-wpan" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
