diff mbox

[PATCHv3,10/10] API and linux-generic: pktio update return codes

Message ID 1417442928-20680-11-git-send-email-maxim.uvarov@linaro.org
State New
Headers show

Commit Message

Maxim Uvarov Dec. 1, 2014, 2:08 p.m. UTC 
Update return codes to readable values.

Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
---
 platform/linux-generic/include/api/odp_packet_io.h | 16 +++++++---------
 platform/linux-generic/odp_packet_io.c             | 12 ++++++------
 2 files changed, 13 insertions(+), 15 deletions(-)
diff mbox

Patch

diff --git a/platform/linux-generic/include/api/odp_packet_io.h b/platform/linux-generic/include/api/odp_packet_io.h
index 1190355..0b9e34b 100644
--- a/platform/linux-generic/include/api/odp_packet_io.h
+++ b/platform/linux-generic/include/api/odp_packet_io.h
@@ -137,8 +137,7 @@  odp_pktio_t odp_pktio_get_input(odp_packet_t pkt);
  *		   use.
  *
  * @retval  0 on success.
- * @retval -1 if specified mtu can not be handled.
- * @retval -1 on any other error or illegal input parameters.
+ * @retval -ERROR on error.
  */
 int odp_pktio_set_mtu(odp_pktio_t id, int mtu);
 
@@ -148,7 +147,7 @@  int odp_pktio_set_mtu(odp_pktio_t id, int mtu);
  * @param[in] id  ODP packet IO handle.
  *
  * @retval MTU value >0 on success.
- * @retval -1 on any error or not existance pktio id.
+ * @retval -ERROR on error.
  */
 int odp_pktio_mtu(odp_pktio_t id);
 
@@ -159,8 +158,7 @@  int odp_pktio_mtu(odp_pktio_t id);
  * @param[in] enable    1 enabled, 0 disabled.
  *
  * @retval  0 on success.
- * @retval -1 on a bad pktio id
- * @retval -1 any other error
+ * @retval -ERROR on error.
  */
 int odp_pktio_promisc_set(odp_pktio_t id, odp_bool_t enable);
 
@@ -171,8 +169,7 @@  int odp_pktio_promisc_set(odp_pktio_t id, odp_bool_t enable);
  *
  * @retval  1 if promiscuous mode is enabled.
  * @retval  0 if promiscuous mode is disabled.
- * @retval -1 on a bad pktio id
- * @retval -1 any other error
+ * @retval -ERROR on error.
 */
 int odp_pktio_promisc_enabled(odp_pktio_t id);
 
@@ -183,8 +180,9 @@  int odp_pktio_promisc_enabled(odp_pktio_t id);
  * @param[in] mac_addr   MAC address to be assigned to the interface.
  * @param[in] addr_size  Size of the address in bytes.
  *
- * @return 0 on success, -ERROR on error.
- */
+ * @retval return 0 on success
+ * @retval -ERROR on error.
+*/
 int odp_pktio_mac_addr_set(odp_pktio_t id, const unsigned char *mac_addr,
 			   size_t addr_size);
 
diff --git a/platform/linux-generic/odp_packet_io.c b/platform/linux-generic/odp_packet_io.c
index 0f2dc33..f7b997b 100644
--- a/platform/linux-generic/odp_packet_io.c
+++ b/platform/linux-generic/odp_packet_io.c
@@ -558,7 +558,7 @@  int odp_pktio_set_mtu(odp_pktio_t id, int mtu)
 	entry = get_entry(id);
 	if (entry == NULL) {
 		ODP_DBG("pktio entry %d does not exist\n", id);
-		return -1;
+		return -ENOENT;
 	}
 
 	if (entry->s.pkt_sock_mmap.sockfd > -1)
@@ -588,7 +588,7 @@  int odp_pktio_mtu(odp_pktio_t id)
 	entry = get_entry(id);
 	if (entry == NULL) {
 		ODP_DBG("pktio entry %d does not exist\n", id);
-		return -1;
+		return -ENOENT;
 	}
 
 	if (entry->s.pkt_sock_mmap.sockfd > -1)
@@ -617,7 +617,7 @@  int odp_pktio_promisc_set(odp_pktio_t id, odp_bool_t enable)
 	entry = get_entry(id);
 	if (entry == NULL) {
 		ODP_DBG("pktio entry %d does not exist\n", id);
-		return -1;
+		return -ENOENT;
 	}
 
 	if (entry->s.pkt_sock_mmap.sockfd > -1)
@@ -642,7 +642,7 @@  int odp_pktio_promisc_set(odp_pktio_t id, odp_bool_t enable)
 	ret = ioctl(sockfd, SIOCSIFFLAGS, &ifr);
 	if (ret < 0) {
 		ODP_DBG("ioctl SIOCSIFFLAGS error\n");
-		return -1;
+		return ret;
 	}
 
 	return 0;
@@ -658,7 +658,7 @@  int odp_pktio_promisc_enabled(odp_pktio_t id)
 	entry = get_entry(id);
 	if (entry == NULL) {
 		ODP_DBG("pktio entry %d does not exist\n", id);
-		return -1;
+		return -ENOENT;
 	}
 
 	if (entry->s.pkt_sock_mmap.sockfd > -1)
@@ -672,7 +672,7 @@  int odp_pktio_promisc_enabled(odp_pktio_t id)
 	ret = ioctl(sockfd, SIOCGIFFLAGS, &ifr);
 	if (ret < 0) {
 		ODP_DBG("ioctl SIOCGIFFLAGS error\n");
-		return -1;
+		return ret;
 	}
 
 	if (ifr.ifr_flags & IFF_PROMISC)