On 11/20/2013 03:15 PM, Al Stone wrote:
On 11/17/2013 03:26 PM, Rafael J. Wysocki wrote:
On Saturday, November 09, 2013 06:36:21 PM al.stone@linaro.org wrote:
From: Al Stone ahs3@redhat.com
I'm reading the patch as "the timer resolution in HW reduced mode is always 32-bit". Is my reading correct?
Whups, I see what I did. You are reading the patch correctly.
However, what the code should ultimately do is ignore the ACPI_FADT_32BIT_TIMER flag completely. Let me look at where the timer_resolution field is being used also since it is not immediately clear what value it should have, or if it should even be used, when in reduced HW mode.
On further investigation, I'll take this patch out of this set. I think this needs better handling in the ACPICA code so I'll work with them on that.
In the meantime, the spec seems to be completely mum on what the timer resolution should be in reduced HW mode. What's more, it looks like acpi_get_system_info() never gets called in the first place, so this change would be irrelevant. Further, it also looks like acpi_get_timer_resolution() (which returns the timer_resolution value) never gets called either, nor is the timer_resolution used outside of the ACPICA code in any way.
So, whenever the right answer gets figured out in ACPICA, it will eventually get pulled into Linux ACPI.
Signed-off-by: Al Stone al.stone@linaro.org
drivers/acpi/acpica/utxface.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/acpi/acpica/utxface.c b/drivers/acpi/acpica/utxface.c index be322c8..fe94b3e 100644 --- a/drivers/acpi/acpica/utxface.c +++ b/drivers/acpi/acpica/utxface.c @@ -187,7 +187,8 @@ acpi_status acpi_get_system_info(struct acpi_buffer * out_buffer)
/* Timer resolution - 24 or 32 bits */
- if (acpi_gbl_FADT.flags & ACPI_FADT_32BIT_TIMER) {
- if (!acpi_gbl_reduced_hardware &&
(acpi_gbl_FADT.flags & ACPI_FADT_32BIT_TIMER)) { info_ptr->timer_resolution = 24; } else { info_ptr->timer_resolution = 32;