Enable checkstyle whitespace rules

Enable checkstyle whitespace rules and fix the violations that are reported.
Also suppressed checkstyle errors from Topology.java, which is machine generated.

Change-Id: I7c608321359f31b3b70c77a20b77e46306d4d2ec
diff --git a/conf/checkstyle/sun_checks.xml b/conf/checkstyle/sun_checks.xml
index 975350e..0f685fa 100644
--- a/conf/checkstyle/sun_checks.xml
+++ b/conf/checkstyle/sun_checks.xml
@@ -170,33 +170,18 @@
 
         <!-- Checks for whitespace                               -->
         <!-- See http://checkstyle.sf.net/config_whitespace.html -->
-        <module name="EmptyForIteratorPad">
-          <property name="severity" value="warning"/>
-        </module>
-
+        <module name="EmptyForIteratorPad"/>
         <module name="GenericWhitespace"/>
         <module name="MethodParamPad"/>
-
-        <module name="NoWhitespaceAfter">
-          <property name="severity" value="warning"/>
-        </module>
-
-        <module name="NoWhitespaceBefore">
-          <property name="severity" value="warning"/>
-        </module>
+        <module name="NoWhitespaceAfter"/>
+        <module name="NoWhitespaceBefore"/>
 
         <!-- Disabled for ONOS.  Default rules specify undesired behavior for the '?' operator -->
         <!-- <module name="OperatorWrap"/> -->
         <module name="ParenPad"/>
         <module name="TypecastParenPad"/>
-
-        <module name="WhitespaceAfter">
-          <property name="severity" value="warning"/>
-        </module>
-
-        <module name="WhitespaceAround">
-          <property name="severity" value="warning"/>
-        </module>
+        <module name="WhitespaceAfter"/>
+        <module name="WhitespaceAround"/>