Skip to content

WAGO device support - #40

Draft
Bra7wurs7 wants to merge 4 commits into
mainfrom
671_672_idents_and_mod
Draft

WAGO device support#40
Bra7wurs7 wants to merge 4 commits into
mainfrom
671_672_idents_and_mod

Conversation

@Bra7wurs7

@Bra7wurs7 Bra7wurs7 commented May 22, 2026

Copy link
Copy Markdown
Contributor

Ported device drivers for 750_553 from control, updated module registrations and added imports for 671 and 672 module idents. Also made slot_devices an Rc and fixed surrounding plumbing

@Bra7wurs7
Bra7wurs7 marked this pull request as draft May 28, 2026 11:58
@Bra7wurs7 Bra7wurs7 changed the title Ported device drivers for 750_553 from control, updated module registrations and added imports for 671 and 672 module idents. 750_354 now uses Rc<RefCell> for subdevices and get_pdo_offsets should work more reliably, some drivers ported, some registrations added Jun 8, 2026
@Bra7wurs7
Bra7wurs7 marked this pull request as ready for review June 8, 2026 14:07
@Bra7wurs7
Bra7wurs7 requested a review from kraemr June 13, 2026 16:07
@Bra7wurs7
Bra7wurs7 requested a review from TheBest6337 June 21, 2026 10:11
@Bra7wurs7 Bra7wurs7 changed the title 750_354 now uses Rc<RefCell> for subdevices and get_pdo_offsets should work more reliably, some drivers ported, some registrations added WAGO device support Jul 17, 2026
Comment on lines +337 to +368
fn pdo_object_steps(product_id: u32, is_tx: bool) -> u32 {
match product_id {
// 4-channel analog input: 4 separate TX objects, no RX
WAGO_750_455_PRODUCT_ID => {
if is_tx {
4
} else {
1
}
}
// 4-channel analog RTD input: 4 separate TX objects, no RX
WAGO_750_460_PRODUCT_ID => {
if is_tx {
4
} else {
1
}
}
// 2-channel analog input: 2 separate TX objects, no RX
WAGO_750_402_PRODUCT_ID => {
if is_tx {
2
} else {
1
}
}
// 4-channel analog output: 4 separate objects in both directions
WAGO_750_553_PRODUCT_ID => 4,
// All digital/byte-based modules: 1 object per direction
_ => 1,
}
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this needed?

@TheBest6337

Copy link
Copy Markdown
Member

@Snacj this can be closed now, with your PR merge?

@Snacj

Snacj commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

@TheBest6337 I'd leave this open for now because im not sure about the Stepper Drivers (750 671 + 672) If they work in qitech_lib. My PR was about the wago coupler not working.

@TheBest6337
TheBest6337 marked this pull request as draft August 19, 2026 11:40
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.

3 participants