<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 25, 2020 at 9:38 AM Tomek The Messenger <<a href="mailto:tomekthemessenger@gmail.com">tomekthemessenger@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi<div>There is <a href="http://checkpatch.pl" target="_blank">checkpatch.pl</a> script where You can check if You wrote code in your kernel module according to linux kernel style. </div><div>However can I ignore warning message?</div><div>WARNING: quoted string split across lines<br>#974: FILE: fpgax67-core.c:974:<br>+               dev_err(&pdev->dev, "registration not done, driver is already "<br>+                                                       "registered\n");<br></div><div><br></div><div>If I don't split line I will have another warning that 80 characters is exceeded.</div><div><br></div></div></blockquote><div>you can put the whole string on next line and/or use "\" for splitting long string.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div></div><div>For sure I can ignore warnings about:</div><div>WARNING: struct  should normally be const<br>#998: FILE: 

fpgax67-core.c

:998:<br>+int 

fpgax67_unregister(struct platform_device *pdev)<br></div><div><br></div><div>For sure all errors must be fixed like:</div><div>const char* tmp -> change to -> const char *tmp;</div><div>if(  => if (   #insert space</div><div><br></div><div>Generally I don't know how much warnings should I correct. If it is mandatory or only good practise and I can omit some if it doesn't make sense.</div></div>
_______________________________________________<br>
Kernelnewbies mailing list<br>
<a href="mailto:Kernelnewbies@kernelnewbies.org" target="_blank">Kernelnewbies@kernelnewbies.org</a><br>
<a href="https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies" rel="noreferrer" target="_blank">https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies</a><br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature">Thank you <br>Warm Regards<br>Anuz<br></div></div>