From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dinh Nguyen Subject: Re: [PATCH 1/2] usb: dwc2: Option to disable dma descriptor mode Date: Tue, 6 May 2014 10:44:30 -0500 Message-ID: <1399391070.11533.7.camel@linux-builds1> References: <1399328585-30949-1-git-send-email-dinguyen@altera.com> <5369022A.2060905@cogentembedded.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <5369022A.2060905-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Sergei Shtylyov Cc: robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, pawel.moll-5wv7dgnIgG8@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org, galak-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, paulz-HKixBCOQz3hWk0Htik3J/w@public.gmane.org, dinh.linux-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org On Tue, 2014-05-06 at 19:39 +0400, Sergei Shtylyov wrote: > Hello. > > On 05/06/2014 02:23 AM, dinguyen-EIB2kfCEclfQT0dZR+AlfA@public.gmane.org wrote: > > > From: Dinh Nguyen > > > 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 > > --- > > 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. > Ah yes, you're right. Thanks for the review. Dinh > WBR, Sergei > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html