Messages in this thread | | | Date | Wed, 17 Jun 2015 08:12:51 -0700 | From | Guenter Roeck <> | Subject | Re: [PATCH v2 1/2] i2c_imc: New driver for Intel's iMC, found on LGA2011 chips |
| |
On Wed, Jun 17, 2015 at 03:18:42PM +0200, Wolfram Sang wrote: > > > > Having said that, I am still not convinced that the driver should be in the kernel > > > to start with. Browsing through Intel's datasheets, the registers are supported > > > in E5-2600 v1, v2, and v3. However, in v3 Intel added a note saying that the registers > > > should not be accessed by the OS directly, but only through the bios. Given that, > > > and if that is possible, it might make more sense to rely on ACPI. It would then > > > be up to the board and/or BIOS vendor to decide if the information should be available > > > to the OS or not. > > > > I think the plan is to add something to ACPI to tell us when we can > > use these registers. Unfortunately I'm not privy to whatever the ACPI > > committee is doing. > > Coming from the embedded world and not knowing much about this, I tend > to wait until the above issue is clear, meaning when and how this driver > is safe to be used. I hope you don't mind. Keeping your code up to date > and sharing it is much appreciated, though! As well as Guenter's comments. > When I suggested that we should rely on ACPI, I meant through ACPI SMBus CMI (i2c-scmi), not through some ACPI properties which would permit direct access.
Thanks, Guenter
| |