On Wed Sep 30 16:57, Mika Westerberg wrote: > On Wed, Sep 30, 2015 at 02:52:28PM +0200, Rafael J. Wysocki wrote: > > > > Device (CH00) { > > > > Name (_ADR, 0x0) > > > > > > > > Device (TMP0) { > > > > Name (_ADR, 0x50) > > > > > > ... and this are not needed. I2cSerialBus already contains the address. > > > > > > Also I think you need to have "PRP0001" here as well. > > > > The idea is to use _ADR kind of instead of "PRP0001" to express the "you > > don't need a driver for this" idea AFAICS. > > But I think it needs a driver and it even includes "compatible" string > below :-) Right, CH00 has just an _ADR, but TMP0 really needs a PRP0001. > > > > Name (_CRS, ResourceTemplate() > > > > { > > > > I2cSerialBus (0x60, ControllerInitiated, I2C_SPEED, > > > > AddressingMode7Bit, "^CH00", 0x00, > > > > ResourceConsumer,,) > > > > }) > > > > Name (_DSD, Package () > > > > { > > > > ToUUID("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), > > > > Package () { > > > > Package (2) { "compatible", "national,lm75" }, > > > > } > > > > }) > > > > } > > > > } > > > > } --Dustin -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html