Dear Stephen Warren,
In message 74CDBE0F657A3D45AFBB94109FB122FF173B23B59F@HQMAIL01.nvidia.com you wrote:
I'm getting a lot of the following when running checkpatch:
WARNING: space prohibited between function name and open parenthesis '('
Should I ignore these?
No, these should be fixed.
So far, I've written completely new code such that it doesn't trigger this warning, but where editing existing code (e.g. just changing the name of a function), I've avoided fixing this checkpatch warning so it's more obvious what my diff is doing; equally, fixing this would cause the lines I edit to be inconsistent with those around them.
Any guidance appreciated. Thanks.
The normal procedure is to submit a [COSMETIC] patch first that fixes the coding style issues of the affected file (and _only_ these), and then submit a chackpatch-clean patch with the actual changes.
Best regards,
Wolfgang Denk