Re: [PATCH] I2O: Lindent run and replacement of printk through osm printing functions

[Jeff Garzik <jgarzik@pobox.com>]

Linux Kernel Mailing List wrote:
> tree da7e51e7204625f21371eac23a931f4fe479e9db
> parent 9e87545f06930c1d294423a8091d1077e7444a47
> author Markus Lidel <Markus.Lidel@shadowconnect.com> Fri, 24 Jun 2005 12:02:23 -0700
> committer Linus Torvalds <torvalds@ppc970.osdl.org> Fri, 24 Jun 2005 14:05:29 -0700
>
> [PATCH] I2O: Lindent run and replacement of printk through osm printing functions
>
> Lindent run and replaced printk() through the corresponding osm_*() function

Please don't combine ANY code changes with an Lindent patch.

Also, you typically want to do a pass through the post-Lindent code, to 
fix crazy word-wrapped lines like

>  				if (copy_from_user
> -				    (p->virt, (void __user *)(unsigned long)sg[i].addr_bus,
> -				     sg_size)) {
> +				    (p->virt,
> +				     (void __user *)(unsigned long)sg[i].
> +				     addr_bus, sg_size)) {
>  					printk(KERN_DEBUG
>  					       "%s: Could not copy SG buf %d FROM user\n",
>  					       c->name, i);

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/