[PATCH] ACPI CMOS region support rev. 6
Anthony Jenkins
Anthony.B.Jenkins at att.net
Fri Jun 26 01:14:58 UTC 2015
Sooo here's the new and improved rev. 6, "new and improved" because it
increases the diff line count by 332%.
> [ajenkins at ajenkins-hplaptop /usr/src]$ wc -l atrtc_c_rev5.diff
> atrtc_rev6.diff
> 220 atrtc_c_rev5.diff
> 731 atrtc_rev6.diff
This is to satisfy the request to split the atrtc.c driver into a "core"
part and "bus" parts:
> Looking at patch 5:
>
> You need to rework this so there’s an atrtc_acpi.c. Put all the ACPI attachment in there. You should also split off the little bit that’s ISA-specific into atrtc_isa. Once you do that, we can talk.
>
> Warner
I actually finished this patch a couple months ago and have been running
it on my laptop, I just don't see the point...maybe Warner could
elaborate on the rationale for his request? At the very least, this
should be two commits - the functional change and the refactorization
request.
I may have also added Ian's request for verbosity tweaks, but it's been
a while... I don't see any CMOS logging noise FWIW.
Thanks,
Anthony
More information about the freebsd-acpi
mailing list