Hello. On 05/06/2014 02:23 AM, dinguyen@xxxxxxxxxx wrote:
From: Dinh Nguyen <dinguyen@xxxxxxxxxx>
Even though the IP supports Descriptor DMA mode, it does not support SPLIT transactions in this mode. Since the driver can get the Descriptor DMA mode support from hardware, the driver in its currently form cannot fully support LS/FS devices connected to a HS Hub when Descriptor DMA mode is enabled in the driver. Thus, we give the option to disable Descriptor DMA from device tree.
Signed-off-by: Dinh Nguyen <dinguyen@xxxxxxxxxx> --- drivers/usb/dwc2/platform.c | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/drivers/usb/dwc2/platform.c b/drivers/usb/dwc2/platform.c index eaba547..cb2133c 100644 --- a/drivers/usb/dwc2/platform.c +++ b/drivers/usb/dwc2/platform.c @@ -123,6 +123,7 @@ static int dwc2_driver_probe(struct platform_device *dev) struct resource *res; int retval; int irq; + u32 prop; if (usb_disabled()) return -ENODEV; @@ -135,6 +136,8 @@ static int dwc2_driver_probe(struct platform_device *dev) dwc2_set_all_params(&defparams, -1); params = &defparams; } + if (!of_property_read_u32(dev->dev.of_node, "dma-desc-enable", &prop)) + defparams.dma_desc_enable = prop;
You don't really need a helper local variable here, the variable receiving the property value won't be changed if there was an error retrieving it.
WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
