On Friday 06 April 2007 03:35, Will Dyson wrote: > On 4/5/07, Michael Buesch <[EMAIL PROTECTED]> wrote: > > On Thursday 05 April 2007 06:03, Will Dyson wrote: > > > @@ -1052,8 +1059,10 @@ static void dma_tx_fragment(struct bcm43xx_dmaring > > > *ring, > > > > > > memcpy(&meta->txstat.control, ctl, sizeof(*ctl)); > > > meta->skb = skb; > > > - meta->dmaaddr = map_descbuffer(ring, skb->data, skb->len, 1); > > > meta->is_last_fragment = 1; > > > + meta->dmaaddr = map_descbuffer(ring, skb->data, skb->len, 1); > > > + if(dma_mapping_error(meta->dmaaddr)) > > > > Codingstyle. Space between if and ( > > Sigh. I did try ./scripts/Lindent, by the way. Pretty useless. It > tries to change almost every line.
Yeah, lindent is useless when patching existing code. > Is there a set of indent args that will actually help with this, > without generating so much noise that I can't see what I messed up? I'm sorry, I don't know. -- Greetings Michael. _______________________________________________ Bcm43xx-dev mailing list [email protected] https://lists.berlios.de/mailman/listinfo/bcm43xx-dev
