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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 522B3C83F1A for ; Fri, 11 Jul 2025 10:12:51 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:MIME-Version:References:In-Reply-To:Message-ID:Subject:CC:To: From:Date:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=OMh9EenLmDvJeXrgwAGeNr+BbCv/E8E4rPJVIDOIt5A=; b=3o4Kz6zD5YMN+YGBZW7evcqKm6 BTkCmlEsZxBavDE/x5aMT+15e83CbbvHc9vAPjgQ4iNHxsGfT0SDvtUuoRX3xpEdtyjx7EhcbCntA M20K8mbcXsR/AXuhVKvXz2eHztMZ4HojEtHH69VqYMrlDWpZ0ZE7ZGcJZI1Z4d2TOXMslIICSplns 3kMq3Uwh7Jkrj41WzcMw4CQ3YTtdvKwp/+F80+Bcet1AijTiPBKP+v58gteSX76DQYttfP9RIQ6DX xCa9FoS7EADvbymqyvYjGqXjrW8XNRIEUzUviB6zfnRIxHq8rFfI+t4IGsis4I5jq+KCRxLKRhTEo T2WwEWAQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1uaAkG-0000000ESND-1Pcl; Fri, 11 Jul 2025 10:12:44 +0000 Received: from frasgout.his.huawei.com ([185.176.79.56]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1uaAAI-0000000EKQP-3I34 for linux-arm-kernel@lists.infradead.org; Fri, 11 Jul 2025 09:35:36 +0000 Received: from mail.maildlp.com (unknown [172.18.186.231]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4bdmkx2Gdrz6GBkR; Fri, 11 Jul 2025 17:34:33 +0800 (CST) Received: from frapeml500008.china.huawei.com (unknown [7.182.85.71]) by mail.maildlp.com (Postfix) with ESMTPS id CDD121402EB; Fri, 11 Jul 2025 17:35:27 +0800 (CST) Received: from localhost (10.203.177.66) by frapeml500008.china.huawei.com (7.182.85.71) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.39; Fri, 11 Jul 2025 11:35:27 +0200 Date: Fri, 11 Jul 2025 10:35:25 +0100 From: Jonathan Cameron To: Alireza Sanaee CC: , , , , , , , , , , , , , , Subject: Re: [PATCH v2 1/5] of: add infra for finding CPU id from phandle Message-ID: <20250711103525.000022d7@huawei.com> In-Reply-To: <20250708151502.561-2-alireza.sanaee@huawei.com> References: <20250708151502.561-1-alireza.sanaee@huawei.com> <20250708151502.561-2-alireza.sanaee@huawei.com> X-Mailer: Claws Mail 4.3.0 (GTK 3.24.42; x86_64-w64-mingw32) MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.203.177.66] X-ClientProxiedBy: lhrpeml100002.china.huawei.com (7.191.160.241) To frapeml500008.china.huawei.com (7.182.85.71) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250711_023535_109483_C495B71F X-CRM114-Status: GOOD ( 27.09 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, 8 Jul 2025 16:14:58 +0100 Alireza Sanaee wrote: > Get CPU id from phandle. Many drivers get do this by getting hold of CPU > node first through a phandle and then find the CPU ID using the relevant > function. This commit encapsulates cpu node finding and improves > readability. > > The API interface requires two parameters, 1) node, 2) pointer to > pointer of CPU node, 3) cpu node index. API sets the pointer to the CPU > node and allows the driver to play with the CPU itself, for logging > purposes for instance. > > Signed-off-by: Alireza Sanaee > --- > drivers/of/cpu.c | 40 ++++++++++++++++++++++++++++++++++++++++ > include/linux/of.h | 9 +++++++++ > 2 files changed, 49 insertions(+) > > diff --git a/drivers/of/cpu.c b/drivers/of/cpu.c > index 5214dc3d05ae..494d47470f94 100644 > --- a/drivers/of/cpu.c > +++ b/drivers/of/cpu.c > @@ -173,6 +173,46 @@ int of_cpu_node_to_id(struct device_node *cpu_node) > } > EXPORT_SYMBOL(of_cpu_node_to_id); > > +/** > + * of_cpu_phandle_to_id: Get the logical CPU number for a given device_node > + * > + * @node: Pointer to the device_node containing CPU phandle. > + * @cpu_np: Pointer to the device_node for CPU. > + * @cpu_idx: The index of the CPU in the list of CPUs. > + * > + * Return: The logical CPU number of the given CPU device_node or -ENODEV if > + * the CPU is not found, or if the node is NULL, it returns -1. On success, > + * cpu_np will always point to the retrieved CPU device_node with refcount > + * incremented, use of_node_put() on it when done. > + */ > +int of_cpu_phandle_to_id(const struct device_node *node, > + struct device_node **cpu_np, > + uint8_t cpu_idx) > +{ > + struct device_node *local_cpu_node; > + int cpu; > + > + if (!node) > + return -1; > + > + local_cpu_node = of_parse_phandle(node, "cpu", 0); Sorry - half asleep in earlier reviews. This is only valid if cpu_idx = 0 I think? struct device_node *local_cpu_node = NULL; ... if (cpu_idx == 0) local_cpu_node = of_parse_phandle(node, "cpu", 0); if (!local_cpu_node) local_cpu_node = of_parse_phandle(node, "cpus", cpu_idx); Is probably the simplest implementation. Very unlikely we'd ever call it with the combination of cpu phandle and an index as that will be constrained by the user (currently just the DSU I think) but we should still make the code not doing crazy things if that happens. > + if (!local_cpu_node) > + local_cpu_node = of_parse_phandle(node, "cpus", cpu_idx); > + > + if (!local_cpu_node) > + return -ENODEV; > + > + cpu = of_cpu_node_to_id(local_cpu_node); > + > + if (cpu_np) > + *cpu_np = local_cpu_node; > + else > + of_node_put(local_cpu_node); > + > + return cpu; > +} > +EXPORT_SYMBOL(of_cpu_phandle_to_id); > + > /** > * of_get_cpu_state_node - Get CPU's idle state node at the given index > * > diff --git a/include/linux/of.h b/include/linux/of.h > index a62154aeda1b..717e55065d99 100644 > --- a/include/linux/of.h > +++ b/include/linux/of.h > @@ -365,6 +365,8 @@ extern const void *of_get_property(const struct device_node *node, > extern struct device_node *of_get_cpu_node(int cpu, unsigned int *thread); > extern struct device_node *of_cpu_device_node_get(int cpu); > extern int of_cpu_node_to_id(struct device_node *np); > +extern int of_cpu_phandle_to_id(const struct device_node *np, > + struct device_node **cpu_np, uint8_t cpu_idx); > extern struct device_node *of_get_next_cpu_node(struct device_node *prev); > extern struct device_node *of_get_cpu_state_node(const struct device_node *cpu_node, > int index); > @@ -680,6 +682,13 @@ static inline int of_cpu_node_to_id(struct device_node *np) > return -ENODEV; > } > > +static inline int of_cpu_phandle_to_id(const struct device_node *np, > + struct device_node **cpu_np, > + uint8_t cpu_idx) > +{ > + return -ENODEV; > +} > + > static inline struct device_node *of_get_next_cpu_node(struct device_node *prev) > { > return NULL;