From: Catalin Marinas <catalin.marinas@arm.com>
To: Lorenzo Pieralisi <lpieralisi@kernel.org>
Cc: Sudeep Holla <sudeep.holla@arm.com>,
will@kernel.org, linux-acpi@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
"Rafael J . Wysocki" <rafael@kernel.org>,
"Rafael J . Wysocki" <rafael.j.wysocki@intel.com>,
Robin Murphy <robin.murphy@arm.com>,
Hanjun Guo <guohanjun@huawei.com>
Subject: Re: [PATCH v2] ACPI: bus: Consolidate all arm specific initialisation into acpi_arm_init()
Date: Thu, 8 Jun 2023 08:07:16 +0100 [thread overview]
Message-ID: <ZIF+JBcEsgjEzPf4@arm.com> (raw)
In-Reply-To: <ZICQIqdbYJlJkI5E@lpieralisi>
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 <lpieralisi@kernel.org>
> > Cc: Rafael J. Wysocki <rafael@kernel.org>
> > Suggested-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> > Reviewed-by: Robin Murphy <robin.murphy@arm.com>
> > Reviewed-by: Hanjun Guo <guohanjun@huawei.com>
> > Link: https://lore.kernel.org/r/CAJZ5v0iBZRZmV_oU+VurqxnVMbFN_ttqrL=cLh0sUH+=u0PYsw@mail.gmail.com
> > Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
> > ---
> > 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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-06-08 7:07 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-06 9:35 [PATCH v2] ACPI: bus: Consolidate all arm specific initialisation into acpi_arm_init() Sudeep Holla
2023-06-06 9:50 ` Lorenzo Pieralisi
2023-06-06 12:02 ` Shaoqin Huang
2023-06-06 14:26 ` Rafael J. Wysocki
2023-06-07 14:11 ` Lorenzo Pieralisi
2023-06-08 7:07 ` Catalin Marinas [this message]
2023-06-08 18:17 ` Catalin Marinas
2023-10-07 0:11 ` D Scott Phillips
2023-10-09 12:29 ` Hanjun Guo
2023-10-09 13:05 ` Sudeep Holla
2023-10-10 7:16 ` Hanjun Guo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZIF+JBcEsgjEzPf4@arm.com \
--to=catalin.marinas@arm.com \
--cc=guohanjun@huawei.com \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=lpieralisi@kernel.org \
--cc=rafael.j.wysocki@intel.com \
--cc=rafael@kernel.org \
--cc=robin.murphy@arm.com \
--cc=sudeep.holla@arm.com \
--cc=will@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).