OpenEtherCATsociety / SOES

Simple Open Source EtherCAT Slave
Other
595 stars 256 forks source link

Cleanup hal alevent #14

Closed sittner closed 8 years ago

sittner commented 8 years ago

Removed ALevents from every HAL ESC_read/ESC_write call and use ESCvar.ALevent in HAL.

nakarlsson commented 8 years ago

What do you say @ArthurKetels? I'm for removing it in the current implementation.

ArthurKetels commented 8 years ago

Also fine with me. There is a reason the ALevents was in ESC_read/ESC_write. I have a private repo with a multi-instance SOES. For an embedded target that has two SOES stacks running next to each other. But I am planning to implement that in a different more logical way. So the ALevents can move to the HAL.

Thanks for the patch.