Bug: Unwanted change resistance in x9c component

[fizista]

What does this implement/fix?

There is an unwanted change in the resistance value.

Just call set_level(20) several times. The resistance will change.

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 n/a

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

Test Environment

• ESP32
• ESP32 IDF
• ESP8266
• RP2040

Example entry for config.yaml:

switch:
  - platform: template
    id: test_repeat
    name: "X9C Const value 20"
    restore_mode: ALWAYS_OFF
    optimistic: True
    on_turn_on:
      - output.set_level: 
          id: x9c_pot
          level: 20
    on_turn_off:
      - output.set_level: 
          id: x9c_pot
          level: 20

output:
  - platform: x9c
    id: x9c_pot
    cs_pin: GPIO15
    inc_pin: GPIO4
    ud_pin: GPIO2
    initial_value: 0.01
    inverted: True

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 @EtienneMD, mind taking a look at this pull request as it has been labeled with an integration (x9c) you are listed as a code owner for? Thanks!
_{^{(message by CodeOwnersMention)}}

[EtienneMD]

I see what you are saying, chip should not be selected with INC pin in High state. It never is thought as the pin is selected high prior to CS pin being lowered. Any advantage in leaving it high all the time when not in use?

[fizista]

You're right. The high state on INC is not needed.