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

[Wolfram Sang]

> > 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.

Thanks!

Attachment: signature.asc
Description: Digital signature