Re: [i2c] [PATCH] Convert i2c-mpc from a platform driver to an of_platform one

[Jean Delvare <khali@linux-fr.org>]

Hi Jon,

On Wed, 11 Jun 2008 12:14:08 -0400, Jon Smirl wrote:
> On 6/11/08, Wolfram Sang <w.sang@pengutronix.de> wrote:
> > On Tue, Jun 10, 2008 at 10:40:45PM -0400, Jon Smirl wrote:
> >  > Convert i2c-mpc from a platform driver into an of_platform driver.
> >  > This patch is much smaller since Jochen already added
> >  > of_find_i2c_driver(). Versions of this have been posted before.
> >  >
> >  > Signed-ff-by: Jon Smirl <jonsmirl@gmail.com>
> >
> > Typo: Signed-off... (I'm curious, do such typos enforce resending the
> >  patch?)

In theory it should. The line means that you've read the "Developer's
Certificate of Origin" and you agree with it, so it's somewhat binding.
I guess a lawyer would argue that the line isn't worth anything if
"Signed-off-by" isn't spelled properly.

Thanks Wolfram for the review, BTW.

> I just cut and pasted this version to get the comments. Next pass I
> will send it using stgit which will add the right signed-off line and
> fix the wrapping.

That's doing things the wrong way around. If you want people to comment
on your patch, the least you can do is make sure they can apply it.

If I can't apply your patch, that means I can't verify if it applies
properly, I can't run checkpatch.pl on it, I can't get quilt to color
it, and I can't compare it to a previous version of the same patch.

Also, it would be a good idea to keep Jochen Friedrich in the loop
(Cc'd), as he proposed a similar patch based on a previous version of
yours back in April. I suppose he has some interest in it.

While mentioning Jochen's version of the patch: it was deleting 123
lines from arch/powerpc/sysdev/fsl_soc.c. Yours doesn't. Should it?
http://lists.lm-sensors.org/pipermail/i2c/2008-April/003314.html

-- 
Jean Delvare