devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <k.kozlowski@samsung.com>
To: Markus Reichl <m.reichl@fivetechno.de>
Cc: Alim Akhtar <alim.akhtar@samsung.com>,
	linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org,
	Krzysztof Kozlowski <k.kozlowski@samsung.com>,
	Javier Martinez Canillas <javier@osg.samsung.com>,
	Tobias Jakobi <tjakobi@math.uni-bielefeld.de>,
	kgene@kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 0/7] Switch to generic syscon regmap based drivers
Date: Tue, 20 Oct 2015 22:56:46 +0900	[thread overview]
Message-ID: <CAJKOXPd74=CuUP_ue=ZqPFX6HbMu9qBCTYtNU+KZ1X4Dvs2ZGA@mail.gmail.com> (raw)
In-Reply-To: <56264505.3090405@fivetechno.de>

2015-10-20 22:43 GMT+09:00 Markus Reichl <m.reichl@fivetechno.de>:
> Am 19.10.2015 um 08:03 schrieb Alim Akhtar:
>> Alim Akhtar (7):
>>   arm: dts: Add syscon-{reboot, poweroff} nodes for exynos3250 SoCs
>>   arm: dts: Add syscon-{reboot, poweroff} nodes for exynos4
>>   arm: dts: Add syscon-{reboot, poweroff} nodes for exynos5
>>   arm: dts: Add syscon-{reboot, poweroff} nodes for exynos5410 SoC
>>   ARM: exynos_defconfig: Normalize exynos defconfig
>>   ARM: exynos_defconfig: Enable generic syscon-{reboot, poweroff}
>>     drivers
>>   ARM: EXYNOS: Remove code for restart and poweroff for exynos SoCs
>>
>>  arch/arm/boot/dts/exynos3250.dtsi |   14 ++++++++++++
>>  arch/arm/boot/dts/exynos4.dtsi    |   14 ++++++++++++
>>  arch/arm/boot/dts/exynos5.dtsi    |   14 ++++++++++++
>>  arch/arm/boot/dts/exynos5410.dtsi |   14 ++++++++++++
>>  arch/arm/configs/exynos_defconfig |   11 +++++-----
>>  arch/arm/mach-exynos/pmu.c        |   43 -------------------------------------
>>  6 files changed, 61 insertions(+), 49 deletions(-)
>>
> Hi,
>
> I have tested this patch on my Odroid U3 and U3+ using 4.3.0-rc6 from
> Tobias' repo at https://github.com/tobiasjakobi/linux-odroid/tree/odroid-4.3.y
>
> and his debug config:
> https://raw.githubusercontent.com/tobiasjakobi/odroid-environment/master/sourcecode/system/vanilla-4.3-debug.conf
>
> halt -p works (power goes down to 0W)
>
> reboot does not work, stuck at 0.5W, right after
> "systemd-shutdown[1]: rebooting".
>
> reboot did work with 4.3.0-rc3 from above repo (without Alim's patch).

Hi Markus,

You replied to v1 so I assume you tested v1. Please test latest version: v2.

Mentioned Tobias' repo does not look like vanilla 4.3.0-rc6. Because
of that it may contain additional patches which can fix something...
or break something (like the reboot).
Please test it on latest mainline release or some recent linux-next.

In my case I tested it on linux-next on Oroid XU3 and tomorrow I will
give it a try on Trats2.

Best regards,
Krzysztof

      reply	other threads:[~2015-10-20 13:56 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-19  6:03 [PATCH 0/7] Switch to generic syscon regmap based drivers Alim Akhtar
2015-10-19  6:03 ` [PATCH 1/7] arm: dts: Add syscon-{reboot, poweroff} nodes for exynos3250 SoCs Alim Akhtar
2015-10-19  8:43   ` [1/7] " Pankaj Dubey
2015-10-19 13:03   ` [PATCH 1/7] " Javier Martinez Canillas
2015-10-20  9:00     ` Alim Akhtar
2015-10-19  6:03 ` [PATCH 2/7] arm: dts: Add syscon-{reboot, poweroff} nodes for exynos4 Alim Akhtar
2015-10-19  8:45   ` [2/7] " Pankaj Dubey
2015-10-19 13:08   ` [PATCH 2/7] " Javier Martinez Canillas
2015-10-19  6:03 ` [PATCH 3/7] arm: dts: Add syscon-{reboot, poweroff} nodes for exynos5 Alim Akhtar
2015-10-19  8:47   ` [3/7] " Pankaj Dubey
2015-10-19 13:09   ` [PATCH 3/7] " Javier Martinez Canillas
2015-10-19  6:03 ` [PATCH 4/7] arm: dts: Add syscon-{reboot, poweroff} nodes for exynos5410 SoC Alim Akhtar
2015-10-19  8:56   ` [4/7] " Pankaj Dubey
2015-10-19 13:10   ` [PATCH 4/7] " Javier Martinez Canillas
2015-10-19  6:03 ` [PATCH 5/7] ARM: exynos_defconfig: Normalize exynos defconfig Alim Akhtar
2015-10-19 13:25   ` Javier Martinez Canillas
2015-10-19 15:58     ` Alim Akhtar
2015-10-19 16:38       ` Javier Martinez Canillas
2015-10-19  6:03 ` [PATCH 6/7] ARM: exynos_defconfig: Enable generic syscon-{reboot, poweroff} drivers Alim Akhtar
2015-10-19  6:03 ` [PATCH 7/7] ARM: EXYNOS: Remove code for restart and poweroff for exynos SoCs Alim Akhtar
2015-10-19  7:00   ` Krzysztof Kozlowski
2015-10-19  8:27     ` Alim Akhtar
2015-10-19  9:56     ` Javier Martinez Canillas
2015-10-19 10:23       ` Alim Akhtar
2015-10-19 13:28       ` Krzysztof Kozlowski
2015-10-19 14:07         ` Javier Martinez Canillas
2015-10-19 16:13           ` Alim Akhtar
2015-10-19 16:42             ` Javier Martinez Canillas
2015-10-19 16:06         ` Alim Akhtar
2015-10-20  0:20           ` Krzysztof Kozlowski
2015-10-19  6:54 ` [PATCH 0/7] Switch to generic syscon regmap based drivers Krzysztof Kozlowski
2015-10-19  7:40   ` Alim Akhtar
2015-10-19 16:08 ` Tobias Jakobi
2015-10-19 16:36   ` Alim Akhtar
2015-10-19 21:18 ` Moritz Fischer
2015-10-20  8:56   ` Alim Akhtar
2015-10-20 13:43 ` Markus Reichl
2015-10-20 13:56   ` Krzysztof Kozlowski [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAJKOXPd74=CuUP_ue=ZqPFX6HbMu9qBCTYtNU+KZ1X4Dvs2ZGA@mail.gmail.com' \
    --to=k.kozlowski@samsung.com \
    --cc=alim.akhtar@samsung.com \
    --cc=devicetree@vger.kernel.org \
    --cc=javier@osg.samsung.com \
    --cc=kgene@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=m.reichl@fivetechno.de \
    --cc=tjakobi@math.uni-bielefeld.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).