feat: Add SD multiblock write support

Add support for CMD25 to write multiple sequential blocks in one go, in an attempt to speed up uploads. It still needs to be benchmarked if this actually results in a meaningful speedup. Also fix logging from SD driver to be visible even when running under micropython. Signed-off-by: Matthias Blankertz <matthias@blankertz.org>
fix: Reduce file upload chunk size to 4k
2026-01-13 22:41:34 +01:00 · 2026-01-13 22:38:37 +01:00 · 2026-01-13 21:31:18 +00:00 · 2026-01-06 14:59:12 +01:00 · 2026-01-06 14:59:12 +01:00 · 2026-01-06 14:59:12 +01:00
20 changed files with 520 additions and 148 deletions
--- a/software/boards/RPI_PICO_W/board.c
+++ b/software/boards/RPI_PICO_W/board.c
@@ -0,0 +1,29 @@
 // SPDX-License-Identifier: MIT
 // Copyright (c) 2025 Matthias Blankertz <matthias@blankertz.org>
 #include "py/obj.h"
 #include "py/runtime.h"
 #include "py/objstr.h"
 #ifndef TONBERRY_GIT_REVISION
 #define TONBERRY_GIT_REVISION "unknown"
 #endif
 #ifndef TONBERRY_VERSION
 #define TONBERRY_VERSION "unknown"
 #endif
 static const MP_DEFINE_STR_OBJ(tonberry_git_revision_obj, TONBERRY_GIT_REVISION);
 static const MP_DEFINE_STR_OBJ(tonberry_version_obj, TONBERRY_VERSION);
 static const mp_rom_map_elem_t board_module_globals_table[] = {
    {MP_ROM_QSTR(MP_QSTR___name__), MP_ROM_QSTR(MP_QSTR_board)},
    {MP_ROM_QSTR(MP_QSTR_revision), MP_ROM_PTR(&tonberry_git_revision_obj)},
    {MP_ROM_QSTR(MP_QSTR_version), MP_ROM_PTR(&tonberry_version_obj)},
 };
 static MP_DEFINE_CONST_DICT(board_module_globals, board_module_globals_table);
 const mp_obj_module_t board_cmodule = {
    .base = {&mp_type_module},
    .globals = (mp_obj_dict_t *)&board_module_globals,
 };
 MP_REGISTER_MODULE(MP_QSTR_board, board_cmodule);
--- a/software/boards/RPI_PICO_W/mpconfigboard.cmake
+++ b/software/boards/RPI_PICO_W/mpconfigboard.cmake
@@ -20,3 +20,22 @@ set(GEN_PINS_CSV_ARG --board-csv "${GEN_PINS_BOARD_CSV}")
 add_link_options("-Wl,--print-memory-usage")
 set(PICO_USE_FASTEST_SUPPORTED_CLOCK 1)
 find_program(GIT git)
 execute_process(COMMAND ${GIT} rev-parse HEAD
    WORKING_DIRECTORY "${CMAKE_CURRENT_LIST_DIR}"
    OUTPUT_VARIABLE TONBERRY_GIT_REVISION
    OUTPUT_STRIP_TRAILING_WHITESPACE
 )
 execute_process(COMMAND ${GIT} describe --match 'v*' --always --dirty
    WORKING_DIRECTORY "${CMAKE_CURRENT_LIST_DIR}"
    OUTPUT_VARIABLE TONBERRY_VERSION
    OUTPUT_STRIP_TRAILING_WHITESPACE
 )
 set(MICROPY_SOURCE_BOARD "${CMAKE_CURRENT_LIST_DIR}/board.c")
 set(MICROPY_DEF_BOARD
    TONBERRY_GIT_REVISION="${TONBERRY_GIT_REVISION}"
    TONBERRY_VERSION="${TONBERRY_VERSION}")
--- a/software/frontend/index.html
+++ b/software/frontend/index.html
@@ -2,7 +2,7 @@
 <html lang="en">
 <head>
 <meta charset="UTF-8">
-<title>Device Admin</title>
+<title>TonBERRY pico</title>
 <style>
    body {
        font-family: Arial, sans-serif;
@@ -131,11 +131,17 @@
        list-style: none;
    }
    .footer {
        font-size: x-small;
        color: gray;
        margin-top: 20px;
    }
 </style>
 </head>
 <body>
-<h1>Device Admin</h1>
+<h1>TonBERRY pico</h1>
 <nav>
    <button onclick="showScreen('menu')">🏠 Main Menu</button>
@@ -154,14 +160,14 @@
        <!-- More screens can be added later -->
    </ul>
    <hr>
-    <button onclick="fetch('/api/v1/reboot/bootloader', {'method': 'POST'});">Reboot to bootloader</button>
+    <button onclick="requestReboot()">Reboot to bootloader</button>
 </div>
 <!-- CONFIG EDITOR SCREEN -->
 <div id="screen-config" class="screen">
    <h2>Configuration Editor</h2>
    <div id="config-container">Loading…</div>
-    <button id="config-save-btn" disabled>Save</button>
+    <button id="config-save-btn" disabled>Save &amp; Reboot</button>
 </div>
 <!-- PLAYLIST EDITOR SCREEN 1: list of playlists -->
@@ -207,6 +213,10 @@
                    <option value="audioplay">Audioplay (no shuffle, start at previous track)</option>
                </select>
            </div>
            <div>
                <label>Playlist name</label>
                <input type="text" placeholder="Playlist name" id="playlist-edit-name" />
            </div>
            <div class="flex-horizontal">
                <div style="flex-grow: 1">
                    <label>Tracks</label>
@@ -257,7 +267,17 @@
        </div>
    </div>
 </div>
-    
+<div class="footer">
    <hr>
    <div class="flex-horizontal">
        <div>
            <a href="https://git.ka.blankertz.org/TonBERRY/tonberry-pico">TonBERRY pico</a>
        </div>
        <div>
            Version: <span id="footer-version">unknown</span>
        </div>
    </div>
 </div>
 <script>
    const Screens = {};
    let activeScreen = null;
@@ -296,7 +316,9 @@
                        return;
                    }
-                    alert("Configuration saved successfully!");
+                    alert("Configuration saved successfully, device will now reboot/shutdown! " +
                          "On battery, press Power button after shutdown to restart.");
                    await fetch('/api/v1/reboot/application', {'method': 'POST'});
                } catch (err) {
                    alert("Error saving configuration: " + err);
                }
@@ -334,7 +356,13 @@
            'root.BUTTON_MAP.PLAY_PAUSE': 'Play/Pause',
            'root.TAG_TIMEOUT_SECS': 'Tag removal timeout (seconds)',
            'root.TAGMODE': 'Tag mode',
-            'root.LED_COUNT': 'Length of WS2182 (Neopixel) LED chain'
+            'root.LED_COUNT': 'Length of WS2182 (Neopixel) LED chain',
            'root.VOLUME_MAX': 'Maximum volume (0-255)',
            'root.VOLUME_BOOT': 'Volume at startup (0-255)',
            'root.LED_MAX': 'Maximum LED brightness (0-255)',
            'root.WIFI.SSID': 'Network name (SSID) (leave empty for AP mode)',
            'root.WIFI.PASSPHRASE': 'Password',
            'root.WIFI.SECURITY': 'Security mode'
        };
        const config_input_override = {
            'root.TAGMODE': {
@@ -344,6 +372,21 @@
                    'tagstartstop': 'Present tag once to start, present again to stop playback'
                }
            },
            'root.BUTTON_MAP.NEXT':  {
                'input-type': 'number'
            },
            'root.BUTTON_MAP.PREV':  {
                'input-type': 'number'
            },
            'root.BUTTON_MAP.VOLUP':  {
                'input-type': 'number'
            },
            'root.BUTTON_MAP.VOLDOWN':  {
                'input-type': 'number'
            },
            'root.BUTTON_MAP.PLAY_PAUSE':  {
                'input-type': 'number'
            },
            'root.IDLE_TIMEOUT_SECS': {
                'input-type': 'number'
            },
@@ -353,6 +396,30 @@
            'root.LED_COUNT': {
                'input-type': 'number'
            },
            'root.WIFI.SSID': {
                'input-type': 'text'
            },
            'root.WIFI.PASSPHRASE': {
                'input-type': 'text'
            },
            'root.WIFI.SECURITY': {
                'element': 'select',
                'values': {
                    'open': 'Open',
                    'wpa_wpa2': 'WPA/WPA2 (PSK Mixed)',
                    'wpa3': 'WPA3 (SAE AES)',
                    'wpa2_wpa3': 'WPA2/WPA3 (PSK AES)'
                }
            },
            'root.VOLUME_MAX': {
                'input-type': 'number'
            },
            'root.VOLUME_BOOT': {
                'input-type': 'number'
            },
            'root.LED_MAX': {
                'input-type': 'number'
            }
        };
        function renderObject(obj, path) {
@@ -423,7 +490,7 @@
                let val = input.value.trim();
                if (val === "") val = null;
-                else if (!isNaN(val)) val = Number(val);
+                else if (!isNaN(val) && input.type != 'text') val = Number(val);
                current[path[path.length - 1]] = val;
            });
@@ -456,7 +523,7 @@
            document.getElementById('playlist-exist-button-delete')
                .addEventListener('click', (e) => {
                    if (lastSelected === null) return;
-                    const tagid = lastSelected.innerText;
+                    const tagid = lastSelected.getAttribute('data-tag');
                    if(confirm(`Really delete playlist ${tagid}?`)) {
                        fetch(`/api/v1/playlist/${tagid}`, {
                            method: 'DELETE'})
@@ -467,7 +534,7 @@
                .addEventListener('click', selectLastTag);
            document.getElementById('playlist-exist-button-edit').addEventListener("click", (e) => {
                if (lastSelected !== null)
-                    showScreen("playlist_edit", {load: lastSelected.innerText});
+                    showScreen("playlist_edit", {load: lastSelected.getAttribute('data-tag')});
            });
            document.getElementById('playlist-exist-list').addEventListener("click", (e) => {
                const node = e.target.closest("li");
@@ -522,8 +589,9 @@
            container.innerHTML = "";
            for (const playlist of playlists) {
                const li = document.createElement("li");
-                li.innerHTML = playlist;
+                li.innerHTML = `${playlist.name} (${playlist.tag})`;
                li.className = "node"
                li.setAttribute('data-tag', playlist.tag);
                container.appendChild(li)
            }
        }
@@ -548,7 +616,7 @@
            document.getElementById('playlist-exist-list')
                    .querySelectorAll("li")
                    .forEach(n => {
-                        if (n.innerText == tagtext) {
+                        if (n.getAttribute('data-tag') == tagtext) {
                            n.classList.add("selected");
                            lastSelected = n;
                        } else {
@@ -650,6 +718,8 @@
            } else if (playlist.persist === "track" && playlist.shuffle === "no") {
                playlisttype.value = "audioplay";
            }
            const playlistname = document.getElementById('playlist-edit-name');
            playlistname.value = playlist.name;
        }
        async function save() {
@@ -672,6 +742,8 @@
                    playlistData.shuffle = "no";
                    break;
            }
            const playlistname = document.getElementById('playlist-edit-name');
            playlistData.name = playlistname.value;
            const container = document.getElementById('playlist-edit-list');
            playlistData.paths = [...container.querySelectorAll("li")].map((node) => node.innerText);
            const saveRes = await fetch(`/api/v1/playlist/${playlistId}`,
@@ -736,6 +808,10 @@
        async function onShow(intent) {
            document.getElementById('playlist-filebrowser-addtrack').disabled = true;
            if (intent !== 'refresh') {
                document.getElementById('playlist-filebrowser-upload-progress').value = 0;
                document.getElementById("playlist-filebrowser-upload-files").value = "";
            }
            tree = document.getElementById("playlist-filebrowser-tree");
            tree.innerHTML = "Loading...";
            fetch('/api/v1/audiofiles')
@@ -770,7 +846,6 @@
            if (type === 'directory') {
                const nested = document.createElement('ul');
                node.appendChild(nested);
                node.classList.add('expanded');
                parent.appendChild(node);
                return nested;
            }
@@ -857,7 +932,7 @@
                    }
                    if (donecount + 1 === totalcount) {
                        // Reload file list from device
-                        onShow();
+                        onShow('refresh');
                    } else {
                        uploadFileHelper(totalcount, donecount + 1, destdir, files.slice(1));
                    }
@@ -884,7 +959,7 @@
            const saveRes = await fetch(`/api/v1/audiofiles?type=directory&location=${location}`,
                  {method: 'POST'});
            // Reload file list from device
-            onShow();
+            onShow('refresh');
        }
        async function deleteItems() {
@@ -905,7 +980,7 @@
                }
            }
            // Reload file list from device
-            onShow();
+            onShow('refresh');
        }
        let tree = (() => {
@@ -966,6 +1041,14 @@
        return { init, onShow };
    })();
 // Misc
    async function requestReboot() {
        const resp = await fetch('/api/v1/reboot/bootloader', {'method': 'POST'});
        if (!resp.ok) {
            alert('Reboot to bootloader failed: ' + await resp.text());
        }
    }
 // Initialization
    Object.values(Screens).forEach(screen => {
@@ -973,6 +1056,13 @@
    });
    showScreen("menu");
    fetch('/api/v1/info')
        .then((resp) => resp.json())
        .then((info) => {
            const version = document.getElementById('footer-version');
            version.innerText = info.version;
        });
 </script>
 </body>
--- a/software/lib/microdot
+++ b/software/lib/microdot
--- a/software/modules/rp2_sd/module.c
+++ b/software/modules/rp2_sd/module.c
@@ -1,11 +1,14 @@
 // SPDX-License-Identifier: MIT
 // Copyright (c) 2025 Matthias Blankertz <matthias@blankertz.org>
 #include <stdarg.h>
 #include "py/obj.h"
 #include "sd.h"
 // Include MicroPython API.
 #include "py/mperrno.h"
 #include "py/mpprint.h"
 #include "py/runtime.h"
 // This module is RP2 specific
@@ -13,6 +16,14 @@
 #include <string.h>
 void sd_printf(const char *fmt, ...)
 {
    va_list ap;
    va_start(ap, fmt);
    mp_vprintf(&mp_sys_stdout_print, fmt, ap);
    va_end(ap);
 }
 const mp_obj_type_t sdcard_type;
 struct sdcard_obj {
    mp_obj_base_t base;
@@ -89,11 +100,14 @@ static mp_obj_t sdcard_writeblocks(mp_obj_t self_obj, mp_obj_t block_obj, mp_obj
    if (bufinfo.len % SD_SECTOR_SIZE != 0)
        mp_raise_ValueError(MP_ERROR_TEXT("Buffer length is invalid"));
    const int nblocks = bufinfo.len / SD_SECTOR_SIZE;
-    for (int block = 0; block < nblocks; block++) {
+    bool ret;
-        // TODO: Implement CMD25 write multiple blocks
+    if (nblocks > 1) {
-        if (!sd_writeblock(&self->sd_context, start_block + block, bufinfo.buf + block * SD_SECTOR_SIZE))
+        ret = sd_writeblocks(&self->sd_context, start_block, nblocks, bufinfo.buf);
-            mp_raise_OSError(MP_EIO);
+    } else {
        ret = sd_writeblock(&self->sd_context, start_block, bufinfo.buf);
    }
    if (!ret)
        mp_raise_OSError(MP_EIO);
    return mp_const_none;
 }
 static MP_DEFINE_CONST_FUN_OBJ_3(sdcard_writeblocks_obj, sdcard_writeblocks);
--- a/software/modules/rp2_sd/sd.c
+++ b/software/modules/rp2_sd/sd.c
@@ -6,7 +6,7 @@
 #include <stdio.h>
 #include <string.h>
-// #define SD_DEBUG
+extern void sd_printf(const char *fmt, ...);
 #define SD_R1_ILLEGAL_COMMAND (1 << 2)
@@ -26,14 +26,14 @@ static bool sd_early_init(void)
    for (int i = 0; i < 500; ++i) {
        if (sd_cmd(0, 0, 1, &buf)) {
 #ifdef SD_DEBUG
-            printf("CMD0 resp %02hhx\n", buf);
+            sd_printf("CMD0 resp %02x\n", buf);
 #endif
            if (buf == 0x01) {
                return true;
            }
        }
 #ifdef SD_DEBUG
-        printf("CMD0 timeout, try again...\n");
+        sd_printf("CMD0 timeout, try again...\n");
 #endif
    }
    return false;
@@ -44,14 +44,14 @@ static bool sd_check_interface_condition(void)
    uint8_t buf[5];
    if (sd_cmd(8, 0x000001AA, 5, buf)) {
        if ((buf[3] & 0xf) != 0x1 || buf[4] != 0xAA) {
-            printf("sd_init: check interface condition failed\n");
+            sd_printf("sd_init: check interface condition failed\n");
            return false;
        }
    } else {
        if (buf[0] & SD_R1_ILLEGAL_COMMAND) {
-            printf("sd_init: check interface condition returned illegal command - old card?\n");
+            sd_printf("sd_init: check interface condition returned illegal command - old card?\n");
        } else {
-            printf("sd_init: check interface condition failed\n");
+            sd_printf("sd_init: check interface condition failed\n");
            return false;
        }
    }
@@ -72,12 +72,12 @@ static bool sd_send_op_cond(void)
        if (!result) {
            if (use_acmd && buf & SD_R1_ILLEGAL_COMMAND) {
 #ifdef SD_DEBUG
-                printf("sd_init: card does not understand ACMD41, try CMD1...\n");
+                sd_printf("sd_init: card does not understand ACMD41, try CMD1...\n");
 #endif
                use_acmd = false;
                continue;
            } else if (buf != 0x01) {
-                printf("sd_init: send_op_cond failed\n");
+                sd_printf("sd_init: send_op_cond failed\n");
                return false;
            } else {
                continue;
@@ -87,7 +87,7 @@ static bool sd_send_op_cond(void)
            return true;
        }
    }
-    printf("sd_init: send_op_cond: timeout waiting for !idle\n");
+    sd_printf("sd_init: send_op_cond: timeout waiting for !idle\n");
    return false;
 }
@@ -107,7 +107,7 @@ static void sd_dump_cid [[maybe_unused]] (void)
        const uint8_t crc = sd_crc7(15, buf);
        const uint8_t card_crc = buf[15] >> 1;
        if (card_crc != crc) {
-            printf("CRC mismatch: Got %02hhx, expected %02hhx\n", card_crc, crc);
+            sd_printf("CRC mismatch: Got %02x, expected %02x\n", card_crc, crc);
            // Some cheap SD cards always report CRC=0, don't fail in that case
            if (card_crc != 0) {
                return;
@@ -122,8 +122,8 @@ static void sd_dump_cid [[maybe_unused]] (void)
        uint32_t psn = buf[9] << 24 | buf[10] << 16 | buf[11] << 8 | buf[12];
        int mdt_year = 2000 + ((buf[13] & 0xf) << 4 | (buf[14] & 0xf0) >> 4);
        int mdt_month = buf[14] & 0x0f;
-        printf("CID: mid: %02hhx, oid: %.2s, pnm: %.5s, prv: %02hhx, psn: %08" PRIx32 ", mdt_year: %d, mdt_month: %d\n",
+        sd_printf("CID: mid: %02x, oid: %.2s, pnm: %.5s, prv: %02x, psn: %08" PRIx32 ", mdt_year: %d, mdt_month: %d\n",
-               mid, oid, pnm, prv, psn, mdt_year, mdt_month);
+                  mid, oid, pnm, prv, psn, mdt_year, mdt_month);
    }
 }
@@ -131,13 +131,13 @@ static bool sd_read_csd(struct sd_context *sd_context)
 {
    uint8_t buf[16];
    if (!sd_cmd_read(9, 0, 16, buf)) {
-        printf("Failed to read CSD\n");
+        sd_printf("Failed to read CSD\n");
        return false;
    }
    const uint8_t crc = sd_crc7(15, buf);
    const uint8_t card_crc = buf[15] >> 1;
    if (card_crc != crc) {
-        printf("CRC mismatch: Got %02hhx, expected %02hhx\n", card_crc, crc);
+        sd_printf("CRC mismatch: Got %02x, expected %02x\n", card_crc, crc);
        // Some cheap SD cards always report CRC=0, don't fail in that case
        if (card_crc != 0) {
            return false;
@@ -151,12 +151,12 @@ static bool sd_read_csd(struct sd_context *sd_context)
    switch (csd_ver) {
    case 0: {
        if (sd_context->sdhc_sdxc) {
-            printf("sd_init: Got CSD v1.0 but card is SDHC/SDXC?\n");
+            sd_printf("sd_init: Got CSD v1.0 but card is SDHC/SDXC?\n");
            return false;
        }
        const unsigned read_bl_len = buf[5] & 0xf;
        if (read_bl_len < 9 || read_bl_len > 11) {
-            printf("Invalid read_bl_len in CSD 1.0\n");
+            sd_printf("Invalid read_bl_len in CSD 1.0\n");
            return false;
        }
        blocksize = 1 << (buf[5] & 0xf);
@@ -176,15 +176,15 @@ static bool sd_read_csd(struct sd_context *sd_context)
        break;
    }
    case 2: {
-        printf("sd_init: Got CSD v3.0, but SDUC does not support SPI.\n");
+        sd_printf("sd_init: Got CSD v3.0, but SDUC does not support SPI.\n");
        return false;
    }
    }
    sd_context->blocks = blocks;
    sd_context->blocksize = blocksize;
 #ifdef SD_DEBUG
-    printf("CSD version %u.0, blocksize %u, blocks %u, capacity %llu MiB, max speed %u\n", version, blocksize, blocks,
+    sd_printf("CSD version %u.0, blocksize %u, blocks %u, capacity %u MiB, max speed %u\n", version, blocksize, blocks,
-           ((uint64_t)blocksize * blocks) / (1024 * 1024), max_speed);
+              (uint32_t)(((uint64_t)blocksize * blocks) / (1024 * 1024)), max_speed);
 #endif
    return true;
 }
@@ -205,11 +205,11 @@ bool sd_init(struct sd_context *sd_context, int mosi, int miso, int sck, int ss,
    uint32_t ocr;
    if (!sd_read_ocr(&ocr)) {
-        printf("sd_init: read OCR failed\n");
+        sd_printf("sd_init: read OCR failed\n");
        goto out_spi;
    }
    if ((ocr & 0x00380000) != 0x00380000) {
-        printf("sd_init: unsupported card voltage range\n");
+        sd_printf("sd_init: unsupported card voltage range\n");
        goto out_spi;
    }
@@ -219,11 +219,11 @@ bool sd_init(struct sd_context *sd_context, int mosi, int miso, int sck, int ss,
    sd_spi_set_bitrate(rate);
    if (!sd_read_ocr(&ocr)) {
-        printf("sd_init: read OCR failed\n");
+        sd_printf("sd_init: read OCR failed\n");
        goto out_spi;
    }
    if (!(ocr & (1 << 31))) {
-        printf("sd_init: card not powered up but !idle?\n");
+        sd_printf("sd_init: card not powered up but !idle?\n");
        goto out_spi;
    }
    sd_context->sdhc_sdxc = (ocr & (1 << 30));
@@ -234,20 +234,20 @@ bool sd_init(struct sd_context *sd_context, int mosi, int miso, int sck, int ss,
    if (sd_context->blocksize != SD_SECTOR_SIZE) {
        if (sd_context->blocksize != 1024 && sd_context->blocksize != 2048) {
-            printf("sd_init: Unsupported block size %u\n", sd_context->blocksize);
+            sd_printf("sd_init: Unsupported block size %u\n", sd_context->blocksize);
            goto out_spi;
        }
        // Attempt SET_BLOCKLEN command
        uint8_t resp[1];
        if (!sd_cmd(16, SD_SECTOR_SIZE, 1, resp)) {
-            printf("sd_init: SET_BLOCKLEN failed\n");
+            sd_printf("sd_init: SET_BLOCKLEN failed\n");
            goto out_spi;
        }
        // Successfully set blocksize to SD_SECTOR_SIZE, adjust context
        sd_context->blocks *= sd_context->blocksize / SD_SECTOR_SIZE;
 #ifdef SD_DEBUG
-        printf("Adjusted blocksize from %u to 512, card now has %u blocks\n", sd_context->blocksize,
+        sd_printf("Adjusted blocksize from %u to 512, card now has %u blocks\n", sd_context->blocksize,
-               sd_context->blocks);
+                  sd_context->blocks);
 #endif
        sd_context->blocksize = SD_SECTOR_SIZE;
    }
@@ -307,7 +307,7 @@ bool sd_readblock_complete(struct sd_context *sd_context)
 bool sd_readblock_is_complete(struct sd_context *sd_context) { return sd_cmd_read_is_complete(); }
-bool sd_writeblock(struct sd_context *sd_context, size_t sector_num, uint8_t buffer[const static SD_SECTOR_SIZE])
+bool sd_writeblock(struct sd_context *sd_context, const size_t sector_num, uint8_t buffer[const static SD_SECTOR_SIZE])
 {
    if (!sd_context->initialized || sector_num >= sd_context->blocks)
        return false;
@@ -319,3 +319,20 @@ bool sd_writeblock(struct sd_context *sd_context, size_t sector_num, uint8_t buf
    }
    return sd_cmd_write(24, addr, SD_SECTOR_SIZE, buffer);
 }
 bool sd_writeblocks(struct sd_context *sd_context, const size_t sector_num, const size_t sectors, uint8_t *const buffer)
 {
    if (!sd_context->initialized || sector_num + sectors >= sd_context->blocks)
        return false;
    if (!sd_context->sdhc_sdxc) {
        // Don't use multi-block writes for SDSC for now
        // Need to configure WRITE_BL_LEN correctly
        for (size_t sector = 0; sector < sectors; ++sector) {
            if (!sd_writeblock(sd_context, sector_num + sector, buffer + sector * SD_SECTOR_SIZE))
                return false;
        }
        return true;
    }
    return sd_cmd_write_multiple(25, sector_num, sectors, SD_SECTOR_SIZE, buffer);
 }
--- a/software/modules/rp2_sd/sd.h
+++ b/software/modules/rp2_sd/sd.h
@@ -24,3 +24,4 @@ bool sd_readblock_complete(struct sd_context *context);
 bool sd_readblock_is_complete(struct sd_context *context);
 bool sd_writeblock(struct sd_context *context, size_t sector_num, uint8_t buffer[const static SD_SECTOR_SIZE]);
 bool sd_writeblocks(struct sd_context *sd_context, const size_t sector_num, const size_t sectors, uint8_t *buffer);
--- a/software/modules/rp2_sd/sd_spi.c
+++ b/software/modules/rp2_sd/sd_spi.c
@@ -12,6 +12,8 @@
 #include <pico/time.h>
 #include <string.h>
 extern void sd_printf(const char *fmt, ...);
 typedef enum { DMA_READ_TOKEN, DMA_READ, DMA_IDLE } sd_dma_state;
 struct sd_dma_context {
@@ -222,7 +224,7 @@ bool sd_cmd_read_complete(void)
    sd_spi_read_blocking(0xff, &buf, 1);
    if (sd_spi_context.sd_dma_context.read_token_buf != 0xfe) {
 #ifdef SD_DEBUG
-        printf("read failed: invalid read token %02hhx\n", sd_spi_context.sd_dma_context.read_token_buf);
+        sd_printf("read failed: invalid read token %02x\n", sd_spi_context.sd_dma_context.read_token_buf);
 #endif
        return false;
    }
@@ -231,7 +233,7 @@ bool sd_cmd_read_complete(void)
    const uint16_t act_crc = sd_spi_context.sd_dma_context.crc_buf[0] << 8 | sd_spi_context.sd_dma_context.crc_buf[1];
    if (act_crc != expect_crc) {
 #ifdef SD_DEBUG
-        printf("read CRC fail: got %04hx, expected %04hx\n", act_crc, expect_crc);
+        sd_printf("read CRC fail: got %04x, expected %04x\n", act_crc, expect_crc);
 #endif
        return false;
    }
@@ -239,10 +241,9 @@ bool sd_cmd_read_complete(void)
    return true;
 }
-bool sd_cmd_write(uint8_t cmd, uint32_t arg, unsigned datalen, uint8_t data[const static datalen])
+static bool sd_cmd_write_begin(uint8_t cmd, uint32_t arg)
 {
-    uint8_t buf[2];
+    uint8_t buf[1];
    const uint16_t crc = sd_crc16(datalen, data);
    sd_spi_cmd_send(cmd, arg);
    // Read up to 8 garbage bytes (0xff), followed by R1 (MSB is zero)
    bool got_r1 = false;
@@ -253,9 +254,20 @@ bool sd_cmd_write(uint8_t cmd, uint32_t arg, unsigned datalen, uint8_t data[cons
            break;
        }
    }
-    if (!got_r1 || buf[0] != 0x00)
+    if (!got_r1 || buf[0] != 0x00) {
-        goto abort;
+#ifdef SD_DEBUG
-    buf[0] = 0xfe;
+        sd_printf("write cmd fail: %02x\n", buf[0]);
 #endif
        return false;
    }
    return true;
 }
 static bool sd_cmd_write_block(uint8_t token, unsigned datalen, uint8_t data[const static datalen])
 {
    uint8_t buf[2];
    const uint16_t crc = sd_crc16(datalen, data);
    buf[0] = token;
    sd_spi_write_blocking(buf, 1);
    sd_spi_write_blocking(data, datalen);
    buf[0] = crc >> 8;
@@ -264,11 +276,16 @@ bool sd_cmd_write(uint8_t cmd, uint32_t arg, unsigned datalen, uint8_t data[cons
    sd_spi_read_blocking(0xff, buf, 1);
    if ((buf[0] & 0x1f) != 0x5) {
 #ifdef SD_DEBUG
-        printf("Write fail: %2hhx\n", buf[0]);
+        sd_printf("Write fail: %2x\n", buf[0]);
 #endif
-        goto abort;
+        return false;
    }
    return true;
 }
 static bool sd_cmd_write_wait_nbusy(void)
 {
    uint8_t buf[1];
    int timeout = 0;
    bool got_done = false;
    for (timeout = 0; timeout < 131072; ++timeout) {
@@ -279,18 +296,73 @@ bool sd_cmd_write(uint8_t cmd, uint32_t arg, unsigned datalen, uint8_t data[cons
        }
    }
 #ifdef SD_DEBUG
-    printf("dbg write end: %d, %2hhx\n", timeout, buf[0]);
+    sd_printf("dbg write end: %d, %2x\n", timeout, buf[0]);
 #endif
-    if (!got_done)
+    return got_done;
 }
 bool sd_cmd_write(uint8_t cmd, uint32_t arg, unsigned datalen, uint8_t data[const static datalen])
 {
 #ifdef SD_DEBUG
    sd_printf("write 1 block at %u\n", arg);
 #endif
    uint8_t buf[2];
    if (!sd_cmd_write_begin(cmd, arg))
        goto abort;
    if (!sd_cmd_write_block(0xfe, datalen, data))
        goto abort;
    if (!sd_cmd_write_wait_nbusy())
        goto abort;
    gpio_put(sd_spi_context.ss, true);
    sd_spi_read_blocking(0xff, buf, 1);
 #ifdef SD_DEBUG
    sd_printf("write ok\n");
 #endif
    return true;
 abort:
    gpio_put(sd_spi_context.ss, true);
    sd_spi_read_blocking(0xff, buf, 1);
 #ifdef SD_DEBUG
    sd_printf("write fail\n");
 #endif
    return false;
 }
 bool sd_cmd_write_multiple(uint8_t cmd, uint32_t arg, unsigned blocks, unsigned datalen, uint8_t *const data)
 {
 #ifdef SD_DEBUG
    sd_printf("write %u blocks at %u\n", blocks, arg);
 #endif
    uint8_t buf[2];
    if (!sd_cmd_write_begin(cmd, arg))
        goto abort;
    for (unsigned i = 0; i < blocks; ++i) {
        if (!sd_cmd_write_block(0b11111100, datalen, data + datalen * i))
            goto abort;
        if (!sd_cmd_write_wait_nbusy())
            goto abort;
    }
    buf[0] = 0b11111101;
    buf[1] = 0xff;
    sd_spi_write_blocking(buf, 2);
    if (!sd_cmd_write_wait_nbusy())
        goto abort;
    gpio_put(sd_spi_context.ss, true);
    sd_spi_read_blocking(0xff, buf, 1);
 #ifdef SD_DEBUG
    sd_printf("write ok\n");
 #endif
    return true;
 abort:
    gpio_put(sd_spi_context.ss, true);
    sd_spi_read_blocking(0xff, buf, 1);
 #ifdef SD_DEBUG
    sd_printf("write fail\n");
 #endif
    return false;
 }
--- a/software/modules/rp2_sd/sd_spi.h
+++ b/software/modules/rp2_sd/sd_spi.h
@@ -27,3 +27,4 @@ bool sd_cmd_read_complete(void);
 bool sd_cmd_read_is_complete(void);
 bool sd_cmd_write(uint8_t cmd, uint32_t arg, unsigned datalen, uint8_t data[const static datalen]);
 bool sd_cmd_write_multiple(uint8_t cmd, uint32_t arg, unsigned blocks, unsigned datalen, uint8_t *const data);
--- a/software/src/app.py
+++ b/software/src/app.py
@@ -9,8 +9,8 @@ from utils import TimerManager
 Dependencies = namedtuple('Dependencies', ('mp3player', 'nfcreader', 'buttons', 'playlistdb', 'hwconfig', 'leds',
                                           'config'))
-# Should be ~ 6dB steps
+# Should be ~ 3dB steps
-VOLUME_CURVE = [1, 2, 4, 8, 16, 32, 63, 126, 251]
+VOLUME_CURVE = [1, 2, 3, 4, 6, 8, 11, 16, 23, 32, 45, 64, 91, 128, 181, 255]
 class PlayerApp:
@@ -52,11 +52,19 @@ class PlayerApp:
        self.hwconfig = deps.hwconfig(self)
        self.leds = deps.leds(self)
        self.tag_mode = self.config.get_tagmode()
        self.volume_max = self.config.get_volume_max()
        self.volume_pos = 3  # fallback if config.get_volume_boot is nonsense
        try:
            for idx, val in enumerate(VOLUME_CURVE):
                if val >= self.config.get_volume_boot():
                    self.volume_pos = idx
                    break
        except (TypeError, ValueError):
            pass
        self.playing_tag = None
        self.playlist = None
        self.buttons = deps.buttons(self) if deps.buttons is not None else None
        self.mp3file = None
        self.volume_pos = 3
        self.paused = False
        self.playing = False
        self.player.set_volume(VOLUME_CURVE[self.volume_pos])
@@ -74,7 +82,7 @@ class PlayerApp:
        uid_str = b''.join('{:02x}'.format(x).encode() for x in new_tag)
        if self.tag_mode == 'tagremains' or (self.tag_mode == 'tagstartstop' and new_tag != self.playing_tag):
            self._set_playlist(uid_str)
-            self.playing_tag = new_tag
+            self.playing_tag = new_tag if self.playlist is not None else None
        elif self.tag_mode == 'tagstartstop':
            print('Tag presented again, stopping playback')
            self._unset_playlist()
@@ -91,8 +99,10 @@ class PlayerApp:
    def onButtonPressed(self, what):
        assert self.buttons is not None
        if what == self.buttons.VOLUP:
-            self.volume_pos = min(self.volume_pos + 1, len(VOLUME_CURVE) - 1)
+            new_volume = min(self.volume_pos + 1, len(VOLUME_CURVE) - 1)
-            self.player.set_volume(VOLUME_CURVE[self.volume_pos])
+            if VOLUME_CURVE[new_volume] <= self.volume_max:
                self.volume_pos = new_volume
                self.player.set_volume(VOLUME_CURVE[self.volume_pos])
        elif what == self.buttons.VOLDOWN:
            self.volume_pos = max(self.volume_pos - 1, 0)
            self.player.set_volume(VOLUME_CURVE[self.volume_pos])
@@ -196,8 +206,5 @@ class PlayerApp:
    def get_playlist_db(self):
        return self.playlist_db
    def get_timer_manager(self):
        return self.timer_manager
    def get_leds(self):
        return self.leds
--- a/software/src/main.py
+++ b/software/src/main.py
@@ -3,13 +3,12 @@
 import aiorepl  # type: ignore
 import asyncio
 from errno import ENOENT
 import machine
 import micropython
 import network
 import os
 import time
 import ubinascii
 import sys
 # Own modules
 import app
@@ -37,14 +36,22 @@ hwconfig.board_init()
 machine.mem32[0x40030000 + 0x00] = 0x10
-def setup_wifi():
+def setup_wifi(ssid='', passphrase='', security=network.WLAN.SEC_WPA_WPA2):
    network.hostname("TonberryPico")
-    wlan = network.WLAN(network.WLAN.IF_AP)
+    if ssid is None or ssid == '':
-    wlan.config(ssid=f"TonberryPicoAP_{machine.unique_id().hex()}", security=wlan.SEC_OPEN)
+        apname = f"TonberryPicoAP_{machine.unique_id().hex()}"
-    wlan.active(True)
+        print(f"Create AP {apname}")
        wlan = network.WLAN(network.WLAN.IF_AP)
        wlan.config(ssid=apname, password=passphrase if passphrase is not None else '', security=security)
        wlan.active(True)
    else:
        print(f"Connect to SSID {ssid} with passphrase {passphrase}...")
        wlan = network.WLAN()
        wlan.active(True)
        wlan.connect(ssid, passphrase if passphrase is not None else '', security=security)
-    # disable power management
+    # configure power management
-    wlan.config(pm=network.WLAN.PM_NONE)
+    wlan.config(pm=network.WLAN.PM_PERFORMANCE)
    mac = ubinascii.hexlify(network.WLAN().config('mac'), ':').decode()
    print(f"     mac: {mac}")
@@ -65,27 +72,37 @@ DB_PATH = '/sd/tonberry.db'
 config = Configuration()
 # Setup LEDs
 np = NeoPixel(hwconfig.LED_DIN, config.get_led_count(), sm=1)
 led_max = config.get_led_max()
 np.fill((led_max, led_max, 0))
 np.write()
 def run():
    asyncio.new_event_loop()
    # Setup LEDs
    np = NeoPixel(hwconfig.LED_DIN, config.get_led_count(), sm=1)
-    # Wifi with default config
+    if machine.Pin(hwconfig.BUTTONS[1], machine.Pin.IN, machine.Pin.PULL_UP).value() == 0:
-    setup_wifi()
+        np.fill((0, 0, led_max))
        np.write()
        # Force default access point
        setup_wifi('', '', network.WLAN.SEC_OPEN)
    else:
        secstring = config.get_wifi_security()
        security = network.WLAN.SEC_WPA_WPA2
        if secstring == 'open':
            security = network.WLAN.SEC_OPEN
        elif secstring == 'wpa_wpa2':
            security = network.WLAN.SEC_WPA_WPA2
        elif secstring == 'wpa3':
            security = network.WLAN.SEC_WPA3
        elif secstring == 'wpa2_wpa3':
            security = network.WLAN.SEC_WPA2_WPA3
        setup_wifi(config.get_wifi_ssid(), config.get_wifi_passphrase(), security)
    # Setup MP3 player
    with SDContext(mosi=hwconfig.SD_DI, miso=hwconfig.SD_DO, sck=hwconfig.SD_SCK, ss=hwconfig.SD_CS,
                   baudrate=hwconfig.SD_CLOCKRATE):
        # Temporary hack: build database from folders if no database exists
        # Can be removed once playlists can be created via API
        try:
            _ = os.stat(DB_PATH)
        except OSError as ex:
            if ex.errno == ENOENT:
                print("No playlist DB found, trying to build DB from tag dirs")
                builddb()
        with BTreeFileManager(DB_PATH) as playlistdb, \
             AudioContext(hwconfig.I2S_DIN, hwconfig.I2S_DCLK, hwconfig.I2S_LRCLK) as audioctx:
@@ -99,7 +116,7 @@ def run():
                                    buttons=lambda the_app: Buttons(the_app, config, hwconfig),
                                    playlistdb=lambda _: playlistdb,
                                    hwconfig=lambda _: hwconfig,
-                                    leds=lambda _: LedManager(np),
+                                    leds=lambda _: LedManager(np, config),
                                    config=lambda _: config)
            the_app = app.PlayerApp(deps)
@@ -112,25 +129,26 @@ def run():
            asyncio.get_event_loop().run_forever()
-def builddb():
+def error_blink():
-    """
+    while True:
-    For testing, build a playlist db based on the previous tag directory format.
+        if machine.Pin(hwconfig.BUTTONS[0], machine.Pin.IN, machine.Pin.PULL_UP).value() == 0:
-    Can be removed once uploading files / playlist via the web api is possible.
+            machine.reset()
-    """
+        np.fill((led_max, 0, 0))
-    try:
+        np.write()
-        os.unlink(DB_PATH)
+        time.sleep_ms(500)
-    except OSError:
+        np.fill((0, 0, 0))
-        pass
+        np.write()
-    with BTreeFileManager(DB_PATH) as db:
+        time.sleep_ms(500)
        for name, type_, _, _ in os.ilistdir(b'/sd'):
            if type_ != 0x4000:
                continue
            fl = [b'/sd/' + name + b'/' + x for x in os.listdir(b'/sd/' + name) if x.endswith(b'.mp3')]
            db.createPlaylistForTag(name, fl)
    os.sync()
 if __name__ == '__main__':
    time.sleep(1)
    if machine.Pin(hwconfig.BUTTONS[0], machine.Pin.IN, machine.Pin.PULL_UP).value() != 0:
-        run()
+        try:
            run()
        except Exception as ex:
            sys.print_exception(ex)
            error_blink()
    else:
        np.fill((led_max, 0, 0))
        np.write()
--- a/software/src/utils/init.py
+++ b/software/src/utils/init.py
@@ -2,6 +2,7 @@
 # Copyright (c) 2025 Matthias Blankertz <matthias@blankertz.org>
 from utils.helpers import safe_callback
 from utils.timer import TimerManager
 from utils.buttons import Buttons
 from utils.config import Configuration
 from utils.leds import LedManager
@@ -9,7 +10,6 @@ from utils.mbrpartition import MBRPartition
 from utils.pinindex import get_pin_index
 from utils.playlistdb import BTreeDB, BTreeFileManager
 from utils.sdcontext import SDContext
 from utils.timer import TimerManager
 __all__ = ["BTreeDB", "BTreeFileManager", "Buttons", "Configuration", "get_pin_index", "LedManager", "MBRPartition",
           "safe_callback", "SDContext", "TimerManager"]
--- a/software/src/utils/buttons.py
+++ b/software/src/utils/buttons.py
@@ -5,7 +5,7 @@ import asyncio
 import machine
 import micropython
 import time
-from utils import safe_callback
+from utils import safe_callback, TimerManager
 try:
    from typing import TYPE_CHECKING  # type: ignore
 except ImportError:
@@ -32,6 +32,7 @@ class Buttons:
    def __init__(self, cb: "ButtonCallback", config, hwconfig):
        self.button_map = config.get_button_map()
        self.hw_buttons = hwconfig.BUTTONS
        self.hwconfig = hwconfig
        self.cb = cb
        self.buttons = dict()
        for key_id, key_name in self.KEYMAP.items():
@@ -42,6 +43,7 @@ class Buttons:
        self.int_flag = asyncio.ThreadSafeFlag()
        self.pressed: list[int] = []
        self.last: dict[int, int] = {}
        self.timer_manager = TimerManager()
        for button in self.buttons.keys():
            button.irq(handler=self._interrupt, trigger=machine.Pin.IRQ_FALLING | machine.Pin.IRQ_RISING)
        asyncio.create_task(self.task())
@@ -69,6 +71,10 @@ class Buttons:
            # print(f'B{keycode} {now}')
            self.pressed.append(keycode)
            self.int_flag.set()
            if keycode == self.VOLDOWN:
                self.timer_manager.schedule(time.ticks_ms() + 5000, self.long_press_shutdown)
        if button.value() == 1 and keycode == self.VOLDOWN:
            self.timer_manager.cancel(self.long_press_shutdown)
    async def task(self):
        while True:
@@ -76,3 +82,9 @@ class Buttons:
            while len(self.pressed) > 0:
                what = self.pressed.pop()
                safe_callback(lambda: self.cb.onButtonPressed(what), "button callback")
    def long_press_shutdown(self):
        if self.hwconfig.get_on_battery():
            self.hwconfig.power_off()
        else:
            machine.reset()
--- a/software/src/utils/config.py
+++ b/software/src/utils/config.py
@@ -22,7 +22,15 @@ class Configuration:
            'PREV': None,
            'NEXT': 1,
        },
-        'TAGMODE': 'tagremains'
+        'TAGMODE': 'tagremains',
        'WIFI': {
            'SSID': '',
            'PASSPHRASE': '',
            'SECURITY': 'wpa_wpa2',
        },
        'VOLUME_MAX': 255,
        'VOLUME_BOOT': 16,
        'LED_MAX': 255,
    }
    def __init__(self, config_path='/config.json'):
@@ -87,6 +95,24 @@ class Configuration:
    def get_tagmode(self) -> str:
        return self._get('TAGMODE')
    def get_wifi_ssid(self) -> str:
        return self._get('WIFI')['SSID']
    def get_wifi_passphrase(self) -> str:
        return self._get('WIFI')['PASSPHRASE']
    def get_wifi_security(self) -> str:
        return self._get('WIFI')['SECURITY']
    def get_volume_max(self) -> int:
        return self._get('VOLUME_MAX')
    def get_led_max(self) -> int:
        return self._get('LED_MAX')
    def get_volume_boot(self) -> int:
        return self._get('VOLUME_BOOT')
    # For the web API
    def get_config(self) -> Mapping[str, Any]:
        return self.config
@@ -104,6 +130,9 @@ class Configuration:
        self._validate(self.DEFAULT_CONFIG, config)
        if 'TAGMODE' in config and config['TAGMODE'] not in ['tagremains', 'tagstartstop']:
            raise ValueError("Invalid TAGMODE: Must be 'tagremains' or 'tagstartstop'")
        if 'WLAN' in config and 'SECURITY' in config['WLAN'] and \
           config['WLAN']['SECURITY'] not in ['open', 'wpa_wpa2', 'wpa3', 'wpa2_wpa3']:
            raise ValueError("Invalid WLAN SECURITY: Must be 'open', 'wpa_wpa2', 'wpa3' or 'wpa2_wpa3'")
        self._merge_configs(self.config, config)
        self.config = config
        self._save()
--- a/software/src/utils/leds.py
+++ b/software/src/utils/leds.py
@@ -12,10 +12,10 @@ class LedManager:
    PLAYING = const(1)
    REBOOTING = const(2)
-    def __init__(self, np):
+    def __init__(self, np, config):
        self.led_state = LedManager.IDLE
        self.brightness = config.get_led_max() / 255
        self.np = np
        self.brightness = 0.1
        self.leds = len(self.np)
        asyncio.create_task(self.run())
--- a/software/src/utils/playlistdb.py
+++ b/software/src/utils/playlistdb.py
@@ -33,12 +33,13 @@ class BTreeDB(IPlaylistDB):
    PERSIST_OFFSET = b'offset'
    class Playlist(IPlaylist):
-        def __init__(self, parent: "BTreeDB", tag: bytes, pos: int, persist, shuffle):
+        def __init__(self, parent: "BTreeDB", tag: bytes, pos: int, persist, shuffle, name):
            self.parent = parent
            self.tag = tag
            self.pos = pos
            self.persist = persist
            self.shuffle = shuffle
            self.name = name
            self.length = self.parent._getPlaylistLength(self.tag)
            self._shuffle()
@@ -168,6 +169,10 @@ class BTreeDB(IPlaylistDB):
        return (b''.join([tag, b'/playlist/']),
                b''.join([tag, b'/playlist0']))
    @staticmethod
    def _keyPlaylistName(tag):
        return b''.join([tag, b'/playlistname'])
    def _flush(self):
        """
        Flush the database and call the flush_func if it was provided.
@@ -222,12 +227,13 @@ class BTreeDB(IPlaylistDB):
            raise RuntimeError("Malformed playlist key")
        return int(elements[2])+1
-    def _savePlaylist(self, tag, entries, persist, shuffle, flush=True):
+    def _savePlaylist(self, tag, entries, persist, shuffle, name, flush=True):
        self._deletePlaylist(tag, False)
        for idx, entry in enumerate(entries):
            self.db[self._keyPlaylistEntry(tag, idx)] = entry
        self.db[self._keyPlaylistPersist(tag)] = persist
        self.db[self._keyPlaylistShuffle(tag)] = shuffle
        self.db[self._keyPlaylistName(tag)] = name.encode()
        if flush:
            self._flush()
@@ -240,7 +246,7 @@ class BTreeDB(IPlaylistDB):
                pass
        for k in (self._keyPlaylistPos(tag), self._keyPlaylistPosOffset(tag),
                  self._keyPlaylistPersist(tag), self._keyPlaylistShuffle(tag),
-                  self._keyPlaylistShuffleSeed(tag)):
+                  self._keyPlaylistShuffleSeed(tag), self._keyPlaylistName(tag)):
            try:
                del self.db[k]
            except KeyError:
@@ -248,15 +254,18 @@ class BTreeDB(IPlaylistDB):
        if flush:
            self._flush()
-    def getPlaylistTags(self):
+    def getPlaylists(self):
        """
-        Get a keys-only dict of all defined playlists. Playlists currently do not have names, but are identified by
+        Get a list of all defined playlists with their tag and names.
        their tag.
        """
        playlist_tags = set()
        for item in self.db:
            playlist_tags.add(item.split(b'/')[0])
-        return playlist_tags
+        playlists = []
        for tag in playlist_tags:
            name = self.db.get(self._keyPlaylistName(tag), b'').decode()
            playlists.append({'tag': tag, 'name': name})
        return playlists
    def getPlaylistForTag(self, tag: bytes):
        """
@@ -275,18 +284,19 @@ class BTreeDB(IPlaylistDB):
            return None
        if self._keyPlaylistEntry(tag, pos) not in self.db:
            pos = 0
        name = self.db.get(self._keyPlaylistName(tag), b'').decode()
        shuffle = self.db.get(self._keyPlaylistShuffle(tag), self.SHUFFLE_NO)
-        return self.Playlist(self, tag, pos, persist, shuffle)
+        return self.Playlist(self, tag, pos, persist, shuffle, name)
    def createPlaylistForTag(self, tag: bytes, entries: typing.Iterable[bytes], persist=PERSIST_TRACK,
-                             shuffle=SHUFFLE_NO):
+                             shuffle=SHUFFLE_NO, name: str = ''):
        """
        Create and save a playlist for 'tag' and return the Playlist object. If a playlist already existed for 'tag' it
        is overwritten.
        """
        assert persist in (self.PERSIST_NO, self.PERSIST_TRACK, self.PERSIST_OFFSET)
        assert shuffle in (self.SHUFFLE_NO, self.SHUFFLE_YES)
-        self._savePlaylist(tag, entries, persist, shuffle)
+        self._savePlaylist(tag, entries, persist, shuffle, name)
        return self.getPlaylistForTag(tag)
    def deletePlaylistForTag(self, tag: bytes):
@@ -370,6 +380,14 @@ class BTreeDB(IPlaylistDB):
                    _ = int(val)
                except ValueError:
                    fail(f' Bad playlistposoffset value for {last_tag}: {val!r}')
            elif fields[1] == b'playlistname':
                val = self.db[k]
                try:
                    name = val.decode()
                    if dump:
                        print(f'\tName: {name}')
                except UnicodeError:
                    fail(f' Bad playlistname for {last_tag}: Not valid unicode')
            else:
                fail(f'Unknown key {k!r}')
        return result
--- a/software/src/webserver.py
+++ b/software/src/webserver.py
@@ -4,10 +4,15 @@ Copyright (c) 2024-2025 Stefan Kratochwil <Kratochwil-LA@gmx.de>
 '''
 import asyncio
 import board
 import errno
 import hwconfig
 import json
 import machine
 import network
 import os
 import time
 import ubinascii
 from array import array
 from microdot import Microdot, redirect, send_file, Request
@@ -27,13 +32,13 @@ Request.max_content_length = 128 * 1024 * 1024  # 128MB requests allowed
 def start_webserver(config_, app_):
    global server, config, app, nfc, playlist_db, leds, timer_manager
-    server = asyncio.create_task(webapp.start_server(port=80))
+    server = asyncio.create_task(webapp.start_server(host='::0', port=80))
    config = config_
    app = app_
    nfc = app.get_nfc()
    playlist_db = app.get_playlist_db()
    leds = app.get_leds()
-    timer_manager = app.get_timer_manager()
+    timer_manager = TimerManager()
@webapp.before_request
@@ -108,7 +113,7 @@ async def static(request, path):
@webapp.route('/api/v1/playlists', methods=['GET'])
 async def playlists_get(request):
-    return sorted(playlist_db.getPlaylistTags())
+    return playlist_db.getPlaylists()
 def is_hex(s):
@@ -129,10 +134,11 @@ async def playlist_get(request, tag):
        return None, 404
    return {
-            'shuffle': playlist.__dict__.get('shuffle'),
+            'shuffle': playlist.shuffle,
-            'persist': playlist.__dict__.get('persist'),
+            'persist': playlist.persist,
            'paths': [(p[len(fsroot):] if p.startswith(fsroot) else p).decode()
                      for p in playlist.getPaths()],
            'name': playlist.name
    }
@@ -152,7 +158,8 @@ async def playlist_put(request, tag):
    playlist_db.createPlaylistForTag(tag.encode(),
                                     (fsroot + path.encode() for path in playlist.get('paths', [])),
                                     playlist.get('persist', 'track').encode(),
-                                     playlist.get('shuffle', 'no').encode())
+                                     playlist.get('shuffle', 'no').encode(),
                                     playlist.get('name', ''))
    return '', 204
@@ -196,6 +203,25 @@ async def audiofiles_get(request):
    return directory_iterator(), {'Content-Type': 'application/json; charset=UTF-8'}
 async def stream_to_file(stream, file_, length):
    data = array('b', range(4096))
    bytes_copied = 0
    while True:
        bytes_read = await stream.readinto(data)
        if bytes_read == 0:
            # End of body
            break
        bytes_written = file_.write(data[:bytes_read])
        if bytes_written != bytes_read:
            # short writes shouldn't happen
            raise OSError(errno.EIO, 'unexpected short write')
        bytes_copied += bytes_written
        if bytes_copied == length:
            break
        app.reset_idle_timeout()
    return bytes_copied
@webapp.route('/api/v1/audiofiles', methods=['POST'])
 async def audiofile_upload(request):
    if 'type' not in request.args or request.args['type'] not in ['file', 'directory']:
@@ -212,20 +238,13 @@ async def audiofile_upload(request):
        os.mkdir(path)
        return '', 204
    with open(path, 'wb') as newfile:
-        data = array('b', range(4096))
+        try:
-        bytes_copied = 0
+            if length > Request.max_body_length:
-        while True:
+                bytes_copied = await stream_to_file(request.stream, newfile, length)
-            bytes_read = await request.stream.readinto(data)
+            else:
-            if bytes_read == 0:
+                bytes_copied = newfile.write(request.body)
-                # End of body
+        except OSError as ex:
-                break
+            return f'error writing data to file: {ex}', 500
            bytes_written = newfile.write(data[:bytes_read])
            if bytes_written != bytes_read:
                # short writes shouldn't happen
                return 'write failure', 500
            bytes_copied += bytes_written
            if bytes_copied == length:
                break
    if bytes_copied == length:
        return '', 204
    else:
@@ -258,11 +277,20 @@ async def audiofile_delete(request):
@webapp.route('/api/v1/reboot/<method>', methods=['POST'])
 async def reboot(request, method):
    if method == 'bootloader':
        if hwconfig.get_on_battery():
            return 'not possible: connect USB first', 403
        leds.set_state(LedManager.REBOOTING)
        timer_manager.schedule(time.ticks_ms() + 1500, machine.bootloader)
-    elif method =='application':
+    elif method == 'application':
        leds.set_state(LedManager.REBOOTING)
        timer_manager.schedule(time.ticks_ms() + 1500, machine.reset)
    else:
        return 'method not supported', 400
    return '', 204
@webapp.route('/api/v1/info', methods=['GET'])
 async def get_info(request):
    mac = ubinascii.hexlify(network.WLAN().config('mac'), ':').decode()
    return {'version': board.version,
            'mac': mac}
--- a/software/tests/test_playerapp.py
+++ b/software/tests/test_playerapp.py
@@ -139,6 +139,12 @@ class FakeConfig:
    def get_tagmode(self):
        return 'tagremains'
    def get_volume_max(self):
        return 255
    def get_volume_boot(self):
        return 16
 def fake_open(filename, mode):
    return FakeFile(filename, mode)
@@ -167,7 +173,7 @@ def test_construct_app(micropythonify, faketimermanager):
    deps = _makedeps(mp3player=fake_mp3)
    dut = app.PlayerApp(deps)
    fake_mp3 = dut.player
-    assert fake_mp3.volume is not None
+    assert fake_mp3.volume is not None and fake_mp3.volume >= 16
 def test_load_playlist_on_tag(micropythonify, faketimermanager, monkeypatch):
--- a/software/tools/standalone-mp3/CMakeLists.txt
+++ b/software/tools/standalone-mp3/CMakeLists.txt
@@ -15,6 +15,8 @@ option(ENABLE_PLAY_TEST "Enable mp3 playback test" OFF)
 option(ENABLE_SD_READ_CRC "Enable crc check when reading from sd card" OFF)
 option(ENABLE_SD_DEBUG "Enable debug output for sd card driver" OFF)
 set(PICO_USE_FASTEST_SUPPORTED_CLOCK 1)
 # initialize the Raspberry Pi Pico SDK
 pico_sdk_init()
@@ -62,7 +64,7 @@ add_subdirectory(../../lib/helix_mp3 helix_mp3)
 target_link_libraries(standalone_mp3 PRIVATE pico_stdlib hardware_dma hardware_spi hardware_sync hardware_pio helix_mp3)
 target_include_directories(standalone_mp3 PRIVATE ${SD_LIB_DIR})
-target_compile_options(standalone_mp3 PRIVATE -Og -DSD_DEBUG)
+target_compile_options(standalone_mp3 PRIVATE -Og)
 pico_add_extra_outputs(standalone_mp3)
--- a/software/tools/standalone-mp3/main.c
+++ b/software/tools/standalone-mp3/main.c
@@ -2,6 +2,7 @@
 #include "sd.h"
 #include <math.h>
 #include <stdarg.h>
 #include <stdio.h>
 #include <string.h>
@@ -20,6 +21,14 @@ extern void sd_spi_dbg_clk(const int div, const int frac);
 extern void sd_spi_dbg_loop(void);
 void sd_printf(const char *fmt, ...)
 {
    va_list ap;
    va_start(ap, fmt);
    vprintf(fmt, ap);
    va_end(ap);
 }
 #define MAX_VOLUME 0x8000u
 void __time_critical_func(volume_adjust)(int16_t *restrict buf, size_t samples, uint16_t scalef)
@@ -157,7 +166,7 @@ static void write_test(struct sd_context *sd_context)
            data_buffer[i] ^= 0xff;
        }
-        if (!sd_writeblock(sd_context, 0, data_buffer)) {
+        if (!sd_writeblocks(sd_context, 0, sizeof(data_buffer) / SD_SECTOR_SIZE, data_buffer)) {
            printf("sd_writeblock failed\n");
            return;
        }
Author	SHA1	Message	Date
Matthias Blankertz	aa41334ba7	feat: Add SD multiblock write support All checks were successful Build RPi Pico firmware image / Build-Firmware (push) Successful in 4m45s Details Check code formatting / Check-C-Format (push) Successful in 6s Details Check code formatting / Check-Python-Flake8 (push) Successful in 10s Details Check code formatting / Check-Bash-Shellcheck (push) Successful in 5s Details Run unit tests on host / Run-Unit-Tests (push) Successful in 8s Details Run pytests / Check-Pytest (push) Successful in 10s Details Add support for CMD25 to write multiple sequential blocks in one go, in an attempt to speed up uploads. It still needs to be benchmarked if this actually results in a meaningful speedup. Also fix logging from SD driver to be visible even when running under micropython. Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2026-01-13 22:41:34 +01:00
Matthias Blankertz	3537a2f1bb	fix: Reduce file upload chunk size to 4k All checks were successful Build RPi Pico firmware image / Build-Firmware (push) Successful in 4m42s Details Check code formatting / Check-C-Format (push) Successful in 7s Details Check code formatting / Check-Python-Flake8 (push) Successful in 10s Details Check code formatting / Check-Bash-Shellcheck (push) Successful in 5s Details Run unit tests on host / Run-Unit-Tests (push) Successful in 8s Details Run pytests / Check-Pytest (push) Successful in 11s Details After seeing occasional memory allocation failures on file upload, reduce chunk size to 4k again. Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2026-01-13 22:38:37 +01:00
Matthias Blankertz	39a9c68aae	Merge pull request 'misc-features' (#62 ) from misc-features into main All checks were successful Build RPi Pico firmware image / Build-Firmware (push) Successful in 4m45s Details Check code formatting / Check-C-Format (push) Successful in 8s Details Check code formatting / Check-Python-Flake8 (push) Successful in 10s Details Check code formatting / Check-Bash-Shellcheck (push) Successful in 5s Details Run unit tests on host / Run-Unit-Tests (push) Successful in 9s Details Run pytests / Check-Pytest (push) Successful in 11s Details Reviewed-on: #62 Reviewed-by: Stefan Kratochwil <kratochwil-la@gmx.de>	2026-01-13 21:31:18 +00:00
Matthias Blankertz	6dee7fff7e	feat: Allow configuring WiFi security All checks were successful Build RPi Pico firmware image / Build-Firmware (push) Successful in 4m44s Details Check code formatting / Check-C-Format (push) Successful in 7s Details Check code formatting / Check-Python-Flake8 (push) Successful in 10s Details Check code formatting / Check-Bash-Shellcheck (push) Successful in 4s Details Run unit tests on host / Run-Unit-Tests (push) Successful in 9s Details Run pytests / Check-Pytest (push) Successful in 11s Details Allow choosing between the three security modes exposed by the micropython cyw43 wifi driver. Also allow setting up security in AP mode. Improve the WiFi section of the configuration UI. Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2026-01-06 14:59:12 +01:00
Matthias Blankertz	6976aa6963	fix[player]: Don't latch tag if no playlist exists When the device is in 'tagstartstop' tag mode, and the user presents a new tag to get the serial number and create a playlist using the web UI, the playerapp still remembered the tag as the currently playing tag even though no playlist was found and no playback is running. After the user saves the playlist in the UI and puts the tag on the device again, they expect the playback to start with the new playlist. Instead, nothing happens, because this is counted as the 'stop' event of the tagstartstop mode. The user would have to remove the tag and present it again (after waiting for the tagtimeout) to play the new playlist. Fix this unexpected behaviour by not storing the current tag into the playing_tag field if no playlist existed for the tag. Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2026-01-06 14:59:12 +01:00
Matthias Blankertz	763305c659	fix[frontend]: Reset upload UI elements, don't expand tree view - Make sure the upload progess bar and file choser are reset when loading the file browser screen. - Don't expand the directories in the tree view by default. Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2026-01-06 14:59:12 +01:00
Matthias Blankertz	6d18437863	fix: Remove directory based db creation Previously, if no tonberry.db existed on the SD card, the database was initialized with a playlist for each directory containing mp3 files, with the tag serial number matching the directory name. This was used during development before the web UI to edit the playlist db existed. It is no longer necessary, and confusing to the user when unusable playlists are created when albums are preloaded onto the SD card before putting it in the tonberry device. Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2026-01-06 14:59:12 +01:00
Matthias Blankertz	2cf88b26ee	fix: button 0 to shutdown device when in error state All checks were successful Build RPi Pico firmware image / Build-Firmware (push) Successful in 4m42s Details Check code formatting / Check-C-Format (push) Successful in 7s Details Check code formatting / Check-Python-Flake8 (push) Successful in 10s Details Check code formatting / Check-Bash-Shellcheck (push) Successful in 5s Details Run unit tests on host / Run-Unit-Tests (push) Successful in 8s Details Run pytests / Check-Pytest (push) Successful in 10s Details Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-27 15:23:48 +01:00
Matthias Blankertz	73da134a12	fix: webserver: file uploading - Fix upload of files <= Request.max_body_length File uploads smaller than the limit were not given to the handler as a stream by microdot, instead the content is directly stored in the request.body. - Refactor stream to file copy into a helper method - Increase the copy buffer size to 16k - Call app.reset_idle_timeout() periodically during file uploads to avoid the device turning off when uploading large files while on battery. Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-27 15:23:48 +01:00
Matthias Blankertz	5c8a61eb27	feat: Allow configuring volume set at startup Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-27 15:23:48 +01:00
Matthias Blankertz	4c85683fcb	feat: Allow limiting LED brightness Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-27 15:23:48 +01:00
Matthias Blankertz	355a8bd345	fix: allow 'reboot' to application when on on battery Having to press the power button again to wake up the device is less annoying to the user than not being able to apply settings when the device is on battery. Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-27 15:23:48 +01:00
Matthias Blankertz	f46c045589	feat: Allow limiting max. volume Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-27 15:23:48 +01:00
Matthias Blankertz	fe1c1eadf7	feat: Add names to playlists Fixes #63. Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-27 15:23:48 +01:00
Matthias Blankertz	743188e1a4	fix: frontend: Replace generic 'Device admin' title Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-27 15:23:48 +01:00
Matthias Blankertz	cd5939f4ee	feat: Enable dualstack IPv4/IPv6 for microdot Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-27 15:23:48 +01:00
Matthias Blankertz	02954cd87c	feat: Visual feedback on LEDs during startup Set the LEDs to a fixed color during bootup to show the different modes: - Orange when the device is booting - Red when button 0 is held and the device goes to a shell - Blue when button 1 is held and the device goes to AP mode instead of joining the configured WiFi network - Red blinking when the run() method throws an exception for any reason Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-27 15:23:48 +01:00
Matthias Blankertz	58f8526d7e	feat: Join an existing WiFi network Add ssid and passphrase to config to configure the network to join. If empty SSID is configured, it will create the "TonberryPicoAP..." network in AP mode as before. Holding the button 1 during startup will revert to AP mode regardless of the current configuration. Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-27 15:23:48 +01:00
Matthias Blankertz	d3aef1be32	fix: frontend: don't convert text that looks like an integer to integers Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-23 12:00:05 +01:00
Matthias Blankertz	b9baa1c7d5	microdot: Update to v2.5.1 The important fix is an urldecode bug causing umlauts to be decoded incorrectly from url arguments. This was fixed in v2.2.0, but let's just update to the latest version. Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-23 12:00:05 +01:00
Matthias Blankertz	67d7650923	fix: webserver: Catch and report IO errors on upload Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-23 12:00:05 +01:00
Matthias Blankertz	43fd68779c	feat: store git version in fw and show in web ui Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-23 12:00:05 +01:00
Matthias Blankertz	704951074b	feat: Long press VOL_DOWN button to shutdown/reset device Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-21 18:31:13 +01:00
Matthias Blankertz	cac61f924f	Merge branch 'file-upload' into mbl-next All checks were successful Build RPi Pico firmware image / Build-Firmware (push) Successful in 4m43s Details Check code formatting / Check-C-Format (push) Successful in 7s Details Check code formatting / Check-Python-Flake8 (push) Successful in 10s Details Check code formatting / Check-Bash-Shellcheck (push) Successful in 5s Details Run unit tests on host / Run-Unit-Tests (push) Successful in 9s Details Run pytests / Check-Pytest (push) Successful in 11s Details	2025-12-21 18:31:04 +01:00
Matthias Blankertz	9320a3cff2	fix: Show reboot request response in UI All checks were successful Build RPi Pico firmware image / Build-Firmware (push) Successful in 4m40s Details Check code formatting / Check-C-Format (push) Successful in 7s Details Check code formatting / Check-Python-Flake8 (push) Successful in 9s Details Check code formatting / Check-Bash-Shellcheck (push) Successful in 4s Details Run unit tests on host / Run-Unit-Tests (push) Successful in 8s Details Run pytests / Check-Pytest (push) Successful in 11s Details Also make response from api more understandable for non-technical users. Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-21 18:30:45 +01:00
Stefan Kratochwil	65efebc5c2	feat: allow reboot commands only if usb cable is inserted	2025-12-21 18:30:45 +01:00
Matthias Blankertz	040ae4a731	fix: fix flake8 complaint Signed-off-by: Matthias Blankertz <matthias@blankertz.org>	2025-12-21 18:30:45 +01:00