Hoernchen has submitted this change. ( https://gerrit.osmocom.org/c/osmo-trx/+/30436 ) Change subject: add checkpatch config ...................................................................... add checkpatch config - do not lint the submodule - do not lint the grgsm code - do not lint headers, checkpatch does not understand that this is a c++ project and any change to existing headers will cause issues. - also do not do arcane C varargs checks that do not apply to C++ Change-Id: Ie7a9fbd021e12a88db30212240af2332c6cdcb37 --- A .checkpatch.conf 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, but someone else must approve pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved diff --git a/.checkpatch.conf b/.checkpatch.conf new file mode 100644 index 0000000..1e4b479 --- /dev/null +++ b/.checkpatch.conf @@ -0,0 +1,4 @@ +--exclude osmocom-bb/.* +--exclude .*h +--exclude Transceiver52M/grgsm_vitac/.* +--ignore FUNCTION_WITHOUT_ARGS \ No newline at end of file -- To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/30436 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-trx Gerrit-Branch: master Gerrit-Change-Id: Ie7a9fbd021e12a88db30212240af2332c6cdcb37 Gerrit-Change-Number: 30436 Gerrit-PatchSet: 2 Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged