Skip to content

Add LG Korean WashTower washer and dryer - #117

Open
tunarabuuu wants to merge 1 commit into
anszom:masterfrom
tunarabuuu:dev-washtower
Open

Add LG Korean WashTower washer and dryer#117
tunarabuuu wants to merge 1 commit into
anszom:masterfrom
tunarabuuu:dev-washtower

Conversation

@tunarabuuu

Copy link
Copy Markdown
Contributor

Fifth of the per-device-class splits from #111.

FAKPK21021 (washer, deviceType 221) and BDH_D39301_KR (dryer, 222) — LG's Korean WashTower, speaking a dedicated 0xEB/0xEC extended-frame protocol (not AA..BB or clip/TLV). Complementary to the NA WashTower in #102, which is a different generation with a different frame format.

Addressing your review note from #111:

  • dropped washtower_common — the washer and dryer share every bit of the decode/control code and differ only in their field/course tables, so both now live in one washtower.ts (two small constructors over a shared class) instead of a _common module plus two wrapper files. I kept both models because a washer's courses and a dryer's genuinely differ — the two tables aren't interchangeable — but there's now a single implementation file.
  • "how was this table created" — added a note: the field maps come from LG's own protocol/convert maps for each model, cross-checked against the appliance's capability dump; a field is included only where both agree.

All labels are English. npm test passes; tsc clean.

FAKPK21021 (washer, deviceType 221) and BDH_D39301_KR (dryer, 222) speak a
dedicated 0xEB/0xEC extended-frame protocol. They share every bit of the
decode and control logic and differ only in their field/course tables, so both
live in one washtower.ts — the washer and dryer classes are two small
constructors over a shared WashTowerDevice, rather than a separate _common
module plus two wrapper files.

The tables are derived from LG's own protocol/convert maps for each model,
cross-checked against the appliance's capability dump.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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.

1 participant