Re: [PATCH 2/2] USB/host: Cleaning up the OK macro in uhci-hub.c

2013-10-05 Thread Alan Stern
On Fri, 4 Oct 2013, Deng-Cheng Zhu wrote: > From: Deng-Cheng Zhu > > The logic "len = (x)" in OK(x) is dead. Clean it up. > > Reviewed-by: James Hogan > Signed-off-by: Deng-Cheng Zhu Getting rid of the OK macro is a good idea. But the way you did it is wrong, because of a pre-existing bug

[PATCH 2/2] USB/host: Cleaning up the OK macro in uhci-hub.c

2013-10-04 Thread Deng-Cheng Zhu
From: Deng-Cheng Zhu The logic "len = (x)" in OK(x) is dead. Clean it up. Reviewed-by: James Hogan Signed-off-by: Deng-Cheng Zhu --- drivers/usb/host/uhci-hub.c | 30 ++ 1 files changed, 14 insertions(+), 16 deletions(-) diff --git a/drivers/usb/host/uhci-hub.c