Re: [PATCH V8 02/10] USB: f81232: implement RX bulk-in EP

[Peter Hung]

Hello,

Johan Hovold æ 2015/3/14 äå 07:48 åé:
> On Thu, Feb 26, 2015 at 06:02:08PM +0800, Peter Hung wrote:
> > -	if (!urb->actual_length)
> > +	if ((urb->actual_length < 2) || (urb->actual_length % 2))
> >   		return;
>
> Not parsing short data (e.g. not divisible by 2) is OK I guess. You
> could also just discard the last odd byte, but that's up to you.
>
> Either way, I think you should add a dev_warn here rather than just
> silently drop it.

With F81232, when it first submit with interrupt URB, it will response 
once with 1 bytes data. The data is hw current LSR, but it useless on 
open. It's should necessary with receiving data. When the device is 
working good, it's should acked with even size data.

To avoid confusing to user, I'll keep it without warning message.

Thanks
--
With Best Regards,
Peter Hung
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/