From dfbe14a6d8cd8aaa830dbd54f5d2d58a39c78b72 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Wed, 12 Oct 2022 18:49:56 +0000 Subject: [PATCH 1/6] Bump geekyeggo/delete-artifact from 1 to 2 Bumps [geekyeggo/delete-artifact](https://github.com/geekyeggo/delete-artifact) from 1 to 2. - [Release notes](https://github.com/geekyeggo/delete-artifact/releases) - [Commits](https://github.com/geekyeggo/delete-artifact/compare/v1...v2) --- updated-dependencies: - dependency-name: geekyeggo/delete-artifact dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] --- .github/workflows/sync-labels.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/sync-labels.yml b/.github/workflows/sync-labels.yml index 986bda6..10abaea 100644 --- a/.github/workflows/sync-labels.yml +++ b/.github/workflows/sync-labels.yml @@ -114,7 +114,7 @@ jobs: path: ${{ env.CONFIGURATIONS_FOLDER }} - name: Remove unneeded artifact - uses: geekyeggo/delete-artifact@v1 + uses: geekyeggo/delete-artifact@v2 with: name: ${{ env.CONFIGURATIONS_ARTIFACT }} From c6d8f98eb9af5bdd41ea6850a5460129fbe7b14a Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Mon, 24 Oct 2022 18:53:43 +0000 Subject: [PATCH 2/6] Bump carlosperate/download-file-action from 1 to 2 Bumps [carlosperate/download-file-action](https://github.com/carlosperate/download-file-action) from 1 to 2. - [Release notes](https://github.com/carlosperate/download-file-action/releases) - [Commits](https://github.com/carlosperate/download-file-action/compare/v1...v2) --- updated-dependencies: - dependency-name: carlosperate/download-file-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] --- .github/workflows/sync-labels.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/sync-labels.yml b/.github/workflows/sync-labels.yml index 10abaea..94938f3 100644 --- a/.github/workflows/sync-labels.yml +++ b/.github/workflows/sync-labels.yml @@ -31,7 +31,7 @@ jobs: - name: Download JSON schema for labels configuration file id: download-schema - uses: carlosperate/download-file-action@v1 + uses: carlosperate/download-file-action@v2 with: file-url: https://raw.githubusercontent.com/arduino/tooling-project-assets/main/workflow-templates/assets/sync-labels/arduino-tooling-gh-label-configuration-schema.json location: ${{ runner.temp }}/label-configuration-schema @@ -65,7 +65,7 @@ jobs: steps: - name: Download - uses: carlosperate/download-file-action@v1 + uses: carlosperate/download-file-action@v2 with: file-url: https://raw.githubusercontent.com/arduino/tooling-project-assets/main/workflow-templates/assets/sync-labels/${{ matrix.filename }} From 5a1bd3feea34447851ec23c506fa79a4a1fde148 Mon Sep 17 00:00:00 2001 From: THE-Spellchecker Date: Mon, 26 Dec 2022 23:14:13 -0600 Subject: [PATCH 3/6] Corrected Comment in AnalogInput.ino --- examples/03.Analog/AnalogInput/AnalogInput.ino | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/examples/03.Analog/AnalogInput/AnalogInput.ino b/examples/03.Analog/AnalogInput/AnalogInput.ino index 735a8b5..923079c 100644 --- a/examples/03.Analog/AnalogInput/AnalogInput.ino +++ b/examples/03.Analog/AnalogInput/AnalogInput.ino @@ -45,6 +45,6 @@ void loop() { delay(sensorValue); // turn the ledPin off: digitalWrite(ledPin, LOW); - // stop the program for for milliseconds: + // stop the program for milliseconds: delay(sensorValue); } From 74be767c762ed3bcd44a3c39729fee4b34173d81 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Mon, 4 Sep 2023 18:36:25 +0000 Subject: [PATCH 4/6] Bump actions/checkout from 3 to 4 Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](https://github.com/actions/checkout/compare/v3...v4) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] --- .github/workflows/code-formatting-check.yml | 2 +- .github/workflows/compile-examples.yml | 2 +- .github/workflows/spell-check.yml | 2 +- .github/workflows/sync-labels.yml | 4 ++-- 4 files changed, 5 insertions(+), 5 deletions(-) diff --git a/.github/workflows/code-formatting-check.yml b/.github/workflows/code-formatting-check.yml index 5406dae..bb3479f 100644 --- a/.github/workflows/code-formatting-check.yml +++ b/.github/workflows/code-formatting-check.yml @@ -29,7 +29,7 @@ jobs: echo "CLANG_FORMAT_INSTALL_PATH=${{ runner.temp }}/clang-format" >> "$GITHUB_ENV" - name: Checkout - uses: actions/checkout@v3 + uses: actions/checkout@v4 - name: Download ClangFormat id: download diff --git a/.github/workflows/compile-examples.yml b/.github/workflows/compile-examples.yml index 646db49..d91dde1 100644 --- a/.github/workflows/compile-examples.yml +++ b/.github/workflows/compile-examples.yml @@ -212,7 +212,7 @@ jobs: steps: - name: Checkout - uses: actions/checkout@v3 + uses: actions/checkout@v4 - name: Compile examples uses: arduino/compile-sketches@v1 diff --git a/.github/workflows/spell-check.yml b/.github/workflows/spell-check.yml index 6a382c5..a3af916 100644 --- a/.github/workflows/spell-check.yml +++ b/.github/workflows/spell-check.yml @@ -13,7 +13,7 @@ jobs: steps: - name: Checkout - uses: actions/checkout@v3 + uses: actions/checkout@v4 # See: https://github.com/codespell-project/actions-codespell/blob/master/README.md - name: Spell check diff --git a/.github/workflows/sync-labels.yml b/.github/workflows/sync-labels.yml index 94938f3..9cde1ac 100644 --- a/.github/workflows/sync-labels.yml +++ b/.github/workflows/sync-labels.yml @@ -27,7 +27,7 @@ jobs: steps: - name: Checkout repository - uses: actions/checkout@v3 + uses: actions/checkout@v4 - name: Download JSON schema for labels configuration file id: download-schema @@ -105,7 +105,7 @@ jobs: echo "::set-output name=flag::--dry-run" - name: Checkout repository - uses: actions/checkout@v3 + uses: actions/checkout@v4 - name: Download configuration files artifact uses: actions/download-artifact@v3 From fdbc6ba8f1041f727428f4c614a6c603b3491d5e Mon Sep 17 00:00:00 2001 From: per1234 Date: Sat, 30 Dec 2023 05:10:14 -0800 Subject: [PATCH 5/6] Add new boards to sketch compilation workflow The repository infrastructure includes a GitHub Actions workflow that compiles the examples for significant official Arduino boards. This provides a automated "smoke test" to easily get some basic validation of the sketches. Since the time the workflow was created, several new official boards have been released. Some of these boards are significantly different from the boards the workflow was compiling the examples for, which meant the workflow was not providing adequate coverage for the new boards. The workflow is hereby configured to also compile the examples for the significant new boards. --- .github/workflows/compile-examples.yml | 30 ++++++++++++++++++++++++++ 1 file changed, 30 insertions(+) diff --git a/.github/workflows/compile-examples.yml b/.github/workflows/compile-examples.yml index d91dde1..b184f9d 100644 --- a/.github/workflows/compile-examples.yml +++ b/.github/workflows/compile-examples.yml @@ -120,18 +120,48 @@ jobs: starter-kit: false tone: true a5: true + - fqbn: arduino:esp32:nano_nora + usb: false + serial1: true + starter-kit: false + tone: true + a5: true - fqbn: arduino:megaavr:uno2018:mode=off usb: false serial1: true starter-kit: false tone: true a5: true + - fqbn: arduino:renesas_uno:minima + usb: true + serial1: true + starter-kit: false + tone: true + a5: true + - fqbn: arduino:renesas_uno:unor4wifi + usb: true + serial1: true + starter-kit: false + tone: true + a5: true + - fqbn: arduino:renesas_portenta:portenta_c33 + usb: true + serial1: true + starter-kit: false + tone: true + a5: true - fqbn: arduino:samd:mkrzero usb: true serial1: true starter-kit: false tone: true a5: true + - fqbn: arduino:mbed_giga:giga + usb: false + serial1: true + starter-kit: false + tone: true + a5: true - fqbn: arduino:mbed_nano:nano33ble usb: false serial1: true From 9806d587c0b5f2b1b286468f0540f60524573373 Mon Sep 17 00:00:00 2001 From: per1234 Date: Sat, 30 Dec 2023 05:14:58 -0800 Subject: [PATCH 6/6] Use distinctive names for SPI pin macros to avoid collisions The "ArduinoISP" example allows the user to define arbitrary pins for use as the signal pins on the programmer board. This is done via a set of macros. Previously, very generic names were used for these macros. This resulted in a name collision. The core for the Renesas boards also defines macros of the same name. Despite what the names might lead us to believe, the values of these macros in the core do not match the Arduino pin numbers of the SPI bus. The result was that, with the default configuration of the sketch where the USE_OLD_STYLE_WIRING macro is not defined, the sketch uses the values of the macros defined by the core as the SPI signal pins instead of the standard SPI signal pins as the user would expect when the sketch is running on an UNO R4 Minima, UNO R4 WiFi, or Portenta C33 board. This causes operations using the programmer to fail with an error like: avrdude: stk500_recv(): programmer is not responding avrdude: stk500_getsync() attempt 1 of 10: not in sync: resp=0x03 [...] The name collision is avoided by adding a "namespace" prefix to the macro names. --- .../11.ArduinoISP/ArduinoISP/ArduinoISP.ino | 54 +++++++++---------- 1 file changed, 27 insertions(+), 27 deletions(-) diff --git a/examples/11.ArduinoISP/ArduinoISP/ArduinoISP.ino b/examples/11.ArduinoISP/ArduinoISP/ArduinoISP.ino index c75d233..bbbcdc9 100644 --- a/examples/11.ArduinoISP/ArduinoISP/ArduinoISP.ino +++ b/examples/11.ArduinoISP/ArduinoISP/ArduinoISP.ino @@ -22,8 +22,8 @@ // using an Uno. (On an Uno this is not needed). // // Alternatively you can use any other digital pin by configuring -// software ('BitBanged') SPI and having appropriate defines for PIN_MOSI, -// PIN_MISO and PIN_SCK. +// software ('BitBanged') SPI and having appropriate defines for ARDUINOISP_PIN_MOSI, +// ARDUINOISP_PIN_MISO and ARDUINOISP_PIN_SCK. // // IMPORTANT: When using an Arduino that is not 5V tolerant (Due, Zero, ...) as // the programmer, make sure to not expose any of the programmer's pins to 5V. @@ -82,9 +82,9 @@ #ifdef USE_OLD_STYLE_WIRING -#define PIN_MOSI 11 -#define PIN_MISO 12 -#define PIN_SCK 13 +#define ARDUINOISP_PIN_MOSI 11 +#define ARDUINOISP_PIN_MISO 12 +#define ARDUINOISP_PIN_SCK 13 #endif @@ -100,20 +100,20 @@ #endif // By default, use hardware SPI pins: -#ifndef PIN_MOSI -#define PIN_MOSI MOSI +#ifndef ARDUINOISP_PIN_MOSI +#define ARDUINOISP_PIN_MOSI MOSI #endif -#ifndef PIN_MISO -#define PIN_MISO MISO +#ifndef ARDUINOISP_PIN_MISO +#define ARDUINOISP_PIN_MISO MISO #endif -#ifndef PIN_SCK -#define PIN_SCK SCK +#ifndef ARDUINOISP_PIN_SCK +#define ARDUINOISP_PIN_SCK SCK #endif // Force bitbanged SPI if not using the hardware SPI pins: -#if (PIN_MISO != MISO) || (PIN_MOSI != MOSI) || (PIN_SCK != SCK) +#if (ARDUINOISP_PIN_MISO != MISO) || (ARDUINOISP_PIN_MOSI != MOSI) || (ARDUINOISP_PIN_SCK != SCK) #undef USE_HARDWARE_SPI #endif @@ -186,11 +186,11 @@ private: class BitBangedSPI { public: void begin() { - digitalWrite(PIN_SCK, LOW); - digitalWrite(PIN_MOSI, LOW); - pinMode(PIN_SCK, OUTPUT); - pinMode(PIN_MOSI, OUTPUT); - pinMode(PIN_MISO, INPUT); + digitalWrite(ARDUINOISP_PIN_SCK, LOW); + digitalWrite(ARDUINOISP_PIN_MOSI, LOW); + pinMode(ARDUINOISP_PIN_SCK, OUTPUT); + pinMode(ARDUINOISP_PIN_MOSI, OUTPUT); + pinMode(ARDUINOISP_PIN_MISO, INPUT); } void beginTransaction(SPISettings settings) { @@ -204,11 +204,11 @@ public: uint8_t transfer(uint8_t b) { for (unsigned int i = 0; i < 8; ++i) { - digitalWrite(PIN_MOSI, (b & 0x80) ? HIGH : LOW); - digitalWrite(PIN_SCK, HIGH); + digitalWrite(ARDUINOISP_PIN_MOSI, (b & 0x80) ? HIGH : LOW); + digitalWrite(ARDUINOISP_PIN_SCK, HIGH); delayMicroseconds(pulseWidth); - b = (b << 1) | digitalRead(PIN_MISO); - digitalWrite(PIN_SCK, LOW); // slow pulse + b = (b << 1) | digitalRead(ARDUINOISP_PIN_MISO); + digitalWrite(ARDUINOISP_PIN_SCK, LOW); // slow pulse delayMicroseconds(pulseWidth); } return b; @@ -408,7 +408,7 @@ void set_parameters() { void start_pmode() { - // Reset target before driving PIN_SCK or PIN_MOSI + // Reset target before driving ARDUINOISP_PIN_SCK or ARDUINOISP_PIN_MOSI // SPI.begin() will configure SS as output, so SPI master mode is selected. // We have defined RESET as pin 10, which for many Arduinos is not the SS pin. @@ -421,9 +421,9 @@ void start_pmode() { // See AVR datasheets, chapter "SERIAL_PRG Programming Algorithm": - // Pulse RESET after PIN_SCK is low: - digitalWrite(PIN_SCK, LOW); - delay(20); // discharge PIN_SCK, value arbitrarily chosen + // Pulse RESET after ARDUINOISP_PIN_SCK is low: + digitalWrite(ARDUINOISP_PIN_SCK, LOW); + delay(20); // discharge ARDUINOISP_PIN_SCK, value arbitrarily chosen reset_target(false); // Pulse must be minimum 2 target CPU clock cycles so 100 usec is ok for CPU // speeds above 20 KHz @@ -439,8 +439,8 @@ void start_pmode() { void end_pmode() { SPI.end(); // We're about to take the target out of reset so configure SPI pins as input - pinMode(PIN_MOSI, INPUT); - pinMode(PIN_SCK, INPUT); + pinMode(ARDUINOISP_PIN_MOSI, INPUT); + pinMode(ARDUINOISP_PIN_SCK, INPUT); reset_target(false); pinMode(RESET, INPUT); pmode = 0;