Re: [PATCH v2 5/7] regulator/mfd: bd718xx: rename bd71837/bd71847 common instances

From: Matti Vaittinen
Date: Fri Sep 14 2018 - 01:42:27 EST


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