On 3 February 2017 at 13:25, srishti sharma <srishtishar@xxxxxxxxx> wrote: > okay , Thanks for your reply > > On Sat, Feb 4, 2017 at 2:53 AM, Jim Davis <jim.epost@xxxxxxxxx> wrote: >> On Fri, Feb 3, 2017 at 2:15 PM, srishti sharma <srishtishar@xxxxxxxxx> wrote: >>> I ran perl scripts/checkpatch.pl -f drivers/staging/comedi/* | less >>> for the file comedi_buf.o there is one error , it says >>> ERROR: Does not appear to be a unified-diff format patch Hi Srishti, The argument to checkpatch should be a patch, not the contents of a directory. Please make a patch like so: $ diff -u file.original file.modified > changes.patch $ checkpatch.pl changes.patch Guru Das Srinagesh. >> >> If it's an *.o file then perhaps it's output from a compiler and not a >> text file that checkpatch could usefully look at. >> >> -- >> Jim > > _______________________________________________ > Kernelnewbies mailing list > Kernelnewbies@xxxxxxxxxxxxxxxxx > https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies _______________________________________________ Kernelnewbies mailing list Kernelnewbies@xxxxxxxxxxxxxxxxx https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies