Implement analogReadResolution

The current released version, was always reading with 12 bits of resolution.
The Arduino analogRead web page said that by default it should default to 10 bits.

The updated code is now ignoring the arduino_adc[idx].resolution
field as this is setup for 12 and you can not change it run time.

Like the MBED version,  this version is using the static variable read_resolution to hold the current resolution.  Which we then pass through to zephyr when we do the reads.

I also changed the parameter buf that we pass through to zephyr from in16_t to uint16_t as if you choose resolution of 16 bits it was returning negative values.

Author: KurtE

cores/arduino/Arduino.h

@@ -100,6 +100,9 @@ enum digitalPins {
 enum analogPins { DT_FOREACH_PROP_ELEM(DT_PATH(zephyr_user),
 				       adc_pin_gpios, AN_ENUMS) };
 
+// We provide analogReadResolution APIs
+void analogReadResolution(int bits);
+
 #endif
 
 void interrupts(void);

cores/arduino/zephyrCommon.cpp

@@ -282,11 +282,11 @@ void analogWrite(pin_size_t pinNumber, int value)
 {
   size_t idx = pwm_pin_index(pinNumber);
 
-  if (!pwm_is_ready_dt(&arduino_pwm[idx])) {
+  if (idx >= ARRAY_SIZE(arduino_pwm) ) {
     return;
   }
 
-  if (idx >= ARRAY_SIZE(arduino_pwm) ) {
+  if (!pwm_is_ready_dt(&arduino_pwm[idx])) {
     return;
   }
 
@@ -319,10 +319,21 @@ void analogReference(uint8_t mode)
   }
 }
 
+// Note: We can not update the arduino_adc structure as it is read only...
+static int read_resolution = 10;
+
+void analogReadResolution(int bits)
+{
+  read_resolution = bits;
+  //for(size_t i=0; i<ARRAY_SIZE(arduino_analog_pins); i++) {
+  //  arduino_adc[i].resolution = bits;
+  // }
+ }
+
 int analogRead(pin_size_t pinNumber)
 {
   int err;
-  int16_t buf;
+  uint16_t buf;
   struct adc_sequence seq = { .buffer = &buf, .buffer_size = sizeof(buf) };
   size_t idx = analog_pin_index(pinNumber);
 
@@ -344,7 +355,8 @@ int analogRead(pin_size_t pinNumber)
   }
 
   seq.channels = BIT(arduino_adc[idx].channel_id);
-  seq.resolution = arduino_adc[idx].resolution;
+  //seq.resolution = arduino_adc[idx].resolution;
+  seq.resolution = read_resolution;
   seq.oversampling = arduino_adc[idx].oversampling;
 
   err = adc_read(arduino_adc[idx].dev, &seq);