From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.1 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,T_DKIMWL_WL_HIGH, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 05F16C28CC0 for ; Wed, 29 May 2019 10:24:42 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id CD3812070D for ; Wed, 29 May 2019 10:24:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1559125481; bh=Zn58nd8T5+65YqD1AnYvF1lnYwDjRaKzeQ7ag4xKOEY=; h=From:To:Cc:Subject:In-Reply-To:References:Date:List-ID:From; b=k41xnEIf4rkO/VLdJJvuXs++fZbkhcXaBXwlRrBtVbbzmZmLkqtopT6uD7gNimLPs fbxcKeSxPUFke1W1rnRKvmpjngkAvPOMbrGmOjylmb9PyCY5rzWYMMtVRNOWqW4ZqQ VB6MKbdGFpGlcsOSxFo8VQuTVg17nJPueV41pPgE= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726741AbfE2KYl (ORCPT ); Wed, 29 May 2019 06:24:41 -0400 Received: from mga09.intel.com ([134.134.136.24]:11595 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725911AbfE2KYk (ORCPT ); Wed, 29 May 2019 06:24:40 -0400 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 29 May 2019 03:24:38 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.60,526,1549958400"; d="scan'208";a="179534587" Received: from pipin.fi.intel.com (HELO pipin) ([10.237.72.175]) by fmsmga002.fm.intel.com with ESMTP; 29 May 2019 03:24:38 -0700 From: Felipe Balbi To: Ran Wang Cc: Greg Kroah-Hartman , "open list\:DESIGNWARE USB3 DRD IP DRIVER" , open list , Rob Herring , "devicetree\@vger.kernel.org" Subject: RE: [PATCH] usb: dwc3: Enable the USB snooping In-Reply-To: References: <20171115060459.45375-1-ran.wang_1@nxp.com> <87ineb9b5v.fsf@linux.intel.com> <87shdfet90.fsf@linux.intel.com> <87k1eaanjw.fsf@linux.intel.com> Date: Wed, 29 May 2019 13:24:37 +0300 Message-ID: <874l5dr21m.fsf@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, Ran Wang writes: >> >> >> c) WHAT does this mean for PCI devices? > > According to DWC3 data book, I think this (PCI) mean to the case of 'master bus type = Native' > The data book describes this feature as 'system bus DMA option for the master bus, > which may be configured as AHB, AXI, or Native.' On Table 6-5, it says when MBUS_TYPE > is Native, the definition of 4 transfer types control bits [3-0] is 'Same as AXI'. > > However, as to the code implementation to be generic to both PCI and AXI, > I admit I don't have a perfect solution so far, only 2 proposals with concerns: > > a. Create another module driver like dwc3-exynos.c (arch/arm/boot/dts/wxynos54xx.dtsi) > to contain above programming code. However, it will touch the same reg range of DWC3 > I think this is not good. I'd prefer avoiding another glue :-) > b. Add #ifdef CONFIG_ARCH_LAYERSCAPE in drivers/usb/dwc3/core.c to constrain hacking code > can only take effect for Layerscape (AXI case). I know it look ugly. > > Do you have any better advice on this (besides changed power on default value from HW perspective)? Maybe we don't need to care, actually. Since this property will only be needed for RTL instantiation that didn't configure these defaults properly during coreConsultant. >> >> >> Another question is: Why wasn't this setup properly during >> >> >> coreConsultant instantiation of the RTL? Do you have devices on >> >> >> the market already that need this or is this some early FPGA model >> >> >> or test-only >> >> ASIC? > > Several Layerscape platforms like LS1043ARDB, LS1046ARDB, etc. are already on > the market and have this issue. So I have to work out a SW patch to fix them. Thank you, now I'm certain that this is not some temporary solution :-) Thanks for going through this again. Please refresh the patch so we can try to get it merged. -- balbi