On Monday, November 30, 2015 06:23:26 PM Andy Lutomirski wrote: > On Nov 30, 2015 5:41 PM, "Rafael J. Wysocki" <rjw@xxxxxxxxxxxxx> wrote: > > > > On Monday, November 30, 2015 03:45:15 PM Andy Lutomirski wrote: > > > On Nov 30, 2015 2:40 PM, "Rafael J. Wysocki" <rjw@xxxxxxxxxxxxx> wrote: > > > > > > > > On Monday, November 30, 2015 12:15:27 PM Andy Lutomirski wrote: > > > > > On Sun, Nov 29, 2015 at 9:19 PM, Andy Lutomirski <luto@xxxxxxxxxxxxxx> wrote: > > > > > > > > [cut] > > > > > > > > > > From inspection of the code, acpi_pnp.c determines that the device is > > > > > > a pnp device because pnp0c02 is in acpi_pnp_device_ids. This causes > > > > > > the device to have a concrete handler, so no platform device is > > > > > > created. However, pnpacpi/core.c rejects it because it has no _CRS > > > > > > method. As a result, it's in limbo. > > > > > > > > > > > > What's the right thing to do here? I could modify acpi_pnp.c to > > > > > > reject the device, which will let the generic platform fallback claim > > > > > > it. > > > > > > > > pnp0c02 is a special device ID. It is defined by MS as "General ID for > > > > reserving resources required by PnP motherboard", so having no _CRS for > > > > that particular device ID doesn't really make sense. > > > > > > > > Moreover, we have a PNP driver that (normally) binds to that device ID, > > > > so a PNP device object should be created for it. > > > > > > Given that the pnpacpi driver is going to reject it anyway, it could > > > make sense to keep it from binding at all (and maybe warn?) so that > > > the platform fallback code picks it up, though. > > > > The point is that we'll create a PNP device for the same device ID on other > > platforms and that's going to be confusing, isn't it? > > > > I guess one option might be to hack the driver in question to accept a > > device without _CRS and then do what you need to be done for that device. > > > > The driver being the one I'm writing or the pnpacpi driver? > > Maybe I should ask the vendor if they're willing to drop the pnp0c02 > cid. This thing is an event-generating device, not a > resource-managing device. [cut] OK, I see what's the problem now. > > > > What exactly do you need that platform device for? > > I want some kind of physical device to bind my driver to. I probably > want suspend/resume callbacks, too. I see. For your use case we could make the PNP layer ignore the pnp0c02 device id if (a) _CRS is missing and (b) pnp0c02 is in the _CID list (not a _HID). We have a deeper problem here, but let me discuss that with Rui. Thanks, Rafael -- 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
