From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sudeep Holla Subject: Re: [RFT PATCH v1 3/4] cpu-topology: Move cpu topology code to common code. Date: Mon, 3 Dec 2018 17:12:24 +0000 Message-ID: <20181203171223.GD17883@e107155-lin> 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> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20181203165833.GA27796@arm.com> Sender: linux-kernel-owner@vger.kernel.org To: Will Deacon , Juri Lelli , Greg Kroah-Hartman Cc: 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. Wysocki" , Rob Herring , Thomas Gleixner List-Id: devicetree@vger.kernel.org (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. I am fine if anyone who has contributed more to this file wants to maintain. -- Regards, Sudeep