Hi Adam
On Fri, 2019-11-08 at 10:37 +0000, Adam Thomson wrote:
> > --- a/drivers/regulator/da9062-regulator.c
> > +++ b/drivers/regulator/da9062-regulator.c
> > @@ -16,6 +16,7 @@
> > #include <linux/regulator/of_regulator.h>
> > #include <linux/mfd/da9062/core.h>
> > #include <linux/mfd/da9062/registers.h>
> > +#include <dt-bindings/regulator/dlg,da906x-regulator.h>
>
> Can we please rename this file to use da9063 instead of da906x
[..]
sure
> > @@ -145,15 +138,14 @@ static unsigned da9062_buck_get_mode(struct
> > regulator_dev *rdev)
> >
> > switch (val) {
> > default:
> > - case BUCK_MODE_MANUAL:
> > mode = REGULATOR_MODE_FAST |
> > REGULATOR_MODE_STANDBY;
> > /* Sleep flag bit decides the mode */
> > break;
>
> I'm not sure your code is based on the latest regulator updates as I believe
> Axel Lin made some improvements to this bit of code. Checkout Mark's regulator
> fork of the kernel.
yes, the line
mode = REGULATOR_MODE_FAST | REGULATOR_MODE_STANDBY;
is now gone by commit
be446f183ae35a8c76
regulator: da9062: Simplify da9062_buck_set_mode for BUCK_MODE_MANUAL case
it's already in linux-next, I'll rebase this patchset
> > diff --git a/include/dt-bindings/regulator/dlg,da906x-regulator.h b/include/dt-
> > bindings/regulator/dlg,da906x-regulator.h
> > new file mode 100644
> > index 00000000..848a4df
> > --- /dev/null
> > +++ b/include/dt-bindings/regulator/dlg,da906x-regulator.h
> > @@ -0,0 +1,16 @@
> > +/* SPDX-License-Identifier: GPL-2.0 */
> > +
> > +#ifndef __DLG_DA906X_REGULATOR_H
> > +#define __DLG_DA906X_REGULATOR_H
>
> Just to echo previous comment, rename from DA906X to DA9063
ok
