On 8/7/07, Joe Perches <[EMAIL PROTECTED]> wrote: > Found these while looking at printk uses. > > Add missing newlines to dev_<level> uses > Add missing KERN_<level> prefixes to multiline dev_<level>s > Fixed a wierd->weird spelling typo > Added a newline to a printk
I think these should have been split logically into 4 groups of patches (except perhaps where they touched the same file for the first two and the last). The third type should go through trivial. And they should have been split logically by subsystem, most likely? <snip diffstat> > diff --git a/arch/ia64/sn/kernel/xpnet.c b/arch/ia64/sn/kernel/xpnet.c > index e58fcad..a5df672 100644 > --- a/arch/ia64/sn/kernel/xpnet.c > +++ b/arch/ia64/sn/kernel/xpnet.c > @@ -269,8 +269,9 @@ xpnet_receive(partid_t partid, int channel, struct > xpnet_message *msg) > skb->protocol = eth_type_trans(skb, xpnet_device); > skb->ip_summed = CHECKSUM_UNNECESSARY; > > - dev_dbg(xpnet, "passing skb to network layer; \n\tskb->head=0x%p " > - "skb->data=0x%p skb->tail=0x%p skb->end=0x%p skb->len=%d\n", > + dev_dbg(xpnet, "passing skb to network layer\n" > + KERN_DEBUG "\tskb->head=0x%p skb->data=0x%p skb->tail=0x%p " > + "skb->end=0x%p skb->len=%d\n", > (void *)skb->head, (void *)skb->data, skb_tail_pointer(skb), > skb_end_pointer(skb), skb->len); <snip> I'm not a maintainer of any code by any means, but it seems odd, redundant and visually confusing to me to use the dev_dbg() function and have KERN_DEBUG's inside the string -- makes me question every time why there isn't one for the first line too. Perhaps these multi-line debug statements should simply be split into multiple dev_dbg() calls? Similar for the other dev_ functions. I think it might look cleaner, at least. Just my two cents, not a NAK or anything. Thanks, Nish - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
