Hello.

On 07/23/2013 06:35 PM, Matthijs Kooijman wrote:

@@ -212,11 +213,19 @@ static int xhci_plat_remove(struct platform_device *dev)
        return 0;
  }

+#ifdef CONFIG_OF
+static const struct of_device_id usb_xhci_of_match[] = {
+       { .compatible = "usb-xhci" },
+       {},
+};

   I wonder if we also need:

MODULE_DEVICE_TABLE(of, usb_xhci_of_match);

+#endif
+
  static struct platform_driver usb_xhci_driver = {
        .probe  = xhci_plat_probe,
        .remove = xhci_plat_remove,
        .driver = {
                .name = "xhci-hcd",
+               .of_match_table = of_match_ptr(usb_xhci_of_match),
        },
  };
  MODULE_ALIAS("platform:xhci-hcd");

This looks like it wouldn't compile without CONFIG_OF?

   No, of_match_ptr() is #defined to NULL when CONFIG_OF=n.

Gr.

Matthijs

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" 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