Merge pull request #4294 from kamtom480/spresense_analogin

tannewt · web-flow · commit 1b3bca14cbb2 · 2021-03-01T13:21:15.000-08:00
spresense: return the correct value for Analog In
diff --git a/ports/cxd56/common-hal/analogio/AnalogIn.c b/ports/cxd56/common-hal/analogio/AnalogIn.c
@@ -105,11 +105,11 @@ bool common_hal_analogio_analogin_deinited(analogio_analogin_obj_t *self) {
 }
 
 uint16_t common_hal_analogio_analogin_get_value(analogio_analogin_obj_t *self) {
-    uint16_t value = 0;
+    int16_t value = 0;
 
     read(analogin_dev[self->number].fd, &value, sizeof(value));
 
-    return value;
+    return (uint16_t) 32768 + (uint16_t) value;
 }
 
 // Reference voltage is a fixed value which is depending on the board.

Original file line number	Diff line number	Diff line change
`@@ -105,11 +105,11 @@ bool common_hal_analogio_analogin_deinited(analogio_analogin_obj_t *self) {`
`105`	`105`	`}`
`106`	`106`
`107`	`107`	`uint16_t common_hal_analogio_analogin_get_value(analogio_analogin_obj_t *self) {`
`108`		`- uint16_t value = 0;`
	`108`	`+ int16_t value = 0;`
`109`	`109`
`110`	`110`	`read(analogin_dev[self->number].fd, &value, sizeof(value));`
`111`	`111`
`112`		`- return value;`
	`112`	`+ return (uint16_t) 32768 + (uint16_t) value;`
`113`	`113`	`}`
`114`	`114`
`115`	`115`	`// Reference voltage is a fixed value which is depending on the board.`