CI: Updated coding style check. (#1738)

* Always run `make cs` to check the coding style.
* Fixed HardwarePWM.

.travis/build.sh

@@ -10,9 +10,18 @@ unset ESPTOOL2
 
 export SMING_HOME=$TRAVIS_BUILD_DIR/Sming
 
+cd $SMING_HOME
+
 # Check coding style
 if [ "$TRAVIS_BUILD_STAGE_NAME" == "Test" ]; then
- 	.travis/tools/clang/format-pr.sh;
+ 	make cs
+ 	DIFFS=$(git diff)
+ 	if [ "$DIFFS" != "" ]; then
+ 	  echo "!!! Coding Style issues Found!!!"
+ 	  echo "    Run: 'make cs' to fix them. "
+ 	  echo "$DIFFS"
+ 	  exit 1
+ 	fi
 fi
 
 # Setup ARCH SDK paths
@@ -28,8 +37,6 @@ fi
 # Full compile checks please
 export STRICT=1
 
-cd $SMING_HOME
-
 # Move samples and tests into directory outside of the Sming repo.
 export SMING_PROJECTS_DIR=$HOME/projects
 mkdir $SMING_PROJECTS_DIR

Sming/Arch/Host/Core/HardwarePWM.cpp

@@ -25,31 +25,38 @@
 
 #include "HardwarePWM.h"
 
-HardwarePWM::HardwarePWM(uint8* pins, uint8 no_of_pins) :
-		channel_count(no_of_pins) {
+HardwarePWM::HardwarePWM(uint8* pins, uint8 no_of_pins) : channel_count(no_of_pins)
+{
 }
 
-HardwarePWM::~HardwarePWM() {
+HardwarePWM::~HardwarePWM()
+{
 }
 
-uint8 HardwarePWM::getChannel(uint8 pin) {
+uint8 HardwarePWM::getChannel(uint8 pin)
+{
 	return PWM_BAD_CHANNEL;
 }
 
-uint32 HardwarePWM::getDutyChan(uint8 chan) {
+uint32 HardwarePWM::getDutyChan(uint8 chan)
+{
 	return 0;
 }
 
-bool HardwarePWM::setDutyChan(uint8 chan, uint32 duty, bool update) {
+bool HardwarePWM::setDutyChan(uint8 chan, uint32 duty, bool update)
+{
 	return false;
 }
 
-uint32 HardwarePWM::getPeriod() {
+uint32 HardwarePWM::getPeriod()
+{
 	return 0;
 }
 
-void HardwarePWM::setPeriod(uint32 period) {
+void HardwarePWM::setPeriod(uint32 period)
+{
 }
 
-void HardwarePWM::update() {
+void HardwarePWM::update()
+{
 }