On Thu, Aug 20, 2020 at 05:31:08PM +0530, Nikunj A. Dadhania wrote: > On 8/19/2020 6:15 PM, Mika Westerberg wrote: > > 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 :) > > > > I meant the second - "disabled || (enabled && !implemented)" > (also non-native speaker). If the comma confuses the reader please remove > it. Removed comma from the comment and applied to fixes, thanks!
