diff options
| author | Patrick Pannuto <ppannuto@codeaurora.org> | 2010-07-23 13:34:18 -0700 |
|---|---|---|
| committer | Stephen Boyd <sboyd@codeaurora.org> | 2013-09-04 14:18:00 -0700 |
| commit | a56b665ad0b4dc67d69393479336185b1fc70922 (patch) | |
| tree | 0f255e01fbf5e2cad9f0abeae1b5a340c41ef9dc /scripts/gcc-wrapper.py | |
| parent | a38797093f1079c9e73fd362be4c6bb656b89c0b (diff) | |
checkpatch: Check for illegal return codes
The only legal integer return is 0, anything else
following "return" should be -ERRCODE or a function.
http://lkml.org/lkml/2010/7/23/318
There's lots of "return -1;" statements in this patch - it's obscene
that this is used to indicate "some error occurred" in kernel space
rather than a real errno value - even when an existing function
(eg, request_irq) gave you an error code already.
Please note this for the future - and please review patches on this
point internally first.
Change-Id: I16268b2ee034f0b3b899115e45c28acfa734ddec
Signed-off-by: Patrick Pannuto <ppannuto@codeaurora.org>
(cherry picked from commit 39531a47164294315b5a7256b520fe22d6e87013)
Diffstat (limited to 'scripts/gcc-wrapper.py')
0 files changed, 0 insertions, 0 deletions