Re: [PATCH v4] Add support for M5MO-LS 8 Mega Pixel camera

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi HeungJun,

On 02/07/2011 12:44 PM, Heungjun Kim wrote:
> Add I2C/V4L2 subdev driver for M5MO-LS camera sensor with integrated
> image processor.
> 

There is something wrong with this patch. It looks like it got mangled by
the mailer. I can see some Korean characters in it and checkpatch.pl 
returns errors:

ERROR: patch seems to be corrupt (line wrapped?)
#122: FILE: drivers/media/video/Kconfig:747:
=20

ERROR: spaces required around that '=' (ctx:WxV)
#208: FILE: drivers/media/video/m5mols/m5mols.h:36:
+	I2C_8BIT	=3D 1,
 	        	^

ERROR: spaces required around that '=' (ctx:WxV)
#209: FILE: drivers/media/video/m5mols/m5mols.h:37:
+	I2C_16BIT	=3D 2,
 	         	^

ERROR: spaces required around that '=' (ctx:WxV)
#210: FILE: drivers/media/video/m5mols/m5mols.h:38:
+	I2C_32BIT	=3D 4,
 	         	^
...

ERROR: spaces required around that '=' (ctx:WxV)
#1500: FILE: drivers/media/video/m5mols/m5mols_core.c:892:
+	.remove		=3D m5mols_remove,
 	       		^

ERROR: spaces required around that '=' (ctx:WxV)
#1501: FILE: drivers/media/video/m5mols/m5mols_core.c:893:
+	.id_table	=3D m5mols_id,
 	         	^

WARNING: please, no space before tabs
#1672: FILE: include/media/m5mols.h:23:
+* ^I^Ito be called after enabling and before disabling$

total: 344 errors, 6 warnings, 1514 lines checked


Regards,
Sylwester
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux