Normal code to register code Arduino UNO

Hello everyone,
I'm trying to optimize my cod as much as possible.
The main problem is at the end, I don't know how to declare digitalWrite(act_pneu,piston_state); in port register. Thank you!

#define sensor_laser_1 2 // pin 2 pentru senzor laser intrare
#define sensor_laser_2 3 // pin 2 pentru senzor laser iesire
#define S0 4 // frecventa scalare
#define S1 5 //frecventa scalare
#define S2 6 // HIGH pentru verde
#define S3 7 // HIGH pentru verde
#define sensorOut1 8 // citire culoare pe pinul 8
#define act_pneu 11 // pin 11 pentru actionare piston

int piston_state = LOW;                   // definire piston stare initiala
unsigned long starTime = 0;               // start arduino
const long interval = 2000;  
bool flag = false;                        // setare interval asteptare dupa 2 sec
int laser1 = digitalRead(sensor_laser_1); // citire senzor laser intrare
int laser2 = digitalRead(sensor_laser_2); // citiare senzor laser iesire
int culoare1 = 0;

void setup() 
  pinMode(sensor_laser_1, INPUT);//define detect input pin
  pinMode(sensor_laser_2, INPUT);//define detect input pin
  pinMode(sensorOut1, INPUT);
  pinMode(act_pneu, OUTPUT);     //define ACTION output pin
  pinMode(S0, OUTPUT);
  pinMode(S1, OUTPUT);
  pinMode(S2, OUTPUT);
  pinMode(S3, OUTPUT);

void loop() 

// Senzor culoare
   culoare1 = pulseIn(sensorOut1, LOW); 
  if(culoare1 <100)
    piston_state= HIGH; // cilindru pe ON
    starTime = millis();
    flag = true;
  if  (flag && millis() - starTime >= interval)
           piston_state = LOW;
           flag = false;

Your post was MOVED to its current location as it is more suitable.

Could you also take a few moments to Learn How To Use The Forum.

Other general help and troubleshooting advice can be found here.
It will help you get the best out of the forum in the future.

What is it not doing, and why do you think it’s not working?

If there are any errors listed, cut & paste them as code in your answer.

There are no errors in code. Just i try to figured out how implementat digitalWrite/ digitalRead with port register configuration. PORTx DDRx and PINx

To do this by direct port addressing is going to save you 4uS at the most. Is it worth it? You seem to be doing it every time round the loop which is wasteful. Try moving it to only execute it when piston_state changes, that will save you a lot more time.

To do this by direct port addressing then find the port and bit position of pin 11. Then read the current port value, apply a mask and store it back.
The mask should be a bit pattern that has a one in the bit position of pin 11 with zeros in all the other bits when you are setting bit 11 high. The mask is applied using an logic OR operation.

The mask should be a bit pattern that has a zero in the bit position of pin 11 with ones in all the other bits when you are setting bit 11 low. The mask is applied using an logic AND operation.

1 Like

Trying to save 6 microseconds on a digitalWrite() won't be much help when pulseIn() can take as long as one second.

1 Like

And I guess it is impossible to change de pulsIn command..

No it is all written in C so you can change it if you want. Perhaps it is better to take the original code, change it and change the name of the function so you have a choice of using it.

Anyway I think you can specify a timeout period in the call, check out the documentation.

1 Like

Thanks for your help guys.

Maybe 'pulseIn()' wasn't the right tool for the job. What is it that you are trying to do? What is producing the pulses? What does it mean when the pulse length is less than 100 microseconds? You have a bunch of input and output pins that you have not used yet so is this just a small part of a larger project?

If you are using a board based on the ATmega328(P), then look at the spec sheet under 13.2.

Is a small project, I’m using a color sensor TCS230 and 2 laser receiver. I want to detect salad with this. I can show you what I want to do. OPTYMA INTER ROW WEEDING HOEING MACHINE BINEUSE SARCHIATRICE INTERFILARE SARCLEUSE HACKMASCHINE - YouTube.

Thank you! But need to be honest and admit that I’m a newbie and still need time to digest all information.

This topic was automatically closed 120 days after the last reply. New replies are no longer allowed.