On Wed, Aug 19, 2020 at 02:54:39PM +0300, Yehezkel Bernat wrote: > > - * @disabled: Disabled by eeprom > > + * @disabled: Disabled by eeprom or enabled, but not implemented > > I'm not a native speaker, so I'm not sure about it, but maybe the comma here is > superfluous and just confuses the reader. To me it looks like it means > "(disabled > || enabled) && !implemented" instead of "disabled || (enabled && !implemented)". > Any opinion? For me (also non-native speaker) I don't see a difference but sure I can remove it :)