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 13A9FA0C5A; Mon, 29 Nov 2021 15:47:40 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 839BC4068A; Mon, 29 Nov 2021 15:47:39 +0100 (CET) Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by mails.dpdk.org (Postfix) with ESMTP id 8381C40689 for ; Mon, 29 Nov 2021 15:47:38 +0100 (CET) X-IronPort-AV: E=McAfee;i="6200,9189,10182"; a="235806049" X-IronPort-AV: E=Sophos;i="5.87,273,1631602800"; d="scan'208";a="235806049" Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 29 Nov 2021 06:47:37 -0800 X-IronPort-AV: E=Sophos;i="5.87,273,1631602800"; d="scan'208";a="743724604" Received: from bricha3-mobl.ger.corp.intel.com ([10.252.12.81]) by fmsmga006-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA; 29 Nov 2021 06:47:35 -0800 Date: Mon, 29 Nov 2021 14:47:31 +0000 From: Bruce Richardson To: Josh Soref Cc: Ferruh Yigit , dev@dpdk.org, David Marchand , Thomas Monjalon Subject: Re: [PATCH] Spelling Message-ID: References: <20211126195851.50167-1-jsoref@users.noreply.github.com> <737b8856-a247-3709-ad4c-a074d4547c86@intel.com> <68412427-b6aa-cb38-110a-14bc90d8fdc0@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: 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 On Mon, Nov 29, 2021 at 09:12:53AM -0500, Josh Soref wrote: > On Mon, Nov 29, 2021, 9:05 AM Ferruh Yigit <[1]ferruh.yigit@intel.com> > wrote: > > On 11/29/2021 1:59 PM, Bruce Richardson wrote: > > On Mon, Nov 29, 2021 at 12:49:56PM +0000, Ferruh Yigit wrote: > >> Hi Josh, Thanks for the patch. > >> > >> The typo fixes for comments and logs are more straightforward, > but I have > >> mixed feeling about the variable / macro typo fixes, let's see > what others > >> think. > >> > >> And I think we should get this patch either one of the first > patch or last > >> patch to prevent conflict with other patches. > > > > +1 to being a first patch to start things off clean. > > > > For the macros, since there is quick a bit of content in the > patch, could > > we have the patch split into a separate patch for macro changes to > review > > separately, allowing the simple doc/printf spelling changes to go > in > > immediately? > > Yeah, this is doable. I'll see if I can do it now. It's a lot of files, > plus I'll need to figure out the command sequence to reply to this > thread. If someone is available on Slack for some handholding, that > might help me get it done faster. Thanks. To reply to this thread with any new patches add "--in-reply-to 20211126195851.50167-1-jsoref@users.noreply.github.com" to your git send-email command. (The long string above is the message id from the email headers of your original patch. you can get it from patchwork if you look at the headers or also the address to your patch in patchwork [1]). [1] http://patches.dpdk.org/project/dpdk/patch/20211126195851.50167-1-jsoref@users.noreply.github.com/