On Sun, Nov 17, 2013 at 11:45 PM, Joe Perches <j...@perches.com> wrote: > On Sun, 2013-11-17 at 20:34 -0200, Erico Nunes wrote: >> Do you mean it as an error in the sparse tool? > > Yes. I think it's a defect in how sparse > treats string concatenation. > > That style: > > printk("%s\n", > #ifdef FOO > "foo" > #endif > #ifdef BAR > "bar" > #endif > "string"); > > is pretty common in the kernel sources. > > The patch itself is otherwise fine, but > perhaps unnecessary.
I agree with Joe, the patch fixes a non-issue. -- Thanks, //richard -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/