From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.90_1) id 1kkp9s-0003oK-VF for mharc-grub-devel@gnu.org; Thu, 03 Dec 2020 09:00:32 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:60868) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kkp9n-0003jz-2P for grub-devel@gnu.org; Thu, 03 Dec 2020 09:00:28 -0500 Received: from dibed.net-space.pl ([84.10.22.86]:37431) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_3DES_EDE_CBC_SHA1:192) (Exim 4.90_1) (envelope-from ) id 1kkp9h-0005vQ-BB for grub-devel@gnu.org; Thu, 03 Dec 2020 09:00:26 -0500 Received: from router-fw.i.net-space.pl ([192.168.52.1]:42394 "EHLO tomti.i.net-space.pl") by router-fw-old.i.net-space.pl with ESMTP id S143839AbgLCOAS (ORCPT ); Thu, 3 Dec 2020 15:00:18 +0100 X-Comment: RFC 2476 MSA function at dibed.net-space.pl logged sender identity as: dkiper Date: Thu, 3 Dec 2020 15:00:14 +0100 From: Daniel Kiper To: Glenn Washburn Cc: grub-devel@gnu.org, Patrick Steinhardt Subject: Re: [PATCH v6 10/12] mips: Enable __clzdi2() Message-ID: <20201203140014.2grzydt2ml22fk45@tomti.i.net-space.pl> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20170113 (1.7.2) Received-SPF: pass client-ip=84.10.22.86; envelope-from=dkiper@net-space.pl; helo=dibed.net-space.pl X-Spam_score_int: -18 X-Spam_score: -1.9 X-Spam_bar: - X-Spam_report: (-1.9 / 5.0 requ) BAYES_00=-1.9, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: The development of GNU GRUB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 03 Dec 2020 14:00:29 -0000 On Fri, Nov 27, 2020 at 03:03:42AM -0600, Glenn Washburn wrote: > This patch is similiar to commit 9dab2f51e (sparc: Enable __clzsi2() and > __clzdi2()) but for MIPS target and __clzdi2 only, __clzsi2 was already > enabled. > > Signed-off-by: Daniel Kiper This should be replaced with: Suggested-by: Daniel Kiper Signed-off-by: Glenn Washburn In general if somebody suggests you something and you use they idea then you should use Suggested-by instead of SOB. Anyway, I can fix it before committing the patch if you are OK with me adding your SOB. Daniel