TriodeGirl / M4_Minima_Test_Code_Fast-PWM_SPI_IRQ_ADC_DAC_SCI

RA4M1 demo code for the Arduino UNO R4 Minima with Fast-PIN, Fast-PWM, SPI, IRQ, ADC, DAC, and SCI operations
2 stars 0 forks source link

Typo in the PFS_4* definitions #1

Open tails4e opened 3 months ago

tails4e commented 3 months ago

The PFS_4 defintions have a typo which alias them back to the PFS_1 locations:

define PFS_P410PFS ((volatile unsigned int )(PORTBASE + P100PFS + (10 4))) // N/C - IRQ5

TriodeGirl commented 3 months ago

Hi @tails4e - thanks for the catch. I wrote a big chunk of these when I had Covid last year - I was not quite ill enough to have to be in bed, so I spent a big chunk of every day working on these RA4M1 code test/examples. These PFS_4* aren't in use on the Minima, but would be an issue with the Renesas EK-RA4M1 dev-board (I have one which I have popped in the Minima Bootloader, it looks like a regular Minima to the Arduino IDE). Thankfully my current include susan_ra4m1_minima_register_defines.h has the correct version. https://github.com/TriodeGirl/RA4M1_Arduino_UNO-R4_Processor_Direct_Register_Addressing_Defines Best, Susan.

tails4e commented 3 months ago

Hi Susan,

You're welcome, and I want to thank you for doing all this work to create the header in the first place. I am using an Uno R4 WiFi, which is how I found this - as the SPI pins use 410 and 411. The Arduino libraries dont support slave mode, so I had to roll my own code, and using your header and example code helped a lot. Its finally working!

All the best, John

On Sat, Jun 1, 2024 at 12:19 PM Susan Parker @.***> wrote:

Hi @tails4e https://github.com/tails4e - thanks for the catch. I wrote a big chunk of these when I had Covid last year - I was not quite ill enough to have to be in bed, so I spent a big chunk of every day working on these RA4M1 code test/examples. These PFS_4 aren't in use on the Minima, but would be an issue with the Renesas EK-RA4M1 dev-board (I have one which I have popped in the Minima Bootloader, it looks like a regular Minima to the Arduino IDE). Thankfully my current include susan_ra4m1_minima_register_defines.h* has the correct version.

https://github.com/TriodeGirl/RA4M1_Arduino_UNO-R4_Processor_Direct_Register_Addressing_Defines http://url Best, Susan.

— Reply to this email directly, view it on GitHub https://github.com/TriodeGirl/M4_Minima_Test_Code_Fast-PWM_SPI_IRQ_ADC_DAC_SCI/issues/1#issuecomment-2143411613, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATB54DH5JCKUXAERFNRBHMLZFGU4VAVCNFSM6AAAAABITL5AO6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGQYTCNRRGM . You are receiving this because you were mentioned.Message ID: <TriodeGirl/M4_Minima_Test_Code_Fast-PWM_SPI_IRQ_ADC_DAC_SCI/issues/1/2143411613 @github.com>