On Tue, May 29, 2018 at 05:35:12PM +0000, Verkamp, Daniel wrote: > Sorry, I should have been clearer - I agree with your position here. Only bits that are valid should be allowed, so for example it is fine to fail commands that set reserved bits, or optional bits that have a mechanism to indicate they are not supported, like Telemetry (which has an associated bit in Identify controller data - LPA). My note above was really just about the health warning bits, which by my reading are not optional. Yes, I think your reading is right and we should not fail those bits. I'll fix it for the next version.
