From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rosia, Nicolae" Subject: Re: [PATCH 4/5] regulator: Add support for TI TWL6032 Date: Sat, 26 Nov 2016 20:24:10 +0000 Message-ID: <1480191847.18423.1.camel@mentor.com> References: <201611270236.UFgW5zDA%fengguang.wu@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <201611270236.UFgW5zDA%fengguang.wu@intel.com> Content-Language: en-US Content-ID: <89D839F4FDDFDE43B67082DFE4914196@mentor.com> Sender: linux-kernel-owner@vger.kernel.org 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" List-Id: linux-omap@vger.kernel.org SGksDQoNCk9uIFN1biwgMjAxNi0xMS0yNyBhdCAwMjo1NSArMDgwMCwga2J1aWxkIHRlc3Qgcm9i b3Qgd3JvdGU6DQo+IEhpIE5pY29sYWUsDQo+IA0KPiBbYXV0byBidWlsZCB0ZXN0IEVSUk9SIG9u IG9tYXAvZm9yLW5leHRdDQo+IFthbHNvIGJ1aWxkIHRlc3QgRVJST1Igb24gdjQuOS1yYzZdDQo+ IFtjYW5ub3QgYXBwbHkgdG8gbmV4dC0yMDE2MTEyNV0NCj4gW2lmIHlvdXIgcGF0Y2ggaXMgYXBw bGllZCB0byB0aGUgd3JvbmcgZ2l0IHRyZWUsIHBsZWFzZSBkcm9wIHVzIGENCj4gbm90ZSB0byBo ZWxwIGltcHJvdmUgdGhlIHN5c3RlbV0NCj4gDQo+IHVybDrCoMKgwqDCoGh0dHBzOi8vZ2l0aHVi LmNvbS8wZGF5LWNpL2xpbnV4L2NvbW1pdHMvTmljb2xhZS1Sb3NpYS9tZmQtdHcNCj4gbC1pbXBy b3ZlbWVudHMtYW5kLW5ldy1yZWd1bGF0b3ItZHJpdmVyLzIwMTYxMTI3LTAyMjIwMQ0KPiBiYXNl OsKgwqDCoGh0dHBzOi8vZ2l0Lmtlcm5lbC5vcmcvcHViL3NjbS9saW51eC9rZXJuZWwvZ2l0L3Rt bGluZC9saW51eC0NCj4gb21hcC5naXQgZm9yLW5leHQNCj4gY29uZmlnOiBpMzg2LWFsbG1vZGNv bmZpZyAoYXR0YWNoZWQgYXMgLmNvbmZpZykNCj4gY29tcGlsZXI6IGdjYy02IChEZWJpYW4gNi4y LjAtMykgNi4yLjAgMjAxNjA5MDENCj4gcmVwcm9kdWNlOg0KPiDCoMKgwqDCoMKgwqDCoMKgIyBz YXZlIHRoZSBhdHRhY2hlZCAuY29uZmlnIHRvIGxpbnV4IGJ1aWxkIHRyZWUNCj4gwqDCoMKgwqDC oMKgwqDCoG1ha2UgQVJDSD1pMzg2wqANCj4gDQo+IEFsbCBlcnJvci93YXJuaW5ncyAobmV3IG9u ZXMgcHJlZml4ZWQgYnkgPj4pOg0KPiANCj4gwqDCoMKgSW4gZmlsZSBpbmNsdWRlZCBmcm9tIGRy aXZlcnMvcmVndWxhdG9yL3R3bDYwMzItcmVndWxhdG9yLmM6MTE6MDoNCj4gPiA+IGRyaXZlcnMv cmVndWxhdG9yL3R3bDYwMzItcmVndWxhdG9yLmM6NTU3OjMxOiBlcnJvcjoNCj4gPiA+ICd0d2w2 MDMyX3JlZ3VsYXRvcl9kcml2ZXJfaWRzJyB1bmRlY2xhcmVkIGhlcmUgKG5vdCBpbiBhDQo+ID4g PiBmdW5jdGlvbikNCj4gDQo+IMKgwqDCoMKgTU9EVUxFX0RFVklDRV9UQUJMRShwbGF0Zm9ybSwg dHdsNjAzMl9yZWd1bGF0b3JfZHJpdmVyX2lkcyk7DQo+IMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC oMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgXg0KPiDCoMKgwqBp bmNsdWRlL2xpbnV4L21vZHVsZS5oOjIxMzoyMTogbm90ZTogaW4gZGVmaW5pdGlvbiBvZiBtYWNy bw0KPiAnTU9EVUxFX0RFVklDRV9UQUJMRScNCj4gwqDCoMKgwqBleHRlcm4gY29uc3QgdHlwZW9m KG5hbWUpDQo+IF9fbW9kXyMjdHlwZSMjX18jI25hbWUjI19kZXZpY2VfdGFibGXCoMKgXA0KPiDC oMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqBefn5+DQo+ID4g PiBpbmNsdWRlL2xpbnV4L21vZHVsZS5oOjIxMzoyNzogZXJyb3I6DQo+ID4gPiAnX19tb2RfcGxh dGZvcm1fX3R3bDYwMzJfcmVndWxhdG9yX2RyaXZlcl9pZHNfZGV2aWNlX3RhYmxlJw0KPiA+ID4g YWxpYXNlZCB0byB1bmRlZmluZWQgc3ltYm9sICd0d2w2MDMyX3JlZ3VsYXRvcl9kcml2ZXJfaWRz Jw0KPiANCj4gwqDCoMKgwqBleHRlcm4gY29uc3QgdHlwZW9mKG5hbWUpDQo+IF9fbW9kXyMjdHlw ZSMjX18jI25hbWUjI19kZXZpY2VfdGFibGXCoMKgXA0KPiDCoMKgwqDCoMKgwqDCoMKgwqDCoMKg wqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqBeDQo+ID4gPiBkcml2ZXJzL3Jl Z3VsYXRvci90d2w2MDMyLXJlZ3VsYXRvci5jOjU1NzoxOiBub3RlOiBpbiBleHBhbnNpb24NCj4g PiA+IG9mIG1hY3JvICdNT0RVTEVfREVWSUNFX1RBQkxFJw0KPiANCj4gwqDCoMKgwqBNT0RVTEVf REVWSUNFX1RBQkxFKHBsYXRmb3JtLCB0d2w2MDMyX3JlZ3VsYXRvcl9kcml2ZXJfaWRzKTsNCj4g wqDCoMKgwqBefn5+fn5+fn5+fn5+fn5+fn5+DQo+IA0KPiB2aW0gKy90d2w2MDMyX3JlZ3VsYXRv cl9kcml2ZXJfaWRzICs1NTcgZHJpdmVycy9yZWd1bGF0b3IvdHdsNjAzMi0NCj4gcmVndWxhdG9y LmMNCj4gDQo+IMKgwqDCoDU1MQkNCj4gwqDCoMKgNTUyCXN0YXRpYyBjb25zdCBzdHJ1Y3Qgb2Zf ZGV2aWNlX2lkIHR3bDYwMzJfZHRfbWF0Y2hbXSA9IHsNCj4gwqDCoMKgNTUzCQl7IC5jb21wYXRp YmxlID0gInRpLHR3bDYwMzItcmVndWxhdG9yIiB9LA0KPiDCoMKgwqA1NTQJCXsgLyogbGFzdCBl bnRyeSAqLyB9DQo+IMKgwqDCoDU1NQl9Ow0KPiDCoMKgwqA1NTYJDQo+IMKgPiA1NTcJTU9EVUxF X0RFVklDRV9UQUJMRShwbGF0Zm9ybSwNCj4gdHdsNjAzMl9yZWd1bGF0b3JfZHJpdmVyX2lkcyk7 DQo+IMKgwqDCoDU1OAkNCj4gwqDCoMKgNTU5CXN0YXRpYyBzdHJ1Y3QgcGxhdGZvcm1fZHJpdmVy IHR3bDYwMzJfcmVndWxhdG9yX2RyaXZlcg0KPiA9IHsNCj4gwqDCoMKgNTYwCQkuZHJpdmVyID0g ew0KDQpUaGFua3MsIEkgZGlkIG5vdCBub3RpY2UgdGhpcyBzaW5jZSBJIHdhcyBvbmx5IHRlc3Rp bmcgdXNpbmcgYnVpbHQtaW4NCm1vZHVsZS4NCkkgd2lsbCB3YWl0IGZvciBjb21tZW50cyBiZWZv cmUgc2VuZGluZyBWMiwgdW50aWxsIHRoZW4gaGVyZSdzIGFuDQppbmxpbmUgcGF0Y2ggd2l0aCB0 aGUgZml4Lg0KDQoNCkJlc3QgcmVnYXJkcywNCk5pY29sYWUNCg0K From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nicolae_Rosia@mentor.com (Rosia, Nicolae) Date: Sat, 26 Nov 2016 20:24:10 +0000 Subject: [PATCH 4/5] regulator: Add support for TI TWL6032 In-Reply-To: <201611270236.UFgW5zDA%fengguang.wu@intel.com> References: <201611270236.UFgW5zDA%fengguang.wu@intel.com> Message-ID: <1480191847.18423.1.camel@mentor.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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