Fix uart to work with new enum definition in esp-idf-v5.2.1

[luar123]

What does this implement/fix?

In esp-idf-v5.2.1 the definition of uart_port_t changed from typedef int to enum. Therefor setting uart_num_=-1 is not valid anymore.

Types of changes

• Bugfix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Other

Related issue or feature (if applicable): fixes esphome/issues#5672

Pull request in esphome-docs with documentation (if applicable):

Test Environment

• ESP32
• ESP32 IDF
• ESP8266
• RP2040
• BK72xx
• RTL87xx

Checklist:

• The code change is tested and works locally.
• Tests have been added to verify that the new code works (under tests/ folder).

If user exposed functionality or configuration variables are added/changed:

• Documentation added/updated in esphome-docs.

[probot-esphome]

Hey there @esphome/core, mind taking a look at this pull request as it has been labeled with an integration (logger) you are listed as a code owner for? Thanks!
_{^{(message by CodeOwnersMention)}}

[dragonnn]

Hope this get merged soon. I couldn't get wifi to connect on ESP32-C6 on older idf version then v5.2.1. Using this patch from a local build of esphome works perfectly fine now but it is pretty cumbersome to have to run a separate esphome instance outside HAOS for C6

[luar123]

@dragonnn https://esphome.io/components/external_components

[dragonnn]

@dragonnn https://esphome.io/components/external_components

Oh, it means that:

external_components:
    - source: github://luar123/esphome@fix_logger
      components: [ logger ]

Would allow me to use regular HA ESPHOME and have the patched logger from your fix?
Didn't find about it, sounds perfect until that gets merged. Thanks!

[jesserockz]

@dragonnn

Hope this get merged soon. I couldn't get wifi to connect on ESP32-C6 on older idf version then v5.2.1. Using this patch from a local build of esphome works perfectly fine now but it is pretty cumbersome to have to run a separate esphome instance outside HAOS for C6

While we will get to this at some point. There is no rush as esp-idf 5 is not recommended or supported, and neither is the c6. We merge future focused PRs like this when we get time to validate and test them but they are lower priority than others.