[lwip-devel] [patch #9364] RFC7668 - 6lowpan over Bluetooth Low Energy -- a new netif

[Simon Goldschmidt]

Follow-up Comment #11, patch #9364 (project lwip):

OK while we're at it, you seem to be missing to set tot_len of the allocated
pbuf. You should pass 'size' to pbuf_alloc() instead of 127. Then you only
have to change p->payload. Also, check for NULL return value!

And I see no reason to call ble_addr_to_eui64() for every RX packet when
'peer_addr' only changes with L2CAP_EVENT_LE_INCOMING_CONNECTION packets...

Next, we'd need a way to pass 'peer_addr' via tcpip_inpkt(), which
unfortunately only takes pbuf and netif pointers. Maybe a good solution is to
somehow set 'peer_addr' as netif-related data. Something like
'rfc7668_if_set_peer_addr(peer_addr)'? Since we only have one peer for BLE for
now, this might work.

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?9364>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/