[RESEND v3 PATCH] usb: gadget: bdc: use readl_poll_timeout() to simplify code

2020-07-21 Thread Chunfeng Yun
Use readl_poll_timeout() to poll register status Cc: Florian Fainelli Signed-off-by: Chunfeng Yun Reviewed-by: Florian Fainelli --- v3 changes: 1. indent code to match open parenthesis suggested by Florian 2. add Reviewed-by Florian v2 changes, suggested by Stephen: 1. use unsigned int i

Re: [v3 PATCH] usb: gadget: bdc: use readl_poll_timeout() to simplify code

2020-07-21 Thread Chunfeng Yun
On Tue, 2020-07-21 at 12:42 +0300, Felipe Balbi wrote: > Hi, > > Chunfeng Yun writes: > > Use readl_poll_timeout() to poll register status > > > > Cc: Florian Fainelli > > Signed-off-by: Chunfeng Yun > > Reviewed-by: Florian Fainelli > > I had a lot of trouble to apply this patch, could you a

Re: [v3 PATCH] usb: gadget: bdc: use readl_poll_timeout() to simplify code

2020-07-21 Thread Felipe Balbi
Hi, Chunfeng Yun writes: > Use readl_poll_timeout() to poll register status > > Cc: Florian Fainelli > Signed-off-by: Chunfeng Yun > Reviewed-by: Florian Fainelli I had a lot of trouble to apply this patch, could you avoid base64 encoding on the patch body next time? Thanks -- balbi sign

Re: [v3 PATCH] usb: gadget: bdc: use readl_poll_timeout() to simplify code

2020-07-15 Thread Stephen Boyd
Quoting Chunfeng Yun (2020-07-12 23:48:01) > diff --git a/drivers/usb/gadget/udc/bdc/bdc_core.c > b/drivers/usb/gadget/udc/bdc/bdc_core.c > index 02a3a77..d567e20 100644 > --- a/drivers/usb/gadget/udc/bdc/bdc_core.c > +++ b/drivers/usb/gadget/udc/bdc/bdc_core.c > @@ -29,24 +30,19 @@ > #include "b

Re: [v2 PATCH] usb: gadget: bdc: use readl_poll_timeout() to simplify code

2020-07-12 Thread Chunfeng Yun
On Sun, 2020-07-12 at 20:53 -0700, Florian Fainelli wrote: > > On 7/12/2020 8:30 PM, Chunfeng Yun wrote: > > Use readl_poll_timeout() to poll register status > > > > Cc: Florian Fainelli > > Signed-off-by: Chunfeng Yun > > --- > > v2 changes, suggested by Stephen: > > 1. use unsigned int inst

[v3 PATCH] usb: gadget: bdc: use readl_poll_timeout() to simplify code

2020-07-12 Thread Chunfeng Yun
Use readl_poll_timeout() to poll register status Cc: Florian Fainelli Signed-off-by: Chunfeng Yun Reviewed-by: Florian Fainelli --- v3 changes: 1. indent code to match open parenthesis suggested by Florian 2. add Reviewed-by Florian v2 changes, suggested by Stephen: 1. use unsigned int i

Re: [v2 PATCH] usb: gadget: bdc: use readl_poll_timeout() to simplify code

2020-07-12 Thread Florian Fainelli
On 7/12/2020 8:30 PM, Chunfeng Yun wrote: > Use readl_poll_timeout() to poll register status > > Cc: Florian Fainelli > Signed-off-by: Chunfeng Yun > --- > v2 changes, suggested by Stephen: > 1. use unsigned int instead of int for @usec parameter > 2. add dev_log() back > 3. drop "Err"

[v2 PATCH] usb: gadget: bdc: use readl_poll_timeout() to simplify code

2020-07-12 Thread Chunfeng Yun
Use readl_poll_timeout() to poll register status Cc: Florian Fainelli Signed-off-by: Chunfeng Yun --- v2 changes, suggested by Stephen: 1. use unsigned int instead of int for @usec parameter 2. add dev_log() back 3. drop "Err" in error log --- drivers/usb/gadget/udc/bdc/bdc_core.c | 26 ++

Re: [PATCH] usb: gadget: bdc: use readl_poll_timeout() to simplify code

2020-07-12 Thread Chunfeng Yun
On Fri, 2020-07-10 at 16:37 -0700, Stephen Boyd wrote: > Quoting Chunfeng Yun (2020-07-09 20:30:56) > > diff --git a/drivers/usb/gadget/udc/bdc/bdc_core.c > > b/drivers/usb/gadget/udc/bdc/bdc_core.c > > index 02a3a77..fa173de 100644 > > --- a/drivers/usb/gadget/udc/bdc/bdc_core.c > > +++ b/drivers

Re: [PATCH] usb: gadget: bdc: use readl_poll_timeout() to simplify code

2020-07-10 Thread Stephen Boyd
Quoting Chunfeng Yun (2020-07-09 20:30:56) > diff --git a/drivers/usb/gadget/udc/bdc/bdc_core.c > b/drivers/usb/gadget/udc/bdc/bdc_core.c > index 02a3a77..fa173de 100644 > --- a/drivers/usb/gadget/udc/bdc/bdc_core.c > +++ b/drivers/usb/gadget/udc/bdc/bdc_core.c > @@ -32,21 +33,14 @@ > static int

[PATCH] usb: gadget: bdc: use readl_poll_timeout() to simplify code

2020-07-09 Thread Chunfeng Yun
Use readl_poll_timeout() to poll register status Signed-off-by: Chunfeng Yun --- drivers/usb/gadget/udc/bdc/bdc_core.c | 22 -- 1 file changed, 8 insertions(+), 14 deletions(-) diff --git a/drivers/usb/gadget/udc/bdc/bdc_core.c b/drivers/usb/gadget/udc/bdc/bdc_core.c index