Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752295AbcKZUYT (ORCPT ); Sat, 26 Nov 2016 15:24:19 -0500 Received: from relay1.mentorg.com ([192.94.38.131]:39589 "EHLO relay1.mentorg.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750947AbcKZUYR (ORCPT ); Sat, 26 Nov 2016 15:24:17 -0500 From: "Rosia, Nicolae" To: "lkp@intel.com" CC: "linux-kernel@vger.kernel.org" , "gg@slimlogic.co.uk" , "robh+dt@kernel.org" , "devicetree@vger.kernel.org" , "lee.jones@linaro.org" , "linux-omap@vger.kernel.org" , "broonie@kernel.org" , "baruch@tkos.co.il" , "mark.rutland@arm.com" , "lgirdwood@gmail.com" , "linux-arm-kernel@lists.infradead.org" , "paul.gortmaker@windriver.com" , "tony@atomide.com" Subject: Re: [PATCH 4/5] regulator: Add support for TI TWL6032 Thread-Topic: [PATCH 4/5] regulator: Add support for TI TWL6032 Thread-Index: AQHSSBDp1nDC8tesn0a7iFvCMGOdK6DrnSuAgAAYxoA= Date: Sat, 26 Nov 2016 20:24:10 +0000 Message-ID: <1480191847.18423.1.camel@mentor.com> References: <201611270236.UFgW5zDA%fengguang.wu@intel.com> In-Reply-To: <201611270236.UFgW5zDA%fengguang.wu@intel.com> Accept-Language: en-US, en-IE Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-messagesentrepresentingtype: 1 x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [137.202.0.76] Content-Type: text/plain; charset="utf-8" Content-ID: <89D839F4FDDFDE43B67082DFE4914196@mentor.com> MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id uAQKOO4X012300 Content-Length: 2588 Lines: 73 Hi, On Sun, 2016-11-27 at 02:55 +0800, kbuild test robot wrote: > Hi Nicolae, > > [auto build test ERROR on omap/for-next] > [also build test ERROR on v4.9-rc6] > [cannot apply to next-20161125] > [if your patch is applied to the wrong git tree, please drop us a > note to help improve the system] > > url:    https://github.com/0day-ci/linux/commits/Nicolae-Rosia/mfd-tw > l-improvements-and-new-regulator-driver/20161127-022201 > base:   https://git.kernel.org/pub/scm/linux/kernel/git/tmlind/linux- > omap.git for-next > config: i386-allmodconfig (attached as .config) > compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 > reproduce: >         # save the attached .config to linux build tree >         make ARCH=i386  > > All error/warnings (new ones prefixed by >>): > >    In file included from drivers/regulator/twl6032-regulator.c:11:0: > > > drivers/regulator/twl6032-regulator.c:557:31: error: > > > 'twl6032_regulator_driver_ids' undeclared here (not in a > > > function) > >     MODULE_DEVICE_TABLE(platform, twl6032_regulator_driver_ids); >                                   ^ >    include/linux/module.h:213:21: note: in definition of macro > 'MODULE_DEVICE_TABLE' >     extern const typeof(name) > __mod_##type##__##name##_device_table  \ >                         ^~~~ > > > include/linux/module.h:213:27: error: > > > '__mod_platform__twl6032_regulator_driver_ids_device_table' > > > aliased to undefined symbol 'twl6032_regulator_driver_ids' > >     extern const typeof(name) > __mod_##type##__##name##_device_table  \ >                               ^ > > > drivers/regulator/twl6032-regulator.c:557:1: note: in expansion > > > of macro 'MODULE_DEVICE_TABLE' > >     MODULE_DEVICE_TABLE(platform, twl6032_regulator_driver_ids); >     ^~~~~~~~~~~~~~~~~~~ > > vim +/twl6032_regulator_driver_ids +557 drivers/regulator/twl6032- > regulator.c > >    551 >    552 static const struct of_device_id twl6032_dt_match[] = { >    553 { .compatible = "ti,twl6032-regulator" }, >    554 { /* last entry */ } >    555 }; >    556 >  > 557 MODULE_DEVICE_TABLE(platform, > twl6032_regulator_driver_ids); >    558 >    559 static struct platform_driver twl6032_regulator_driver > = { >    560 .driver = { Thanks, I did not notice this since I was only testing using built-in module. I will wait for comments before sending V2, untill then here's an inline patch with the fix. Best regards, Nicolae