Received: by 2002:a05:6a11:4021:0:0:0:0 with SMTP id ky33csp683784pxb; Thu, 23 Sep 2021 08:33:48 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxomJwAFJOvcZgIPTeNnpDVQkK54TeNcYzHq/nCv4krnoL8UWKYEDr2WUPeCRU33ydUfyz2 X-Received: by 2002:a50:cf08:: with SMTP id c8mr6235850edk.86.1632411228497; Thu, 23 Sep 2021 08:33:48 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1632411228; cv=none; d=google.com; s=arc-20160816; b=AYcphdQh2LAvlw7/Vr2w8ZuBhO8JKP/jomuPIBYp5ZGxL3LdzdiYHeFd93ZgD/t9al ll+YEL0m7/9aCJL6YQhAWhBlZUyEJeiNn7ENjWEgXRj0G2hSm6rsrKTiAwUopshK7/BZ pokNMF9Id+6NALGUJmo4clSvd2i2UvnZ1MAKK+8YyDytSWk4zbfb0gwBV10PTBrbTZq5 239Pr8bu7Yp5L+aMan3wH7SMzFNqjxBJ5KrTdP3E53EjjJQQklCJ2m6SuMdoU/gwBTVy iBGIQ78EaEyXuo6SxSP3V2Txoojy3rct7BEytEWFAHJo+NYvs58e5qUWLaxK+fPbzX45 eU0w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject:dkim-signature; bh=AnEeQ76X0azmr4+uOVT20QGvRc60URiBSo8uDfTzt2g=; b=DCMXBhDqoqisZaBN79vQlU4NKSi//Bll3YTOyhZ9IWcp8HKOGf9Guiq9pZmAjN3Ye2 r7IhOgK+srFV6UJSeaMR6Dr9lDShqzu+noVV/PD4LKnfAiOsnTdZWWgCu9pq7CcEZU+f jvF0qhyi3NZ/M5aJtbrjotNXutk/F3yI9mFoqvwB9i5F0QHf0VxzIeBqjvd0lEnaABbO XrGfksRZ+6VgJSdocfZHFQ34ma41sqhibYPc6LfZO2vxWLn5uq59tQrHcOdfEdt82oWj +GcRTyamjG/nkKTkVBMAVyPZxQepomgBl8H7O6LqZeDxBqpdAVKhJbaLfAXhIsedQ2n+ U7tA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@infradead.org header.s=bombadil.20210309 header.b="K/qxyniV"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id 28si6180455ejg.200.2021.09.23.08.33.21; Thu, 23 Sep 2021 08:33:48 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@infradead.org header.s=bombadil.20210309 header.b="K/qxyniV"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S241991AbhIWPas (ORCPT + 99 others); Thu, 23 Sep 2021 11:30:48 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34062 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S241979AbhIWPar (ORCPT ); Thu, 23 Sep 2021 11:30:47 -0400 Received: from bombadil.infradead.org (bombadil.infradead.org [IPv6:2607:7c80:54:e::133]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9DA5EC061574 for ; Thu, 23 Sep 2021 08:29:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=Content-Transfer-Encoding: Content-Type:In-Reply-To:MIME-Version:Date:Message-ID:From:References:Cc:To: Subject:Sender:Reply-To:Content-ID:Content-Description; bh=AnEeQ76X0azmr4+uOVT20QGvRc60URiBSo8uDfTzt2g=; b=K/qxyniVuxCWpbQZFwzJnDj0g8 UD9+E0DJYK+kXB0jVGFVEtRT3fDOSmlX5AjPSX3i/p6iZEaMsEsiS+rGxg6s7IsA+LGfFU/M0KzCJ JR8X6lPsjX/odIxhW7K5+Sn1BDLW5g0mO6lA+x4N8wpQOTJxpigQUOXhhf752+1wSeWY7ymbAujyE ucoHogmA6JnUmU/HRai/FFTPwLNaX68eVCCKGwQQoE5BSz0/iTa+r0JucFPTlIzPCTZOvrsEH+DZL jKkaSQh838yyIhIGQ86lIBA/XyHV2yGEmJUR58NmXFzYYc2AjSe56vAVkqi567Wo0c+0gYiLzilRn MaFMkWeg==; Received: from [2601:1c0:6280:3f0::aa0b] by bombadil.infradead.org with esmtpsa (Exim 4.94.2 #2 (Red Hat Linux)) id 1mTQet-00C3Wc-7f; Thu, 23 Sep 2021 15:29:11 +0000 Subject: Re: [PATCH] clocksource: arc_timer: eliminate redefined macro error To: Shahab Vahedi , "linux-kernel@vger.kernel.org" Cc: Vineet Gupta , "linux-snps-arc@lists.infradead.org" , Daniel Lezcano , Thomas Gleixner References: <20210923031808.5945-1-rdunlap@infradead.org> <61fef534-36df-2ed9-5662-ccebefef494e@synopsys.com> From: Randy Dunlap Message-ID: Date: Thu, 23 Sep 2021 08:29:09 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: <61fef534-36df-2ed9-5662-ccebefef494e@synopsys.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 9/23/21 2:42 AM, Shahab Vahedi wrote: > Hi Randy, > > I have one minor comment. > > On 9/23/21 5:18 AM, Randy Dunlap wrote: >> >> --- linux-next-20210917.orig/include/soc/arc/timers.h >> +++ linux-next-20210917/include/soc/arc/timers.h >> @@ -17,7 +17,7 @@ >> #define ARC_REG_TIMER1_CNT 0x100 /* timer 1 count */ >> >> /* CTRL reg bits */ >> -#define TIMER_CTRL_IE (1 << 0) /* Interrupt when Count reaches limit */ >> +#define TIMER_CTRL_IEN (1 << 0) /* Interrupt when Count reaches limit */ > > Could you change the name to "ARC_TIMER_CTRL_IE" instead? and while at it, please rename > "TIMER_CTRL_NH" to "ARC_TIMER_CTRL_NH" for obvious consistency reasons and saving us > from future clashes. Thank you! > OK, no problem. Thanks for the feedback. -- ~Randy