Re: [PATCH 1/3] hpet: allow fixed_mem32 ACPI resource type

[Bjorn Helgaas <bjorn.helgaas@hp.com>]

On Wednesday 12 October 2005 12:58 pm, Randy Dunlap wrote:
> Allow the ACPI HPET description table to use a resource type
> of FIXED_MEM32 for the HPET reource.  Use the fixed resoure
> size of 1 KB for the HPET resource as per the HPET spec.

I have a patch in my tree to convert HPET from an ACPI
driver to a PNP driver, using PNPACPI.  That should take
care of issues like this.

But my patch is waiting on some PNP work by Adam to allow
PNPACPI devices to have more than 2 IRQs.

In the meantime, I think your patch is fine.

> +#define HPET_RANGE_SIZE		1024	/* from HPET spec */

Out of curiosity, why do you need this?  ACPI_RSTYPE_FIXED_MEM32
contains a length field, and my patch uses it.  Did you run
into some firmware that supplies incorrect information about the
size of the HPET MMIO area?

Another minor HPET nit I fixed is that it currently doesn't
use request_mem_region().  I did it in PNP terms, so it's
waiting on Adam's work, but maybe it'd be worth an interim
patch until that's ready.