From 8b66920165f1c2332e8074e7a19a6d86edf14811 Mon Sep 17 00:00:00 2001
From: aurelienhess <38730075+aurelienhess@users.noreply.github.com>
Date: Tue, 7 Jan 2025 18:13:22 +0100
Subject: [PATCH] Update Somfy_Remote.ino

Changed PORT_TX Low operations to bitwise OR instead of logical OR
---
 Somfy_Remote.ino | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/Somfy_Remote.ino b/Somfy_Remote.ino
index d57a506..d70eae1 100644
--- a/Somfy_Remote.ino
+++ b/Somfy_Remote.ino
@@ -42,7 +42,7 @@ void SendCommand(byte *frame, byte sync);
 void setup() {
   Serial.begin(115200);
   DDRD |= 1<<PORT_TX; // Pin 5 an output
-  PORTD &= !(1<<PORT_TX); // Pin 5 LOW
+  PORTD &= ~(1<<PORT_TX); // Pin 5 LOW
 
   if (EEPROM.get(EEPROM_ADDRESS, rollingCode) < newRollingCode) {
     EEPROM.put(EEPROM_ADDRESS, newRollingCode);
@@ -148,24 +148,24 @@ void BuildFrame(byte *frame, byte button) {
 void SendCommand(byte *frame, byte sync) {
   if(sync == 2) { // Only with the first frame.
   //Wake-up pulse & Silence
-    PORTD |= 1<<PORT_TX;
-    delayMicroseconds(9415);
-    PORTD &= !(1<<PORT_TX);
-    delayMicroseconds(89565);
+    PORTD |= 1<<PORT_TX; // Set PORT_TX High
+    delayMicroseconds(9415); 
+    PORTD &= ~(1<<PORT_TX); // Set PORT_TX Low 
+    delayMicroseconds(89565); 
   }
 
 // Hardware sync: two sync for the first frame, seven for the following ones.
   for (int i = 0; i < sync; i++) {
-    PORTD |= 1<<PORT_TX;
-    delayMicroseconds(4*SYMBOL);
-    PORTD &= !(1<<PORT_TX);
+    PORTD |= 1<<PORT_TX; // Set PORT_TX High
+    delayMicroseconds(4*SYMBOL); 
+    PORTD &= ~(1<<PORT_TX); // Set PORT_TX Low 
     delayMicroseconds(4*SYMBOL);
   }
 
 // Software sync
   PORTD |= 1<<PORT_TX;
   delayMicroseconds(4550);
-  PORTD &= !(1<<PORT_TX);
+  PORTD &= ~(1<<PORT_TX);
   delayMicroseconds(SYMBOL);
   
   
@@ -185,6 +185,6 @@ void SendCommand(byte *frame, byte sync) {
     }
   }
   
-  PORTD &= !(1<<PORT_TX);
+  PORTD &= ~(1<<PORT_TX);
   delayMicroseconds(30415); // Inter-frame silence
 }