From f5a44b0d149001541dc92a7c6b128aaa2090468c Mon Sep 17 00:00:00 2001 From: Tim Newsome Date: Mon, 31 Aug 2020 13:20:46 -0700 Subject: [PATCH] Make checkpatch require Signed-off-by (#516) * Make checkpatch require Signed-off-by This will make it easier to send changes contributed here to mainline OpenOCD. (Intentionally not including the required line here to make sure I can't just merge this.) Change-Id: I089084d38f3e08859d62cf7eface405f37af4799 * Whitespace fix. This PR isn't building on travis. Maybe because I only changed .travis.yml. Here's a source change to force a build (hopefully). Change-Id: I8a828fe1d56a1960bc4bfb91d3d2f3a0790ad571 * Can't check for signoff on sources alone. Change-Id: I741a299b64bf14857a4e1807b254a7d270b2e466 Signed-off-by: Tim Newsome * Actual whitespace fixes. Why didn't this fail to build before? Change-Id: I339c03c4ef96546dbef5f16e635921a4fdaf9b35 Signed-off-by: Tim Newsome --- .travis.yml | 2 ++ src/target/riscv/batch.c | 2 +- src/target/riscv/encoding.h | 4 ++-- 3 files changed, 5 insertions(+), 3 deletions(-) diff --git a/.travis.yml b/.travis.yml index 719144508..78bdc8b75 100644 --- a/.travis.yml +++ b/.travis.yml @@ -58,6 +58,8 @@ matrix: - binutils-mingw-w64-i686 gcc-mingw-w64-i686 g++-mingw-w64-i686 patchutils script: + # This is here for the signoff check. + - ./tools/checkpatch.sh # Ideally we'd diff back to where we either branched off OpenOCD or master, # or riscv. But that's tricky, and the default git clone only gets the last # 50 changes any case. Most merges won't consist of more than 40 changes, diff --git a/src/target/riscv/batch.c b/src/target/riscv/batch.c index 81d0358ec..b2c7e834d 100644 --- a/src/target/riscv/batch.c +++ b/src/target/riscv/batch.c @@ -11,7 +11,7 @@ #define get_field(reg, mask) (((reg) & (mask)) / ((mask) & ~((mask) << 1))) #define set_field(reg, mask, val) (((reg) & ~(mask)) | (((val) * ((mask) & ~((mask) << 1))) & (mask))) -#define DTM_DMI_MAX_ADDRESS_LENGTH ((1<