Skip to content

[WWSTCERT-12195, WWSTCERT-12199, WWSTCERT-12203, WWSTCERT-12207] fingerprints for Sonoff valves#3027

Open
KKlimczukS wants to merge 1 commit into
mainfrom
fingerprints_for_sonoff_valves
Open

[WWSTCERT-12195, WWSTCERT-12199, WWSTCERT-12203, WWSTCERT-12207] fingerprints for Sonoff valves#3027
KKlimczukS wants to merge 1 commit into
mainfrom
fingerprints_for_sonoff_valves

Conversation

@KKlimczukS

Copy link
Copy Markdown
Contributor

Check all that apply

Type of Change

  • WWST Certification Request
    • If this is your first time contributing code:
      • I have reviewed the README.md file
      • I have reviewed the CODE_OF_CONDUCT.md file
      • I have signed the CLA
    • I plan on entering a WWST Certification Request or have entered a request through the WWST Certification console at developer.smartthings.com
  • Bug fix
  • New feature
  • Refactor
  • Fingerprints

Checklist

  • I have performed a self-review of my code
  • I have commented my code in hard-to-understand areas
  • I have verified my changes by testing with a device or have communicated a plan for testing
  • I am adding new behavior, such as adding a sub-driver, and have added and run new unit tests to cover the new behavior

Description of Change

WWSTCERT-12195, WWSTCERT-12199, WWSTCERT-12203, WWSTCERT-12207
New fingerprints for Sonoff valves

Summary of Completed Tests

@github-actions

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown

Test Results

   73 files    515 suites   0s ⏱️
2 904 tests 2 904 ✅ 0 💤 0 ❌
4 800 runs  4 800 ✅ 0 💤 0 ❌

Results for commit 5dcb4c7.

@github-actions

Copy link
Copy Markdown

File Coverage
All files 97%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-valve/src/lazy_load_subdriver.lua 57%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-valve/src/sinope/init.lua 97%

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against 5dcb4c7

@ldeora

ldeora commented Jul 16, 2026

Copy link
Copy Markdown

Which firmware versions were used for certification, and was the valve tested while remaining continuously open for more than ten minutes? If so, how was manual_default_settings configured?

Relevant thread:

https://community.smartthings.com/t/st-edge-sonoff-hydro-one-duo-family-swv-zfe-swv-zfu-swv-zne-swv-znu-swv-zf2e-swv-zf2u/309672/66

@ldeora

ldeora commented Jul 18, 2026

Copy link
Copy Markdown

Following up on my question above, I would like to explain why the firmware version and the >10-minute test are important.

This PR currently adds fingerprints for SWV-ZNU, SWV-ZNE, SWV-ZFU and SWV-ZFE to the generic valve-battery-powerSource profile. That may provide basic pairing and On/Off control, but these devices are not simply generic Zigbee On/Off valves.

Independent hardware testing has now confirmed the following for current SWV-ZFE/SWV-ZFU firmware 0x00001007 / 1.0.7:

  • Every normal watering start — including the physical button and plain Zigbee On — uses the configuration stored in private cluster 0xFC11, attribute 0x501D (manualDefaultSettings).
  • The factory configuration normally limits such sessions to approximately ten minutes.
  • OnWithTimedOff, on_time and off_wait_time are unsupported or ignored on this firmware.
  • A different duration requires writing the complete 12-byte Array<Uint8> configuration to 0x501D, followed by a normal Zigbee On.
  • Read responses for this attribute are malformed by the device firmware (48 48 0C ... instead of 48 20 0C ...) and need special handling.

The full Hydro ONE variants also expose flow measurements, irrigation statistics and several private alarm and safety settings which the generic valve driver does not handle.

Therefore, fingerprints alone appear to provide only partial compatibility: the valve may pair and open, but it may still close automatically after about ten minutes, and significant device functionality will remain unavailable.

Could you clarify the intended scope of the certification? If this is deliberately limited to basic valve control, I think the ten-minute limitation and missing device-specific features should at least be documented. If complete product support is intended, these models likely need a SONOFF-specific subdriver or confirmed firmware behavior that provides equivalent standard Zigbee functionality.

Relevant independent hardware investigations:

zigpy/zha-device-handlers#4927

Koenkk/zigbee2mqtt#32479

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants