Dear Lukasz Majewski,
In message 20111018122416.483205d6@lmajewski.digital.local you wrote:
Joe Hershberger has prepared a following patch:
http://patchwork.ozlabs.org/patch/119083/
Is this THE ONE checkpatch version which we shall use (including its config file)?
It shall become it, yes.
I've used it for my patch sets and I can say that warnings like:
WARNING: min() should probably be min_t(unsigned int, amount_left, mod_data.buflen)
and
WARNING: consider using kstrto* in preference to simple_strtoul #4358:
are still present.
Those are Linux kernel specific.
Shall I not care about them and submit patches with non zero warnings output from checkpatch?
Neither one nore the other. Help adapting the checkpatch config file to our needs, so that things like simple_strtoul() don't throw such warnings in U-Boot context.
Then submit zero warning patches...
Best regards,
Wolfgang Denk