From f9afd8ba4a3fae69fdee00f87d9f000cfe65223d Mon Sep 17 00:00:00 2001
From: Brendan <2bndy5@gmail.com>
Date: Mon, 4 Dec 2023 20:24:29 -0800
Subject: [PATCH] [RF24 wrapper] use `rf24_gpio_pin_t` and `RF24_PIN_INVALID`
 (#28)

This is actually an overlooked follow up to nRF24/RF24#898

The problematic inconsistency was primarily affecting builds with the RPi driver.
---
 src/pyRF24.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/pyRF24.cpp b/src/pyRF24.cpp
index 18787da..3d7847e 100644
--- a/src/pyRF24.cpp
+++ b/src/pyRF24.cpp
@@ -42,7 +42,7 @@ class RF24Wrapper : public RF24
 {
 
 public:
-    RF24Wrapper(uint16_t _ce_pin, uint16_t _csn_pin, uint32_t _spi_speed = 10000000) : RF24(_ce_pin, _csn_pin, _spi_speed)
+    RF24Wrapper(rf24_gpio_pin_t _ce_pin, rf24_gpio_pin_t _csn_pin, uint32_t _spi_speed = 10000000) : RF24(_ce_pin, _csn_pin, _spi_speed)
     {
     }
 
@@ -851,7 +851,7 @@ PYBIND11_MODULE(rf24, m)
         // *****************************************************************************
         // ************************** functions that have overloads
 
-        .def(py::init<uint8_t, uint8_t, uint32_t>(), R"docstr(
+        .def(py::init<rf24_gpio_pin_t, rf24_gpio_pin_t, uint32_t>(), R"docstr(
             __init__(ce_pin: int, csn_pin: int, spi_speed: int = 10000000)
             __init__(spi_speed: int = 10000000)
 
@@ -861,7 +861,7 @@ PYBIND11_MODULE(rf24, m)
             :param int csn_pin: The pin number connected to the radio's CSN pin.
             :param int spi_speed: The SPI bus speed (in Hz). Defaults to 10 MHz when not specified.
         )docstr",
-             py::arg("ce_pin") = 0xFFFF, py::arg("csn_pin") = 0xFFFF, py::arg("spi_speed") = 10000000)
+             py::arg("ce_pin") = RF24_PIN_INVALID, py::arg("csn_pin") = RF24_PIN_INVALID, py::arg("spi_speed") = 10000000)
 
         // *****************************************************************************
 
@@ -885,7 +885,7 @@ PYBIND11_MODULE(rf24, m)
 
         // *****************************************************************************
 
-        .def("begin", static_cast<bool (RF24Wrapper::*)(uint16_t, uint16_t)>(&RF24Wrapper::begin), R"docstr(
+        .def("begin", static_cast<bool (RF24Wrapper::*)(rf24_gpio_pin_t, rf24_gpio_pin_t)>(&RF24Wrapper::begin), R"docstr(
             If configuring the radio's CE & CSN pins dynamically, then the respective pin numbers must be passed to this function.
 
             :param int ce_pin: The pin number connected to the radio's CE pin.