Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Board support for Adafruit Grand Central M4 #7

Open
wants to merge 1,416 commits into
base: master
Choose a base branch
from
Open

Board support for Adafruit Grand Central M4 #7

wants to merge 1,416 commits into from

Conversation

graeme-winter
Copy link

Adds:

  • Adafruit grand central M4 board based on itsybitsy
  • Full pin list for board
  • Bug fixes for SAMD51 A0, A1 DAC

@graeme-winter graeme-winter force-pushed the adafruit-grand-central branch 11 times, most recently from 8cf4aee to 3315632 Compare March 27, 2023 08:49
@graeme-winter graeme-winter force-pushed the adafruit-grand-central branch 2 times, most recently from 598b551 to 1506cfb Compare April 4, 2023 07:25
@graeme-winter graeme-winter force-pushed the adafruit-grand-central branch 2 times, most recently from 73d09f1 to e348705 Compare April 27, 2023 10:39
@graeme-winter graeme-winter force-pushed the adafruit-grand-central branch 4 times, most recently from 77d211e to 8e97011 Compare June 10, 2023 06:27
sylvinus and others added 11 commits September 21, 2024 08:00
@sylvinus @graeme-winter
extmod/mbedtls: Enable GCM and ECDHE-RSA in common mbedtls config.
ea54ee3 
Enable support for cipher suites like
TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256, as suggested in
micropython#14204 (comment)
and micropython#10485 (comment)

Tests have been run on the top 500 domains from moz.com.  Without this
patch, 155 out of 500 fail to connect because of TLS issues.  This patch
fixes them all.  And it seems all existing mbedtls flags are needed to get
good coverage of those top 500 domains.

The `ssl_poll.py` test has the cipher bits increased from 512 to 1024 in
its test key/cert so that it can work with ECDHE-RSA which is now the
chosen cipher.

Signed-off-by: Sylvain Zimmer <[email protected]>
Signed-off-by: Damien George <[email protected]>
@dpgeorge @graeme-winter
rp2/mbedtls: Remove config options that are now in the common settings.
a0fbead 
Signed-off-by: Damien George <[email protected]>
@Gadgetoid @graeme-winter
rp2/rp2_pio: Replace PIO_NUM macro with pio_get_index.
e8cdd8c 
The `PIO_NUM` macro was defined when `rp2_pio.c` was first conceived.
There's now a Pico SDK function for this, `pio_get_index()`, which is
already used in some parts of the code.

This commit removes `PIO_NUM` in favour of using `pio_get_index()`
everywhere.

Signed-off-by: Phil Howard <[email protected]>
@Gadgetoid @graeme-winter
rp2/rp2_pio: Replace explicit pio ternary expression with pio_get_index.
794ad02 
There are three changes here:
- Fix `rp2_pio_print` to use `pio_get_index()` too, since it had its own
  copy of the ternary expression.
- Remove a ternary from `rp2_pio_state_machine` and calculate it from
  `pio_get_index`.
- Remove a ternary on `GPIO_FUNC_PIO0` vs `GPIO_FUNC_PIO1`.  These
  constants are sequentially ordered so we can calculate them too.

Signed-off-by: Phil Howard <[email protected]>
@robert-hh @graeme-winter
rp2/machine_adc: Initialise ADC GPIO when a Pin is referenced by int.
d758616 
The change closes the gap in the API when an integer is used as Pin
reference.  With the change, e.g. ADC(26), ADC(Pin(26)) and ADC("GP26")
behave identically and the GPIO is initialised in ACD/high-Z mode.

Only when using ADC channel numbers 0-3 are the corresponding GPIO left
uninitialised, and then the user is responsible for configuring the GPIO.

Signed-off-by: robert-hh <[email protected]>
@robert-hh @graeme-winter
docs/rp2/quickref: Document the use of channel numbers for ADC.
e54d171 
Signed-off-by: robert-hh <[email protected]>
@jonfoster @graeme-winter
py/objstr: Add new mp_obj_new_str_from_cstr() helper function.
0d476e0 
There were lots of places where this pattern was duplicated, to convert a
standard C string to a MicroPython string:

    x = mp_obj_new_str(s, strlen(s));

This commit provides a simpler method that removes this code duplication:

    x = mp_obj_new_str_from_cstr(s);

This gives clearer, and probably smaller, code.

Signed-off-by: Jon Foster <[email protected]>
@jonfoster @graeme-winter
all: Use new mp_obj_new_str_from_cstr() function.
741332a 
Use new function mp_obj_new_str_from_cstr() where appropriate.  It
simplifies the code, and makes it smaller too.

Signed-off-by: Jon Foster <[email protected]>
@dpgeorge @graeme-winter
esp32,esp8266: Use new mp_obj_new_str_from_cstr() function.
5cd28c1 
These were missed in the previous commit.

Signed-off-by: Damien George <[email protected]>
@felixdoerre @graeme-winter
docs: Update docs to replace ifconfig with ipconfig.
25814be 
Follow up to 1c6012b

Signed-off-by: Felix Dörre <[email protected]>
@dpgeorge @graeme-winter
tests/thread/stress_aes.py: Fix logic waiting for finished threads.
7f2107d 
Because the main thread executes `thread_entry()` it means there's an
additional one added to `count`, so the test must wait for the count to
reach `n_thread + 1`.

Signed-off-by: Damien George <[email protected]>
higstar and others added 29 commits September 21, 2024 08:00
@higstar @graeme-winter
stm32: Update STM32L452xx Multi OTG handling.
8cad2f0 
Added custom Multi OTG handling for STM32L452xx, based on STM32L432xx
handling.

Fixes issue micropython#15795.

Signed-off-by: Adrian Higgins <[email protected]>
@mattytrentini @graeme-winter
stm32/boards/STM32F429DISC: Add DAC support to the STM32F429DISC.
52a2e08 
The STM32F429DISC board definition did not have DAC enabled, however the
micro/board supports it so this commit enables the feature.

Signed-off-by: Matt Trentini <[email protected]>
@mattytrentini @graeme-winter
examples/natmod: Fix URL links in README.md.
8fbfd02 
Signed-off-by: Matt Trentini <[email protected]>
@pdg137 @graeme-winter
docs/esp32: Update pin access example with addresses for ESP32-S3.
698d236 
Signed-off-by: Paul Grayson <[email protected]>
Signed-off-by: Angus Gratton <[email protected]>
@dpgeorge @graeme-winter
docs/reference: Fix pyboard.py filesystem cp example with three files.
7e1913c 
Fix documentation to match behaviour (directories are not preserved).

Fixes issue micropython#11101.

Signed-off-by: Damien George <[email protected]>
@dpgeorge @graeme-winter
docs/library: Document math.log with two arguments.
299effd 
The functionality is there but was not documented.

Signed-off-by: Damien George <[email protected]>
@projectgus @graeme-winter
docs/library: Document machine.Pin.toggle() method.
d35f399 
Original commit was by @millosolomillo from 2022, but CI no longer accepts
their auto-generated GitHub commit email...

Signed-off-by: Angus Gratton <[email protected]>
@ZodiusInfuser @graeme-winter
rp2: Add board-level hooks to main, and MICROPY_SOURCE_BOARD cmake var.
e8bb349 
Signed-off-by: Christopher Parrott <[email protected]>
@ZodiusInfuser @graeme-winter
rp2: Increase ext pins from 10 to 32.
8769aac 
To allow more pins when other ways are used to provide external GPIO (ie
not via cyw43).

Signed-off-by: Christopher Parrott <[email protected]>
@ZodiusInfuser @graeme-winter
rp2/machine_pin_cyw43: Include check for CYW43_GPIO.
2f566ad 
The `#if` check only checks that `MICROPY_PY_NETWORK_CYW43` and
`MICROPY_HW_PIN_EXT_COUNT` are defined.  This is a reasonable assumption
for the Pico W, but causes conflicts if someone wants to attach an external
IO expander to their Pico W and have its pins appear as Pin objects.

This commit addresses this by adding the additional checks, letting board
builds include wireless but separately choose whether the external IO pins
come from the cyw43 or not.

Signed-off-by: Christopher Parrott <[email protected]>
@iabdalkader @graeme-winter
rp2/modules: Fix FatFS boot script to detect invalid FAT filesystem.
6fb79f8 
This change helps detect if the filesystem is invalid, by also including
the first mount attempt within the try-except.  Then the FAT is reformatted
if needed.

Fixes issue micropython#15779.

Signed-off-by: iabdalkader <[email protected]>
@projectgus @graeme-winter
py/scheduler: Only run callbacks on the main thread if GIL is disabled.
59e2b06 
Otherwise it's very difficult to reason about thread safety in a
scheduler callback, as it can run at any time on any thread - including
racing against any bytecode operation on any thread.

This work was funded through GitHub Sponsors.

Signed-off-by: Angus Gratton <[email protected]>
@projectgus @graeme-winter
shared/tinyusb: Only run TinyUSB on the main thread if GIL is disabled.
4a8a6fa 
If GIL is disabled then there's threat of a race condition if some other
code specifically requests USB processing (i.e. to unblock stdio), while
a scheduled TinyUSB callback is already running on another thread.

Relies on the change in the parent commit, where scheduler is restricted
to main thread if GIL is disabled.

Fixes micropython#15390 - "TinyUSB callback can't recurse" exceptions on rp2 when
using _thread module and USB serial I/O.

Adds a unit test for stdin functioning correctly in threads (fails on rp2
port without this fix).

This work was funded through GitHub Sponsors.

Signed-off-by: Angus Gratton <[email protected]>
@dpgeorge @graeme-winter
all: Remove tinytest component.
8688f4b 
With the recent qemu (d9a0fdd and
0426934) and zephyr
(05cad7b) changes to how their tests are
run, two things became unused:

- The tinytest framework, which embedded a set of tests and their expected
  output within firmware, so these tests could be run stand-alone.

- The `--write-exp` and `--list-tests` options to `tests/run-tests.py`,
  which were needed primarily to generated the expected test output for
  tinytest (also the associated `tests/run-tests-exp.py/.sh` scripts are
  now unused).

This commit removes the tinytest component and all its helper code.  This
eliminates a maintenance burden.

Signed-off-by: Damien George <[email protected]>
@dpgeorge @graeme-winter
tests/run-tests.py: Remove --write-exp and --list-tests options.
2cc4131 
Removing the now-unused (see previous commit for details) `--write-exp` and
`--list-tests` options helps to simplify the rather complex logic in
`run-tests.py`.

Signed-off-by: Damien George <[email protected]>
@agatti @graeme-winter
tools/ci.sh: Clean up the Unix port's MIPS target.
e6811d4 
The Unix port's MIPS target CI steps have been updated to be more in
line with the other targets (the MicroPython binary now runs as a
dynamic executable), and the test exceptions for ffi have been lifted.

Signed-off-by: Alessandro Gatti <[email protected]>
@agatti @graeme-winter
tools/ci.sh: Add missing FFI helper for CI RV64 Unix builds.
67c17fa 
The FFI helper definition was accidentally omitted when committing the
necessary shell code for building RV64 Unix builds in the CI
environment.

Signed-off-by: Alessandro Gatti <[email protected]>
@agatti @graeme-winter
tools/ci.sh: Clean up the Unix port's Arm target.
376e050 
The Unix port's Arm target CI steps have been updated to be more in
line with the other targets (the MicroPython binary doesn't need an
environment variable to be set in order to run now).

Signed-off-by: Alessandro Gatti <[email protected]>
@dpgeorge @graeme-winter
tools/mpy_ld.py: Support jumping more than 2k on armv6m architectures.
efc6e5e 
Native .mpy files targetting armv6m (eg RP2040) cannot currently have more
than about 2kiB of native code (between the start of the file and the init
function).

This commit fixes that by using bigger jumps to jump to the init function.

Signed-off-by: Damien George <[email protected]>
@mchlswyr @graeme-winter
unix/modffi: Fix signed integer cast in return_ffi_value.
2f36be7 
Casting an ffi_arg to a signed int may truncate the value. E.g., when the
ffi_arg is 64-bit and the signed int is 32-bit. Also, casting an ffi_arg
to a larger signed type will not sign extend the value. E.g., when the
ffi_arg is 32-bit and the larger signed type is int64_t. If the value is
signed, it should be cast to ffi_sarg, which is the same size as ffi_arg.

Signed-off-by: Michael Sawyer <[email protected]>
@mchlswyr @graeme-winter
tests/ports/unix: Update and extend the modffi integer tests.
d34fe1a 
Added the "long" modffi tests. The tests could not be added to the existing
ffi_types test because two .exp files were required for the 32-bit and
64-bit results. Code common to both the ffi_types and type "long" tests was
factored into ffi_int_base. ffi_types was renamed to ffi_int_types to group
the related tests under the "ffi_int" prefix.

Signed-off-by: Michael Sawyer <[email protected]>
@UnexpectedMaker @graeme-winter
esp32/boards/UM_FEATHERS3NEO: Add FeatherS3 Neo board definition.
b09f80e 
Signed-off-by: Seon Rozenblum <[email protected]>
@UnexpectedMaker @graeme-winter
esp32/boards: Add UM_OMGS3 and UM_RGBTOUCH_MINI board definitions.
0f045de 
This adds two new UM boards: OMGS3 and RGB Touch Mini.  Also fixed the
NanoS3 deploy info.

Signed-off-by: Seon Rozenblum <[email protected]>
@graeme-winter
esp32/boards/OLIMEX_ESP32_EVB: Add Olimex ESP32 EVB board definition.
7b6d3c3 
This is for boards not covered by the Olimex ESP32 PoE implementation.  The
major setting is about the PHY interface configuration.

Tested with esp-idf v5.0.4 and Olimex ESP32 EVB boards.

Signed-off-by: shiggy <[email protected]>
@pi-anl @graeme-winter
py/objringio: Add micropython.RingIO() interface for general use.
d482a41 
This commit adds a new `RingIO` type which exposes the internal ring-buffer
code for general use in Python programs.  It has the stream interface
making it similar to `StringIO` and `BytesIO`, except `RingIO` has a fixed
buffer size and is automatically safe when reads and writes are in
different threads or an IRQ.

This new type is enabled at the "extra features" ROM level.

Signed-off-by: Andrew Leech <[email protected]>
@UnexpectedMaker @graeme-winter
esp32/boards/UM_RGBTOUCH_MINI: Fix compile error with missing modules.
9953155 
Signed-off-by: Seon Rozenblum <[email protected]>
@graeme-winter
STATIC -> static
c2abf2a
@graeme-winter
STATIC -> static
2f82f19
@graeme-winter
Merge remote-tracking branch 'upstream/master' into adafruit-grand-ce…
b495a46 
…ntral
@github-actions GitHub Actions
Copy link

Code size report:

   bare-arm:    +0 +0.000% 
minimal x86:    +0 +0.000% 
   unix x64:    +0 +0.000% standard
      stm32:    +0 +0.000% PYBV10
     mimxrt:    +0 +0.000% TEENSY40
        rp2:    +0 +0.000% RPI_PICO_W
       samd:   +52 +0.019% ADAFRUIT_ITSYBITSY_M4_EXPRESS[incl +4(bss)]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.