diff --git a/of/api/src/main/java/org/onlab/onos/of/controller/PacketListener.java b/of/api/src/main/java/org/onlab/onos/of/controller/PacketListener.java
index 5c6d73f..a14f9e1 100644
--- a/of/api/src/main/java/org/onlab/onos/of/controller/PacketListener.java
+++ b/of/api/src/main/java/org/onlab/onos/of/controller/PacketListener.java
@@ -6,8 +6,9 @@
 public interface PacketListener {
 
     /**
-     * Handle the packet.
-     * @param pktCtx the packet context ({@link }
+     * Handles the packet.
+     *
+     * @param pktCtx the packet context
      */
     public void handlePacket(PacketContext pktCtx);
 }
diff --git a/of/api/src/main/java/org/onlab/onos/of/controller/driver/RoleHandler.java b/of/api/src/main/java/org/onlab/onos/of/controller/driver/RoleHandler.java
index ac1e1ab..bfded52 100644
--- a/of/api/src/main/java/org/onlab/onos/of/controller/driver/RoleHandler.java
+++ b/of/api/src/main/java/org/onlab/onos/of/controller/driver/RoleHandler.java
@@ -45,7 +45,7 @@
      *
      * @param role
      * @throws IOException
-     * @returns false if and only if the switch does not support role-request
+     * @return false if and only if the switch does not support role-request
      * messages, according to the switch driver; true otherwise.
      */
     public boolean sendRoleRequest(RoleState role, RoleRecvStatus exp)
@@ -53,8 +53,7 @@
 
     /**
      * Extract the role information from an OF1.3 Role Reply Message.
-     * @param h
-     * @param rrmsg
+     * @param rrmsg role reply message
      * @return RoleReplyInfo object
      * @throws SwitchStateException
      */