Ok, merged!
Maxim.

On 12/18/2014 03:17 PM, Stuart Haslam wrote:
ping - this can be merged now.

On Tue, Dec 16, 2014 at 03:55:07PM +0000, Savolainen, Petri (NSN - FI/Espoo) 
wrote:
Reviewed-by: Petri Savolainen <[email protected]>

-----Original Message-----
From: ext Stuart Haslam [mailto:[email protected]]
Sent: Tuesday, December 16, 2014 4:55 PM
To: [email protected]
Cc: Savolainen, Petri (NSN - FI/Espoo); Stuart Haslam
Subject: [PATCH] api: pktio: remove odp_pktio_get_input() and
odp_pktio_set_input()

odp_pktio_get_input() has been replaced by odp_packet_input()

odp_pktio_set_input() has no valid use case outside of the
implementation.

Signed-off-by: Stuart Haslam <[email protected]>
---
(This code contribution is provided under the terms of agreement LES-LTM-
21309)

Note: depends on odp_packet_input() from the Packet API patches that
       are not yet in mainline.

  example/l2fwd/odp_l2fwd.c                          |  2 +-
  example/packet/odp_pktio.c                         |  2 +-
  platform/linux-generic/include/api/odp_packet_io.h | 19 -----------------
--
  platform/linux-generic/odp_packet_io.c             | 12 +-----------
  4 files changed, 3 insertions(+), 32 deletions(-)

diff --git a/example/l2fwd/odp_l2fwd.c b/example/l2fwd/odp_l2fwd.c
index 9721ab7..3e11b60 100644
--- a/example/l2fwd/odp_l2fwd.c
+++ b/example/l2fwd/odp_l2fwd.c
@@ -229,7 +229,7 @@ static void *pktio_queue_thread(void *arg)
                        continue;
                }

-               pktio_tmp = odp_pktio_get_input(pkt);
+               pktio_tmp = odp_packet_input(pkt);
                outq_def = odp_pktio_outq_getdef(dstpktio[pktio_tmp]);
                if (outq_def == ODP_QUEUE_INVALID) {
                        EXAMPLE_ERR("  [%02i] Error: def output-Q query\n",
diff --git a/example/packet/odp_pktio.c b/example/packet/odp_pktio.c
index ea72e58..446aac5 100644
--- a/example/packet/odp_pktio.c
+++ b/example/packet/odp_pktio.c
@@ -209,7 +209,7 @@ static void *pktio_queue_thread(void *arg)
                        continue;
                }

-               pktio_tmp = odp_pktio_get_input(pkt);
+               pktio_tmp = odp_packet_input(pkt);
                outq_def = odp_pktio_outq_getdef(pktio_tmp);

                if (outq_def == ODP_QUEUE_INVALID) {
diff --git a/platform/linux-generic/include/api/odp_packet_io.h
b/platform/linux-generic/include/api/odp_packet_io.h
index 63c047c..a1ad754 100644
--- a/platform/linux-generic/include/api/odp_packet_io.h
+++ b/platform/linux-generic/include/api/odp_packet_io.h
@@ -111,25 +111,6 @@ int odp_pktio_inq_remdef(odp_pktio_t id);
  odp_queue_t odp_pktio_outq_getdef(odp_pktio_t id);

  /**
- * Store packet input handle into packet
- *
- * @param pkt  ODP packet buffer handle
- * @param id   ODP packet IO handle
- *
- * @return
- */
-void odp_pktio_set_input(odp_packet_t pkt, odp_pktio_t id);
-
-/**
- * Get stored packet input handle from packet
- *
- * @param pkt  ODP packet buffer handle
- *
- * @return Packet IO handle
- */
-odp_pktio_t odp_pktio_get_input(odp_packet_t pkt);
-
-/**
   * Configure the MTU for a packet IO interface.
   *
   * @param[in] id   ODP packet IO handle.
diff --git a/platform/linux-generic/odp_packet_io.c b/platform/linux-
generic/odp_packet_io.c
index aa37179..1b795d3 100644
--- a/platform/linux-generic/odp_packet_io.c
+++ b/platform/linux-generic/odp_packet_io.c
@@ -284,16 +284,6 @@ int odp_pktio_close(odp_pktio_t id)
        return 0;
  }

-void odp_pktio_set_input(odp_packet_t pkt, odp_pktio_t pktio)
-{
-       odp_packet_hdr(pkt)->input = pktio;
-}
-
-odp_pktio_t odp_pktio_get_input(odp_packet_t pkt)
-{
-       return odp_packet_hdr(pkt)->input;
-}
-
  int odp_pktio_recv(odp_pktio_t id, odp_packet_t pkt_table[], unsigned
len)
  {
        pktio_entry_t *pktio_entry = get_pktio_entry(id);
@@ -327,7 +317,7 @@ int odp_pktio_recv(odp_pktio_t id, odp_packet_t
pkt_table[], unsigned len)
                return pkts;

        for (i = 0; i < pkts; ++i)
-               odp_pktio_set_input(pkt_table[i], id);
+               odp_packet_hdr(pkt_table[i])->input = id;

        return pkts;
  }
--
2.1.1





_______________________________________________
lng-odp mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to