Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue with compiling example code #72

Open
worrallkj opened this issue Sep 11, 2024 · 3 comments
Open

Issue with compiling example code #72

worrallkj opened this issue Sep 11, 2024 · 3 comments

Comments

@worrallkj
Copy link

Hello all.

using the Adafruit Feather 2040 RP2040 DVI, Arduino IDE 2.3.2, current version of PicoDVI and associated libraries, and I am getting the following error:

\Documents\Arduino\libraries\PicoDVI_-Adafruit_Fork\src\libdvi\dvi.c: In function 'dvi_dma_irq_handler':
\Documents\Arduino\libraries\PicoDVI
-_Adafruit_Fork\src\libdvi\dvi.c:194:68: error: 'dma_debug_channel_hw_t' has no member named 'tcr'
194 | while (dma_debug_hw->ch[inst->dma_cfg[i].chan_data].tcr != inst->timing->h_active_pixels / DVI_SYMBOLS_PER_WORD)
| ^

exit status 1

Compilation error: exit status 1

Other RP2040 examples work but anything related to PicoDVI does not compile, all examples flagging the same error. Gone hunting about both the code and various forums but no one seems to have something similar. Happy to be at fault with this but I can't find the issue.

@SharifRafid
Copy link

Facing same issue, did you find a solution?

@HisashiKato
Copy link

HisashiKato commented Sep 20, 2024

Open 'dvi.c' in the ' \Arduino\libraries\PicoDVI_-_Adafruit_Fork\src\libdvi\ ' folder with an editor.

In line 194 of 'dvi.c', change 'tcr' to 'dbg_tcr' in ' ch[inst->dma_cfg[i].chan_data].tcr '.

- while (dma_debug_hw->ch[inst->dma_cfg[i].chan_data].tcr != inst->timing->h_active_pixels / DVI_SYMBOLS_PER_WORD)
+ while (dma_debug_hw->ch[inst->dma_cfg[i].chan_data].dbg_tcr != inst->timing->h_active_pixels / DVI_SYMBOLS_PER_WORD)

Save the file.

This is probably fine, sorry if it's not.

(This text was translated into English using a translation site)

@worrallkj
Copy link
Author

Brilliant - that was the change needed. Works well. tested a couple of the examples and is working. Thanks HisashiKato

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants