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 1E73AA0562 for ; Tue, 23 Mar 2021 09:45:41 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 1991C4014F; Tue, 23 Mar 2021 09:45:41 +0100 (CET) Received: from dpdk.org (dpdk.org [92.243.14.124]) by mails.dpdk.org (Postfix) with ESMTP id 147034014D for ; Tue, 23 Mar 2021 09:45:40 +0100 (CET) Received: by dpdk.org (Postfix, from userid 1017) id 11CB3F64; Tue, 23 Mar 2021 09:45:40 +0100 (CET) In-Reply-To: <20210323084439.3898-2-pbhagavatula@marvell.com> References: <20210323084439.3898-2-pbhagavatula@marvell.com> To: test-report@dpdk.org Cc: Message-Id: <20210323084540.11CB3F64@dpdk.org> Date: Tue, 23 Mar 2021 09:45:40 +0100 (CET) From: checkpatch@dpdk.org Subject: [dpdk-test-report] |WARNING| pw89668 [PATCH v3 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/89668 _coding style issues_ WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #117: 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'? #117: 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 #139: FILE: drivers/event/octeontx2/otx2_tim_worker.h:255: + asm volatile(" ldxr %[hbt], [%[w1]] " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #140: FILE: drivers/event/octeontx2/otx2_tim_worker.h:256: + " tbz %[hbt], 33, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #140: FILE: drivers/event/octeontx2/otx2_tim_worker.h:256: + " tbz %[hbt], 33, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #141: FILE: drivers/event/octeontx2/otx2_tim_worker.h:257: + " sevl " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #142: FILE: drivers/event/octeontx2/otx2_tim_worker.h:258: + "rty%=: wfe " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #143: FILE: drivers/event/octeontx2/otx2_tim_worker.h:259: + " ldxr %[hbt], [%[w1]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #144: FILE: drivers/event/octeontx2/otx2_tim_worker.h:260: + " tbnz %[hbt], 33, rty%= " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #145: FILE: drivers/event/octeontx2/otx2_tim_worker.h:261: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #145: FILE: drivers/event/octeontx2/otx2_tim_worker.h:261: + "dne%=: " WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #165: FILE: drivers/event/octeontx2/otx2_tim_worker.h:292: + tim_bkt_dec_lock(bkt); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #179: 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'? #180: FILE: drivers/event/octeontx2/otx2_tim_worker.h:308: + tim_bkt_inc_nent(bkt); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #181: 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 #201: FILE: drivers/event/octeontx2/otx2_tim_worker.h:337: + asm volatile(" ldxr %[hbt], [%[w1]] " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #202: FILE: drivers/event/octeontx2/otx2_tim_worker.h:338: + " tbz %[hbt], 33, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #202: FILE: drivers/event/octeontx2/otx2_tim_worker.h:338: + " tbz %[hbt], 33, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #203: FILE: drivers/event/octeontx2/otx2_tim_worker.h:339: + " sevl " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #204: FILE: drivers/event/octeontx2/otx2_tim_worker.h:340: + "rty%=: wfe " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #205: FILE: drivers/event/octeontx2/otx2_tim_worker.h:341: + " ldxr %[hbt], [%[w1]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #206: FILE: drivers/event/octeontx2/otx2_tim_worker.h:342: + " tbnz %[hbt], 33, rty%= " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #207: FILE: drivers/event/octeontx2/otx2_tim_worker.h:343: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #207: FILE: drivers/event/octeontx2/otx2_tim_worker.h:343: + "dne%=: " WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #223: 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 #238: FILE: drivers/event/octeontx2/otx2_tim_worker.h:366: + asm volatile(" ldxr %[w1], [%[crem]] " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #239: FILE: drivers/event/octeontx2/otx2_tim_worker.h:367: + " tbz %[w1], 63, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #239: FILE: drivers/event/octeontx2/otx2_tim_worker.h:367: + " tbz %[w1], 63, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #240: FILE: drivers/event/octeontx2/otx2_tim_worker.h:368: + " sevl " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #241: FILE: drivers/event/octeontx2/otx2_tim_worker.h:369: + "rty%=: wfe " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #242: FILE: drivers/event/octeontx2/otx2_tim_worker.h:370: + " ldxr %[w1], [%[crem]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #243: FILE: drivers/event/octeontx2/otx2_tim_worker.h:371: + " tbnz %[w1], 63, rty%= " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #244: FILE: drivers/event/octeontx2/otx2_tim_worker.h:372: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #244: FILE: drivers/event/octeontx2/otx2_tim_worker.h:372: + "dne%=: " WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #269: FILE: drivers/event/octeontx2/otx2_tim_worker.h:393: + tim_bkt_set_rem(bkt, 0); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #270: FILE: drivers/event/octeontx2/otx2_tim_worker.h:394: + tim_bkt_dec_lock(bkt); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #278: 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'? #282: 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'? #298: FILE: drivers/event/octeontx2/otx2_tim_worker.h:416: + __atomic_store_n(&tim->state, RTE_EVENT_TIMER_ARMED, __ATOMIC_RELEASE); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #299: FILE: drivers/event/octeontx2/otx2_tim_worker.h:417: + tim_bkt_inc_nent(bkt); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #300: FILE: drivers/event/octeontx2/otx2_tim_worker.h:418: + tim_bkt_dec_lock_relaxed(bkt); WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #309: FILE: drivers/event/octeontx2/otx2_tim_worker.h:468: + asm volatile(" ldxrb %w[lock_cnt], [%[lock]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #310: FILE: drivers/event/octeontx2/otx2_tim_worker.h:469: + " tst %w[lock_cnt], 255 " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #311: FILE: drivers/event/octeontx2/otx2_tim_worker.h:470: + " beq dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #311: FILE: drivers/event/octeontx2/otx2_tim_worker.h:470: + " beq dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #312: FILE: drivers/event/octeontx2/otx2_tim_worker.h:471: + " sevl " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #313: FILE: drivers/event/octeontx2/otx2_tim_worker.h:472: + "rty%=: wfe " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #314: FILE: drivers/event/octeontx2/otx2_tim_worker.h:473: + " ldxrb %w[lock_cnt], [%[lock]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #315: FILE: drivers/event/octeontx2/otx2_tim_worker.h:474: + " tst %w[lock_cnt], 255 " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #316: FILE: drivers/event/octeontx2/otx2_tim_worker.h:475: + " bne rty%= " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #317: FILE: drivers/event/octeontx2/otx2_tim_worker.h:476: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #317: FILE: drivers/event/octeontx2/otx2_tim_worker.h:476: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #344: FILE: drivers/event/octeontx2/otx2_tim_worker.h:492: + asm volatile(" ldxr %[hbt], [%[w1]] " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #345: FILE: drivers/event/octeontx2/otx2_tim_worker.h:493: + " tbz %[hbt], 33, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #345: FILE: drivers/event/octeontx2/otx2_tim_worker.h:493: + " tbz %[hbt], 33, dne%= " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #346: FILE: drivers/event/octeontx2/otx2_tim_worker.h:494: + " sevl " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #347: FILE: drivers/event/octeontx2/otx2_tim_worker.h:495: + "rty%=: wfe " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #348: FILE: drivers/event/octeontx2/otx2_tim_worker.h:496: + " ldxr %[hbt], [%[w1]] " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #349: FILE: drivers/event/octeontx2/otx2_tim_worker.h:497: + " tbnz %[hbt], 33, rty%= " WARNING:TYPO_SPELLING: 'dne' may be misspelled - perhaps 'done'? #350: FILE: drivers/event/octeontx2/otx2_tim_worker.h:498: + "dne%=: " WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline #350: FILE: drivers/event/octeontx2/otx2_tim_worker.h:498: + "dne%=: " WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #369: FILE: drivers/event/octeontx2/otx2_tim_worker.h:584: + tim_bkt_dec_lock(bkt); WARNING:TYPO_SPELLING: 'tim' may be misspelled - perhaps 'time'? #380: FILE: drivers/event/octeontx2/otx2_tim_worker.h:593: + tim_bkt_dec_lock(bkt); total: 0 errors, 63 warnings, 272 lines checked