Why is the button on the STM32 micocontroller not responding once I am in the while loop?

Question

0.00/5 (No votes)

See more:

I am trying to make a program where two different LEDs are blinking every 1000ms but I also want to be able to exit the loop anytime I want. Once I get in the while loop, the button on the STM32 is not responding.

I am working with STM32CubeIDE and TouchGFX, C language.

C

void Screen1View::relay_1()
{
if(toggleButton1.getState()){
 while(1){
   HAL_GPIO_WritePin(GPIOD, GPIO_PIN_3, GPIO_PIN_SET);
   HAL_Delay(1000);
   HAL_GPIO_WritePin(GPIOD, GPIO_PIN_3, GPIO_PIN_RESET);
   HAL_Delay(1000);
   HAL_GPIO_WritePin(GPIOJ, GPIO_PIN_2, GPIO_PIN_SET);
   HAL_Delay(1000);
   HAL_GPIO_WritePin(GPIOJ, GPIO_PIN_2, GPIO_PIN_RESET);
   HAL_Delay(1000);
if(toggleButton1.getState() == 0){
   break;
       }
   }
}
else HAL_GPIO_WritePin(GPIOD, GPIO_PIN_3, GPIO_PIN_RESET);
}

What I have tried:

I have tried to set toggleButton1.getState() != 0 and just play with the code around but nothing helps.

If I use that part of the code without while, I can interact with the button.

Posted 18-Aug-23 0:54am

Member 16074047

Updated 18-Aug-23 7:21am

Deeksha Shenoy

v2

Add a Solution

Comments

jeron1 18-Aug-23 10:09am

Possibly setup a timer that triggers an interrupt at the proper time or have no interrupts and simply poll the timer status bit in your while loop and alter your LED outputs when the timer status flag is set, all the while still being able to poll for a button press?

Member 16074047 21-Aug-23 1:59am

So for example I could (sum) +1 everytime the while goes trough and once it adds up to 10 the while loop stops?

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

OriginalGriff · Answer 1 · 2023-08-18T01:04:00

Solution 1

You have 5 Delay statements in there: each of them probably waits around 1 second - so unless you hold the button down for at least 5 seconds, probably longer, there is no way that the code will see the button being pressed since it reads the current state of an IO pin - which is almost certainly not connected to a latch since there is no sign of a latch reset in your code ...

Posted 18-Aug-23 1:04am

OriginalGriff

Comments

Member 16074047 18-Aug-23 7:11am

Would adding an additional function inside the loop that checks if the button is 1 or 0 help?

OriginalGriff 18-Aug-23 7:32am

Probably not: the check itself takes an unnoticeable amount of time in human terms, and each delay is a second - think about how long it takes you to "press a button" and you'd see what I mean.

What you probably need to do is change the whole way you do things: I have no idea if you have interrupts on that board or how you tie C code to them, but if you do and can, then I'd set up a state machine to decide what to do next and let timer and button interrupts change the state.

Member 16074047 18-Aug-23 7:36am

Ill do some more research about your suggestion. Thank you for your help :)

edit:I can interupt the code anytime by pressing a physical button on my board. But is that the "legit way" to say that the program is working?

OriginalGriff 18-Aug-23 8:22am

No, the interrupts I means are hardware signals that cause code to run when they change: Interrupt - Wikipedia[^]

Member 16074047 18-Aug-23 9:23am

alright, Ill do something else.

Andre Oosthuizen · Answer 2 · 2023-08-18T02:15:00

You need to add a pause within your loop because your current loop is continuously running and not giving the system a chance to process other events, including button presses -

C

void Screen1View::relay_1()
{
    if (toggleButton1.getState()) {
        while (1) {
            //Turn LED1 on...
            HAL_GPIO_WritePin(GPIOD, GPIO_PIN_3, GPIO_PIN_SET);
            HAL_Delay(1000);

            //Turn LED1 off...
            HAL_GPIO_WritePin(GPIOD, GPIO_PIN_3, GPIO_PIN_RESET);
            HAL_Delay(1000);

            //Turn LED2 on...
            HAL_GPIO_WritePin(GPIOJ, GPIO_PIN_2, GPIO_PIN_SET);
            HAL_Delay(1000);

            //Turn LED2 off...
            HAL_GPIO_WritePin(GPIOJ, GPIO_PIN_2, GPIO_PIN_RESET);
            HAL_Delay(1000);

            //Check if the toggle button state has changed...
            if (!toggleButton1.getState()) {
                break; //If so, exit the loop if button is not pressed...
            }
        }
    } else {
        //If the button is not pressed, turn off LED1...
        HAL_GPIO_WritePin(GPIOD, GPIO_PIN_3, GPIO_PIN_RESET);
    }
}