> 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.