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