Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754493AbaDYUZn (ORCPT ); Fri, 25 Apr 2014 16:25:43 -0400 Received: from top.free-electrons.com ([176.31.233.9]:49835 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751866AbaDYUZl (ORCPT ); Fri, 25 Apr 2014 16:25:41 -0400 Date: Fri, 25 Apr 2014 22:25:36 +0200 From: Thomas Petazzoni To: Arnd Bergmann Cc: linux-arm-kernel@lists.infradead.org, Gregory CLEMENT , Mathias Nyman , Greg Kroah-Hartman , Felipe Balbi , linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, Jason Cooper , Andrew Lunn , Sebastian Hesselbarth , Lior Amsalem , Tawfik Bayouk , devicetree@vger.kernel.org, Nadav Haklai , Rob Herring , Ezequiel Garcia , Grant Likely Subject: Re: [PATCH v2 03/18] usb: host: xhci-plat: Add support for the Armada 38x Message-ID: <20140425222536.430d5c4d@skate> In-Reply-To: <5392399.GiPjM0CIZ0@wuerfel> References: <1398434836-18908-1-git-send-email-gregory.clement@free-electrons.com> <1398434836-18908-4-git-send-email-gregory.clement@free-electrons.com> <5392399.GiPjM0CIZ0@wuerfel> Organization: Free Electrons X-Mailer: Claws Mail 3.9.1 (GTK+ 2.24.20; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Dear Arnd Bergmann, On Fri, 25 Apr 2014 22:01:51 +0200, Arnd Bergmann wrote: > I think you're doing it the wrong way around: You have a specialized > version of the generic xhci-plat driver. The normal way to handle this > is to have a loadable module that contains all the Armada specific > code and that registers a platform_driver. In the probe() function of > that driver, you can do the platform specific setup and then call > the generic xhci_plat_probe() function, which of course has to > be provided using EXPORT_SYMBOL_GPL. You should have a look at the v1 Gregory sent: it was implementing exactly what you suggest here, but Felipe explicitly requested the patches to be changed like is now proposed in v2. Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/