Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754341AbcJUIWd (ORCPT ); Fri, 21 Oct 2016 04:22:33 -0400 Received: from mail-qk0-f181.google.com ([209.85.220.181]:32943 "EHLO mail-qk0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754593AbcJUIWa (ORCPT ); Fri, 21 Oct 2016 04:22:30 -0400 MIME-Version: 1.0 In-Reply-To: <1476720825-30442-1-git-send-email-javier@osg.samsung.com> References: <1476720825-30442-1-git-send-email-javier@osg.samsung.com> From: Ulf Hansson Date: Fri, 21 Oct 2016 10:22:28 +0200 Message-ID: Subject: Re: [PATCH 1/2] mmc: mediatek: Fix module autoload To: Javier Martinez Canillas Cc: "linux-kernel@vger.kernel.org" , linux-mmc , Geert Uytterhoeven , Douglas Anderson , linux-mediatek@lists.infradead.org, "linux-arm-kernel@lists.infradead.org" , Chaotian Jing , Nicolas Boichat , Matthias Brugger Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1276 Lines: 46 On 17 October 2016 at 18:13, Javier Martinez Canillas wrote: > If the driver is built as a module, autoload won't work because the module > alias information is not filled. So user-space can't match the registered > device with the corresponding module. > > Export the module alias information using the MODULE_DEVICE_TABLE() macro. > > Before this patch: > > $ modinfo drivers/mmc/host/mtk-sd.ko | grep alias > $ > > After this patch: > > $ modinfo drivers/mmc/host/mtk-sd.ko | grep alias > alias: of:N*T*Cmediatek,mt8135-mmcC* > alias: of:N*T*Cmediatek,mt8135-mmc > > Signed-off-by: Javier Martinez Canillas Thanks, applied for next! Kind regards Uffe > --- > > drivers/mmc/host/mtk-sd.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/mmc/host/mtk-sd.c b/drivers/mmc/host/mtk-sd.c > index 84e9afcb5c09..86af0b199a54 100644 > --- a/drivers/mmc/host/mtk-sd.c > +++ b/drivers/mmc/host/mtk-sd.c > @@ -1713,6 +1713,7 @@ static const struct of_device_id msdc_of_ids[] = { > { .compatible = "mediatek,mt8135-mmc", }, > {} > }; > +MODULE_DEVICE_TABLE(of, msdc_of_ids); > > static struct platform_driver mt_msdc_driver = { > .probe = msdc_drv_probe, > -- > 2.7.4 >