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

Can't connect the paddle to pin-IO0? #129

Open
jh4vaj opened this issue May 19, 2022 · 2 comments
Open

Can't connect the paddle to pin-IO0? #129

jh4vaj opened this issue May 19, 2022 · 2 comments

Comments

@jh4vaj
Copy link

jh4vaj commented May 19, 2022

My hardware has the left paddle connected to pin-IO1 and the right paddle connected to pin-IO0.
And it works fine with the 2021.12.17.01 software.

Yesterday, I downloaded the new source code (2022.01.28.01) and compiled it, but I got a compilation error.
When I checked the history of the source code, it was as follows.

Added compiler macro to error out if paddle_left or paddle_right is defined as 0 (disabled)

I have keyer_pin_settings.h like this to fit my hardware:

#define paddle_left 1
#define paddle_right 0

I thought that this "0" was not the meaning of disable, but the definition of the pin number (IO0).

Is it no longer possible to assign a pin-IO0 to a paddle with current software?

@jefflogullo
Copy link

jefflogullo commented May 19, 2022 via email

@jh4vaj
Copy link
Author

jh4vaj commented May 20, 2022

Thank you very much, Jeff.

I understand the situation.

I have just completed my variant.
It has been working fine with the previous software (paddles are defined on pin-IO0 and pin-IO1).
I ran into this trouble when I tried to replace it with the latest software.

I understand that changing the pin assignments is a good idea, but doing that now is difficult.
Because it would require a redesign of the PCB.

I compared the previous software with the latest one and it seems that the following code has just been added.

#if (paddle_left == 0) || (paddle_right == 0)
  #error "You cannot define paddle_left or paddle_right as 0 to disable"
#endif

This time, this code will be disabled.
If someday I ever redesign the PCB, then I will change the paddle pin definitions.

If this trouble is limited to cases with CLI functionality, how about the following code?

#if defined(FEATURE_COMMAND_LINE_INTERFACE) && ((paddle_left == 0) || (paddle_right == 0))

Mit / JH4VAJ

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

2 participants