linux-acpi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH AUTOSEL 4.4 24/35] ACPICA: iASL: Fix for WPBT table with no command-line arguments
       [not found] <20210909120116.150912-1-sashal@kernel.org>
@ 2021-09-09 12:01 ` Sasha Levin
  2021-09-10  7:45   ` Pavel Machek
  0 siblings, 1 reply; 5+ messages in thread
From: Sasha Levin @ 2021-09-09 12:01 UTC (permalink / raw)
  To: linux-kernel, stable
  Cc: Bob Moore, Rafael J . Wysocki, Sasha Levin, linux-acpi, devel

From: Bob Moore <robert.moore@intel.com>

[ Upstream commit 87b8ec5846cb81747088d1729acaf55a1155a267 ]

Handle the case where the Command-line Arguments table field
does not exist.

ACPICA commit d6487164497fda170a1b1453c5d58f2be7c873d6

Link: https://github.com/acpica/acpica/commit/d6487164
Signed-off-by: Bob Moore <robert.moore@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
---
 include/acpi/actbl3.h | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/include/acpi/actbl3.h b/include/acpi/actbl3.h
index 1df891660f43..81d5ebc886f5 100644
--- a/include/acpi/actbl3.h
+++ b/include/acpi/actbl3.h
@@ -738,6 +738,10 @@ struct acpi_table_wpbt {
 	u16 arguments_length;
 };
 
+struct acpi_wpbt_unicode {
+	u16 *unicode_string;
+};
+
 /*******************************************************************************
  *
  * XENV - Xen Environment Table (ACPI 6.0)
-- 
2.30.2


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH AUTOSEL 4.4 24/35] ACPICA: iASL: Fix for WPBT table with no command-line arguments
  2021-09-09 12:01 ` [PATCH AUTOSEL 4.4 24/35] ACPICA: iASL: Fix for WPBT table with no command-line arguments Sasha Levin
@ 2021-09-10  7:45   ` Pavel Machek
  2021-09-11 14:38     ` Sasha Levin
  2021-09-17 19:21     ` Moore, Robert
  0 siblings, 2 replies; 5+ messages in thread
From: Pavel Machek @ 2021-09-10  7:45 UTC (permalink / raw)
  To: Sasha Levin
  Cc: linux-kernel, stable, Bob Moore, Rafael J . Wysocki, linux-acpi, devel

[-- Attachment #1: Type: text/plain, Size: 813 bytes --]

Hi!

> Handle the case where the Command-line Arguments table field
> does not exist.
> 
> ACPICA commit d6487164497fda170a1b1453c5d58f2be7c873d6

I'm not sure what is going on here, but adding unused definition will
not make any difference for 4.4 users, so we don't need this in
-stable...?

Best regards,
								Pavel

> +++ b/include/acpi/actbl3.h
> @@ -738,6 +738,10 @@ struct acpi_table_wpbt {
>  	u16 arguments_length;
>  };
>  
> +struct acpi_wpbt_unicode {
> +	u16 *unicode_string;
> +};
> +
>  /*******************************************************************************
>   *
>   * XENV - Xen Environment Table (ACPI 6.0)

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH AUTOSEL 4.4 24/35] ACPICA: iASL: Fix for WPBT table with no command-line arguments
  2021-09-10  7:45   ` Pavel Machek
@ 2021-09-11 14:38     ` Sasha Levin
  2021-09-13 12:20       ` Rafael J. Wysocki
  2021-09-17 19:21     ` Moore, Robert
  1 sibling, 1 reply; 5+ messages in thread
From: Sasha Levin @ 2021-09-11 14:38 UTC (permalink / raw)
  To: Pavel Machek
  Cc: linux-kernel, stable, Bob Moore, Rafael J . Wysocki, linux-acpi, devel

On Fri, Sep 10, 2021 at 09:45:36AM +0200, Pavel Machek wrote:
>Hi!
>
>> Handle the case where the Command-line Arguments table field
>> does not exist.
>>
>> ACPICA commit d6487164497fda170a1b1453c5d58f2be7c873d6
>
>I'm not sure what is going on here, but adding unused definition will
>not make any difference for 4.4 users, so we don't need this in
>-stable...?

Ugh, dropped, thanks!

I wonder what this patch actually does upstream.

-- 
Thanks,
Sasha

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH AUTOSEL 4.4 24/35] ACPICA: iASL: Fix for WPBT table with no command-line arguments
  2021-09-11 14:38     ` Sasha Levin
@ 2021-09-13 12:20       ` Rafael J. Wysocki
  0 siblings, 0 replies; 5+ messages in thread
From: Rafael J. Wysocki @ 2021-09-13 12:20 UTC (permalink / raw)
  To: Sasha Levin
  Cc: Pavel Machek, Linux Kernel Mailing List, Stable, Bob Moore,
	Rafael J . Wysocki, ACPI Devel Maling List,
	open list:ACPI COMPONENT ARCHITECTURE (ACPICA)

On Sat, Sep 11, 2021 at 4:38 PM Sasha Levin <sashal@kernel.org> wrote:
>
> On Fri, Sep 10, 2021 at 09:45:36AM +0200, Pavel Machek wrote:
> >Hi!
> >
> >> Handle the case where the Command-line Arguments table field
> >> does not exist.
> >>
> >> ACPICA commit d6487164497fda170a1b1453c5d58f2be7c873d6
> >
> >I'm not sure what is going on here, but adding unused definition will
> >not make any difference for 4.4 users, so we don't need this in
> >-stable...?
>
> Ugh, dropped, thanks!
>
> I wonder what this patch actually does upstream.

There are AML compiler changes in it, but the compiler is not included
into the Linux kernel.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* RE: [PATCH AUTOSEL 4.4 24/35] ACPICA: iASL: Fix for WPBT table with no command-line arguments
  2021-09-10  7:45   ` Pavel Machek
  2021-09-11 14:38     ` Sasha Levin
@ 2021-09-17 19:21     ` Moore, Robert
  1 sibling, 0 replies; 5+ messages in thread
From: Moore, Robert @ 2021-09-17 19:21 UTC (permalink / raw)
  To: Pavel Machek, Sasha Levin
  Cc: linux-kernel, stable, Wysocki, Rafael J, linux-acpi, devel

This structure is used by the iASL data table disassembler.
Bob


-----Original Message-----
From: Pavel Machek <pavel@denx.de> 
Sent: Friday, September 10, 2021 12:46 AM
To: Sasha Levin <sashal@kernel.org>
Cc: linux-kernel@vger.kernel.org; stable@vger.kernel.org; Moore, Robert <robert.moore@intel.com>; Wysocki, Rafael J <rafael.j.wysocki@intel.com>; linux-acpi@vger.kernel.org; devel@acpica.org
Subject: Re: [PATCH AUTOSEL 4.4 24/35] ACPICA: iASL: Fix for WPBT table with no command-line arguments

Hi!

> Handle the case where the Command-line Arguments table field does not 
> exist.
> 
> ACPICA commit d6487164497fda170a1b1453c5d58f2be7c873d6

I'm not sure what is going on here, but adding unused definition will not make any difference for 4.4 users, so we don't need this in -stable...?

Best regards,
								Pavel

> +++ b/include/acpi/actbl3.h
> @@ -738,6 +738,10 @@ struct acpi_table_wpbt {
>  	u16 arguments_length;
>  };
>  
> +struct acpi_wpbt_unicode {
> +	u16 *unicode_string;
> +};
> +
>  /*******************************************************************************
>   *
>   * XENV - Xen Environment Table (ACPI 6.0)

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2021-09-17 19:22 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20210909120116.150912-1-sashal@kernel.org>
2021-09-09 12:01 ` [PATCH AUTOSEL 4.4 24/35] ACPICA: iASL: Fix for WPBT table with no command-line arguments Sasha Levin
2021-09-10  7:45   ` Pavel Machek
2021-09-11 14:38     ` Sasha Levin
2021-09-13 12:20       ` Rafael J. Wysocki
2021-09-17 19:21     ` Moore, Robert

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).