In the datasheet, it is mentioned that bits labeled as T must always be written with logic 0. The unused bit is labeled as T so I am checking it all the time.
In general I'm not a fan of having parameter name when passing a value to it. I know some languages like old lady VisualBasic had it, but it's confusing and also give the ability to reorder the functions parameters when passing values which I believe is not good and is making the code hard to understand and read. As you used a very good naming for the values it shows which parameter is used.
Resolving the case in not enough, you also need to reply to Behrouz's last comment and mention the word "RESOLVED", otherwise, Dara's Crucible script won't pick this code review up. We will review this next week during the training.
This is just an example test script using the HD library. I imagined a tester wanting to set a specific interval and I don't imagine testers will be creating constants or enums for their code.
