On 7/22/24 10:23 PM, Zixun LI wrote:
- Rename atmel_usba_start() / atmel_usba_stop() to usba_udc_enable() / usba_udc_disable(), remove atmel_ prefix to be inline with other functions. Also avoid confusion with DM start() / stop() functions. - Replace printf() by log_err() - Spelling paramter -> parameter
I think I have to push a bit more here, can you please split each bullet point into a separate patch ? I think that'd be ideal, because it seems that each bullet point is one type of change, and one type of change should be in one patch.
