On Mon, Jun 26, 2023 at 10:28:26AM +0800, Guiting Shen wrote: > On Mon, Jun 26, 2023 at 04:04:29AM GMT+8, Alan Stern wrote: > > This comment doesn't say why the code uses !ohci_at91->wakeup. It > > should explain that. For example: > > > > /* > > * According to the comment in ohci_hcd_at91_drv_suspend() > > * we need to do a reset if the 48-MHz clock was stopped, > > * that is, if ohci_at91->wakeup is clear. Tell ohci_resume() > > * to reset in this case by setting its "hibernated" flag. > > */ > > > > Ok, Thank you! > Do I send the v3 patch after the merge window close? You can send the patch at any time. It doesn't matter whether the merge window is open or closed. Alan Stern