On Tue, Aug 26, 2014 at 12:33:28PM +0200, Takashi Iwai wrote:
> At Tue, 26 Aug 2014 15:43:24 +0530,
> Sudip Mukherjee wrote:
> >
> > printk replaced with corresponding pr_err and pr_alert
> >
> > Signed-off-by: Sudip Mukherjee <su...@vectorindia.org>
> > ---
> >
> > Initially started by copy-paste but as Takashi suggested it was done by a
> > script.
> > bash script is as follows :
> >
> > OLD="printk(KERN_ERR \?"
> > OLD1="printk(KERN_ALERT \?"
> > NEW="pr_err("
> > NEW1="pr_alert("
> > TFILE="/tmp/out.tmp.$$"
> > for f in *.c
> > do
> > sed "s/$OLD/$NEW/g" "$f" > $TFILE && mv $TFILE "$f"
> > sed "s/$OLD1/$NEW1/g" "$f" > $TFILE && mv $TFILE "$f"
> > done
>
> All these can be a single call of sed :)
is it? I will try to see ..
>
> > this patch will generate a warning from checkpatch for an unnecessary space
> > before new line and has not been fixed as this patch is only for printk
> > replacement.
>
> Better to put such information into the changelog.
>
> Do you have the patch reformatting the broken lines, too?
> If yes, I'll wait for it, as we should apply both such patches at
> once.
>
I will send you the broken string reformatting patch .
thanks
sudip
>
> Takashi
--
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/
