As discussed in card10/firmware!357, we should increase the number of
enabled compiler warnings to catch a lot more silent bugs.  Increase the
meson-global warning level to 2, but excluding the `unused-parameter`
and `old-style-declaration` warnings.  Additionally, disable warnings
for libraries as we cannot fix that code anyway.

Signed-off-by: Rahix <rahix@rahix.de>

Signed-off-by: Rahix <rahix@rahix.de>

See merge request card10/firmware!339

As opposed to defining all possible keys at compile time.

In preparation for next commit

Mostly changing things like if { ... return} else { ... } to not have
an extra indentation level of the else part.

The diff is probably very weird even with stripped whitespace.

Added a page to the wiki which describes how to setup NimBLE on the ESP32.

See merge request card10/firmware!358

Fix faulty error handling due to always-false unsigned comparisons.
`-Wtype-limits` revealed three errors in the firmware:

    ../pycardium/modules/sys_leds.c: In function 'mp_leds_get_rocket':
    ../pycardium/modules/sys_leds.c:207:10: warning: comparison is
                always false due to limited range of data type [-Wtype-limits]
      207 |  if (ret == -EINVAL) {
          |          ^~


    ../pycardium/mphalport.c: In function 'mp_hal_set_interrupt_char':
    ../pycardium/mphalport.c:115:8: warning: comparison is always true
                due to limited range of data type [-Wtype-limits]
      115 |  if (c != -1) {
          |        ^~

    ../epicardium/modules/bhi.c: In function 'epic_bhi160_enable_sensor':
    ../epicardium/modules/bhi.c:134:12: warning: comparison is always
                false due to limited range of data type [-Wtype-limits]
      134 |  if (vs_id < 0) {
          |            ^
    ../epicardium/modules/bhi.c: In function 'epic_bhi160_disable_sensor':
    ../epicardium/modules/bhi.c:191:12: warning: comparison is always
                false due to limited range of data type [-Wtype-limits]
      191 |  if (vs_id < 0) {
          |            ^

Note that on the target platform `char` and `enum`s are unsigned.  The
compiler has probably removed all of these `if`s so far because the
conditions were determined to always be false.

See merge request card10/firmware!357

Implement the hal functions necessary for utime.ticks_ms() and
utime.ticks_us().  This enables much more accurate measurements of time
taken in Pycardium (useful for profiling).

Please note that the tick counter will reset whenever a new app is
loaded.

Signed-off-by: Rahix <rahix@rahix.de>

Add a short delay after opening USB-Mode to prevent immediate exit which
happened when pressing the select-button for a very short time.

Also exit the menu-app after exiting USB-Mode to force a reload and thus
immediately display changes in the apps/ folder.

Signed-off-by: Rahix <rahix@rahix.de>

Fix the display backlight staying off while the pmic task prints its
messages (power-off/sleep & battery critial).

Signed-off-by: Rahix <rahix@rahix.de>

fgross authored 5 years ago and rahix committed 5 years ago

The framebuf module provides a framebuffer implementation running purely
in MicroPython which will increase drawing performance by a lot.  A
follow-up change will be necessary to connect this framebuffer
implementation to the display API.

This commit is implements a part of MR !211.

Closes #177

See merge request !324

Rewrite the delay implementation to be more accurate and properly handle
incoming MicroPython interrupts (See issue #177).

For this, the delay is split into two functions:

- systick_delay_precise(): Delay with microsecond accuracy for delays
      less than 1 ms.
- systick_delay_sleep(): Sleep for a potentially very long time, using
      WFI to put the CPU to sleep.  Whenever the CPU wakes up, this
      function will poll the MicroPython scheduler so it can execute
      pending interrupts.

Ontop, this implementation allows for a future, fast, tick-based
monotonic time implementation (e.g. mp_hal_ticks_ms).

Signed-off-by: Rahix <rahix@rahix.de>

Signed-off-by: Rahix <rahix@rahix.de>

Enable the input() builtin which allows reading from the serial console.

Signed-off-by: Rahix <rahix@rahix.de>

Add an option to use the top right and bottom right buttons to move
in the menu.  The option would ideally be added to the card10.cfg.

Related: #178

See merge request card10/firmware!330

See merge request card10/firmware!355

Rewrite image-convert script to more easily change the version splash
screen.

Signed-off-by: Rahix <rahix@rahix.de>

Closes #53 and #188

See merge request card10/firmware!354

See merge request card10/firmware!353

RFC: Proper Mutexes

See merge request card10/firmware!345

Allow calling buttons.read() without an argument.  This is equivalent to
a mask with all bits set (that is, buttons.read(255)).  Additionally,
add a check that the mask is within the limits (< 256).

Signed-off-by: Rahix <rahix@rahix.de>