From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756039AbdJJJiR (ORCPT ); Tue, 10 Oct 2017 05:38:17 -0400 Received: from usa-sjc-mx-foss1.foss.arm.com ([217.140.101.70]:41008 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751420AbdJJJiP (ORCPT ); Tue, 10 Oct 2017 05:38:15 -0400 Subject: Re: [PATCH v7 3/6] coresight: of: Use of_cpu_node_to_id helper To: Leo Yan Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, mark.rutland@arm.com, robh@kernel.org, will.deacon@arm.com, sudeep.holla@arm.com, frowand.list@gmail.com, devicetree@vger.kernel.org, Jonathan.Cameron@huawei.com, marc.zyngier@arm.com, peterz@infradead.org, mathieu.poirier@linaro.org References: <20171006103610.11853-1-suzuki.poulose@arm.com> <20171006103610.11853-4-suzuki.poulose@arm.com> <20171009025842.GA3146@leoy-linaro> From: Suzuki K Poulose Message-ID: <4518662b-01ce-30fd-a9ea-c517b2d1c5ff@arm.com> Date: Tue, 10 Oct 2017 10:38:11 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <20171009025842.GA3146@leoy-linaro> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09/10/17 03:58, Leo Yan wrote: > Hi Suzuki, > > On Fri, Oct 06, 2017 at 11:36:07AM +0100, Suzuki K Poulose wrote: >> Reuse the new generic helper, of_cpu_node_to_id() to map a >> given CPU phandle to a logical CPU number. >> >> Cc: Leo Yan >> Acked-by: Mathieu Poirier >> Signed-off-by: Suzuki K Poulose > > Tested-by: Leo Yan > > BTW, I think the function get_cpu_for_node() in the file > arch/arm64/kernel/topology.c we also can refactor with the same > function, just reminding for this. > Leo, Thanks for the testing and the pointer. Yes we can move that to use the new helper. On another take, I found that the arm_pmu can also use it for pmu_parse_irq_affinity(). I will add them to the series. Cheers Suzuki