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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id E0112C7EE25 for ; Thu, 8 Jun 2023 07:07:24 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235195AbjFHHHY (ORCPT ); Thu, 8 Jun 2023 03:07:24 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47218 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235092AbjFHHHX (ORCPT ); Thu, 8 Jun 2023 03:07:23 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D39CE269A for ; Thu, 8 Jun 2023 00:07:21 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 5960661FD1 for ; Thu, 8 Jun 2023 07:07:21 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 37EECC433EF; Thu, 8 Jun 2023 07:07:19 +0000 (UTC) Date: Thu, 8 Jun 2023 08:07:16 +0100 From: Catalin Marinas To: Lorenzo Pieralisi Cc: Sudeep Holla , will@kernel.org, linux-acpi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, "Rafael J . Wysocki" , "Rafael J . Wysocki" , Robin Murphy , Hanjun Guo Subject: Re: [PATCH v2] ACPI: bus: Consolidate all arm specific initialisation into acpi_arm_init() Message-ID: References: <20230606093531.2746732-1-sudeep.holla@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org On Wed, Jun 07, 2023 at 04:11:46PM +0200, Lorenzo Pieralisi wrote: > [+Catalin, Will] > > On Tue, Jun 06, 2023 at 10:35:31AM +0100, Sudeep Holla wrote: > > Move all of the ARM-specific initialization into one function namely > > acpi_arm_init(), so it is not necessary to modify/update bus.c every > > time a new piece of it is added. > > > > Cc: Lorenzo Pieralisi > > Cc: Rafael J. Wysocki > > Suggested-by: Rafael J. Wysocki > > Reviewed-by: Robin Murphy > > Reviewed-by: Hanjun Guo > > Link: https://lore.kernel.org/r/CAJZ5v0iBZRZmV_oU+VurqxnVMbFN_ttqrL=cLh0sUH+=u0PYsw@mail.gmail.com > > Signed-off-by: Sudeep Holla > > --- > > drivers/acpi/arm64/Makefile | 2 +- > > drivers/acpi/arm64/agdi.c | 2 +- > > drivers/acpi/arm64/apmt.c | 2 +- > > drivers/acpi/arm64/init.c | 13 +++++++++++++ > > drivers/acpi/arm64/init.h | 6 ++++++ > > drivers/acpi/arm64/iort.c | 1 + > > drivers/acpi/bus.c | 7 +------ > > include/linux/acpi.h | 6 ++++++ > > include/linux/acpi_agdi.h | 13 ------------- > > include/linux/acpi_apmt.h | 19 ------------------- > > include/linux/acpi_iort.h | 2 -- > > 11 files changed, 30 insertions(+), 43 deletions(-) > > create mode 100644 drivers/acpi/arm64/init.c > > create mode 100644 drivers/acpi/arm64/init.h > > delete mode 100644 include/linux/acpi_agdi.h > > delete mode 100644 include/linux/acpi_apmt.h > > Hi Catalin/Will, > > this patch has got all tags required and it is best to send > arm64 ACPI changes upstream through the arm64 tree for > consistency, can you pick it up for v6.5 please ? Happy to pick it up. Thanks Lorenzo. -- Catalin