Re: [PATCH v2 1/2] i2c_imc: New driver for Intel's iMC, found on LGA2011 chips

[Guenter Roeck <linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>]

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