aboutsummaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorLoek Le Blansch <loek@pipeframe.xyz>2024-06-10 16:40:07 +0200
committerLoek Le Blansch <loek@pipeframe.xyz>2024-06-10 16:40:07 +0200
commit0e6fb935a8f079089e7d524ff2aa98d9be798a15 (patch)
treec880ae8466761d34f5790be98d61ef4c879e9917 /lib
parenta3ecfc2a6e6ace7bcb7666afc37ff97f9ce401b4 (diff)
rp2040 driver works (ugly)
Diffstat (limited to 'lib')
-rw-r--r--lib/pbdrv/CMakeLists.txt2
-rw-r--r--lib/pbdrv/drv/rp2040/cfg.cmake7
-rw-r--r--lib/pbdrv/drv/rp2040/mod.c73
-rw-r--r--lib/pbdrv/drv/rp2040/mod.h13
4 files changed, 1 insertions, 94 deletions
diff --git a/lib/pbdrv/CMakeLists.txt b/lib/pbdrv/CMakeLists.txt
index ca85b2b..fe464a9 100644
--- a/lib/pbdrv/CMakeLists.txt
+++ b/lib/pbdrv/CMakeLists.txt
@@ -24,5 +24,5 @@ target_include_directories(pbdrv-mod SYSTEM INTERFACE .)
# supported puzzle bus drivers
include(drv/arduino/cfg.cmake)
-include(drv/rp2040/cfg.cmake)
+# include(drv/rp2040/cfg.cmake) # moved to /main/pbdrv.c
diff --git a/lib/pbdrv/drv/rp2040/cfg.cmake b/lib/pbdrv/drv/rp2040/cfg.cmake
deleted file mode 100644
index 0fbad18..0000000
--- a/lib/pbdrv/drv/rp2040/cfg.cmake
+++ /dev/null
@@ -1,7 +0,0 @@
-if(NOT PICO_PLATFORM STREQUAL "rp2040")
- return()
-endif()
-
-target_sources(pbdrv-mod PRIVATE "${CMAKE_CURRENT_LIST_DIR}/mod.c")
-target_link_libraries(pbdrv-mod hardware_i2c pico_i2c_slave)
-
diff --git a/lib/pbdrv/drv/rp2040/mod.c b/lib/pbdrv/drv/rp2040/mod.c
deleted file mode 100644
index 6becdee..0000000
--- a/lib/pbdrv/drv/rp2040/mod.c
+++ /dev/null
@@ -1,73 +0,0 @@
-#include "mod.h"
-
-#include "../../pb.h"
-#include "../../pb-types.h"
-#include "../../pb-mod.h"
-
-#include <hardware/i2c.h>
-#include <hardware/gpio.h>
-#include <pico/i2c_slave.h>
-
-#define PBDRV_I2C i2c0
-#define BUF_SIZE 256
-
-static volatile bool pbdrv_i2c_msg_avail = false;
-static uint8_t data[BUF_SIZE];
-static size_t size = 0;
-
-// TODO: create event group instead of pbdrv_i2c_msg_avail
-// TODO: create freertos task that monitors the pbdrv_i2c_msg_avail flag and
-// calls pbdrv_i2c_recv when a message is received
-
-
-/**
- * \note this function is called from the I2C ISR, and should return as quickly
- * as possible.
- */
-static void recv_event(i2c_inst_t *i2c, i2c_slave_event_t event) {
- // message needs to be handled first
- if (pbdrv_i2c_msg_avail) return;
-
- switch (event) {
- case I2C_SLAVE_RECEIVE: {
- if (size == BUF_SIZE) return;
- data[size++] = i2c_read_byte_raw(PBDRV_I2C);
- break;
- }
- case I2C_SLAVE_FINISH: {
- // TODO: handle this w/ queue mechanism instead?
- // pbdrv_i2c_recv(data, size);
- pbdrv_i2c_msg_avail = true;
- size = 0;
- break;
- }
- default: break;
- }
-}
-
-void pbdrv_setup() {
- i2c_init(PBDRV_I2C, PB_CLOCK_SPEED_HZ);
- i2c_slave_init(PBDRV_I2C, PBDRV_MOD_ADDR, &recv_event);
-}
-
-/**
- * While the RP2040's datasheet claims it supports multi-master configurations
- * by implementing bus arbitration, it does not natively support a mode where
- * it is configured as a (multi-)master with a slave address, such that it can
- * be addressed by other multi-masters. This function includes a hacky
- * workaround that teporarily sets the RP2040 to I2C master mode to send a
- * message, and then restores it back to slave mode.
- *
- * This approach results in some received frames being (partially) dropped in
- * the time period between the invocation of this function and the bus becoming
- * idle (and the message is sent).
- */
-__weak void pbdrv_i2c_send(i2c_addr_t addr, const uint8_t * buf, size_t sz) {
- i2c_set_slave_mode(PBDRV_I2C, false, PBDRV_MOD_ADDR);
-
- // false to write stop condition to i2c bus
- i2c_write_timeout_us(PBDRV_I2C, addr, buf, sz, false, PB_TIMEOUT_US);
-
- i2c_set_slave_mode(PBDRV_I2C, true, PBDRV_MOD_ADDR);
-}
-
diff --git a/lib/pbdrv/drv/rp2040/mod.h b/lib/pbdrv/drv/rp2040/mod.h
deleted file mode 100644
index 0cf2e63..0000000
--- a/lib/pbdrv/drv/rp2040/mod.h
+++ /dev/null
@@ -1,13 +0,0 @@
-#pragma once
-
-#ifdef __cplusplus
-extern "C" {
-#endif
-
-//! puzzle bus driver setup
-void pbdrv_setup();
-
-#ifdef __cplusplus
-}
-#endif
-