* [PATCH v2] ACPI: Fixes device power states array overflow
@ 2011-06-01 15:54 Lin Ming
2011-06-14 2:53 ` Lin Ming
2011-07-14 3:54 ` Len Brown
0 siblings, 2 replies; 3+ messages in thread
From: Lin Ming @ 2011-06-01 15:54 UTC (permalink / raw)
To: Len Brown; +Cc: linux-acpi, Dan Carpenter, Old
Commit 28c2103 added new state ACPI_STATE_D3_COLD, so the device power
states array must be expanded by one also.
v2: Use ACPI_D_STATE_COUNT instead of number 5 for the array size.
Reported-by: Dan Carpenter <error27@gmail.com>
Suggested-by: Oldřich Jedlička <oldium.pro@seznam.cz>
Signed-off-by: Lin Ming <ming.m.lin@intel.com>
---
include/acpi/acpi_bus.h | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/include/acpi/acpi_bus.h b/include/acpi/acpi_bus.h
index 3a10ef5..6cd5b64 100644
--- a/include/acpi/acpi_bus.h
+++ b/include/acpi/acpi_bus.h
@@ -210,7 +210,7 @@ struct acpi_device_power_state {
struct acpi_device_power {
int state; /* Current state */
struct acpi_device_power_flags flags;
- struct acpi_device_power_state states[4]; /* Power states (D0-D3) */
+ struct acpi_device_power_state states[ACPI_D_STATE_COUNT]; /* Power states (D0-D3Cold) */
};
/* Performance Management */
--
1.7.2.3
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v2] ACPI: Fixes device power states array overflow
2011-06-01 15:54 [PATCH v2] ACPI: Fixes device power states array overflow Lin Ming
@ 2011-06-14 2:53 ` Lin Ming
2011-07-14 3:54 ` Len Brown
1 sibling, 0 replies; 3+ messages in thread
From: Lin Ming @ 2011-06-14 2:53 UTC (permalink / raw)
To: Lin Ming; +Cc: Len Brown, linux-acpi, Dan Carpenter, Old
On Wed, Jun 1, 2011 at 11:54 PM, Lin Ming <ming.m.lin@intel.com> wrote:
> Commit 28c2103 added new state ACPI_STATE_D3_COLD, so the device power
> states array must be expanded by one also.
>
> v2: Use ACPI_D_STATE_COUNT instead of number 5 for the array size.
>
> Reported-by: Dan Carpenter <error27@gmail.com>
> Suggested-by: Oldřich Jedlička <oldium.pro@seznam.cz>
> Signed-off-by: Lin Ming <ming.m.lin@intel.com>
> ---
> include/acpi/acpi_bus.h | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/include/acpi/acpi_bus.h b/include/acpi/acpi_bus.h
> index 3a10ef5..6cd5b64 100644
> --- a/include/acpi/acpi_bus.h
> +++ b/include/acpi/acpi_bus.h
> @@ -210,7 +210,7 @@ struct acpi_device_power_state {
> struct acpi_device_power {
> int state; /* Current state */
> struct acpi_device_power_flags flags;
> - struct acpi_device_power_state states[4]; /* Power states (D0-D3) */
> + struct acpi_device_power_state states[ACPI_D_STATE_COUNT]; /* Power states (D0-D3Cold) */
> };
>
> /* Performance Management */
> --
> 1.7.2.3
Len,
Would you apply this one?
Thanks,
Lin Ming
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2] ACPI: Fixes device power states array overflow
2011-06-01 15:54 [PATCH v2] ACPI: Fixes device power states array overflow Lin Ming
2011-06-14 2:53 ` Lin Ming
@ 2011-07-14 3:54 ` Len Brown
1 sibling, 0 replies; 3+ messages in thread
From: Len Brown @ 2011-07-14 3:54 UTC (permalink / raw)
To: Lin Ming; +Cc: linux-acpi, Dan Carpenter, Old
applied
thanks,
Len Brown, Intel Open Source Technology Center
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-07-14 3:54 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-06-01 15:54 [PATCH v2] ACPI: Fixes device power states array overflow Lin Ming
2011-06-14 2:53 ` Lin Ming
2011-07-14 3:54 ` Len Brown
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.