* [PATCH] ARM: dts: e60k02: fix power button
@ 2019-11-11 20:29 Andreas Kemnade
2019-11-11 21:05 ` H. Nikolaus Schaller
2019-12-04 12:36 ` Shawn Guo
0 siblings, 2 replies; 5+ messages in thread
From: Andreas Kemnade @ 2019-11-11 20:29 UTC (permalink / raw)
To: robh+dt, mark.rutland, shawnguo, devicetree, linux-arm-kernel
Cc: Andreas Kemnade, H . Nikolaus Schaller
The power button was only producing irqs, but no key events,
Forced power down with long key press works, so probably
only a short spike arrives at the SoC.
Further investigation shows that LDORTC2 is off after boot
of the vendor kernel. LDORTC2 is shared with a GPIO at the pmic
which probably transfers the button press to the SoC.
That regulator off at boot, so "regulator-boot-on" is definitively
wrong. So remove that.
Reported-by: H. Nikolaus Schaller <hns@goldelico.com>
Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
---
arch/arm/boot/dts/e60k02.dtsi | 5 -----
1 file changed, 5 deletions(-)
diff --git a/arch/arm/boot/dts/e60k02.dtsi b/arch/arm/boot/dts/e60k02.dtsi
index a0ade2ba7a21..33e02bd2b59d 100644
--- a/arch/arm/boot/dts/e60k02.dtsi
+++ b/arch/arm/boot/dts/e60k02.dtsi
@@ -356,11 +356,6 @@
regulator-name = "LDORTC1";
regulator-boot-on;
};
-
- ldortc2_reg: LDORTC2 {
- regulator-name = "LDORTC2";
- regulator-boot-on;
- };
};
};
};
--
2.20.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: dts: e60k02: fix power button
2019-11-11 20:29 [PATCH] ARM: dts: e60k02: fix power button Andreas Kemnade
@ 2019-11-11 21:05 ` H. Nikolaus Schaller
2019-12-04 12:36 ` Shawn Guo
1 sibling, 0 replies; 5+ messages in thread
From: H. Nikolaus Schaller @ 2019-11-11 21:05 UTC (permalink / raw)
To: Andreas Kemnade
Cc: robh+dt, mark.rutland, shawnguo, devicetree, linux-arm-kernel
> Am 11.11.2019 um 21:29 schrieb Andreas Kemnade <andreas@kemnade.info>:
>
> The power button was only producing irqs, but no key events,
> Forced power down with long key press works, so probably
> only a short spike arrives at the SoC.
> Further investigation shows that LDORTC2 is off after boot
> of the vendor kernel. LDORTC2 is shared with a GPIO at the pmic
> which probably transfers the button press to the SoC.
> That regulator off at boot, so "regulator-boot-on" is definitively
> wrong. So remove that.
>
> Reported-by: H. Nikolaus Schaller <hns@goldelico.com>
> Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
> ---
> arch/arm/boot/dts/e60k02.dtsi | 5 -----
> 1 file changed, 5 deletions(-)
>
> diff --git a/arch/arm/boot/dts/e60k02.dtsi b/arch/arm/boot/dts/e60k02.dtsi
> index a0ade2ba7a21..33e02bd2b59d 100644
> --- a/arch/arm/boot/dts/e60k02.dtsi
> +++ b/arch/arm/boot/dts/e60k02.dtsi
> @@ -356,11 +356,6 @@
> regulator-name = "LDORTC1";
> regulator-boot-on;
> };
> -
> - ldortc2_reg: LDORTC2 {
> - regulator-name = "LDORTC2";
> - regulator-boot-on;
> - };
> };
> };
> };
> --
> 2.20.1
>
Also:
Tested-by: H. Nikolaus Schaller <hns@goldelico.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: dts: e60k02: fix power button
2019-11-11 20:29 [PATCH] ARM: dts: e60k02: fix power button Andreas Kemnade
2019-11-11 21:05 ` H. Nikolaus Schaller
@ 2019-12-04 12:36 ` Shawn Guo
2019-12-05 6:06 ` Andreas Kemnade
1 sibling, 1 reply; 5+ messages in thread
From: Shawn Guo @ 2019-12-04 12:36 UTC (permalink / raw)
To: Andreas Kemnade
Cc: robh+dt, mark.rutland, devicetree, linux-arm-kernel,
H . Nikolaus Schaller
On Mon, Nov 11, 2019 at 09:29:59PM +0100, Andreas Kemnade wrote:
> The power button was only producing irqs, but no key events,
> Forced power down with long key press works, so probably
> only a short spike arrives at the SoC.
> Further investigation shows that LDORTC2 is off after boot
> of the vendor kernel. LDORTC2 is shared with a GPIO at the pmic
> which probably transfers the button press to the SoC.
> That regulator off at boot, so "regulator-boot-on" is definitively
> wrong. So remove that.
>
> Reported-by: H. Nikolaus Schaller <hns@goldelico.com>
> Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
Do you want this go into 5.5-rc as a fix? In that case, we may need a
Fixes tag here.
Shawn
> ---
> arch/arm/boot/dts/e60k02.dtsi | 5 -----
> 1 file changed, 5 deletions(-)
>
> diff --git a/arch/arm/boot/dts/e60k02.dtsi b/arch/arm/boot/dts/e60k02.dtsi
> index a0ade2ba7a21..33e02bd2b59d 100644
> --- a/arch/arm/boot/dts/e60k02.dtsi
> +++ b/arch/arm/boot/dts/e60k02.dtsi
> @@ -356,11 +356,6 @@
> regulator-name = "LDORTC1";
> regulator-boot-on;
> };
> -
> - ldortc2_reg: LDORTC2 {
> - regulator-name = "LDORTC2";
> - regulator-boot-on;
> - };
> };
> };
> };
> --
> 2.20.1
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: dts: e60k02: fix power button
2019-12-04 12:36 ` Shawn Guo
@ 2019-12-05 6:06 ` Andreas Kemnade
2019-12-09 0:56 ` Shawn Guo
0 siblings, 1 reply; 5+ messages in thread
From: Andreas Kemnade @ 2019-12-05 6:06 UTC (permalink / raw)
To: Shawn Guo
Cc: robh+dt, mark.rutland, devicetree, linux-arm-kernel,
H . Nikolaus Schaller
[-- Attachment #1: Type: text/plain, Size: 1393 bytes --]
On Wed, 4 Dec 2019 20:36:47 +0800
Shawn Guo <shawnguo@kernel.org> wrote:
> On Mon, Nov 11, 2019 at 09:29:59PM +0100, Andreas Kemnade wrote:
> > The power button was only producing irqs, but no key events,
> > Forced power down with long key press works, so probably
> > only a short spike arrives at the SoC.
> > Further investigation shows that LDORTC2 is off after boot
> > of the vendor kernel. LDORTC2 is shared with a GPIO at the pmic
> > which probably transfers the button press to the SoC.
> > That regulator off at boot, so "regulator-boot-on" is definitively
> > wrong. So remove that.
> >
> > Reported-by: H. Nikolaus Schaller <hns@goldelico.com>
> > Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
>
> Do you want this go into 5.5-rc as a fix? In that case, we may need a
> Fixes tag here.
>
well, now there is
regulator: core: Let boot-on regulators be powered off
so the power button should work after regulator_late_cleanup()
is called but that is usually some time after userspace has started,
so it would be better to have that in. And of course dt should
correctly describe the hardware. That would be
Fixes: c100ea86e6ab ("ARM: dts: add Netronix E60K02 board common file")
BTW: I have not seen your 5.5 stuff in Linus' tree yet. Seems that Olof
has not sent his pull request yet or is there bigger trouble?
Regards,
Andreas
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] ARM: dts: e60k02: fix power button
2019-12-05 6:06 ` Andreas Kemnade
@ 2019-12-09 0:56 ` Shawn Guo
0 siblings, 0 replies; 5+ messages in thread
From: Shawn Guo @ 2019-12-09 0:56 UTC (permalink / raw)
To: Andreas Kemnade
Cc: robh+dt, mark.rutland, devicetree, linux-arm-kernel,
H . Nikolaus Schaller
On Thu, Dec 05, 2019 at 07:06:12AM +0100, Andreas Kemnade wrote:
> On Wed, 4 Dec 2019 20:36:47 +0800
> Shawn Guo <shawnguo@kernel.org> wrote:
>
> > On Mon, Nov 11, 2019 at 09:29:59PM +0100, Andreas Kemnade wrote:
> > > The power button was only producing irqs, but no key events,
> > > Forced power down with long key press works, so probably
> > > only a short spike arrives at the SoC.
> > > Further investigation shows that LDORTC2 is off after boot
> > > of the vendor kernel. LDORTC2 is shared with a GPIO at the pmic
> > > which probably transfers the button press to the SoC.
> > > That regulator off at boot, so "regulator-boot-on" is definitively
> > > wrong. So remove that.
> > >
> > > Reported-by: H. Nikolaus Schaller <hns@goldelico.com>
> > > Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
> >
> > Do you want this go into 5.5-rc as a fix? In that case, we may need a
> > Fixes tag here.
> >
> well, now there is
> regulator: core: Let boot-on regulators be powered off
> so the power button should work after regulator_late_cleanup()
> is called but that is usually some time after userspace has started,
> so it would be better to have that in. And of course dt should
> correctly describe the hardware. That would be
>
> Fixes: c100ea86e6ab ("ARM: dts: add Netronix E60K02 board common file")
Applied with the tag.
>
> BTW: I have not seen your 5.5 stuff in Linus' tree yet. Seems that Olof
> has not sent his pull request yet or is there bigger trouble?
Now they are in v5.5-rc1.
Shawn
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-12-09 0:56 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-11 20:29 [PATCH] ARM: dts: e60k02: fix power button Andreas Kemnade
2019-11-11 21:05 ` H. Nikolaus Schaller
2019-12-04 12:36 ` Shawn Guo
2019-12-05 6:06 ` Andreas Kemnade
2019-12-09 0:56 ` Shawn Guo
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).