You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Because enabling the dynamic payloads feature on pipe 1 is not needed to transmit ACK payloads, the configuration of pipe 1 should be left up to the user in this context. Of course, the examples should demonstrate proper pipe usage rather than letting the library hide (& probably confuse people about) the requirement to enable the dynamic payloads feature for pipes using ACK payloads in RX mode.
The following line of code from
enableAckPayload()
is a bit erroneous.write_register(DYNPD, read_register(DYNPD) | _BV(DPL_P1) | _BV(DPL_P0))
should be changed to
write_register(DYNPD, read_register(DYNPD) | _BV(DPL_P0))
Because enabling the dynamic payloads feature on pipe 1 is not needed to transmit ACK payloads, the configuration of pipe 1 should be left up to the user in this context. Of course, the examples should demonstrate proper pipe usage rather than letting the library hide (& probably confuse people about) the requirement to enable the dynamic payloads feature for pipes using ACK payloads in RX mode.
Examples/tests affected:
The note about this line of code in the docs should also be amended accordingly.
The text was updated successfully, but these errors were encountered: