Jia-Ju Bai <baijiaju1990@xxxxxxxxx> wrote: > zd1201_probe() is never called in atomic context. > > zd1201_probe() is only set as ".probe" in struct usb_driver. > > Despite never getting called from atomic context, zd1201_probe() > calls mdelay() to busily wait. > This is not necessary and can be replaced with msleep() to > avoid busy waiting. > > This is found by a static analysis tool named DCNS written by myself. > And I also manually check it. > > Signed-off-by: Jia-Ju Bai <baijiaju1990@xxxxxxxxx> I need a review from someone else before I'm willing to take this. -- https://patchwork.kernel.org/patch/10333189/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches