On Fri, Sep 14, 2018 at 04:39:28AM +0800, kbuild test robot wrote: > Hi Matti, > > Thank you for the patch! Yet something to improve: > > [auto build test ERROR on regulator/for-next] > [also build test ERROR on next-20180913] > [cannot apply to v4.19-rc3] > [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/Matti-Vaittinen/regulator-mfd-dt_bindings-bd718x7-Support-ROHM-bd71847/20180914-000129 > base: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-next > config: i386-randconfig-s0-201836 (attached as .config) > compiler: gcc-6 (Debian 6.4.0-9) 6.4.0 20171026 > reproduce: > # save the attached .config to linux build tree > make ARCH=i386 > > Note: the linux-review/Matti-Vaittinen/regulator-mfd-dt_bindings-bd718x7-Support-ROHM-bd71847/20180914-000129 HEAD 0df8e57cacf59e85ad40a25fbb1bd7e842608be8 builds fine. > It only hurts bisectibility. > > All errors (new ones prefixed by >>): > > >> drivers/regulator/bd71837-regulator.c:407:18: error: 'BD718XX_4TH_NODVS_BUCK_VOLTAGE_NUM' undeclared here (not in a function) > .n_voltages = BD718XX_4TH_NODVS_BUCK_VOLTAGE_NUM, > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > drivers/regulator/bd71837-regulator.c:735:18: error: initializer element is not constant > .n_voltages = BD718XX_4TH_NODVS_BUCK_VOLTAGE_NUM, > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Sorry all. I've messed up the series. The BD718XX_4TH_NODVS_BUCK_VOLTAGE_NUM definition is used already in [PATCH v2 5/7] but only added to header in patch [PATCH v2 6/7]. I'll fix this and send out v3. Br, Matti Vaittinen