From mboxrd@z Thu Jan 1 00:00:00 1970 From: Juri Lelli Subject: Re: [RFT PATCH v1 3/4] cpu-topology: Move cpu topology code to common code. Date: Tue, 4 Dec 2018 10:50:43 +0100 Message-ID: <20181204095043.GD23553@localhost.localdomain> References: <1543534100-3654-1-git-send-email-atish.patra@wdc.com> <1543534100-3654-4-git-send-email-atish.patra@wdc.com> <20181203165833.GA27796@arm.com> <20181203171223.GD17883@e107155-lin> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20181203171223.GD17883@e107155-lin> Sender: linux-kernel-owner@vger.kernel.org To: Sudeep Holla Cc: Will Deacon , Greg Kroah-Hartman , Atish Patra , linux-kernel@vger.kernel.org, Albert Ou , Anup Patel , Ard Biesheuvel , Catalin Marinas , devicetree@vger.kernel.org, Dmitriy Cherkasov , Ingo Molnar , Jeremy Linton , "moderated list:ARM64 PORT (AARCH64 ARCHITECTURE)" , linux-riscv@lists.infradead.org, Mark Rutland , Morten Rasmussen , Palmer Dabbelt , "Peter Zijlstra (Intel)" , Rafael J. Wys List-Id: devicetree@vger.kernel.org Hi, On 03/12/18 17:12, Sudeep Holla wrote: > (Fixed Juri Lelli's email) > > On Mon, Dec 03, 2018 at 04:58:34PM +0000, Will Deacon wrote: > > On Thu, Nov 29, 2018 at 03:28:19PM -0800, Atish Patra wrote: > > > Both RISC-V & ARM64 are using cpu-map device tree to describe > > > their cpu topology. It's better to move the relevant code to > > > a common place instead of duplicate code. > > > > > > Signed-off-by: Atish Patra > > > --- > > > arch/arm64/include/asm/topology.h | 22 --- > > > arch/arm64/kernel/topology.c | 303 +------------------------------------- > > > drivers/base/arch_topology.c | 294 ++++++++++++++++++++++++++++++++++++ > > > > Whilst I'm happy seeing this code moved out, I'd like to have an entry in > > MAINTAINERS so that somebody can review changes from the arm64 side without > > having to poll a load of different lists. I assume patches would still go > > via Rafael and Greg, but if we could have Sudeep added as Reviewer for this > > file, then I'd really appreciate it. > > > > Thanks Will for pointing this. I agree with you and like to be added as > reviewer for this. > > Since arch_topology.c was mostly added for ARM/ARM64 and by Juri Lelli > (who is no longer with ARM) and I did review the initial version, I would > like to assume maintenance for this file if both Juri and Greg are OK > with it so that I can keep an eye on this as we expect more changes to > this both for ARM and RISC-V. Sure, this is OK with me. Thanks for keeping an eye on this. Best, - Juri