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=-3.8 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=no 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 67D9EC4724C for ; Thu, 30 Apr 2020 14:53:06 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 3A9D82076D for ; Thu, 30 Apr 2020 14:53:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1588258386; bh=3jNY7Z1CXHUeLNkZam1xF4SaPvI1RqKiMPWejLFrHe4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:List-ID:From; b=XPtqoX/o8aAzFbDaNjLtH/BZ10DnwsiCHwToQIHd60QmbLA/XVSCmyYGptjAYfZ89 55+B7IEnFHsJuYY3Zo46PiNEIAjlYuiNyeQ2ANlRdELDefhRPB8gN2dH09D00/KWfK X8RL35058qMKcYG4SpiwIOmuFdqnPI1XuePToNEE= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726503AbgD3OxF (ORCPT ); Thu, 30 Apr 2020 10:53:05 -0400 Received: from mail.kernel.org ([198.145.29.99]:50506 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726357AbgD3OxF (ORCPT ); Thu, 30 Apr 2020 10:53:05 -0400 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 70A572074A; Thu, 30 Apr 2020 14:53:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1588258385; bh=3jNY7Z1CXHUeLNkZam1xF4SaPvI1RqKiMPWejLFrHe4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=uEF3pPQNsAva6ERqoXN3GPqn/YP2A19BkDE70CMdUpfUnzPq6MwtOfXpPa6ETzAzy KtlsGxEAo9l+RWIHMUj1/2vo6BmV0P6TVfAGB1Y064snxw/wEADbL+MlbpZMsoqI5/ ag7+OSQouLJuxkZOEWDRkchLaPXmMSNryI9kAkRI= Date: Thu, 30 Apr 2020 16:53:02 +0200 From: Greg Kroah-Hartman To: Vinod Koul Cc: Mathias Nyman , Mathias Nyman , linux-arm-msm@vger.kernel.org, Bjorn Andersson , Christian Lamparter , Yoshihiro Shimoda , John Stultz , Alan Stern , Andreas =?iso-8859-1?Q?B=F6hler?= , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v11 2/5] usb: renesas-xhci: Add the renesas xhci driver Message-ID: <20200430145302.GB3495619@kroah.com> References: <20200430101019.1130956-1-vkoul@kernel.org> <20200430101019.1130956-3-vkoul@kernel.org> <8ffd119b-192f-8fcc-46cc-3a405e30338c@linux.intel.com> <20200430144641.GJ948789@vkoul-mobl.Dlink> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200430144641.GJ948789@vkoul-mobl.Dlink> Sender: linux-arm-msm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org On Thu, Apr 30, 2020 at 08:16:41PM +0530, Vinod Koul wrote: > > > diff --git a/drivers/usb/host/Makefile b/drivers/usb/host/Makefile > > > index b191361257cc..c3a79f626393 100644 > > > --- a/drivers/usb/host/Makefile > > > +++ b/drivers/usb/host/Makefile > > > @@ -70,7 +70,8 @@ obj-$(CONFIG_USB_OHCI_HCD_DAVINCI) += ohci-da8xx.o > > > obj-$(CONFIG_USB_UHCI_HCD) += uhci-hcd.o > > > obj-$(CONFIG_USB_FHCI_HCD) += fhci.o > > > obj-$(CONFIG_USB_XHCI_HCD) += xhci-hcd.o > > > -obj-$(CONFIG_USB_XHCI_PCI) += xhci-pci.o > > > +usb-xhci-pci-objs := xhci-pci.o xhci-pci-renesas.o > > > +obj-$(CONFIG_USB_XHCI_PCI) += usb-xhci-pci.o > > > > I don't think it's a good idea to rename the xhci-pci module to usb-xhci-pci > > > > does > > > > xhci-pci-y := xhci-pci.o xhci-pci-renesas.o > > obj-$(CONFIG_USB_XHCI_PCI) += xhci-pci.o > > > > cause some kbuild issues? > > Yes with this version I get the warning: > make[4]: Circular drivers/usb/host/xhci-pci.o <- drivers/usb/host/xhci-pci.o dependency dropped. > > I don't speak enough Kbuild, but I guess it does make sense that we have > xhci-pci.o as target for both xhci-pci.o xhci-pci-renesas.o! That was > the reason for adding usb tag to this to resolve the conflict. > > I am okay for any other mechanism which can work well here. Btw what > issues do you foresee with adding usb tag to module name. It will break all sorts of things. Happens every time we rename modules, let's not do it unless we absolutely have to. greg k-h