On Tue, 14 Oct 2014, Kevin Cernekee wrote: > On Tue, Oct 14, 2014 at 6:58 AM, Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> wrote: > > On Sat, 11 Oct 2014, Kevin Cernekee wrote: > > > >> These quirks are currently set through platform_data; allow DT-based SoCs > >> to use them too. > >> > >> Signed-off-by: Kevin Cernekee <cernekee@xxxxxxxxx> > > > >> --- a/drivers/usb/host/ohci-platform.c > >> +++ b/drivers/usb/host/ohci-platform.c > >> @@ -175,6 +175,12 @@ static int ohci_platform_probe(struct platform_device *dev) > >> if (of_property_read_bool(dev->dev.of_node, "big-endian")) > >> ohci->flags |= OHCI_QUIRK_BE_MMIO | OHCI_QUIRK_BE_DESC; > >> > >> + if (of_property_read_bool(dev->dev.of_node, "no-big-frame-no")) > >> + ohci->flags |= OHCI_QUIRK_FRAME_NO; > >> + > >> + of_property_read_u32(dev->dev.of_node, "num-ports", > >> + &ohci->num_ports); > > > > Does this do the right thing if the property isn't defined? > > In that case, of_property_read_u32() will just return -EINVAL or > -ENODATA without writing anything to ohci->num_ports, leaving the > default of 0 in place. Okay, in that case: Acked-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> -- 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
