From: Caesar Wang <caesar.upstream@gmail.com>
To: Andy Yan <andy.yan@rock-chips.com>
Cc: robh+dt@kernel.org, john.stultz@linaro.org, arnd@arndb.de,
heiko@sntech.de, mark.rutland@arm.com, catalin.marinas@arm.com,
will.deacon@arm.com, sre@kernel.org,
alexandre.belloni@free-electrons.com, f.fainelli@gmail.com,
linux@arm.linux.org.uk, dbaryshkov@gmail.com, richard@nod.at,
linux-rockchip@lists.infradead.org, wxt@rock-chips.com,
devicetree@vger.kernel.org, pawel.moll@arm.com,
ijc+devicetree@hellion.org.uk, edubezval@gmail.com,
lorenzo.pieralisi@arm.com, matthias.bgg@gmail.com,
moritz.fischer@ettus.com, linux-kernel@vger.kernel.org,
galak@codeaurora.org, olof@lixom.net, jun.nie@linaro.org,
dwmw2@infradead.org
Subject: Re: [PATCH v4 0/4] add reboot mode driver
Date: Thu, 3 Mar 2016 09:52:49 +0800 [thread overview]
Message-ID: <56D798F1.9060709@gmail.com> (raw)
In-Reply-To: <1456834172-5425-1-git-send-email-andy.yan@rock-chips.com>
Hi Andy,
This series patches ware verified on my github with RK3368 evb and
RK30236 board.
https://github.com/Caesar-github/rockchip/tree/for-reboot-mode-tests
在 2016年03月01日 20:09, Andy Yan 写道:
> This driver parse the reboot commands like "reboot loader"
> and "reboot recovery" to get a boot mode described in the
> device tree , then call the corresponding write interfae
> to store the boot mode in some place like special register
> or ram , which can be read by the bootloader after system
> reboot.
>
> This is commonly done on Android based devices, in order to
> reboot the device into fastboot or recovery mode.
>
>
> Changes in v4:
> - remove mode-maskrom
> - rename mode-fastboot to mode-bootloader to keep compatible with the exiting Android device
> - make this driver depends on OF to avoid kbuild test error
>
> Changes in v3:
> - descirbe all reboot mode as properity instead of subnode
> - scan multi properities
> - add mask value for some platform which only use some bits of the register
> to store boot mode magic value
> - descirbe all reboot mode as property instead of subnode
> - add rk3036 support
> - descirbe all reboot mode as properity instead of subnode
>
> Changes in v2:
> - move to dir drivers/power/reset/
> - make syscon-reboot-mode a generic driver
> - make this node as a subnode of PMU
> - make this node as a subnode of pmugrf
>
> Changes in v1:
> - fix the embarrassed compile warning
> - correct the maskrom magic number
> - check for the normal reboot
> - correct the maskrom magic number
> - use macro defined in rockchip_boot-mode.h for reboot-mode DT node
>
> Andy Yan (4):
> dt-bindings: power: reset: add document for reboot-mode driver
> power: reset: add reboot mode driver
> ARM: dts: rockchip: add syscon-reboot-mode DT node
> ARM64: dts: rockchip: add syscon-reboot-mode DT node
Tested-by: Caesar Wang <wxt@rock-chips.com>
>
> .../bindings/power/reset/reboot-mode.txt | 24 +++++
> .../bindings/power/reset/syscon-reboot-mode.txt | 35 +++++++
> arch/arm/boot/dts/rk3036.dtsi | 11 ++-
> arch/arm/boot/dts/rk3288.dtsi | 10 ++
> arch/arm/boot/dts/rk3xxx.dtsi | 12 ++-
> arch/arm64/boot/dts/rockchip/rk3368.dtsi | 13 ++-
> drivers/power/reset/Kconfig | 16 ++++
> drivers/power/reset/Makefile | 2 +
> drivers/power/reset/reboot-mode.c | 105 +++++++++++++++++++++
> drivers/power/reset/reboot-mode.h | 6 ++
> drivers/power/reset/syscon-reboot-mode.c | 64 +++++++++++++
> include/dt-bindings/soc/rockchip_boot-mode.h | 30 ++++++
> 12 files changed, 325 insertions(+), 3 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/power/reset/reboot-mode.txt
> create mode 100644 Documentation/devicetree/bindings/power/reset/syscon-reboot-mode.txt
> create mode 100644 drivers/power/reset/reboot-mode.c
> create mode 100644 drivers/power/reset/reboot-mode.h
> create mode 100644 drivers/power/reset/syscon-reboot-mode.c
> create mode 100644 include/dt-bindings/soc/rockchip_boot-mode.h
>
--
Thanks,
Caesar
prev parent reply other threads:[~2016-03-03 1:52 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-01 12:09 [PATCH v4 0/4] add reboot mode driver Andy Yan
2016-03-01 12:09 ` Andy Yan
2016-03-01 12:12 ` [PATCH v4 2/4] power: reset: " Andy Yan
2016-03-01 18:29 ` John Stultz
2016-03-01 18:29 ` John Stultz
[not found] ` <1456834172-5425-1-git-send-email-andy.yan-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
2016-03-01 12:11 ` [PATCH v4 1/4] dt-bindings: power: reset: add document for reboot-mode driver Andy Yan
2016-03-01 12:11 ` Andy Yan
2016-03-01 12:12 ` [PATCH v4 3/4] ARM: dts: rockchip: add syscon-reboot-mode DT node Andy Yan
2016-03-01 12:12 ` Andy Yan
2016-03-01 12:14 ` [PATCH v4 4/4] ARM64: " Andy Yan
2016-03-01 12:14 ` Andy Yan
2016-03-03 1:52 ` Caesar Wang [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=56D798F1.9060709@gmail.com \
--to=caesar.upstream@gmail.com \
--cc=alexandre.belloni@free-electrons.com \
--cc=andy.yan@rock-chips.com \
--cc=arnd@arndb.de \
--cc=catalin.marinas@arm.com \
--cc=dbaryshkov@gmail.com \
--cc=devicetree@vger.kernel.org \
--cc=dwmw2@infradead.org \
--cc=edubezval@gmail.com \
--cc=f.fainelli@gmail.com \
--cc=galak@codeaurora.org \
--cc=heiko@sntech.de \
--cc=ijc+devicetree@hellion.org.uk \
--cc=john.stultz@linaro.org \
--cc=jun.nie@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=linux@arm.linux.org.uk \
--cc=lorenzo.pieralisi@arm.com \
--cc=mark.rutland@arm.com \
--cc=matthias.bgg@gmail.com \
--cc=moritz.fischer@ettus.com \
--cc=olof@lixom.net \
--cc=pawel.moll@arm.com \
--cc=richard@nod.at \
--cc=robh+dt@kernel.org \
--cc=sre@kernel.org \
--cc=will.deacon@arm.com \
--cc=wxt@rock-chips.com \
/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 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.