Hi, On Thu, Apr 30, 2015 at 11:32:50AM +0200, Greg Kroah-Hartman wrote: > We should not be doing assignments within an if () block > so fix up the code to not do this. > > change was created using Coccinelle. > > CC: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> > Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> For the entire series: Reviewed-by: Felipe Balbi <balbi@xxxxxx> > --- > drivers/usb/host/ehci-dbg.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/usb/host/ehci-dbg.c b/drivers/usb/host/ehci-dbg.c > index 524cbf26d992..b26b96e25a13 100644 > --- a/drivers/usb/host/ehci-dbg.c > +++ b/drivers/usb/host/ehci-dbg.c > @@ -628,7 +628,8 @@ static ssize_t fill_periodic_buffer(struct debug_buffer *buf) > unsigned i; > __hc32 tag; > > - if (!(seen = kmalloc (DBG_SCHED_LIMIT * sizeof *seen, GFP_ATOMIC))) > + seen = kmalloc(DBG_SCHED_LIMIT * sizeof *seen, GFP_ATOMIC); > + if (!seen) > return 0; > seen_count = 0; > > -- > 2.3.7 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-usb" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- balbi
Attachment:
signature.asc
Description: Digital signature
