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.
