Re: [PATCH 9/9] regulator: bd71837: Build BD71837 regulator driver

[kbuild test robot]

Hi Matti,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on ljones-mfd/for-mfd-next]
[also build test WARNING on v4.17-rc6]
[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/mfd-regulator-clk-bd71837-ROHM-BD71837-PMIC-driver/20180526-192351
base:   https://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git for-mfd-next
reproduce:
# apt-get install sparse
make ARCH=x86_64 allmodconfig
make C=1 CF=-D__CHECK_ENDIAN__


Please review and possibly fold the followup patch.

---
0-DAY kernel test infrastructureOpen Source Technology Center
https://lists.01.org/pipermail/kbuild-all   Intel Corporation

Re: [PATCH 9/9] regulator: bd71837: Build BD71837 regulator driver

[kbuild test robot]

Hi Matti,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on ljones-mfd/for-mfd-next]
[also build test WARNING on v4.17-rc6]
[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/mfd-regulator-clk-bd71837-ROHM-BD71837-PMIC-driver/20180526-192351
base:   https://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git for-mfd-next
reproduce:
# apt-get install sparse
make ARCH=x86_64 allmodconfig
make C=1 CF=-D__CHECK_ENDIAN__


Please review and possibly fold the followup patch.

---
0-DAY kernel test infrastructureOpen Source Technology Center
https://lists.01.org/pipermail/kbuild-all   Intel Corporation

RE: [PATCH 9/9] regulator: bd71837: Build BD71837 regulator driver

[Vaittinen, Matti]

> On Thu, May 24, 2018 at 09:01:14AM +0300, Matti Vaittinen wrote:
> > Configurations and Makefile for BD71837 regulator driver
> >
> > Signed-off-by: Matti Vaittinen 
> > ---
> >  drivers/regulator/Kconfig  | 11 +++
> >  drivers/regulator/Makefile |  1 +
> >  2 files changed, 12 insertions(+)
>
> Just squash this into the single patch adding the driver, it makes life
> easier.

Will do. I was wondering if it is nicer to have smaller patches and thus
decided to split this but if it is easier to read them in one patch
- no problem.

Br,
Matti Vaittinen

RE: [PATCH 9/9] regulator: bd71837: Build BD71837 regulator driver

[Vaittinen, Matti]

> On Thu, May 24, 2018 at 09:01:14AM +0300, Matti Vaittinen wrote:
> > Configurations and Makefile for BD71837 regulator driver
> >
> > Signed-off-by: Matti Vaittinen 
> > ---
> >  drivers/regulator/Kconfig  | 11 +++
> >  drivers/regulator/Makefile |  1 +
> >  2 files changed, 12 insertions(+)
>
> Just squash this into the single patch adding the driver, it makes life
> easier.

Will do. I was wondering if it is nicer to have smaller patches and thus
decided to split this but if it is easier to read them in one patch
- no problem.

Br,
Matti Vaittinen

Re: [PATCH 9/9] regulator: bd71837: Build BD71837 regulator driver

[Mark Brown]

On Thu, May 24, 2018 at 09:01:14AM +0300, Matti Vaittinen wrote:
> Configurations and Makefile for BD71837 regulator driver
> 
> Signed-off-by: Matti Vaittinen 
> ---
>  drivers/regulator/Kconfig  | 11 +++
>  drivers/regulator/Makefile |  1 +
>  2 files changed, 12 insertions(+)

Just squash this into the single patch adding the driver, it makes life
easier.


signature.asc
Description: PGP signature

Re: [PATCH 9/9] regulator: bd71837: Build BD71837 regulator driver

[Mark Brown]

On Thu, May 24, 2018 at 09:01:14AM +0300, Matti Vaittinen wrote:
> Configurations and Makefile for BD71837 regulator driver
> 
> Signed-off-by: Matti Vaittinen 
> ---
>  drivers/regulator/Kconfig  | 11 +++
>  drivers/regulator/Makefile |  1 +
>  2 files changed, 12 insertions(+)

Just squash this into the single patch adding the driver, it makes life
easier.


signature.asc
Description: PGP signature