From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934227AbcAZK22 (ORCPT ); Tue, 26 Jan 2016 05:28:28 -0500 Received: from mga01.intel.com ([192.55.52.88]:51291 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933952AbcAZK2X (ORCPT ); Tue, 26 Jan 2016 05:28:23 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,349,1449561600"; d="scan'208";a="901294744" Subject: Re: [PATCH v6 03/12] usb: dbc: probe and setup xhci debug capability To: Greg Kroah-Hartman References: <1453781665-4714-1-git-send-email-baolu.lu@linux.intel.com> <1453781665-4714-4-git-send-email-baolu.lu@linux.intel.com> <20160126073653.GB24272@kroah.com> Cc: linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org From: Lu Baolu Message-ID: <56A74A23.2010108@linux.intel.com> Date: Tue, 26 Jan 2016 18:27:47 +0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <20160126073653.GB24272@kroah.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/26/2016 03:36 PM, Greg Kroah-Hartman wrote: > On Tue, Jan 26, 2016 at 12:14:16PM +0800, Lu Baolu wrote: >> xHCI debug capability (DbC) is an optional functionality provided >> by an xHCI host controller. Software learns this capability by >> walking through the extended capability list in mmio of the host. >> >> This patch introduces the code to probe and initialize the debug >> capability hardware during early boot. With hardware initialization >> done, the debug target (system under debug which has DbC enabled) >> will present a debug device through the debug port. The debug device >> is fully compliant with the USB framework and provides the equivalent >> of a very high performance (USB3) full-duplex serial link between the >> debug host and target. >> >> Signed-off-by: Lu Baolu >> --- >> MAINTAINERS | 7 + >> arch/x86/Kconfig.debug | 12 + >> drivers/usb/early/Makefile | 1 + > Why did you make a whole new subdirectory, just for your one driver? > Please don't do that, just put it in the host controller directory where > it belongs... I am sorry, but driver/usb/early is an existing subdirectory. $ ls drivers/usb/early/ ehci-dbgp.c Makefile If I understand it correctly, the "early" subdirectory is used to keep drivers which should be loaded during early boot stage. We already have code for EHCI debug port there with the filename ehci-dbgp.c. It is used to aid the developer to get kernel messages during early boot through EHCI debug port. What I have done is to implement the same thing with xHCI debug port. It's not part of xHCI host controller driver and it is executed even before xHCI driver is loaded. Thanks, Baolu > >