On Wednesday 16 December 2009 07:40:31 am yakui.zhao@xxxxxxxxx wrote: > From: Zhao Yakui <yakui.zhao@xxxxxxxxx> > > V2->V3: According to Bjorn's comment, this IPMI opregion code is put into > the IPMI subsystem(ipmi_si_intf.c). In such case the part of IPMI opregion > code will be installed automatically when IPMI detection module is loaded. > When the IPMI system interface is detected by loading PNP device driver, we > will record every IPMI system interface defined in ACPI namespace and then > install the corresponding IPMI opregion space handler, which is used to enable > ACPI AML code to access the BMC controller. > > V1->V2: According to Bjorn's comment, we won't install the IPMI space handler > by loading an ACPI device driver. Instead we will enumerate the ACPI IPMI > device directly in ACPI device tree and then install the IPMI space handler. > Then ACPI AML code and access the BMC controller through the IPMI space > handler. Oh, BTW, it's handy if you add a [0/n] email with the overview, and put the V2->V3 stuff in there. The [0/n] text doesn't become part of the commit, so the logs don't get cluttered with all the development history. Also, please include the version in the subject somewhere, e.g., "[PATCH v3 0/n] ..." Bjorn -- 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