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 X-Spam-Level: X-Spam-Status: No, score=-23.3 required=3.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_IN_DEF_DKIM_WL autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D19D8C433DB for ; Wed, 17 Mar 2021 03:36:47 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id A08AB64F8F for ; Wed, 17 Mar 2021 03:36:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229537AbhCQDgP (ORCPT ); Tue, 16 Mar 2021 23:36:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53812 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229727AbhCQDgH (ORCPT ); Tue, 16 Mar 2021 23:36:07 -0400 Received: from mail-pf1-x42d.google.com (mail-pf1-x42d.google.com [IPv6:2607:f8b0:4864:20::42d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 720DDC06175F for ; Tue, 16 Mar 2021 20:36:07 -0700 (PDT) Received: by mail-pf1-x42d.google.com with SMTP id y5so166020pfn.1 for ; Tue, 16 Mar 2021 20:36:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=date:subject:in-reply-to:cc:from:to:message-id:mime-version :content-transfer-encoding; bh=W8H72798it9OolNWp4JLWdlboyq7CodcHmCStG5d+3I=; b=sDlO2kksnw7mXcOE//Cv43BSuy2P0zSuKgFBddBhhf2tC6IkJ7W8BGkCRbwRJ7nIUo GqgCZLicz29lrk2L/QY+ycBUwQ54L+COiFtCVbAyhfPue9O9HLmE+3MXXZWSfPwiSYPY Juh3VAVgr/ox9Y3wvQyITyQch8YaTdycI/X0HFkTlRZOFgXr66N5OfEamn59CD52L4xV 546oJYhNFVDOsMVGi5ukeBqmCKNsBQLxV5HgVceq72qCEt+B0/CgEZ0RenvKSykQrKNB Ou5cUqZ8xgftMk0+KbL/8zGMZmKpbssPZAtAAI9o03G+MT6TrF1jGGLVuoHrqGxEwbzU nQ+A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:subject:in-reply-to:cc:from:to:message-id :mime-version:content-transfer-encoding; bh=W8H72798it9OolNWp4JLWdlboyq7CodcHmCStG5d+3I=; b=pbTqRRm6oA2gVP9+6zwd0vuOr6mQMHx/GqQhiRYjIEhLA9f9irhD4HlroTvHMmcLvy 3vbbukwZmhX+9sB0h9xK/NHW2euQ3UJK3fzlEa5mmIpt8ihg3JoQqq4pWdcpjiKRD186 76oZcOD80+yl0QxpbYX1+GIiRjWnv3TX8sW9/b8mhwQKY9SQYdgYsZcNmfSrsgKuPsg1 SwwipvfF15gbV51dt8pJPXD1t5MfpuaGxeyG6olTmVPh0HV2tx19ss3UeiZM1reLiq9W YmrE+36VuA8nZxSVurvwJ42ouuXAfnurjRRz60yY9HcNnbfCej+0CeTuE9sQ1bUBZ5TB Un/Q== X-Gm-Message-State: AOAM531m1sOcfMXpxvCR79EDfxhb4XE94I+GMOAZnwYsQkkcLDip15RD xrRT4qhgq7ODGZWLThTTbOtipg== X-Google-Smtp-Source: ABdhPJwl5btd1ncxM6rY0CAXQzLiRZjgQVJn8fY2NbH9I7FpzCjOQ2Za5LRVxptThmI9urwNGwsHQw== X-Received: by 2002:a63:6dca:: with SMTP id i193mr837247pgc.81.1615952166796; Tue, 16 Mar 2021 20:36:06 -0700 (PDT) Received: from localhost (76-210-143-223.lightspeed.sntcca.sbcglobal.net. [76.210.143.223]) by smtp.gmail.com with ESMTPSA id s22sm724266pjs.42.2021.03.16.20.36.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 16 Mar 2021 20:36:05 -0700 (PDT) Date: Tue, 16 Mar 2021 20:36:05 -0700 (PDT) X-Google-Original-Date: Tue, 16 Mar 2021 20:33:16 PDT (-0700) Subject: Re: [PATCH 1/4] irqchip: riscv: Using CPUHP_AP_ONLINE_DYN In-Reply-To: <1614608902-85038-1-git-send-email-guoren@kernel.org> CC: guoren@kernel.org, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org, linux-csky@vger.kernel.org, linux-arch@vger.kernel.org, guoren@linux.alibaba.com, peterz@infradead.org, tglx@linutronix.de, Arnd Bergmann , Linus Torvalds , Anup Patel , Atish Patra , Christoph Hellwig From: Palmer Dabbelt To: guoren@kernel.org Message-ID: Mime-Version: 1.0 (MHng) Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-csky@vger.kernel.org On Mon, 01 Mar 2021 06:28:19 PST (-0800), guoren@kernel.org wrote: > From: Guo Ren > > Remove RISC-V irqchip custom definitions in hotplug.h: > - CPUHP_AP_IRQ_RISCV_STARTING > - CPUHP_AP_IRQ_SIFIVE_PLIC_STARTING > > For coding convention. > > Cc: Peter Zijlstra > Cc: Thomas Gleixner > Cc: Arnd Bergmann > Cc: Linus Torvalds > Cc: Palmer Dabbelt > Cc: Anup Patel > Cc: Atish Patra > Cc: Christoph Hellwig > Tested-by: Guo Ren > Signed-off-by: Guo Ren > Link: https://lore.kernel.org/lkml/CAHk-=wjM+kCsKqNdb=c0hKsv=J7-3Q1zmM15vp6_=8S5XfGMtA@mail.gmail.com/ > --- > drivers/irqchip/irq-riscv-intc.c | 2 +- > drivers/irqchip/irq-sifive-plic.c | 2 +- > include/linux/cpuhotplug.h | 2 -- > 3 files changed, 2 insertions(+), 4 deletions(-) > > diff --git a/drivers/irqchip/irq-riscv-intc.c b/drivers/irqchip/irq-riscv-intc.c > index 8017f6d..2c37f3a 100644 > --- a/drivers/irqchip/irq-riscv-intc.c > +++ b/drivers/irqchip/irq-riscv-intc.c > @@ -125,7 +125,7 @@ static int __init riscv_intc_init(struct device_node *node, > return rc; > } > > - cpuhp_setup_state(CPUHP_AP_IRQ_RISCV_STARTING, > + cpuhp_setup_state(CPUHP_AP_ONLINE_DYN, > "irqchip/riscv/intc:starting", > riscv_intc_cpu_starting, > riscv_intc_cpu_dying); > diff --git a/drivers/irqchip/irq-sifive-plic.c b/drivers/irqchip/irq-sifive-plic.c > index 6f432d2..f499f1b 100644 > --- a/drivers/irqchip/irq-sifive-plic.c > +++ b/drivers/irqchip/irq-sifive-plic.c > @@ -375,7 +375,7 @@ static int __init plic_init(struct device_node *node, > */ > handler = this_cpu_ptr(&plic_handlers); > if (handler->present && !plic_cpuhp_setup_done) { > - cpuhp_setup_state(CPUHP_AP_IRQ_SIFIVE_PLIC_STARTING, > + cpuhp_setup_state(CPUHP_AP_ONLINE_DYN, > "irqchip/sifive/plic:starting", > plic_starting_cpu, plic_dying_cpu); > plic_cpuhp_setup_done = true; > diff --git a/include/linux/cpuhotplug.h b/include/linux/cpuhotplug.h > index f14adb8..14f49fd 100644 > --- a/include/linux/cpuhotplug.h > +++ b/include/linux/cpuhotplug.h > @@ -103,8 +103,6 @@ enum cpuhp_state { > CPUHP_AP_IRQ_ARMADA_XP_STARTING, > CPUHP_AP_IRQ_BCM2836_STARTING, > CPUHP_AP_IRQ_MIPS_GIC_STARTING, > - CPUHP_AP_IRQ_RISCV_STARTING, > - CPUHP_AP_IRQ_SIFIVE_PLIC_STARTING, > CPUHP_AP_ARM_MVEBU_COHERENCY, > CPUHP_AP_MICROCODE_LOADER, > CPUHP_AP_PERF_X86_AMD_UNCORE_STARTING, Acked-by: Palmer Dabbelt I'm going to assume this is going in through an irqchip tree, but LMK if you want me to take it via mine. This isn't really my sort of thing, so I'd prefer at least an Ack. Thanks!