From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id C3D8DA034F for ; Thu, 25 Feb 2021 13:23:48 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id B17AD16087B; Thu, 25 Feb 2021 13:23:48 +0100 (CET) Received: from dpdk.org (dpdk.org [92.243.14.124]) by mails.dpdk.org (Postfix) with ESMTP id D8322160877 for ; Thu, 25 Feb 2021 13:23:46 +0100 (CET) Received: by dpdk.org (Postfix, from userid 1017) id CF06C2B92; Thu, 25 Feb 2021 13:23:46 +0100 (CET) In-Reply-To: <20210225122315.6350-2-pbhagavatula@marvell.com> References: <20210225122315.6350-2-pbhagavatula@marvell.com> To: test-report@dpdk.org Cc: Message-Id: <20210225122346.CF06C2B92@dpdk.org> Date: Thu, 25 Feb 2021 13:23:46 +0100 (CET) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw88221 [PATCH 2/4] event/octeontx2: optimize timer arm routine X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: automatic DPDK test reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: test-report-bounces@dpdk.org Sender: "test-report" Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/88221 _coding style issues_ WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #112: FILE: drivers/event/octeontx2/otx2_tim_worker.h:91: +tim_bkt_dec_lock_relaxed(struct otx2_tim_bkt *bktp) WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #112: FILE: drivers/event/octeontx2/otx2_tim_worker.h:91: +tim_bkt_dec_lock_relaxed(struct otx2_tim_bkt *bktp) WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #134: FILE: drivers/event/octeontx2/otx2_tim_worker.h:255: + asm volatile(" ldxr %[hbt], [%[w1]] " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #135: FILE: drivers/event/octeontx2/otx2_tim_worker.h:256: + " tbz %[hbt], 33, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #135: FILE: drivers/event/octeontx2/otx2_tim_worker.h:256: + " tbz %[hbt], 33, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #136: FILE: drivers/event/octeontx2/otx2_tim_worker.h:257: + " sevl " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #137: FILE: drivers/event/octeontx2/otx2_tim_worker.h:258: + "rty%=: wfe " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #138: FILE: drivers/event/octeontx2/otx2_tim_worker.h:259: + " ldxr %[hbt], [%[w1]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #139: FILE: drivers/event/octeontx2/otx2_tim_worker.h:260: + " tbnz %[hbt], 33, rty%= " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #140: FILE: drivers/event/octeontx2/otx2_tim_worker.h:261: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #140: FILE: drivers/event/octeontx2/otx2_tim_worker.h:261: + "dne%=: " WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #160: FILE: drivers/event/octeontx2/otx2_tim_worker.h:292: + tim_bkt_dec_lock(bkt); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #175: FILE: drivers/event/octeontx2/otx2_tim_worker.h:309: + tim_bkt_inc_nent(bkt); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #176: FILE: drivers/event/octeontx2/otx2_tim_worker.h:310: + tim_bkt_dec_lock_relaxed(bkt); WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #196: FILE: drivers/event/octeontx2/otx2_tim_worker.h:338: + asm volatile(" ldxr %[hbt], [%[w1]] " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #197: FILE: drivers/event/octeontx2/otx2_tim_worker.h:339: + " tbz %[hbt], 33, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #197: FILE: drivers/event/octeontx2/otx2_tim_worker.h:339: + " tbz %[hbt], 33, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #198: FILE: drivers/event/octeontx2/otx2_tim_worker.h:340: + " sevl " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #199: FILE: drivers/event/octeontx2/otx2_tim_worker.h:341: + "rty%=: wfe " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #200: FILE: drivers/event/octeontx2/otx2_tim_worker.h:342: + " ldxr %[hbt], [%[w1]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #201: FILE: drivers/event/octeontx2/otx2_tim_worker.h:343: + " tbnz %[hbt], 33, rty%= " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #202: FILE: drivers/event/octeontx2/otx2_tim_worker.h:344: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #202: FILE: drivers/event/octeontx2/otx2_tim_worker.h:344: + "dne%=: " WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #218: FILE: drivers/event/octeontx2/otx2_tim_worker.h:364: + tim_bkt_dec_lock(bkt); WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #232: FILE: drivers/event/octeontx2/otx2_tim_worker.h:366: + asm volatile(" ldxr %[rem], [%[crem]] " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #233: FILE: drivers/event/octeontx2/otx2_tim_worker.h:367: + " tbz %[rem], 63, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #233: FILE: drivers/event/octeontx2/otx2_tim_worker.h:367: + " tbz %[rem], 63, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #234: FILE: drivers/event/octeontx2/otx2_tim_worker.h:368: + " sevl " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #235: FILE: drivers/event/octeontx2/otx2_tim_worker.h:369: + "rty%=: wfe " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #236: FILE: drivers/event/octeontx2/otx2_tim_worker.h:370: + " ldxr %[rem], [%[crem]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #237: FILE: drivers/event/octeontx2/otx2_tim_worker.h:371: + " tbnz %[rem], 63, rty%= " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #238: FILE: drivers/event/octeontx2/otx2_tim_worker.h:372: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #238: FILE: drivers/event/octeontx2/otx2_tim_worker.h:372: + "dne%=: " WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #263: FILE: drivers/event/octeontx2/otx2_tim_worker.h:393: + tim_bkt_set_rem(bkt, 0); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #264: FILE: drivers/event/octeontx2/otx2_tim_worker.h:394: + tim_bkt_dec_lock(bkt); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #272: FILE: drivers/event/octeontx2/otx2_tim_worker.h:398: + if (tim_bkt_fetch_lock(lock_sema)) { WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #276: FILE: drivers/event/octeontx2/otx2_tim_worker.h:402: + } while (tim_bkt_fetch_lock(lock_sema) - 1); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #293: FILE: drivers/event/octeontx2/otx2_tim_worker.h:418: + tim_bkt_inc_nent(bkt); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #294: FILE: drivers/event/octeontx2/otx2_tim_worker.h:419: + tim_bkt_dec_lock_relaxed(bkt); WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #303: FILE: drivers/event/octeontx2/otx2_tim_worker.h:469: + asm volatile(" ldxrb %w[lock_cnt], [%[lock]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #304: FILE: drivers/event/octeontx2/otx2_tim_worker.h:470: + " tst %w[lock_cnt], 255 " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #305: FILE: drivers/event/octeontx2/otx2_tim_worker.h:471: + " beq dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #305: FILE: drivers/event/octeontx2/otx2_tim_worker.h:471: + " beq dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #306: FILE: drivers/event/octeontx2/otx2_tim_worker.h:472: + " sevl " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #307: FILE: drivers/event/octeontx2/otx2_tim_worker.h:473: + "rty%=: wfe " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #308: FILE: drivers/event/octeontx2/otx2_tim_worker.h:474: + " ldxrb %w[lock_cnt], [%[lock]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #309: FILE: drivers/event/octeontx2/otx2_tim_worker.h:475: + " tst %w[lock_cnt], 255 " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #310: FILE: drivers/event/octeontx2/otx2_tim_worker.h:476: + " bne rty%= " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #311: FILE: drivers/event/octeontx2/otx2_tim_worker.h:477: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #311: FILE: drivers/event/octeontx2/otx2_tim_worker.h:477: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #338: FILE: drivers/event/octeontx2/otx2_tim_worker.h:493: + asm volatile(" ldxr %[hbt], [%[w1]] " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #339: FILE: drivers/event/octeontx2/otx2_tim_worker.h:494: + " tbz %[hbt], 33, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #339: FILE: drivers/event/octeontx2/otx2_tim_worker.h:494: + " tbz %[hbt], 33, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #340: FILE: drivers/event/octeontx2/otx2_tim_worker.h:495: + " sevl " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #341: FILE: drivers/event/octeontx2/otx2_tim_worker.h:496: + "rty%=: wfe " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #342: FILE: drivers/event/octeontx2/otx2_tim_worker.h:497: + " ldxr %[hbt], [%[w1]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #343: FILE: drivers/event/octeontx2/otx2_tim_worker.h:498: + " tbnz %[hbt], 33, rty%= " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #344: FILE: drivers/event/octeontx2/otx2_tim_worker.h:499: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #344: FILE: drivers/event/octeontx2/otx2_tim_worker.h:499: + "dne%=: " WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #363: FILE: drivers/event/octeontx2/otx2_tim_worker.h:585: + tim_bkt_dec_lock(bkt); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #374: FILE: drivers/event/octeontx2/otx2_tim_worker.h:594: + tim_bkt_dec_lock(bkt); total: 0 errors, 61 warnings, 271 lines checked Warning in drivers/event/octeontx2/otx2_tim_worker.h: Using rte_smp_[r/w]mb Warning in drivers/event/octeontx2/otx2_tim_worker.c: Using __atomic_thread_fence