* [PATCH 1/1] Revert "mfd: da9061: Fix to remove BBAT_CONT register from chip model"
@ 2017-08-15 8:38 Lee Jones
2017-08-21 7:49 ` Lee Jones
0 siblings, 1 reply; 4+ messages in thread
From: Lee Jones @ 2017-08-15 8:38 UTC (permalink / raw)
To: stwiss.opensource; +Cc: linux-arm-kernel, linux-kernel, Lee Jones
This patch was applied to the MFD twice, causing unwanted behavour.
This reverts commit b77eb79acca3203883e8d8dbc7f2b842def1bff8.
Fixes: b77eb79acca3 ("mfd: da9061: Fix to remove BBAT_CONT register from chip model")
Reported-by: Steve Twiss <stwiss.opensource@diasemi.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
drivers/mfd/da9062-core.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/drivers/mfd/da9062-core.c b/drivers/mfd/da9062-core.c
index fbe0f245ce8e..fe1811523e4a 100644
--- a/drivers/mfd/da9062-core.c
+++ b/drivers/mfd/da9062-core.c
@@ -645,6 +645,9 @@ static const struct regmap_range da9062_aa_readable_ranges[] = {
.range_min = DA9062AA_VLDO1_B,
.range_max = DA9062AA_VLDO4_B,
}, {
+ .range_min = DA9062AA_BBAT_CONT,
+ .range_max = DA9062AA_BBAT_CONT,
+ }, {
.range_min = DA9062AA_INTERFACE,
.range_max = DA9062AA_CONFIG_E,
}, {
@@ -721,6 +724,9 @@ static const struct regmap_range da9062_aa_writeable_ranges[] = {
.range_min = DA9062AA_VLDO1_B,
.range_max = DA9062AA_VLDO4_B,
}, {
+ .range_min = DA9062AA_BBAT_CONT,
+ .range_max = DA9062AA_BBAT_CONT,
+ }, {
.range_min = DA9062AA_GP_ID_0,
.range_max = DA9062AA_GP_ID_19,
},
--
2.11.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 1/1] Revert "mfd: da9061: Fix to remove BBAT_CONT register from chip model"
2017-08-15 8:38 [PATCH 1/1] Revert "mfd: da9061: Fix to remove BBAT_CONT register from chip model" Lee Jones
@ 2017-08-21 7:49 ` Lee Jones
2017-08-21 12:32 ` Steve Twiss
0 siblings, 1 reply; 4+ messages in thread
From: Lee Jones @ 2017-08-21 7:49 UTC (permalink / raw)
To: stwiss.opensource; +Cc: linux-arm-kernel, linux-kernel
On Tue, 15 Aug 2017, Lee Jones wrote:
> This patch was applied to the MFD twice, causing unwanted behavour.
>
> This reverts commit b77eb79acca3203883e8d8dbc7f2b842def1bff8.
>
> Fixes: b77eb79acca3 ("mfd: da9061: Fix to remove BBAT_CONT register from chip model")
> Reported-by: Steve Twiss <stwiss.opensource@diasemi.com>
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> ---
> drivers/mfd/da9062-core.c | 6 ++++++
> 1 file changed, 6 insertions(+)
Steve, It would be good to obtain a {Reviewed|Acked}-by from you
before applying.
> diff --git a/drivers/mfd/da9062-core.c b/drivers/mfd/da9062-core.c
> index fbe0f245ce8e..fe1811523e4a 100644
> --- a/drivers/mfd/da9062-core.c
> +++ b/drivers/mfd/da9062-core.c
> @@ -645,6 +645,9 @@ static const struct regmap_range da9062_aa_readable_ranges[] = {
> .range_min = DA9062AA_VLDO1_B,
> .range_max = DA9062AA_VLDO4_B,
> }, {
> + .range_min = DA9062AA_BBAT_CONT,
> + .range_max = DA9062AA_BBAT_CONT,
> + }, {
> .range_min = DA9062AA_INTERFACE,
> .range_max = DA9062AA_CONFIG_E,
> }, {
> @@ -721,6 +724,9 @@ static const struct regmap_range da9062_aa_writeable_ranges[] = {
> .range_min = DA9062AA_VLDO1_B,
> .range_max = DA9062AA_VLDO4_B,
> }, {
> + .range_min = DA9062AA_BBAT_CONT,
> + .range_max = DA9062AA_BBAT_CONT,
> + }, {
> .range_min = DA9062AA_GP_ID_0,
> .range_max = DA9062AA_GP_ID_19,
> },
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 4+ messages in thread
* RE: [PATCH 1/1] Revert "mfd: da9061: Fix to remove BBAT_CONT register from chip model"
2017-08-21 7:49 ` Lee Jones
@ 2017-08-21 12:32 ` Steve Twiss
2017-08-23 7:57 ` Lee Jones
0 siblings, 1 reply; 4+ messages in thread
From: Steve Twiss @ 2017-08-21 12:32 UTC (permalink / raw)
To: Lee Jones; +Cc: linux-arm-kernel, linux-kernel
On 21 August 2017 08:50, Lee Jones wrote:
Hi Lee,
> To: Steve Twiss
> Cc: linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH 1/1] Revert "mfd: da9061: Fix to remove BBAT_CONT register from chip model"
>
> On Tue, 15 Aug 2017, Lee Jones wrote:
>
> > This patch was applied to the MFD twice, causing unwanted behavour.
> >
> > This reverts commit b77eb79acca3203883e8d8dbc7f2b842def1bff8.
> >
> > Fixes: b77eb79acca3 ("mfd: da9061: Fix to remove BBAT_CONT register from chip model")
> > Reported-by: Steve Twiss <stwiss.opensource@diasemi.com>
> > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> > ---
> > drivers/mfd/da9062-core.c | 6 ++++++
> > 1 file changed, 6 insertions(+)
>
> Steve, It would be good to obtain a {Reviewed|Acked}-by from you
> before applying.
>
Sure.
> > diff --git a/drivers/mfd/da9062-core.c b/drivers/mfd/da9062-core.c
> > index fbe0f245ce8e..fe1811523e4a 100644
> > --- a/drivers/mfd/da9062-core.c
> > +++ b/drivers/mfd/da9062-core.c
> > @@ -645,6 +645,9 @@ static const struct regmap_range da9062_aa_readable_ranges[] = {
> > .range_min = DA9062AA_VLDO1_B,
> > .range_max = DA9062AA_VLDO4_B,
> > }, {
> > + .range_min = DA9062AA_BBAT_CONT,
> > + .range_max = DA9062AA_BBAT_CONT,
> > + }, {
> > .range_min = DA9062AA_INTERFACE,
> > .range_max = DA9062AA_CONFIG_E,
> > }, {
> > @@ -721,6 +724,9 @@ static const struct regmap_range da9062_aa_writeable_ranges[] = {
> > .range_min = DA9062AA_VLDO1_B,
> > .range_max = DA9062AA_VLDO4_B,
> > }, {
> > + .range_min = DA9062AA_BBAT_CONT,
> > + .range_max = DA9062AA_BBAT_CONT,
> > + }, {
> > .range_min = DA9062AA_GP_ID_0,
> > .range_max = DA9062AA_GP_ID_19,
> > },
This looks good. Thanks for the changes.
Reviewed-by: Steve Twiss <stwiss.opensource@diasemi.com>
Acked-by: Steve Twiss <stwiss.opensource@diasemi.com>
Regards,
Steve
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 1/1] Revert "mfd: da9061: Fix to remove BBAT_CONT register from chip model"
2017-08-21 12:32 ` Steve Twiss
@ 2017-08-23 7:57 ` Lee Jones
0 siblings, 0 replies; 4+ messages in thread
From: Lee Jones @ 2017-08-23 7:57 UTC (permalink / raw)
To: Steve Twiss; +Cc: linux-arm-kernel, linux-kernel
On Mon, 21 Aug 2017, Steve Twiss wrote:
> On 21 August 2017 08:50, Lee Jones wrote:
>
> Hi Lee,
>
> > To: Steve Twiss
> > Cc: linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org
> > Subject: Re: [PATCH 1/1] Revert "mfd: da9061: Fix to remove BBAT_CONT register from chip model"
> >
> > On Tue, 15 Aug 2017, Lee Jones wrote:
> >
> > > This patch was applied to the MFD twice, causing unwanted behavour.
> > >
> > > This reverts commit b77eb79acca3203883e8d8dbc7f2b842def1bff8.
> > >
> > > Fixes: b77eb79acca3 ("mfd: da9061: Fix to remove BBAT_CONT register from chip model")
> > > Reported-by: Steve Twiss <stwiss.opensource@diasemi.com>
> > > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> > > ---
> > > drivers/mfd/da9062-core.c | 6 ++++++
> > > 1 file changed, 6 insertions(+)
> >
> > Steve, It would be good to obtain a {Reviewed|Acked}-by from you
> > before applying.
> >
>
> Sure.
>
> > > diff --git a/drivers/mfd/da9062-core.c b/drivers/mfd/da9062-core.c
> > > index fbe0f245ce8e..fe1811523e4a 100644
> > > --- a/drivers/mfd/da9062-core.c
> > > +++ b/drivers/mfd/da9062-core.c
> > > @@ -645,6 +645,9 @@ static const struct regmap_range da9062_aa_readable_ranges[] = {
> > > .range_min = DA9062AA_VLDO1_B,
> > > .range_max = DA9062AA_VLDO4_B,
> > > }, {
> > > + .range_min = DA9062AA_BBAT_CONT,
> > > + .range_max = DA9062AA_BBAT_CONT,
> > > + }, {
> > > .range_min = DA9062AA_INTERFACE,
> > > .range_max = DA9062AA_CONFIG_E,
> > > }, {
> > > @@ -721,6 +724,9 @@ static const struct regmap_range da9062_aa_writeable_ranges[] = {
> > > .range_min = DA9062AA_VLDO1_B,
> > > .range_max = DA9062AA_VLDO4_B,
> > > }, {
> > > + .range_min = DA9062AA_BBAT_CONT,
> > > + .range_max = DA9062AA_BBAT_CONT,
> > > + }, {
> > > .range_min = DA9062AA_GP_ID_0,
> > > .range_max = DA9062AA_GP_ID_19,
> > > },
>
> This looks good. Thanks for the changes.
>
> Reviewed-by: Steve Twiss <stwiss.opensource@diasemi.com>
> Acked-by: Steve Twiss <stwiss.opensource@diasemi.com>
Patch now send to Mainline for v4.13 fixes.
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-08-23 7:57 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-15 8:38 [PATCH 1/1] Revert "mfd: da9061: Fix to remove BBAT_CONT register from chip model" Lee Jones
2017-08-21 7:49 ` Lee Jones
2017-08-21 12:32 ` Steve Twiss
2017-08-23 7:57 ` Lee Jones
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).