rp2/boards/PIMORONI_PICOLIPO: Refactor Pico LiPo to use board variants.

Combine the 4MB and 16MB "PIMORONI_PICOLIPO" variants into a single board. Signed-off-by: Phil Howard <[email protected]>
IhorNehrutsa · Aug 2, 2024 · cd1ab76 · cd1ab76
1 parent 11becbe
commit cd1ab76
Show file tree

Hide file tree

Showing 11 changed files with 50 additions and 54 deletions.
diff --git a/.../boards/PIMORONI_PICOLIPO_16MB/board.json → ...s/rp2/boards/PIMORONI_PICOLIPO/board.json b/.../boards/PIMORONI_PICOLIPO_16MB/board.json → ...s/rp2/boards/PIMORONI_PICOLIPO/board.json
@@ -14,8 +14,11 @@
         "PimoroniPicoLipo_1of3_1024x1024.jpg"
     ],
     "mcu": "rp2040",
-    "product": "Pico LiPo (16MiB)",
+    "product": "Pico LiPo",
     "thumbnail": "",
     "url": "https://shop.pimoroni.com/products/pimoroni-pico-lipo",
+    "variants": {
+        "FLASH_16M": "16 MiB Flash"
+    },
     "vendor": "Pimoroni"
 }
diff --git a/...PIMORONI_PICOLIPO_4MB/mpconfigboard.cmake → ...rds/PIMORONI_PICOLIPO/mpconfigboard.cmake b/...PIMORONI_PICOLIPO_4MB/mpconfigboard.cmake → ...rds/PIMORONI_PICOLIPO/mpconfigboard.cmake
diff --git a/...rds/PIMORONI_PICOLIPO_4MB/mpconfigboard.h → .../boards/PIMORONI_PICOLIPO/mpconfigboard.h b/...rds/PIMORONI_PICOLIPO_4MB/mpconfigboard.h → .../boards/PIMORONI_PICOLIPO/mpconfigboard.h
@@ -1,7 +1,9 @@
 // https://shop.pimoroni.com/products/pimoroni-pico-lipo?variant=39386149093459
 
+#ifndef MICROPY_HW_BOARD_NAME
 #define MICROPY_HW_BOARD_NAME          "Pimoroni Pico LiPo 4MB"
-#define MICROPY_HW_FLASH_STORAGE_BYTES (3 * 1024 * 1024)
+#endif
+#define MICROPY_HW_FLASH_STORAGE_BYTES (PICO_FLASH_SIZE_BYTES - (1 * 1024 * 1024))
 
 #define MICROPY_HW_USB_VID (0x2E8A)
 #define MICROPY_HW_USB_PID (0x1002)
@@ -10,11 +12,3 @@
 #define MICROPY_HW_UART1_RX  (9)
 #define MICROPY_HW_UART1_CTS (10)
 #define MICROPY_HW_UART1_RTS (11)
-
-// User LED GPIO25
-
-// VBUS_SENSE GPIO24
-
-// BAT_SENSE GPIO29
-
-// Boot button GPIO23
diff --git a/ports/rp2/boards/PIMORONI_PICOLIPO/mpconfigvariant.cmake b/ports/rp2/boards/PIMORONI_PICOLIPO/mpconfigvariant.cmake
@@ -0,0 +1,6 @@
+set(PICO_BOARD "pimoroni_picolipo_4mb")
+
+# Override the MicroPython board name
+list(APPEND MICROPY_DEF_BOARD
+    MICROPY_HW_BOARD_NAME="Pimoroni Pico LiPo 4MB"
+)
diff --git a/ports/rp2/boards/PIMORONI_PICOLIPO/mpconfigvariant_FLASH_16M.cmake b/ports/rp2/boards/PIMORONI_PICOLIPO/mpconfigvariant_FLASH_16M.cmake
@@ -0,0 +1,6 @@
+set(PICO_BOARD "pimoroni_picolipo_16mb")
+
+# Override the MicroPython board name
+list(APPEND MICROPY_DEF_BOARD
+    MICROPY_HW_BOARD_NAME="Pimoroni Pico LiPo 16MB"
+)
diff --git a/ports/rp2/boards/PIMORONI_PICOLIPO/pins.csv b/ports/rp2/boards/PIMORONI_PICOLIPO/pins.csv
@@ -0,0 +1,31 @@
+GP0,GPIO0
+GP1,GPIO1
+GP2,GPIO2
+GP3,GPIO3
+GP4,GPIO4
+GP5,GPIO5
+GP6,GPIO6
+GP7,GPIO7
+GP8,GPIO8
+GP9,GPIO9
+GP10,GPIO10
+GP11,GPIO11
+GP12,GPIO12
+GP13,GPIO13
+GP14,GPIO14
+GP15,GPIO15
+GP16,GPIO16
+GP17,GPIO17
+GP18,GPIO18
+GP19,GPIO19
+GP20,GPIO20
+GP21,GPIO21
+GP22,GPIO22
+GP25,GPIO25
+GP26,GPIO26
+GP27,GPIO27
+GP28,GPIO28
+BOOT,GPIO23
+VBUS_SENSE,GPIO24
+LED,GPIO25
+BAT_SENSE,GPIO29
diff --git a/ports/rp2/boards/PIMORONI_PICOLIPO_16MB/mpconfigboard.cmake b/ports/rp2/boards/PIMORONI_PICOLIPO_16MB/mpconfigboard.cmake
diff --git a/ports/rp2/boards/PIMORONI_PICOLIPO_16MB/mpconfigboard.h b/ports/rp2/boards/PIMORONI_PICOLIPO_16MB/mpconfigboard.h
diff --git a/ports/rp2/boards/PIMORONI_PICOLIPO_16MB/pins.csv b/ports/rp2/boards/PIMORONI_PICOLIPO_16MB/pins.csv
diff --git a/ports/rp2/boards/PIMORONI_PICOLIPO_4MB/board.json b/ports/rp2/boards/PIMORONI_PICOLIPO_4MB/board.json
diff --git a/ports/rp2/boards/PIMORONI_PICOLIPO_4MB/pins.csv b/ports/rp2/boards/PIMORONI_PICOLIPO_4MB/pins.csv