Re: [PATCH 11/12] ACPI: use of ACPI_FADT_32BIT_TIMER is not allowed in HW reduced mode

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx wrote:
From: Al Stone <ahs3@xxxxxxxxxx>

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@xxxxxxxxxx>
---
  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;





--
ciao,
al
-----------------------------------
Al Stone
Software Engineer
Linaro Enterprise Group
al.stone@xxxxxxxxxx
-----------------------------------
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux