> Does this look better?

> Index: arch/arm/xscale/pxa27x_udc.c

>       if (xfer != NULL)
>               SIMPLEQ_REMOVE_HEAD(&sc->sc_free_xfers, next);
> -     else
> -             xfer = malloc(sizeof(struct pxaudc_xfer), M_USB, M_NOWAIT);
> -     if (xfer != NULL)
>               bzero(xfer, sizeof(struct pxaudc_xfer));
> +     else
> +             xfer = malloc(sizeof(struct pxaudc_xfer), M_USB,
> +                     M_NOWAIT | M_ZERO);

No, until you add a pair of { } braces.

Reply via email to