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 4672FA0C43; Fri, 17 Sep 2021 18:15:47 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D621A410EC; Fri, 17 Sep 2021 18:15:46 +0200 (CEST) Received: from wnew2-smtp.messagingengine.com (wnew2-smtp.messagingengine.com [64.147.123.27]) by mails.dpdk.org (Postfix) with ESMTP id 294EF410EB for ; Fri, 17 Sep 2021 18:15:46 +0200 (CEST) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailnew.west.internal (Postfix) with ESMTP id 4C2A02B008DA; Fri, 17 Sep 2021 12:15:44 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute1.internal (MEProxy); Fri, 17 Sep 2021 12:15:45 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:content-type; s=fm2; bh= FIxMBZfJ4EVWfZwXGr9Ie12SLs8K9aMEUpJtiMVOLQ4=; b=ey1lm8QYEncjufRv 34LA6Q4JN2AnCO0ywYFFxN6w1r0nyIQ57VV2NlzsVCwEj644GykJ3QL7xScr2sv9 HziidPruOj2zhgBd0D6uOJyFtex/cEtqfrg+LaIVLvh+3L+e3onc88BbUOCeZ8WP hen8TcvVzwQu2ehzXbRpmvcVeUhjyfZA+1uNMvhGd2tlXkTV79VB6abWTsSlYtl3 YeDpiOE55GsPKGV2i/jWDDCSmPcbQdMKdPprausqM9QUOIoLgjkOap6ey664cZxR r7VhDYeI5/RABQ/ZnZeLtwa9oq4TjOdIlcHGn/SMtSYxFdZJjiiRSaGbp7wqflhQ 5HMqEQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm3; bh=FIxMBZfJ4EVWfZwXGr9Ie12SLs8K9aMEUpJtiMVOL Q4=; b=rMS4Q2k4oZLxEQZLS4qG/ch9EFxBb8giWCOySSDPvzOTKxNgUnzjaf5/u i0D6gK4LDSt3J0JgbASAQV7wM6q4qHvmlOo9DxHNhIC8uEmQMy9oxMjWgXH3KpVT cMrAD0/mflUbEKAwrs2Y1+YIlj1AcqGXQZTMo/hBZMr5XZUPPNyrr+EoAfWOEZge 6+q1KP51GiNAWsebAJ7ohX4zBCTcoiS4g9VHw5Y0Q6HOVn40WZ8zUhg7Z9CVQ8JL dzH9EL72V0V7NG7Zzc8Pb9afjQCPUDoL6uindQB5ic06JUZNiehNiTfngeXo3q64 kwhau/6QlgNifmj2uqu8slgJLx/jQ== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvtddrudehiedgleegucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhephffvufffkfgjfhgggfgtsehtufertddttddvnecuhfhrohhmpefvhhhomhgr shcuofhonhhjrghlohhnuceothhhohhmrghssehmohhnjhgrlhhonhdrnhgvtheqnecugg ftrfgrthhtvghrnhepffdvffejueetleefieeludduuefgteejleevfeekjeefieegheet ffdvkeefgedunecuffhomhgrihhnpeguphgukhdrohhrghenucevlhhushhtvghrufhiii gvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpehthhhomhgrshesmhhonhhjrghlohhn rdhnvght X-ME-Proxy: Received: by mail.messagingengine.com (Postfix) with ESMTPA; Fri, 17 Sep 2021 12:15:41 -0400 (EDT) From: Thomas Monjalon To: Ashwin Sekhar Thalakalath Kottilveetil Cc: "dev@dpdk.org" , Jerin Jacob Kollanukkaran , Sunil Kumar Kori , Satha Koteswara Rao Kottidi , Pavan Nikhilesh Bhagavatula , Kiran Kumar Kokkilagadda , Satheesh Paul , Anoob Joseph , Akhil Goyal , "ferruh.yigit@intel.com" , "david.marchand@redhat.com" Date: Fri, 17 Sep 2021 18:15:39 +0200 Message-ID: <1890938.tUBTFp5uOv@thomas> In-Reply-To: References: <20210917100330.269797-1-asekhar@marvell.com> <4154556.qFCnVS4LHx@thomas> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [EXT] Re: [PATCH v3] devtools: add acronyms in dictionary for commit checks X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 17/09/2021 16:53, Ashwin Sekhar Thalakalath Kottilveetil: > From: Thomas Monjalon > > 17/09/2021 15:54, Ashwin Sekhar Thalakalath Kottilveetil: > > > From: Thomas Monjalon > > > > 17/09/2021 12:58, Ashwin Sekhar T K: > > > > > Update word list with Marvell specific acronyms. [...] > > Please add details in the commit log so we understand they are Marvell > > acronyms. > Commit log already mentions these are Marvell specific acronyms. I did not > add explanation for each of them as this would make the message too long. Oh yes, I missed it, sorry. > > One more question: why is useful to add? Some people forget uppercases? > > Upper case is desired but not really mandatory. This was a suggestion put > forth to me In one of the reviews. > https://patches.dpdk.org/project/dpdk/patch/20210830135231.2610152-1-asekhar@marvell.com/ > > I can abandon this change if you feel it is not appropriate to put many > device specific acronyms in the top level word list. No strong opinion, but I think the patch is OK. David, Ferruh, opinions?