On 9/23/2015 8:41 PM, David Woodhouse wrote:
On Wed, 2015-08-12 at 17:06 -0500, Jeremy Linton wrote:
+static const struct acpi_device_id smsc911x_acpi_match[] = {
+       { "ARMH9118", 0 },
+       { }
+};
+MODULE_DEVICE_TABLE(acpi, smsc911x_acpi_match);
+
  static struct platform_driver smsc911x_driver = {
         .probe = smsc911x_drv_probe,
         .remove = smsc911x_drv_remove,
@@ -2661,6 +2656,7 @@ static struct platform_driver smsc911x_driver =
{
                 .name   = SMSC_CHIPNAME,
                 .pm     = SMSC911X_PM_OPS,
                 .of_match_table = of_match_ptr(smsc911x_dt_ids),
+               .acpi_match_table = ACPI_PTR(smsc911x_acpi_match),
         },
  };
Hm, surely you shouldn't need this part? If the ACPI device has a HID
of PRP0001, and an appropriate "compatible" property, then it ought to
be loaded anyway. If that doesn't work, we should fix that.

But if the device ID is assigned already, why would it hurt to actually use it?

Thanks,
Rafael

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to