[riot-notifications] [RIOT-OS/RIOT] Missing drop implementations in netdev_driver_t::recv (#10410)

Marian Buschsieweke notifications at github.com
Fri Nov 16 11:07:35 CET 2018


#### Description
Some implementations of `netdev_driver_t::recv(netdev_t *dev, void *buf, size_t len, void *info)` do not implement the drop case (`(buf == NULL) && (len != 0)`). This issue should track the effort to provide the missing implementations.

In case the upper layer used for the driver never uses the drop feature of `recv`, an implementation of the drop feature is not required, if instead corresponding `assert()`s are added and this is documented properly.

#### Tracking

| Driver          | State                         | Pull Request                               |
|-----------------|-------------------------------|--------------------------------------------|
| `ata8520e`      | Not affected? (**)            | -                                          |
| `at86rf2xx`     | Not affected                  | -                                          |
| `cc110x`        | Rewrite contains fix          | https://github.com/RIOT-OS/RIOT/pull/10340 |
| `cc2420`        | AFFECTED                      |                                            |
| `enc28j60`      | PR open                       | https://github.com/RIOT-OS/RIOT/pull/9806  |
| `encx24j600`    | AFFECTED                      |                                            |
| `ethos`         | Not affected                  | -                                          |
| `kw2xrf`        | AFFECTED                      |                                            |
| `mrf24j50`      | Not affected? (*)             | -                                          |
| `nrfble`        | asserts present, doc requried |                                            |
| `nrfmin`        | Not affected                  | -                                          |
| `rn2xx3`        | Not affected? (**)            | -                                          |
| `slipdev`       | Not affected                  |                                            |
| `sx127x`        | AFFECTED                      |                                            |
| `socket_zep`    | AFFECTED                      |                                            |
| `w5100`         | AFFECTED                      |                                            |
| `xbee`          | Not affected                  | -                                          |

(*)  Driver seems not to be affected, but I'm not 100% sure
(**) Driver seem to use the `netdev_driver_t` API, but I only briefly looked in the source. Maybe I missed something

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/issues/10410
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20181116/636fd57e/attachment-0001.html>


More information about the notifications mailing list