Should this function verify that the specified channel is disabled before 
incrementing `m_cb.active_channels`?  I don't know how important this is, but I 
noticed that the nrfx code performs this check:
```
    if (m_cb.psel[channel].pselp == NRF_SAADC_INPUT_DISABLED) 
    {
        ++m_cb.active_channels;
    }
```

A similar comment applies to the disable case.

[ Full content available at: https://github.com/apache/mynewt-core/pull/1432 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to