On Mon, Jun 27, 2016 at 12:13:19PM -0700, Alison Schofield wrote: [ ... ] > > > > Great. Can you send a patch for the jc42 driver ? > > > > Thanks, > > Guenter > > > Guenther, > > I have the patch, but am stumbling on the changelog. > > I hoped to simply state in the changelog that MCP9808 is a jc-42 > compliant sensor, but I don't think it's 100% true, and I'm not > sure of where it falls short of compliance. > > jc42 currently supports these Microchip sensors: > MCP9804, MCP9805, MCP98242, MCP98243, MCP98244, MCP9843 > > All, with the exception of 9804 (and my 9808), clearly state they > are jc-42 compliant in their datasheet. > > 9804 is 9808's twin and I've tested with 9808. > > Wondering if the non-compliance has to do with the lack of a capability > register for 9804 & 9808, which doesn't affect the drivers functioning. > The capabilities register is there, it is just not officially supported. It does report useful values, though, at least as far as we need it. The chip does have a resolution register, which doesn't exist in JC-42. " hwmon: (jc42) Add support for Microchip MCP9808 MCP9808 is not officially compliant to JC-42, similar to MCP9804, but its registers are compatible to JC-42. " is good enough. If you want to be fancy, add "The sensor resolution register is not currently supported". Hope this helps, Guenter -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
