On Wed, Aug 26, 2015 at 08:22:33AM +0000, Zha, Qipeng wrote: > On Tue, Aug 18, 2015 at 04:50:56PM +0100, Matt Fleming wrote: > > On Mon, 2015-08-10 at 16:04 +0100, Shevchenko, Andriy wrote: > > > > > > Matt, by the way intel_pmc_ipc.c module creates iTCO device (LPC bus). > > > Does it look correct? > > > > Yeah, it looks OK to me. Note there are changes queued up in Lee Jones' > > tree for changing the lpc_ich_info into itco_wdt_platform_data. > > > > Btw, is there a reason that intel_pmc_ipc doesn't use the mfd_* API > > since it's concerned with multiple functions? > > > Qipeng? > > > I saw the v2 come in, but didn't see this question addressed in the changelog. > > When design P-unit driver, we got feedback from BIOS that there will allocate a dedicate > ACPI id for P-unit controller, and finally BIOS decide not do that, just put some necessary > Resource in PMC acpi table. > After that, we got request to do same thing for iTco. > > I agree it's better to update pmc driver to use mfd_add_devices to create punit and iTco device. Does this mean there sill be a v4 I should be waiting for? -- Darren Hart Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
