diff for duplicates of <1497969136.3177.10.camel@nxp.com> diff --git a/a/1.txt b/N1/1.txt index 2582643..bb5d249 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -6,15 +6,15 @@ On Tue, 2017-06-20 at 06:55 +0200, Oleksij Rempel wrote: > > > external > > > PMIC is the following: > > > ... -> > > 3. SoC is programming PMIC for power off when standby is asserted. -> > > 4. In CCM STOP mode, Standby is asserted, PMIC gates SoC supplies. +> > > 3.??SoC is programming PMIC for power off when standby is asserted. +> > > 4.??In CCM STOP mode, Standby is asserted, PMIC gates SoC supplies. > > > > > > This patch implements step 4. of this sequence. > > > > > > Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de> > > > --- -> > > arch/arm/mach-imx/pm-imx6.c | 25 +++++++++++++++++++++++++ -> > > 1 file changed, 25 insertions(+) +> > > ?arch/arm/mach-imx/pm-imx6.c | 25 +++++++++++++++++++++++++ +> > > ?1 file changed, 25 insertions(+) > > > > > > diff --git a/arch/arm/mach-imx/pm-imx6.c b/arch/arm/mach-imx/pm- > > > imx6.c @@ -23,8 +23,8 @@ On Tue, 2017-06-20 at 06:55 +0200, Oleksij Rempel wrote: > > > +++ b/arch/arm/mach-imx/pm-imx6.c > > > @@ -602,6 +602,28 @@ static void __init imx6_pm_common_init(const > > > struct imx6_pm_socdata -> > > IMX6Q_GPR1_GINT); -> > > } +> > > ? ???IMX6Q_GPR1_GINT); +> > > ?} > > > > > > +static void imx6_pm_poweroff(void) > > > +{ @@ -33,9 +33,9 @@ On Tue, 2017-06-20 at 06:55 +0200, Oleksij Rempel wrote: > > It's a bit strange that you're using parts of the suspend code for > > this. Perhaps you could just write your bits to CLPCR and then execute > > a WFI? -> Well, every thing in this patch set, yell about using standby/suspend -> path in HW for power off. Even iMX code use STOP_POWER_OFF for -> configuring this regs. How can it be more strange? :) Should i really +> Well, every thing in this patch set, yell about using standby/suspend? +> path in HW for power off. Even iMX code use STOP_POWER_OFF for? +> configuring this regs. How can it be more strange? :) Should i really? > duplicate code to hide this? > This code is used to set the power state the soc enters on WFI. These diff --git a/a/content_digest b/N1/content_digest index fecbbd5..b318bd5 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -2,25 +2,10 @@ "ref\020170619050245.2632-3-o.rempel@pengutronix.de\0" "ref\01497872133.5380.11.camel@nxp.com\0" "ref\07f2db0a8-a316-5ecb-24bf-46bcb625e123@pengutronix.de\0" - "From\0Leonard Crestez <leonard.crestez@nxp.com>\0" - "Subject\0Re: [PATCH v5 2/6] ARM: imx6: register pm_power_off handler if \"fsl,pmic-stby-poweroff\" is set\0" + "From\0leonard.crestez@nxp.com (Leonard Crestez)\0" + "Subject\0[PATCH v5 2/6] ARM: imx6: register pm_power_off handler if \"fsl,pmic-stby-poweroff\" is set\0" "Date\0Tue, 20 Jun 2017 17:32:16 +0300\0" - "To\0Oleksij Rempel <o.rempel@pengutronix.de>\0" - "Cc\0Andrew Morton <akpm@linux-foundation.org>" - <devicetree@vger.kernel.org> - Fabio Estevam <fabio.estevam@nxp.com> - <kernel@pengutronix.de> - Liam Girdwood <lgirdwood@gmail.com> - <linux-arm-kernel@lists.infradead.org> - <linux-clk@vger.kernel.org> - <linux-kernel@vger.kernel.org> - Mark Brown <broonie@kernel.org> - Mark Rutland <mark.rutland@arm.com> - Michael Turquette <mturquette@baylibre.com> - Rob Herring <robh+dt@kernel.org> - Russell King <linux@armlinux.org.uk> - Shawn Guo <shawnguo@kernel.org> - " Stephen Boyd <sboyd@codeaurora.org>\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "On Tue, 2017-06-20 at 06:55 +0200, Oleksij Rempel wrote:\n" @@ -31,15 +16,15 @@ "> > > external\n" "> > > PMIC is the following:\n" "> > > ...\n" - "> > > 3.\302\240\302\240SoC is programming PMIC for power off when standby is asserted.\n" - "> > > 4.\302\240\302\240In CCM STOP mode, Standby is asserted, PMIC gates SoC supplies.\n" + "> > > 3.??SoC is programming PMIC for power off when standby is asserted.\n" + "> > > 4.??In CCM STOP mode, Standby is asserted, PMIC gates SoC supplies.\n" "> > > \n" "> > > This patch implements step 4. of this sequence.\n" "> > > \n" "> > > Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>\n" "> > > ---\n" - "> > > \302\240arch/arm/mach-imx/pm-imx6.c | 25 +++++++++++++++++++++++++\n" - "> > > \302\2401 file changed, 25 insertions(+)\n" + "> > > ?arch/arm/mach-imx/pm-imx6.c | 25 +++++++++++++++++++++++++\n" + "> > > ?1 file changed, 25 insertions(+)\n" "> > > \n" "> > > diff --git a/arch/arm/mach-imx/pm-imx6.c b/arch/arm/mach-imx/pm-\n" "> > > imx6.c\n" @@ -48,8 +33,8 @@ "> > > +++ b/arch/arm/mach-imx/pm-imx6.c\n" "> > > @@ -602,6 +602,28 @@ static void __init imx6_pm_common_init(const\n" "> > > struct imx6_pm_socdata\n" - "> > > \302\240\t\t\t\t\302\240\302\240\302\240IMX6Q_GPR1_GINT);\n" - "> > > \302\240}\n" + "> > > ?\t\t\t\t???IMX6Q_GPR1_GINT);\n" + "> > > ?}\n" "> > > \n" "> > > +static void imx6_pm_poweroff(void)\n" "> > > +{\n" @@ -58,9 +43,9 @@ "> > It's a bit strange that you're using parts of the suspend code for\n" "> > this. Perhaps you could just write your bits to CLPCR and then execute\n" "> > a WFI?\n" - "> Well, every thing in this patch set, yell about using standby/suspend\302\240\n" - "> path in HW for power off. Even iMX code use STOP_POWER_OFF for\302\240\n" - "> configuring this regs. How can it be more strange? :) Should i really\302\240\n" + "> Well, every thing in this patch set, yell about using standby/suspend?\n" + "> path in HW for power off. Even iMX code use STOP_POWER_OFF for?\n" + "> configuring this regs. How can it be more strange? :) Should i really?\n" "> duplicate code to hide this?\n" "> \n" "This code is used to set the power state the soc enters on WFI. These\n" @@ -70,4 +55,4 @@ "However I can't think of any obvious way this could misbehave so I\n" guess it's fine. -e3b0e5fc2899624b0f39cacfe1cc5a30e2cbcb4526771683f887dc991bf1de5c +4cbc97b2474b666a4eb9371bbefc08cbc7041e97dfe0d5881c356abac44d6733
diff --git a/a/content_digest b/N2/content_digest index fecbbd5..70f6c9b 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -7,13 +7,13 @@ "Date\0Tue, 20 Jun 2017 17:32:16 +0300\0" "To\0Oleksij Rempel <o.rempel@pengutronix.de>\0" "Cc\0Andrew Morton <akpm@linux-foundation.org>" - <devicetree@vger.kernel.org> + devicetree@vger.kernel.org Fabio Estevam <fabio.estevam@nxp.com> - <kernel@pengutronix.de> + kernel@pengutronix.de Liam Girdwood <lgirdwood@gmail.com> - <linux-arm-kernel@lists.infradead.org> - <linux-clk@vger.kernel.org> - <linux-kernel@vger.kernel.org> + linux-arm-kernel@lists.infradead.org + linux-clk@vger.kernel.org + linux-kernel@vger.kernel.org Mark Brown <broonie@kernel.org> Mark Rutland <mark.rutland@arm.com> Michael Turquette <mturquette@baylibre.com> @@ -70,4 +70,4 @@ "However I can't think of any obvious way this could misbehave so I\n" guess it's fine. -e3b0e5fc2899624b0f39cacfe1cc5a30e2cbcb4526771683f887dc991bf1de5c +69c1559ec6c1fe42b81c27e4ea595f1b95446e900ae13fd134abbb26a645063b
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.