Hey Petri, I like the API changes. A very minor nit, looks like a few places where extra whitespace got injected if you happen to be making other changes. The review is for the patch set.
Reviewed-by: Robbie King <[email protected]> -----Original Message----- From: lng-odp [mailto:[email protected]] On Behalf Of Petri Savolainen Sent: Thursday, April 23, 2015 6:29 AM To: [email protected] Subject: [lng-odp] [API-NEXT PATCH v2 1/5] api: packet_io: change word instance to interface Signed-off-by: Petri Savolainen <[email protected]> --- include/odp/api/packet_io.h | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/include/odp/api/packet_io.h b/include/odp/api/packet_io.h index e960f10..5446999 100644 --- a/include/odp/api/packet_io.h +++ b/include/odp/api/packet_io.h @@ -45,11 +45,12 @@ extern "C" { */ /** - * Open an ODP packet IO instance + * Open an ODP packet IO interface * - * Packet IO handles are single instance per device, attempts to open an already - * open device will fail, returning ODP_PKTIO_INVALID with errno set. - * odp_pktio_lookup() may be used to obtain a handle to an already open device. + * An ODP program can open a single packet IO interface per device, attempts + * to open an already open device will fail, returning ODP_PKTIO_INVALID with + * errno set. Use odp_pktio_lookup() to obtain a handle to an already open + * device. * * @param dev Packet IO device name * @param pool Default pool from which to allocate buffers for storing packets @@ -72,7 +73,7 @@ extern "C" { odp_pktio_t odp_pktio_open(const char *dev, odp_pool_t pool); /** - * Close an ODP packet IO instance + * Close an ODP packet IO interface * * @param pktio ODP packet IO handle * -- 2.3.5 _______________________________________________ lng-odp mailing list [email protected] https://lists.linaro.org/mailman/listinfo/lng-odp _______________________________________________ lng-odp mailing list [email protected] https://lists.linaro.org/mailman/listinfo/lng-odp
