Commit fc59ed70 authored by Tomasz Moń's avatar Tomasz Moń Committed by Carles Cufi
Browse files

nrf_usbd_common: Disable started interrupt handling 



There is no point in calling an empty function on every single DMA
transfer start. While the empty function is just BX LR the fact that is
is called on every DMA transfer makes its impact visible. This change
improves CDC ACM echo throughput by approximately 2%.

Signed-off-by: default avatarTomasz Moń <tomasz.mon@nordicsemi.no>
parent 0efdff96

drivers/usb/common/nrf_usbd_common/nrf_usbd_common.c

+1 −10
Original line number Diff line number Diff line
@@ -18,10 +18,6 @@ 
#include <zephyr/logging/log.h>

LOG_MODULE_REGISTER(nrf_usbd_common, CONFIG_NRF_USBD_COMMON_LOG_LEVEL);



#ifndef NRF_USBD_COMMON_STARTED_EV_ENABLE

#define NRF_USBD_COMMON_STARTED_EV_ENABLE 0

#endif



#ifndef NRF_USBD_COMMON_ISO_DEBUG

/* Also generate information about ISOCHRONOUS events and transfers.

 * Turn this off if no ISOCHRONOUS transfers are going to be debugged and this
@@ -806,11 +802,6 @@ static void ev_usbreset_handler(void) 


static void ev_started_handler(void)

{

#if NRF_USBD_COMMON_STARTED_EV_ENABLE

	/* Handler not used by the stack.

	 * May be used for debugging.

	 */

#endif

}



/**
@@ -1716,7 +1707,7 @@ void nrf_usbd_common_start(bool enable_sof) 
	__ASSERT_NO_MSG(m_drv_state == NRFX_DRV_STATE_POWERED_ON);

	m_bus_suspend = false;



	uint32_t ints_to_enable = NRF_USBD_INT_USBRESET_MASK | NRF_USBD_INT_STARTED_MASK |

	uint32_t ints_to_enable = NRF_USBD_INT_USBRESET_MASK |

				  NRF_USBD_INT_ENDEPIN0_MASK | NRF_USBD_INT_EP0DATADONE_MASK |

				  NRF_USBD_INT_ENDEPOUT0_MASK | NRF_USBD_INT_USBEVENT_MASK |

				  NRF_USBD_INT_EP0SETUP_MASK | NRF_USBD_INT_DATAEP_MASK;

CRA Git | Maintained and supported by SUSTech CRA and CCSE