Skip to content

Conversation

@uis246
Copy link
Contributor

@uis246 uis246 commented Aug 16, 2022

And also rename hdmi pins

@yrabbit
Copy link
Collaborator

yrabbit commented Aug 16, 2022

Ignoring the LED on this board is not a mistake, but a precautionary measure: any failure when using the MODE pin as a GPIO can turn your FPGA into a brick, and on this board the LED is just on the MODE pin (10).
The ability to use the MODE pin as a GPIO is not even supported by gowin_pack.
So this board requires external LEDs to be connected, well since we are connecting external LEDs anyway, why 1 and not 6? :)
I don't think renaming these lines is a good idea - it would break some examples.

@uis246
Copy link
Contributor Author

uis246 commented Aug 16, 2022

But my board didn't brick. Only write to flash bricks?

The ability to use the MODE pin as a GPIO is not even supported by gowin_pack.

IDK. Worked for me.

why 1 and not 6? :)
I don't think renaming these lines is a good idea - it would break some examples.

I was thinking about on-board led. Maybe separate cst for blinky with it?

@yrabbit
Copy link
Collaborator

yrabbit commented Aug 16, 2022

But my board didn't brick. Only write to flash bricks?

The ability to use the MODE pin as a GPIO is not even supported by gowin_pack.

IDK. Worked for me.
Now that's a real issue :) We should take a look at the code and fix it.

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

Successfully merging this pull request may close these issues.

2 participants