Scott, I don't see ther new PID as an issue. It's a fixed response, like E120_SUPPORTED_PARAMETERS PID and implemented instead of a "get count" PID. Its just that "get count" feels like wallpapering over a crack - the crack is still there.
Eric, I'm only suggesting the SELF_TEST_DESCRIPTION PID is mandatory when the responder supports the new "get test codes" PID, in the same way that
SUPPORTED_PARAMETERS is only required when actually supporting additional PID's. All existing responders are compatible since they won't support "get test codes". Without it, the controller is blind to what the test codes are for. Having said that, I wouldn't worry too much about it being should, just not optional.
|