Wow, this is clearly wrong (found in esa_set_params):         if (p->channels < 1 && p->channels > 2)                 return EINVAL; Change the 'and' to an 'or' so the check actually does something.
Wow, this is clearly wrong (found in esa_set_params):
        if (p->channels < 1 && p->channels > 2)
                return EINVAL;
Change the 'and' to an 'or' so the check actually does something.

From jmcneill NetBSD

ok krw@

orumin/openbsd-efivars,orumin/openbsd-efivars,orumin/openbsd-efivars,orumin/openbsd-efivars