Re: [RFC PATCH 04/14] ACPI: Add ACPI 5.0 Time and Alarm Device driver

[Rafael J. Wysocki]

On Thursday, January 09, 2014 11:47:39 AM joeyli wrote:
> æ äï2014-01-08 æ 09:56 -0800ïH. Peter Anvin æåï
> [...]
> > > Document of Windows XP:
> > >
> > http://www.freelists.org/post/windows_errors/what-error-messages-really-mean-WinXP-IO-Ports-Blocked-from-Bios-AML-on-Windows-XP
> > > 
> > > If just for ACPI TAD testing, we can remove the port protection
> > check of
> > > RTC ports in hwvalid.c. I have read 0x70/0x71 port success after
> > removed
> > > the checking in acpica/hwvalid.c.
> > > 
> > > I will try to write RTC port in AML after remove acpica check, maybe
> > > have other unpredictable situation.
> > > 
> > 
> > Now *THERE* is a good use of the "no RTC bit".  In the case that bit
> > is
> > set we should presumably remove these ports from the block list.
> 
> Thanks for your suggestion, I will put a testing patch on this.
> 
> > 
> > Otherwise we should use the CMOS address space, not the I/O port
> > address
> > space.
> > 
> >         -hpa 
> 
> Unfortunately current acpica leaks the SystemCMOS handler:
> 
> ACPI Error: Region SystemCMOS (ID=5) has no handler (20131115/exfldio-299)

This is not an ACPICA problem.  We have no CMOS address space handler in Linux
and arguably there should be one.  So I'd suggest implementing that to start
with or we risk conflicts between kernel code and AML accessing registers in
that space.

Thanks!

-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/