From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932683AbcGDJRf (ORCPT ); Mon, 4 Jul 2016 05:17:35 -0400 Received: from hqemgate16.nvidia.com ([216.228.121.65]:1857 "EHLO hqemgate16.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932539AbcGDJRd (ORCPT ); Mon, 4 Jul 2016 05:17:33 -0400 X-PGP-Universal: processed; by hqpgpgate101.nvidia.com on Mon, 04 Jul 2016 02:17:15 -0700 Subject: Re: [PATCH 3/4] bus: tegra-aconnect: make it explicitly non-modular To: Paul Gortmaker , References: <20160703173040.23612-1-paul.gortmaker@windriver.com> <20160703173040.23612-4-paul.gortmaker@windriver.com> CC: Stephen Warren , Thierry Reding , Alexandre Courbot , , From: Jon Hunter Message-ID: <577A2995.6020504@nvidia.com> Date: Mon, 4 Jul 2016 10:17:09 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.8.0 MIME-Version: 1.0 In-Reply-To: <20160703173040.23612-4-paul.gortmaker@windriver.com> X-Originating-IP: [10.21.132.110] X-ClientProxiedBy: UKMAIL102.nvidia.com (10.26.138.15) To UKMAIL102.nvidia.com (10.26.138.15) 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 Hi Paul, On 03/07/16 18:30, Paul Gortmaker wrote: > The Kconfig currently controlling compilation of this code is: > > drivers/bus/Kconfig:config TEGRA_ACONNECT > drivers/bus/Kconfig: bool "Tegra ACONNECT Bus Driver" > > ...meaning that it currently is not being built as a module by anyone. > > Lets remove the modular code that is essentially orphaned, so that > when reading the driver there is no doubt it is builtin-only. > > We explicitly disallow a driver unbind, since that doesn't have a > sensible use case anyway, and it allows us to drop the ".remove" > code for non-modular drivers. > > Since module_platform_driver() uses the same init level priority as > builtin_platform_driver() the init ordering remains unchanged with > this commit. > > Also note that MODULE_DEVICE_TABLE is a no-op for non-modular code. > > We also delete the MODULE_LICENSE tag etc. since all that information > was (or is now) contained at the top of the file in the comments. In version 3 of the aconnect series [0] I had made this a tristate because we allowed it to be removed and you had submitted a patch to export the PM_CLK APIs. However, when discussing with Thierry he said that we were unable to merge with tristate because of the dependency on your patch. So he suggested we merge with bool for now and then change it back to tristate for v4.9. I understand that we should not do this, but we do plan to make this modular in the future. Cheers Jon [0] http://marc.info/?l=linux-tegra&m=146616753627760&w=2 -- nvpublic