[E3-hacking] [PATCH v2 1/6] SoC Camera: add driver for OMAP1 camera interface

hermann pitton hermann-pitton at arcor.de
Thu Sep 23 00:31:15 BST 2010


Am Mittwoch, den 22.09.2010, 08:08 +0200 schrieb Guennadi Liakhovetski:
> On Wed, 22 Sep 2010, hermann pitton wrote:
> 
> > Am Mittwoch, den 22.09.2010, 01:23 +0200 schrieb Guennadi Liakhovetski:
> > > On Sat, 11 Sep 2010, Janusz Krzysztofik wrote:
> > > 
> > > > This is a V4L2 driver for TI OMAP1 SoC camera interface.
> > 
> > [snip]
> > 
> > > > +
> > > > +	} else {
> > > > +		dev_warn(dev, "%s: unhandled camera interrupt, status == "
> > > > +				"0x%0x\n", __func__, it_status);
> > > 
> > > Please, don't split strings
> > 
> > sorry for any OT interference.
> > 
> > But, are there any new rules out?
> > 
> > Maybe I missed them.
> > 
> > Either way, the above was forced during the last three years.
> > 
> > Not at all ?
> 
> No. Splitting print strings has always been discouraged, because it makes 
> tracking back kernel logs difficult. The reason for this has been the 80 
> character line length limit, which has now been effectively lifted. I'm 
> sure you can find enough links on the internet for any of these topics.
> 
> Thanks
> Guennadi
> ---
> Guennadi Liakhovetski, Ph.D.
> Freelance Open-Source Software Developer
> http://www.open-technology.de/

Guennadi,

thanks for the update!

If somebody ever would like to waste time on it, lots of patches and
whole drivers have been forced into this limitations for strings too.

In fact, fixing only a few lines, including the offset, you almost
always did hit it.

I'm pleased to hear now, that this problem never did exist ;))

Cheers,
Hermann










More information about the e3-hacking mailing list