Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S938541AbcKKVWo (ORCPT ); Fri, 11 Nov 2016 16:22:44 -0500 Received: from smtprelay.synopsys.com ([198.182.47.9]:49461 "EHLO smtprelay.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S936602AbcKKVWl (ORCPT ); Fri, 11 Nov 2016 16:22:41 -0500 Subject: Re: [PATCH 1/2] usb: dwc2: add amcc,dwc-otg support To: Christian Lamparter , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-usb@vger.kernel.org" , "linuxppc-dev@lists.ozlabs.org" References: From: John Youn CC: John Youn , Mark Rutland , Rob Herring , Greg Kroah-Hartman , Felipe Balbi Message-ID: Date: Fri, 11 Nov 2016 13:22:16 -0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.4.120.69] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1079 Lines: 30 On 11/11/2016 12:59 PM, Christian Lamparter wrote: > This patch adds support for the "amcc,usb-otg" device > which is found in the PowerPC Canyonlands' dts. > > The device definition was added by: > commit c89b3458d8cc ("powerpc/44x: Add USB DWC DTS entry to Canyonlands board")' > but without any driver support as the dwc2 driver wasn't > available at that time. > > Note: The system can't use the generic "snps,dwc2" compatible > because of the special ahbcfg configuration. The default > GAHBCFG_HBSTLEN_INCR4 of snps,dwc2 can cause a system hang > when the USB and SATA is used concurrently. Hi, I don't want to add any more of these param structures to the driver unless really necessary. We're trying to remove usage of them in favor of using auto-detected defaults and device properties to override them. The AHB Burst is actually one of the ones we were going to do next because our platform also doesn't work well with INCR4. In fact I'm thinking of making the default INCR. If that's all you need then a devicetree binding should be enough right? Regards, John