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-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
