fix: HWCDC removed clearing USB Serial/JTAG interrupts during begin (#9316)

[kiran-desilva]

Description of Change

Fixes (#9316) where ESP may not respond over the USB Serial/JTAG interface if bytes are sent before HWCDC::begin(...) is called. Clearing of the interrupt status has been removed from HWCDC::begin(...) so that if the USB_SERIAL_JTAG_INTR_SERIAL_OUT_RECV_PKT is already set, the isr handler can process the interrupt status correctly.

Tests scenarios

Tested with ESP32-S3:

#include <Arduino.h>
#include <HWCDC.h>
#include <hal/usb_serial_jtag_ll.h>
#include <string>


uint32_t prevTime = 0;


void setup() {
  
  delay(5000);

  Serial.begin(115200);
  Serial.println("begin");
}

void loop() {

  if (millis() - prevTime > 500)
  {
    prevTime = millis();
    std::string output;
    output += "rxFifo: " + std::to_string(usb_serial_jtag_ll_rxfifo_data_available());
    output += " Serial avaliable: " + std::to_string(Serial.available());
    Serial.println(output.c_str());

  }

  while (Serial.available())
  {
    uint8_t b = Serial.read();
    Serial.print(Serial.available());
    Serial.print(" - ");
    Serial.write(b);
    Serial.println();
  }
  vTaskDelay(1); // watchdog
}

Related links

Closes (#9316)

[CLAassistant]

All committers have signed the CLA.

[P-R-O-C-H-Y]

@kiran-desilva Please use release/v2.x. branch as a base for the PR. Thanks

[SuGlider]

@kiran-desilva - We have another problem now... the rebasing added a thousand changed files.
Can you fix it? If necessary, you can close it and open a new PR directly forked from "release/v2.x".

[kiran-desilva]

@kiran-desilva - We have another problem now... the rebasing added a thousand changed files.
Can you fix it? If necessary, you can close it and open a new PR directly forked from "release/v2.x".

Yep I'll close and re-open a new pr.