From 4093ec5979ce09b2cdbaa2960dfc472fad139637 Mon Sep 17 00:00:00 2001
From: Ben Pfaff <blp@nicira.com>
Date: Tue, 15 Mar 2011 10:08:54 -0700
Subject: [PATCH] ofproto: Move hton_ofp_phy_port() to ofp-util.

This removes some code from ofproto.c that doesn't really seem to
belong there to begin with.
---
 lib/ofp-util.c    | 13 +++++++++++++
 lib/ofp-util.h    |  2 ++
 ofproto/ofproto.c | 13 -------------
 3 files changed, 15 insertions(+), 13 deletions(-)

diff --git a/lib/ofp-util.c b/lib/ofp-util.c
index e42abc50..d91fbb11 100644
--- a/lib/ofp-util.c
+++ b/lib/ofp-util.c
@@ -1863,6 +1863,19 @@ make_echo_reply(const struct ofp_header *rq)
     return out;
 }
 
+/* Converts the members of 'opp' from host to network byte order. */
+void
+hton_ofp_phy_port(struct ofp_phy_port *opp)
+{
+    opp->port_no = htons(opp->port_no);
+    opp->config = htonl(opp->config);
+    opp->state = htonl(opp->state);
+    opp->curr = htonl(opp->curr);
+    opp->advertised = htonl(opp->advertised);
+    opp->supported = htonl(opp->supported);
+    opp->peer = htonl(opp->peer);
+}
+
 const struct ofp_flow_stats *
 flow_stats_first(struct flow_stats_iterator *iter,
                  const struct ofp_stats_reply *osr)
diff --git a/lib/ofp-util.h b/lib/ofp-util.h
index b4fe6423..75e76e24 100644
--- a/lib/ofp-util.h
+++ b/lib/ofp-util.h
@@ -257,6 +257,8 @@ struct ofpbuf *make_unbuffered_packet_out(const struct ofpbuf *packet,
 struct ofpbuf *make_echo_request(void);
 struct ofpbuf *make_echo_reply(const struct ofp_header *rq);
 
+void hton_ofp_phy_port(struct ofp_phy_port *);
+
 struct flow_stats_iterator {
     const uint8_t *pos, *end;
 };
diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
index 699985cf..bd81a556 100644
--- a/ofproto/ofproto.c
+++ b/ofproto/ofproto.c
@@ -107,7 +107,6 @@ struct ofport {
 static void ofport_free(struct ofport *);
 static void ofport_run(struct ofproto *, struct ofport *);
 static void ofport_wait(struct ofport *);
-static void hton_ofp_phy_port(struct ofp_phy_port *);
 
 struct action_xlate_ctx {
 /* action_xlate_ctx_init() initializes these members. */
@@ -2633,18 +2632,6 @@ send_error_oh(const struct ofconn *ofconn, const struct ofp_header *oh,
     }
 }
 
-static void
-hton_ofp_phy_port(struct ofp_phy_port *opp)
-{
-    opp->port_no = htons(opp->port_no);
-    opp->config = htonl(opp->config);
-    opp->state = htonl(opp->state);
-    opp->curr = htonl(opp->curr);
-    opp->advertised = htonl(opp->advertised);
-    opp->supported = htonl(opp->supported);
-    opp->peer = htonl(opp->peer);
-}
-
 static int
 handle_echo_request(struct ofconn *ofconn, const struct ofp_header *oh)
 {
-- 
2.30.2