From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754530AbbICHlh (ORCPT ); Thu, 3 Sep 2015 03:41:37 -0400 Received: from arroyo.ext.ti.com ([192.94.94.40]:56707 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752837AbbICHlg (ORCPT ); Thu, 3 Sep 2015 03:41:36 -0400 Subject: Re: [PATCH] mtd: nand: omap2: Rename shippable module to omap2_nand To: Brian Norris References: <1440153935-28927-1-git-send-email-rogerq@ti.com> <20150902210637.GY81844@google.com> CC: , , , , , , Robert Nelson From: Roger Quadros Message-ID: <55E7F992.6050908@ti.com> Date: Thu, 3 Sep 2015 10:41:06 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: <20150902210637.GY81844@google.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 03/09/15 00:06, Brian Norris wrote: > On Fri, Aug 21, 2015 at 01:45:35PM +0300, Roger Quadros wrote: >> As both omap2 onenand and omap2 nand driver modules are >> named the same i.e. "omap2.ko", only one of them gets shipped >> during MODPOST if both are configured as loadable modules. >> >> To avoid this ambiguity let's ship the omap2 nand >> driver as "omap2_nand.ko" >> >> Reported by Pierre Neyron via github >> https://github.com/beagleboard/linux/issues/40 >> >> Cc: Robert Nelson >> Signed-off-by: Roger Quadros >> --- >> drivers/mtd/nand/Makefile | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/mtd/nand/Makefile b/drivers/mtd/nand/Makefile >> index 1f897ec..075a027 100644 >> --- a/drivers/mtd/nand/Makefile >> +++ b/drivers/mtd/nand/Makefile >> @@ -26,7 +26,8 @@ obj-$(CONFIG_MTD_NAND_CS553X) += cs553x_nand.o >> obj-$(CONFIG_MTD_NAND_NDFC) += ndfc.o >> obj-$(CONFIG_MTD_NAND_ATMEL) += atmel_nand.o >> obj-$(CONFIG_MTD_NAND_GPIO) += gpio.o >> -obj-$(CONFIG_MTD_NAND_OMAP2) += omap2.o >> +omap2_nand-objs := omap2.o >> +obj-$(CONFIG_MTD_NAND_OMAP2) += omap2_nand.o > > Pushed to l2-mtd.git. I suppose this one should go out for 4.3, as > there's little downside to it, and it's kind of a bugfix. That is perfectly fine. Thanks. cheers, -roger > >> obj-$(CONFIG_MTD_NAND_OMAP_BCH_BUILD) += omap_elm.o >> obj-$(CONFIG_MTD_NAND_CM_X270) += cmx270_nand.o >> obj-$(CONFIG_MTD_NAND_PXA3xx) += pxa3xx_nand.o >> -- >> 2.1.4 >>