diff --git a/src/main/java/net/floodlightcontroller/counter/CountSeries.java b/src/main/java/net/floodlightcontroller/counter/CountSeries.java
index 9d2081a08b6f14bfb9575fae89512b2556a6aede..c94e5bc613b66767903165ce74d8dec26ce8e4d0 100644
--- a/src/main/java/net/floodlightcontroller/counter/CountSeries.java
+++ b/src/main/java/net/floodlightcontroller/counter/CountSeries.java
@@ -63,7 +63,6 @@ public class CountSeries {
   /**
    * Return a long that is the number of milliseconds in a ds (second/minute/hour/day/week).  (Utility method.)
    * 
-   * @param d
    * @param ds
    * @return
    */
diff --git a/src/main/java/net/floodlightcontroller/counter/CounterStore.java b/src/main/java/net/floodlightcontroller/counter/CounterStore.java
index a3d6bbb87c98720b7aede246f0111bdd7efa089d..0da77e20088bf59851a427b6bd686884885b8495 100644
--- a/src/main/java/net/floodlightcontroller/counter/CounterStore.java
+++ b/src/main/java/net/floodlightcontroller/counter/CounterStore.java
@@ -139,7 +139,8 @@ public class CounterStore {
      * Create a new ICounter and set the title.  Note that the title must be unique, otherwise this will
      * throw an IllegalArgumentException.
      * 
-     * @param title
+     * @param key
+     * @param type
      * @return
      */
     public ICounter createCounter(String key, CounterValue.CounterType type) {
diff --git a/src/main/java/net/floodlightcontroller/counter/ICounter.java b/src/main/java/net/floodlightcontroller/counter/ICounter.java
index 55fa1de0559d442a386b6db033b8bf8c28a280cc..625bebdc34fb26bee9c3bd3f279de70fe0867a97 100644
--- a/src/main/java/net/floodlightcontroller/counter/ICounter.java
+++ b/src/main/java/net/floodlightcontroller/counter/ICounter.java
@@ -63,7 +63,7 @@ public interface ICounter {
    * Returns a CountSeries that is a snapshot of the counter's values for the given dateSpan.  (Further changes
    * to this counter won't be reflected in the CountSeries that comes  back.)
    * 
-   * @param startDate
+   * @param dateSpan
    * @return
    */
   public CountSeries snapshot(DateSpan dateSpan);
diff --git a/src/main/java/net/floodlightcontroller/devicemanager/internal/DeviceManagerImpl.java b/src/main/java/net/floodlightcontroller/devicemanager/internal/DeviceManagerImpl.java
index c42ce554d2a37cadecfdc8d988d52e9eb2afa058..472a8b13d8c3b337964d773c29d664b494780c83 100755
--- a/src/main/java/net/floodlightcontroller/devicemanager/internal/DeviceManagerImpl.java
+++ b/src/main/java/net/floodlightcontroller/devicemanager/internal/DeviceManagerImpl.java
@@ -1285,7 +1285,7 @@ public class DeviceManagerImpl implements IDeviceManager, IOFMessageListener,
      * Puts an update in queue to indicate the Device moved.  Must be called
      * from within the write lock.
      * @param device The device that has moved.
-     * @param oldPort The old switchport
+     * @param oldSwPort The old switchport
      * @param newDap The new attachment point
      */
     protected void updateMoved(Device device, SwitchPortTuple oldSwPort, 
@@ -1341,7 +1341,7 @@ public class DeviceManagerImpl implements IDeviceManager, IOFMessageListener,
     /**
      * Removes any attachment points that are in the same 
      * {@link net.floodlightcontroller.topology.SwitchCluster SwitchCluster} 
-     * @param The device to update the attachment points
+     * @param d The device to update the attachment points
      */
     public void cleanupAttachmentPoints(Device d) {
         // The long here is the SwitchCluster ID