Teensy 3.0 and above support is needed

[vamsee227]

Error compiling the test code for teensy board Adafruit_PCD8544.cpp: In member function 'void Adafruit_PCD8544::begin(uint8_t, uint8_t)': Adafruit_PCD8544.cpp:287:13: error: cannot convert 'volatile uint8_t {aka volatile unsigned char}' to 'PortReg {aka volatile long unsigned int}' in assignment clkport = portOutputRegister(digitalPinToPort(_sclk)); Adafruit_PCD8544.cpp:289:14: error: cannot convert 'volatile uint8_t {aka volatile unsigned char}' to 'PortReg {aka volatile long unsigned int}' in assignment mosiport = portOutputRegister(digitalPinToPort(_din)); Error compiling for board Teensy 3.2 / 3.1.

• Teensy board used: Teensy 3.2

• Arduino IDE version **Arduino IDE ver. 1.8.13

[vamsee227]

It worked after editing the header file (H file). After changing "uint32_t" in these lines

else

typedef volatile uint32_t PortReg; ///< PortReg for other chips typedef uint32_t PortMask; ///< PortMask for other chips

endif

Changed to "uint8_t"

else

typedef volatile uint8_t PortReg; ///< PortReg for other chips typedef uint8_t PortMask; ///< PortMask for other chips

endif

I request you to update the library by providing the support for Teensy 3.0 and above boards

Thanks Vamsee

[drak7]

By default the example uses software SPI, switching to hardware SPI should fix it.

The port registers on the Teensy 3 are emulated as 8-bit for compatibility with Arduino so we could check for that board and use the 8-bit defines. The actual port registers for the teensy are 32-bit and can be used directly, that would be the better way to handle it. See this teensy forum post and this one for more info.

[vamsee227]

Okay, thanks.

[FerGT50]

Same problem here. Applying @vamsee227 corrections worked! An update to the library to automatically account for 32 bit Teensies would be nice indeed!

Like:

if defined(SAM3X8E) || defined(ARDUINO_ARCH_SAMD)

typedef volatile RwReg PortReg; ///< PortReg for SAMD typedef uint32_t PortMask; ///< PortMask for SAMD

elif defined(AVR) || defined (TEENSYDUINO)

typedef volatile uint8_t PortReg; ///< PortReg for AVR typedef uint8_t PortMask; ///< PortMask for AVR

else

typedef volatile uint32_t PortReg; ///< PortReg for other chips typedef uint32_t PortMask; ///< PortMask for other chips

endif

[ladyada]

this should be fixed now with the move to busio!