Marek Vasut [mailto:ma...@denx.de]
> Sent: Friday, May 22, 2020 9:22 PM
[...]
> > - if ((ep_addr & USB_DIR_IN) && !ep_in_found) {
> > - ss->ep_in = ep_addr &
> > - USB_ENDPOINT_NUMBER_MASK;
> > - ep_in_found = 1;
> > +
> > + if (ep_addr & USB_DIR_IN) {
> > + if (!ep_in_found) {
> > + ss->ep_in = ep_addr &
> > + USB_ENDPOINT_NUMBER_MASK;
> > + ep_in_found = 1;
> > + }
>
> So why don't you rework the code this way instead, to make it easier to
> understand:
Ok, I'll do it and resend this patch. Thanks.
> if ((ep_addr & USB_DIR_IN) && !ep_in_found) {
> ... do in stuff ...
> }
>
> if ((ep_addr & USB_DIR_OUT) && !ep_out_found) {
> ... do out stuff ...
> }
>
> Would that work ?
Yes.
Best Regards,
Hayes
