mbox series

[v2,0/4] ACPI: Enable Platform Runtime Mechanism(PRM) support on ARM64

Message ID 20220628125346.693304-1-sudeep.holla@arm.com
Headers show
Series ACPI: Enable Platform Runtime Mechanism(PRM) support on ARM64 | expand

Message

Sudeep Holla June 28, 2022, 12:53 p.m. UTC
Hi,

This tiny series is to enable Platform Runtime Mechanism(PRM) support on
ARM64. Not much changes. Just enabling the build and moving the config
option so that it is not listed under the topmost menu for ARM64.

Regards,
Sudeep

v1[1]->v2:
	- Changed arch_efi_call_virt() to use typeof instead of forced
	  typecasting to void pointer
	- Added change in PRMT driver to move handler_addr type to void
	  pointer

[1] https://lore.kernel.org/linux-acpi/20220624152331.4009502-1-sudeep.holla@arm.com

Sudeep Holla (4):
  ACPI: PRM: Change handler_addr type to void pointer
  arm64: efi: Simplify arch_efi_call_virt() macro by using typeof()
  ACPI: Enable Platform Runtime Mechanism(PRM) support on ARM64
  ACPI: Move PRM config option under the main ACPI config

 arch/arm64/include/asm/efi.h |  3 +--
 drivers/acpi/Kconfig         | 30 +++++++++++++++---------------
 drivers/acpi/prmt.c          |  4 ++--
 3 files changed, 18 insertions(+), 19 deletions(-)

--
2.37.0

Comments

Sudeep Holla June 29, 2022, 9:06 a.m. UTC | #1
Hi Rafael,

On Tue, Jun 28, 2022 at 01:53:42PM +0100, Sudeep Holla wrote:
> Hi,
> 
> This tiny series is to enable Platform Runtime Mechanism(PRM) support on
> ARM64. Not much changes. Just enabling the build and moving the config
> option so that it is not listed under the topmost menu for ARM64.
> 

Ard has slightly modified version of patch 2/4 which updates few other
arch code. Since the PRMT driver build is dependent on that, is it OK
for you to ACK if you agree with other 3 patches(1,3,4) so that it can
be routed via efi tree. Alternatively, Ard has offered to provide a stable
branch if needed.
Sudeep Holla June 29, 2022, 9:21 a.m. UTC | #2
On Wed, Jun 29, 2022 at 10:06:35AM +0100, Sudeep Holla wrote:
> Hi Rafael,
> 
> On Tue, Jun 28, 2022 at 01:53:42PM +0100, Sudeep Holla wrote:
> > Hi,
> > 
> > This tiny series is to enable Platform Runtime Mechanism(PRM) support on
> > ARM64. Not much changes. Just enabling the build and moving the config
> > option so that it is not listed under the topmost menu for ARM64.
> > 
> 
> Ard has slightly modified version of patch 2/4 which updates few other
> arch code. Since the PRMT driver build is dependent on that, is it OK
> for you to ACK if you agree with other 3 patches(1,3,4) so that it can
> be routed via efi tree. Alternatively, Ard has offered to provide a stable
> branch if needed.

Sorry pressed enter too early. I wanted to add reference to the commit in
Ard's efi queue I was referring[1] and the email where we discussed how
to deal with these PRMT patches[2]
Rafael J. Wysocki June 29, 2022, 7:12 p.m. UTC | #3
On Wed, Jun 29, 2022 at 11:22 AM Sudeep Holla <sudeep.holla@arm.com> wrote:
>
> On Wed, Jun 29, 2022 at 10:06:35AM +0100, Sudeep Holla wrote:
> > Hi Rafael,
> >
> > On Tue, Jun 28, 2022 at 01:53:42PM +0100, Sudeep Holla wrote:
> > > Hi,
> > >
> > > This tiny series is to enable Platform Runtime Mechanism(PRM) support on
> > > ARM64. Not much changes. Just enabling the build and moving the config
> > > option so that it is not listed under the topmost menu for ARM64.
> > >
> >
> > Ard has slightly modified version of patch 2/4 which updates few other
> > arch code. Since the PRMT driver build is dependent on that, is it OK
> > for you to ACK if you agree with other 3 patches(1,3,4) so that it can
> > be routed via efi tree. Alternatively, Ard has offered to provide a stable
> > branch if needed.
>
> Sorry pressed enter too early. I wanted to add reference to the commit in
> Ard's efi queue I was referring[1] and the email where we discussed how
> to deal with these PRMT patches[2]

The ACPI changes in this series are fine with me, so please feel free to add

Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

to the patches and route them through the EFI tree.

Thanks!
Sudeep Holla June 30, 2022, 1:02 p.m. UTC | #4
On Wed, Jun 29, 2022 at 09:12:39PM +0200, Rafael J. Wysocki wrote:
> On Wed, Jun 29, 2022 at 11:22 AM Sudeep Holla <sudeep.holla@arm.com> wrote:
> >
> > On Wed, Jun 29, 2022 at 10:06:35AM +0100, Sudeep Holla wrote:
> > > Hi Rafael,
> > >
> > > On Tue, Jun 28, 2022 at 01:53:42PM +0100, Sudeep Holla wrote:
> > > > Hi,
> > > >
> > > > This tiny series is to enable Platform Runtime Mechanism(PRM) support on
> > > > ARM64. Not much changes. Just enabling the build and moving the config
> > > > option so that it is not listed under the topmost menu for ARM64.
> > > >
> > >
> > > Ard has slightly modified version of patch 2/4 which updates few other
> > > arch code. Since the PRMT driver build is dependent on that, is it OK
> > > for you to ACK if you agree with other 3 patches(1,3,4) so that it can
> > > be routed via efi tree. Alternatively, Ard has offered to provide a stable
> > > branch if needed.
> >
> > Sorry pressed enter too early. I wanted to add reference to the commit in
> > Ard's efi queue I was referring[1] and the email where we discussed how
> > to deal with these PRMT patches[2]
> 
> The ACPI changes in this series are fine with me, so please feel free to add
> 
> Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> 
> to the patches and route them through the EFI tree.
> 

Thanks Rafael.

Hi Ard,

I assume you will pick up the patches(except 2/4 which is already in your
-next) with Rafael's ack.
Ard Biesheuvel June 30, 2022, 3:36 p.m. UTC | #5
On Thu, 30 Jun 2022 at 15:03, Sudeep Holla <sudeep.holla@arm.com> wrote:
>
> On Wed, Jun 29, 2022 at 09:12:39PM +0200, Rafael J. Wysocki wrote:
> > On Wed, Jun 29, 2022 at 11:22 AM Sudeep Holla <sudeep.holla@arm.com> wrote:
> > >
> > > On Wed, Jun 29, 2022 at 10:06:35AM +0100, Sudeep Holla wrote:
> > > > Hi Rafael,
> > > >
> > > > On Tue, Jun 28, 2022 at 01:53:42PM +0100, Sudeep Holla wrote:
> > > > > Hi,
> > > > >
> > > > > This tiny series is to enable Platform Runtime Mechanism(PRM) support on
> > > > > ARM64. Not much changes. Just enabling the build and moving the config
> > > > > option so that it is not listed under the topmost menu for ARM64.
> > > > >
> > > >
> > > > Ard has slightly modified version of patch 2/4 which updates few other
> > > > arch code. Since the PRMT driver build is dependent on that, is it OK
> > > > for you to ACK if you agree with other 3 patches(1,3,4) so that it can
> > > > be routed via efi tree. Alternatively, Ard has offered to provide a stable
> > > > branch if needed.
> > >
> > > Sorry pressed enter too early. I wanted to add reference to the commit in
> > > Ard's efi queue I was referring[1] and the email where we discussed how
> > > to deal with these PRMT patches[2]
> >
> > The ACPI changes in this series are fine with me, so please feel free to add
> >
> > Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> >
> > to the patches and route them through the EFI tree.
> >
>
> Thanks Rafael.
>
> Hi Ard,
>
> I assume you will pick up the patches(except 2/4 which is already in your
> -next) with Rafael's ack.
>

OK, I've queued them up now.