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 727A3A0548 for ; Sun, 21 Mar 2021 09:50:15 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 6020C140E39; Sun, 21 Mar 2021 09:50:15 +0100 (CET) Received: from dpdk.org (dpdk.org [92.243.14.124]) by mails.dpdk.org (Postfix) with ESMTP id E3C4D140E10 for ; Sun, 21 Mar 2021 09:50:13 +0100 (CET) Received: by dpdk.org (Postfix, from userid 1017) id DFEF92BA9; Sun, 21 Mar 2021 09:50:12 +0100 (CET) In-Reply-To: <20210321084915.2649-2-pbhagavatula@marvell.com> References: <20210321084915.2649-2-pbhagavatula@marvell.com> To: test-report@dpdk.org Cc: Message-Id: <20210321085012.DFEF92BA9@dpdk.org> Date: Sun, 21 Mar 2021 09:50:12 +0100 (CET) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw89599 [PATCH v2 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/89599 _coding style issues_ WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #111: 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'? #111: 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 #133: FILE: drivers/event/octeontx2/otx2_tim_worker.h:255: + asm volatile(" ldxr %[hbt], [%[w1]] " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #134: FILE: drivers/event/octeontx2/otx2_tim_worker.h:256: + " tbz %[hbt], 33, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #134: 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:257: + " sevl " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #136: FILE: drivers/event/octeontx2/otx2_tim_worker.h:258: + "rty%=: wfe " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #137: FILE: drivers/event/octeontx2/otx2_tim_worker.h:259: + " ldxr %[hbt], [%[w1]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #138: FILE: drivers/event/octeontx2/otx2_tim_worker.h:260: + " tbnz %[hbt], 33, rty%= " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #139: FILE: drivers/event/octeontx2/otx2_tim_worker.h:261: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #139: FILE: drivers/event/octeontx2/otx2_tim_worker.h:261: + "dne%=: " WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #159: FILE: drivers/event/octeontx2/otx2_tim_worker.h:292: + tim_bkt_dec_lock(bkt); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #173: FILE: drivers/event/octeontx2/otx2_tim_worker.h:307: + __atomic_store_n(&tim->state, RTE_EVENT_TIMER_ARMED, __ATOMIC_RELEASE); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #174: FILE: drivers/event/octeontx2/otx2_tim_worker.h:308: + tim_bkt_inc_nent(bkt); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #175: FILE: drivers/event/octeontx2/otx2_tim_worker.h:309: + tim_bkt_dec_lock_relaxed(bkt); WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #195: FILE: drivers/event/octeontx2/otx2_tim_worker.h:337: + asm volatile(" ldxr %[hbt], [%[w1]] " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #196: FILE: drivers/event/octeontx2/otx2_tim_worker.h:338: + " tbz %[hbt], 33, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #196: FILE: drivers/event/octeontx2/otx2_tim_worker.h:338: + " 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: + " sevl " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #198: FILE: drivers/event/octeontx2/otx2_tim_worker.h:340: + "rty%=: wfe " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #199: FILE: drivers/event/octeontx2/otx2_tim_worker.h:341: + " ldxr %[hbt], [%[w1]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #200: FILE: drivers/event/octeontx2/otx2_tim_worker.h:342: + " tbnz %[hbt], 33, rty%= " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #201: FILE: drivers/event/octeontx2/otx2_tim_worker.h:343: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #201: FILE: drivers/event/octeontx2/otx2_tim_worker.h:343: + "dne%=: " WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #217: FILE: drivers/event/octeontx2/otx2_tim_worker.h:363: + tim_bkt_dec_lock(bkt); WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #231: FILE: drivers/event/octeontx2/otx2_tim_worker.h:365: + asm volatile(" ldxr %[rem], [%[crem]] " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #232: FILE: drivers/event/octeontx2/otx2_tim_worker.h:366: + " tbz %[rem], 63, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #232: FILE: drivers/event/octeontx2/otx2_tim_worker.h:366: + " 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: + " sevl " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #234: FILE: drivers/event/octeontx2/otx2_tim_worker.h:368: + "rty%=: wfe " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #235: FILE: drivers/event/octeontx2/otx2_tim_worker.h:369: + " ldxr %[rem], [%[crem]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #236: FILE: drivers/event/octeontx2/otx2_tim_worker.h:370: + " tbnz %[rem], 63, rty%= " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #237: FILE: drivers/event/octeontx2/otx2_tim_worker.h:371: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #237: FILE: drivers/event/octeontx2/otx2_tim_worker.h:371: + "dne%=: " WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #262: FILE: drivers/event/octeontx2/otx2_tim_worker.h:392: + tim_bkt_set_rem(bkt, 0); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #263: FILE: drivers/event/octeontx2/otx2_tim_worker.h:393: + tim_bkt_dec_lock(bkt); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #271: FILE: drivers/event/octeontx2/otx2_tim_worker.h:397: + if (tim_bkt_fetch_lock(lock_sema)) { WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #275: FILE: drivers/event/octeontx2/otx2_tim_worker.h:401: + } while (tim_bkt_fetch_lock(lock_sema) - 1); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #291: FILE: drivers/event/octeontx2/otx2_tim_worker.h:415: + __atomic_store_n(&tim->state, RTE_EVENT_TIMER_ARMED, __ATOMIC_RELEASE); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #292: FILE: drivers/event/octeontx2/otx2_tim_worker.h:416: + tim_bkt_inc_nent(bkt); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #293: FILE: drivers/event/octeontx2/otx2_tim_worker.h:417: + tim_bkt_dec_lock_relaxed(bkt); WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #302: FILE: drivers/event/octeontx2/otx2_tim_worker.h:467: + asm volatile(" ldxrb %w[lock_cnt], [%[lock]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #303: FILE: drivers/event/octeontx2/otx2_tim_worker.h:468: + " tst %w[lock_cnt], 255 " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #304: FILE: drivers/event/octeontx2/otx2_tim_worker.h:469: + " beq dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #304: FILE: drivers/event/octeontx2/otx2_tim_worker.h:469: + " beq dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #305: FILE: drivers/event/octeontx2/otx2_tim_worker.h:470: + " sevl " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #306: FILE: drivers/event/octeontx2/otx2_tim_worker.h:471: + "rty%=: wfe " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #307: FILE: drivers/event/octeontx2/otx2_tim_worker.h:472: + " ldxrb %w[lock_cnt], [%[lock]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #308: FILE: drivers/event/octeontx2/otx2_tim_worker.h:473: + " tst %w[lock_cnt], 255 " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #309: FILE: drivers/event/octeontx2/otx2_tim_worker.h:474: + " bne rty%= " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #310: FILE: drivers/event/octeontx2/otx2_tim_worker.h:475: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #310: FILE: drivers/event/octeontx2/otx2_tim_worker.h:475: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #337: FILE: drivers/event/octeontx2/otx2_tim_worker.h:491: + asm volatile(" ldxr %[hbt], [%[w1]] " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #338: FILE: drivers/event/octeontx2/otx2_tim_worker.h:492: + " tbz %[hbt], 33, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #338: FILE: drivers/event/octeontx2/otx2_tim_worker.h:492: + " tbz %[hbt], 33, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #339: FILE: drivers/event/octeontx2/otx2_tim_worker.h:493: + " sevl " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #340: FILE: drivers/event/octeontx2/otx2_tim_worker.h:494: + "rty%=: wfe " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #341: FILE: drivers/event/octeontx2/otx2_tim_worker.h:495: + " ldxr %[hbt], [%[w1]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #342: FILE: drivers/event/octeontx2/otx2_tim_worker.h:496: + " tbnz %[hbt], 33, rty%= " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #343: FILE: drivers/event/octeontx2/otx2_tim_worker.h:497: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #343: FILE: drivers/event/octeontx2/otx2_tim_worker.h:497: + "dne%=: " WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #362: FILE: drivers/event/octeontx2/otx2_tim_worker.h:583: + tim_bkt_dec_lock(bkt); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #373: FILE: drivers/event/octeontx2/otx2_tim_worker.h:592: + tim_bkt_dec_lock(bkt); total: 0 errors, 63 warnings, 271 lines checked