On Tue, 5 Nov 2013 10:04:02 Alan Stern wrote:
[snip]
> >
> > + /* Platforms using DT don't always provide platform data.
> > + * This should provide reasonable defaults. */
>
> /*
> * The accepted format for multi-line
> * comments is like this.
> */
>
Ok, I'll fix that for the next revision.
> > + if (!pdata)
> > + dev->dev.platform_data = pdata = &ohci_platform_defaults;
> > +
> >
> > irq = platform_get_irq(dev, 0);
> > if (irq < 0) {
> >
> > dev_err(&dev->dev, "no irq provided");
> >
> > @@ -171,6 +175,11 @@ static int ohci_platform_resume(struct device *dev)
> >
> > #define ohci_platform_resume NULL
> > #endif /* CONFIG_PM */
> >
> > +static const struct of_device_id ohci_of_match[] = {
> > + { .compatible = "ibm,akebono-ohci", },
> > + {},
> > +};
> > +
> >
> > static const struct platform_device_id ohci_platform_table[] = {
> >
> > { "ohci-platform", 0 },
> > { }
> >
> > @@ -191,6 +200,7 @@ static struct platform_driver ohci_platform_driver = {
> >
> > .owner = THIS_MODULE,
> > .name = "ohci-platform",
> > .pm = &ohci_platform_pm_ops,
> >
> > + .of_match_table = ohci_of_match,
> >
> > }
> >
> > };
>
> Update the comment formatting, and then you can resubmit with
>
> Acked-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
Thanks. Based on the discussion for the EHCI driver I would like to change the
compatibility string to "usb-ochi" (instead of "ibm,akebono-ohci"). Are you
still happy for me to add the Acked-by with the alternate compatibility (and
of course the formatting fix)? No other drivers currently use "usb-ochi" so it
shouldn't require any merging of drivers.
Regards,
Alistair
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
