Messages in this thread | | | From | Bjorn Helgaas <> | Subject | Re: [PATCH 1/3] hpet: allow fixed_mem32 ACPI resource type | Date | Wed, 12 Oct 2005 14:08:32 -0600 |
| |
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. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |