From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3EA71C4321E for ; Thu, 10 Nov 2022 18:47:59 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230338AbiKJSr5 (ORCPT ); Thu, 10 Nov 2022 13:47:57 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41408 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231139AbiKJSrx (ORCPT ); Thu, 10 Nov 2022 13:47:53 -0500 Received: from mail-ot1-x32d.google.com (mail-ot1-x32d.google.com [IPv6:2607:f8b0:4864:20::32d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 418294E41F for ; Thu, 10 Nov 2022 10:47:50 -0800 (PST) Received: by mail-ot1-x32d.google.com with SMTP id w26-20020a056830061a00b0066c320f5b49so1619697oti.5 for ; Thu, 10 Nov 2022 10:47:50 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=content-disposition:mime-version:message-id:subject:cc:to:from:date :sender:from:to:cc:subject:date:message-id:reply-to; bh=67MozYRSpJPpL2qvP40XMg8+ncMPDhHLkklopg68FPM=; b=DGOqQFjBHxx03rJYrbTo40AR5n6tjLUsad66QPFNAFhJ4qyC4ccQz6TyZIbUikxtLG dtAu3RYqPELF+BDr1a266RUcVCQG2GmNXf34zJPrHEMVfHOqbrMW+Nhx5STjjW52u2/n 2lpVQXfEhgTX4ZKKz5zGbhficlSsnhbfTn2e97RNib89wZ6fbkhAp2f5ONcOSEXZ+tvT BEnqXLR4KAwT/+07paWO7bUDGEDYqgAPljDbKpL1XINYk3NqHxN9DtTajCLcgz3RjiRR rmmL32EY8NVO6Jn+NIFpep0UeAolqUYCM3fU4KdOIAGCR4SdrfGjyEIS4zBVkDMXN3oU 1qDg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-disposition:mime-version:message-id:subject:cc:to:from:date :sender:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=67MozYRSpJPpL2qvP40XMg8+ncMPDhHLkklopg68FPM=; b=LFgJU1kyDyO4tZvPT05dleppM0GfToXGlY8azj6ZMM9VOWlMmGhfJBV/QzOpqXzJW5 N4SWRmBUbNlSKxFcFyh0TyHlwHdh+iAJz4uzl4tXPm77AxDPJ5STT8fd380N9yipwQeo 4YcRLOjWqjXki5yWIDH1gz2XATzyNdK7ranPjFbnpiN35xMrHiaklgFoQzw0aUBxnjhE b++TiV/mJGLZBhjZbMGH4rhCiAEP7VmwWhJsaLTIHc4+7+U20pboOXOoLk+FWIHkA/ml CdUOF4t9LqUy01UnkWvYHlcCFJIpwKrOmCJPH2ZbfrwrFOktnv6+k0pWC+mVPN4fF/+2 4U+A== X-Gm-Message-State: ACrzQf0ZJFSzCX68tdCUD++A4Bk+CcHdWveu5Z2oJ2NhA0vdr0ho+amu /eHairpqpbaV21TKerbaIhg= X-Google-Smtp-Source: AMsMyM63cAmW6nFvVFkw1Fs/CoJiriADIHk3UOQ3Sq/5CktzIOUZsKXa3bup8iKZeMVs3LUL5N7f3Q== X-Received: by 2002:a9d:6b90:0:b0:66c:4584:dd91 with SMTP id b16-20020a9d6b90000000b0066c4584dd91mr1891180otq.86.1668106069455; Thu, 10 Nov 2022 10:47:49 -0800 (PST) Received: from server.roeck-us.net ([2600:1700:e321:62f0:329c:23ff:fee3:9d7c]) by smtp.gmail.com with ESMTPSA id i2-20020a4ac502000000b00480816a5b8csm74846ooq.18.2022.11.10.10.47.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 10 Nov 2022 10:47:48 -0800 (PST) Sender: Guenter Roeck Date: Thu, 10 Nov 2022 10:47:46 -0800 From: Guenter Roeck To: Steven Rostedt Cc: linux-kernel@vger.kernel.org, Linus Torvalds , Thomas Gleixner , Stephen Boyd , Anna-Maria Gleixner , Andrew Morton , Julia Lawall , Viresh Kumar , Shiraz Hashim , Russell King , soc@kernel.org, linux-arm-kernel@lists.infradead.org, Arnd Bergmann , Viresh Kumar Subject: Re: [PATCH v6 1/6] ARM: spear: Do not use timer namespace for timer_shutdown() function Message-ID: <20221110184746.GA3084458@roeck-us.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Nov 10, 2022 at 01:41:02AM -0500, Steven Rostedt wrote: > From: "Steven Rostedt (Google)" > > A new "shutdown" timer state is being added to the generic timer code. One > of the functions to change the timer into the state is called > "timer_shutdown()". This means that there can not be other functions > called "timer_shutdown()" as the timer code owns the "timer_*" name space. > > Rename timer_shutdown() to spear_timer_shutdown() to avoid this conflict. > > Link: https://lkml.kernel.org/r/20221106212701.822440504@goodmis.org > Link: https://lore.kernel.org/all/20221105060155.228348078@goodmis.org/ Looks like those links are obsolete. Guenter > > Cc: Viresh Kumar > Cc: Linus Torvalds > Cc: Thomas Gleixner > Cc: Stephen Boyd > Cc: Anna-Maria Gleixner > Cc: Andrew Morton > Cc: Julia Lawall > Cc: Shiraz Hashim > Cc: Russell King > Cc: soc@kernel.org > Cc: linux-arm-kernel@lists.infradead.org > Acked-by: Arnd Bergmann > Acked-by: Viresh Kumar > Reviewed-by: Guenter Roeck > Tested-by: Guenter Roeck > Signed-off-by: Steven Rostedt (Google) > --- > arch/arm/mach-spear/time.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/arch/arm/mach-spear/time.c b/arch/arm/mach-spear/time.c > index e979e2197f8e..5371c824786d 100644 > --- a/arch/arm/mach-spear/time.c > +++ b/arch/arm/mach-spear/time.c > @@ -90,7 +90,7 @@ static void __init spear_clocksource_init(void) > 200, 16, clocksource_mmio_readw_up); > } > > -static inline void timer_shutdown(struct clock_event_device *evt) > +static inline void spear_timer_shutdown(struct clock_event_device *evt) > { > u16 val = readw(gpt_base + CR(CLKEVT)); > > @@ -101,7 +101,7 @@ static inline void timer_shutdown(struct clock_event_device *evt) > > static int spear_shutdown(struct clock_event_device *evt) > { > - timer_shutdown(evt); > + spear_timer_shutdown(evt); > > return 0; > } > @@ -111,7 +111,7 @@ static int spear_set_oneshot(struct clock_event_device *evt) > u16 val; > > /* stop the timer */ > - timer_shutdown(evt); > + spear_timer_shutdown(evt); > > val = readw(gpt_base + CR(CLKEVT)); > val |= CTRL_ONE_SHOT; > @@ -126,7 +126,7 @@ static int spear_set_periodic(struct clock_event_device *evt) > u16 val; > > /* stop the timer */ > - timer_shutdown(evt); > + spear_timer_shutdown(evt); > > period = clk_get_rate(gpt_clk) / HZ; > period >>= CTRL_PRESCALER16; > -- > 2.35.1