Merge pull request sipeed#9 from ReinForce-II/cleanup_code

generic cleanup and fix

Author: bitsk

cores/arduino/wiring_analog.c

@@ -52,6 +52,7 @@ int analogRead(pin_size_t pinNumber) {
         return -1;
     }
 #endif
+    return -1;
 }
 
 void analogWrite(pin_size_t pinNumber, int value) {
@@ -63,7 +64,7 @@ void analogWrite(pin_size_t pinNumber, int value) {
     rcu_periph_clock_enable(PIN_MAP[pinNumber].timer_device->clk_id);
     timer_deinit(PIN_MAP[pinNumber].timer_device->timer_dev);
     timer_initpara.prescaler
-        = rcu_clock_freq_get(CK_APB1) == rcu_clock_freq_get(CK_AHB)
+        = rcu_clock_freq_get(CK_APB1) != rcu_clock_freq_get(CK_AHB)
         ? 2 * rcu_clock_freq_get(CK_APB1) / 1000000 - 1
         : rcu_clock_freq_get(CK_APB1) / 1000000 - 1;
     timer_initpara.alignedmode = TIMER_COUNTER_EDGE;

cores/arduino/wiring_pulse.c

@@ -24,16 +24,14 @@ unsigned long pulseIn(pin_size_t pin, uint8_t state, unsigned long timeout) {
     }
     uint64_t timeoutInCycles = timeout * _ClockCyclesPerMicrosecond();
     uint64_t start = get_mcycle();
-    while (gpio_input_bit_get(
-               PIN_MAP[pin].gpio_device->gpio_port, PIN_MAP[pin].gpio_bit)
+    while (gpio_input_bit_get(digitalPinToPort(pin), digitalPinToBitMask(pin))
         != state) {
         if (get_mcycle() - start > timeoutInCycles) {
             return 0;
         }
     }
     uint64_t pulseBegin = get_mcycle();
-    while (gpio_input_bit_get(
-               PIN_MAP[pin].gpio_device->gpio_port, PIN_MAP[pin].gpio_bit)
+    while (gpio_input_bit_get(digitalPinToPort(pin), digitalPinToBitMask(pin))
         == state) {
         if (get_mcycle() - start > timeoutInCycles) {
             return 0;

cores/arduino/wiring_shift.c

@@ -10,23 +10,21 @@ void shiftOut(
     if (clockPin > VARIANT_GPIO_NUM) {
         return;
     }
-    gpio_bit_reset(
-        PIN_MAP[clockPin].gpio_device->gpio_port, PIN_MAP[clockPin].gpio_bit);
+    gpio_bit_reset(digitalPinToPort(clockPin), digitalPinToBitMask(clockPin));
     for (size_t i = 0; i < 8; i++) {
         if (bitOrder == LSBFIRST) {
-            gpio_bit_write(PIN_MAP[dataPin].gpio_device->gpio_port,
-                PIN_MAP[dataPin].gpio_bit, (val & 0x01) == 0 ? RESET : SET);
+            gpio_bit_write(digitalPinToPort(dataPin),
+                digitalPinToBitMask(dataPin), (val & 0x01) == 0 ? RESET : SET);
             val = val >> 1;
         }
         else {
-            gpio_bit_write(PIN_MAP[dataPin].gpio_device->gpio_port,
-                PIN_MAP[dataPin].gpio_bit, (val & 0x80) == 0 ? RESET : SET);
+            gpio_bit_write(digitalPinToPort(dataPin),
+                digitalPinToBitMask(dataPin), (val & 0x80) == 0 ? RESET : SET);
             val = val << 1;
         }
-        gpio_bit_set(PIN_MAP[clockPin].gpio_device->gpio_port,
-            PIN_MAP[clockPin].gpio_bit);
-        gpio_bit_reset(PIN_MAP[clockPin].gpio_device->gpio_port,
-            PIN_MAP[clockPin].gpio_bit);
+        gpio_bit_set(digitalPinToPort(clockPin), digitalPinToBitMask(clockPin));
+        gpio_bit_reset(
+            digitalPinToPort(clockPin), digitalPinToBitMask(clockPin));
     }
 }
 
@@ -38,26 +36,24 @@ pin_size_t shiftIn(pin_size_t dataPin, pin_size_t clockPin, BitOrder bitOrder) {
     if (clockPin > VARIANT_GPIO_NUM) {
         return 0;
     }
-    gpio_bit_reset(
-        PIN_MAP[clockPin].gpio_device->gpio_port, PIN_MAP[clockPin].gpio_bit);
+    gpio_bit_reset(digitalPinToPort(clockPin), digitalPinToBitMask(clockPin));
     for (size_t i = 0; i < 8; i++) {
         if (bitOrder == LSBFIRST) {
             ret = ret >> 1;
             ret = ret
-                & (gpio_input_bit_get(PIN_MAP[dataPin].gpio_device->gpio_port,
-                       PIN_MAP[dataPin].gpio_bit)
+                & (gpio_input_bit_get(
+                       digitalPinToPort(dataPin), digitalPinToBitMask(dataPin))
                     << 7);
         }
         else {
             ret = ret << 1;
             ret = ret
-                & gpio_input_bit_get(PIN_MAP[dataPin].gpio_device->gpio_port,
-                    PIN_MAP[dataPin].gpio_bit);
+                & gpio_input_bit_get(
+                    digitalPinToPort(dataPin), digitalPinToBitMask(dataPin));
         }
-        gpio_bit_set(PIN_MAP[clockPin].gpio_device->gpio_port,
-            PIN_MAP[clockPin].gpio_bit);
-        gpio_bit_reset(PIN_MAP[clockPin].gpio_device->gpio_port,
-            PIN_MAP[clockPin].gpio_bit);
+        gpio_bit_set(digitalPinToPort(clockPin), digitalPinToBitMask(clockPin));
+        gpio_bit_reset(
+            digitalPinToPort(clockPin), digitalPinToBitMask(clockPin));
     }
     return ret;
 }