Code review comment for lp:~dirk.zimoch/epics-base/fix-aai-and-aao

Revision history for this message
Andrew Johnson (anj) wrote :

Hi Dirk,

When SIMM=YES, the aai record is put into alarm using the SIMS field for the severity. The aao record always goes to INVALID_ALARM in simulation mode.

Do you know under what circumstances a device support might want to set BPTR itself? The comment about calling the pdset->init_record() in pass 0 implies that run-time changes of the device address wouldn't be possible in that case (which is in the device support's power to allow if it uses devExtend(), but prevented if it doesn't).

Have you ever seen any written documentation for the aai/aao record types? There are no chapters for them in the Record Reference wiki yet.

I haven't started actually testing this code yet (that's my next task), but so far I think this work should be merged into R3.14.12.

- Andrew

« Back to merge proposal