Support get/set of analog period.

Author: microbit-matt-hillsdon

src/board/index.ts

@@ -8,13 +8,14 @@ import {
   MICROBIT_HAL_PIN_P0,
   MICROBIT_HAL_PIN_P1,
   MICROBIT_HAL_PIN_P2,
+  MICROBIT_HAL_PIN_P3,
 } from "./constants";
 import * as conversions from "./conversions";
 import { DataLogging } from "./data-logging";
 import { Display } from "./display";
 import { FileSystem } from "./fs";
 import { Microphone } from "./microphone";
-import { Pin } from "./pins";
+import { Pin, StubPin, TouchPin } from "./pins";
 import { Radio } from "./radio";
 import { RangeSensor, State } from "./state";
 import { ModuleWrapper } from "./wasm";
@@ -121,17 +122,24 @@ export class Board {
       ),
     ];
     this.pins = Array(33);
-    this.pins[MICROBIT_HAL_PIN_FACE] = new Pin(
+    this.pins[MICROBIT_HAL_PIN_FACE] = new TouchPin(
       "pinLogo",
       {
         element: this.svg.querySelector("#Logo")!,
         label: () => this.formattedMessage({ id: "touch-logo" }),
       },
       onChange
     );
-    this.pins[MICROBIT_HAL_PIN_P0] = new Pin("pin0", null, onChange);
-    this.pins[MICROBIT_HAL_PIN_P1] = new Pin("pin1", null, onChange);
-    this.pins[MICROBIT_HAL_PIN_P2] = new Pin("pin2", null, onChange);
+    this.pins[MICROBIT_HAL_PIN_P0] = new TouchPin("pin0", null, onChange);
+    this.pins[MICROBIT_HAL_PIN_P1] = new TouchPin("pin1", null, onChange);
+    this.pins[MICROBIT_HAL_PIN_P2] = new TouchPin("pin2", null, onChange);
+    for (let pin = 3; pin <= 19; ++pin) {
+      if (pin === 17 || pin === 18) {
+        continue;
+      }
+      this.pins[pin] = new StubPin(`pin${pin}`);
+    }
+
     this.audio = new Audio();
     this.temperature = new RangeSensor("temperature", -5, 50, 21, "°C");
     this.accelerometer = new Accelerometer(onChange);

src/board/pins.ts

@@ -1,8 +1,61 @@
 import { RangeSensor, State } from "./state";
 
-export class Pin {
+export interface Pin {
   state: RangeSensor;
 
+  updateTranslations(): void;
+
+  setValue(value: any): void;
+
+  isTouched(): boolean;
+
+  boardStopped(): void;
+
+  setAnalogPeriodUs(period: number): number;
+
+  getAnalogPeriodUs(): number;
+
+}
+
+abstract class BasePin implements Pin {
+  state: RangeSensor;
+
+  // For now we just allow get/set of this value
+  // but don't do anything with it.
+  private analogPeriodUs: number = -1;
+
+  constructor(id: string) {
+    this.state = new RangeSensor(id, 0, 1, 0, undefined)
+  }
+
+  updateTranslations() {
+  }
+
+  setValue(value: any): void {
+    this.state.value = value;
+  }
+
+  setAnalogPeriodUs(period: number) {
+    this.analogPeriodUs = period;
+    return 0;
+  }
+
+  getAnalogPeriodUs() {
+    return this.analogPeriodUs;
+  }
+
+  isTouched(): boolean {
+    return false;
+  }
+
+  boardStopped() {
+  }
+}
+
+export class StubPin extends BasePin {
+}
+
+export class TouchPin extends BasePin {
   private _mouseDown: boolean = false;
 
   private keyListener: (e: KeyboardEvent) => void;
@@ -16,7 +69,7 @@ export class Pin {
     private ui: { element: SVGElement; label: () => string } | null,
     private onChange: (changes: Partial<State>) => void
   ) {
-    this.state = new RangeSensor(id, 0, 1, 0, undefined);
+    super(id);
 
     if (this.ui) {
       const { element, label } = this.ui;
@@ -76,7 +129,8 @@ export class Pin {
   }
 
   private setValueInternal(value: any, internalChange: boolean) {
-    this.state.setValue(value);
+    super.setValue(value);
+
     if (internalChange) {
       this.onChange({
         [this.id]: this.state,

src/jshal.h

@@ -48,6 +48,8 @@ int mp_js_hal_button_get_presses(int button);
 bool mp_js_hal_button_is_pressed(int button);
 
 bool mp_js_hal_pin_is_touched(int pin);
+int mp_js_hal_pin_get_analog_period_us(int pin);
+int mp_js_hal_pin_set_analog_period_us(int pin, int period);
 
 int mp_js_hal_display_get_pixel(int x, int y);
 void mp_js_hal_display_set_pixel(int x, int y, int value);

src/jshal.js

@@ -127,6 +127,14 @@ mergeInto(LibraryManager.library, {
     return Module.board.pins[pin].isTouched();
   },
 
+  mp_js_hal_pin_get_analog_period_us: function (/** @type {number} */ pin) {
+    return Module.board.pins[pin].getAnalogPeriodUs();
+  },
+
+  mp_js_hal_pin_set_analog_period_us: function (/** @type {number} */ pin, /** @type {number} */ period) {
+    return Module.board.pins[pin].setAnalogPeriodUs(period);
+  },
+
   mp_js_hal_display_get_pixel: function (
     /** @type {number} */ x,
     /** @type {number} */ y

src/microbithal_js.c

@@ -113,21 +113,11 @@ int microbit_hal_pin_set_analog_period_us(int pin, int period) {
         mp_js_hal_audio_period_us(period);
         return 0;
     }
-    // We don't support other pins yet but more practical to claim success as
-    // this allows programs using servos to otherwise function.
-    return 0;
+    return mp_js_hal_pin_set_analog_period_us(pin, period);
 }
 
 int microbit_hal_pin_get_analog_period_us(int pin) {
-    /*
-    int period = pin_obj[pin]->getAnalogPeriodUs();
-    if (period != DEVICE_NOT_SUPPORTED) {
-        return period;
-    } else {
-        return -1;
-    }
-    */
-    return -1;
+    return mp_js_hal_pin_get_analog_period_us(pin);
 }
 
 void microbit_hal_pin_set_touch_mode(int pin, int mode) {