From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752153AbcA0AiK (ORCPT ); Tue, 26 Jan 2016 19:38:10 -0500 Received: from mail-pf0-f170.google.com ([209.85.192.170]:34084 "EHLO mail-pf0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751891AbcA0AiF (ORCPT ); Tue, 26 Jan 2016 19:38:05 -0500 From: John Stultz To: lkml Cc: John Stultz , Andy Yan , Rob Herring , Arnd Bergmann , Thierry Reding , =?UTF-8?q?Heiko=20St=C3=BCbner?= , Caesar Wang , Kees Cook , Guodong Xu , Haojian Zhuang , Vishal Bhoj , Bjorn Andersson , devicetree@vger.kernel.org, Android Kernel Team Subject: [RFC][PATCH 0/3] SRAM reboot mode driver Date: Tue, 26 Jan 2016 16:37:57 -0800 Message-Id: <1453855080-17760-1-git-send-email-john.stultz@linaro.org> X-Mailer: git-send-email 1.9.1 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This patchset extends on Andy Yan's reboot mode driver work from here: https://lkml.org/lkml/2016/1/12/315 It adds reboot mode/reason support for devices that use an SRAM location to communicate with the bootloader. Doing this via an SRAM subnode was a suggestion from Arnd, but I worry this implementation isn't yet ideal, since I spent quite a bit of time futzing with getting the sram dts entry to work properly. So I suspect there will be a number of suggestions for improvements. Again, this series dependson Andy's patch set above, but also was developed & tested against the 4.1 based hikey tree, so at least the hikey dts patch won't apply. I'm mostly sending this out for just a rough initial review of the dts and conceptual usage of sram subnodes. Also, it was pointed out that the hikey dts entry for this really should be added by the UEFI firmware, since alternative bootloaders may be used which do not support this feature. So the hikey dts patch isn't likely to ever go upstream, but its a useful illustration for how other devices might use this driver. Cc: Andy Yan Cc: Rob Herring Cc: Arnd Bergmann Cc: Thierry Reding Cc: Heiko Stübner Cc: Caesar Wang Cc: Kees Cook Cc: Guodong Xu Cc: Haojian Zhuang Cc: Vishal Bhoj Cc: Bjorn Andersson Cc: devicetree@vger.kernel.org Cc: Android Kernel Team John Stultz (3): dt-bindings: power: reset: Add document for sram-reboot-mode driver power: reset: Add sram-reboot-mode driver dts: hikey: Add hikey support for sram-reboot-mode .../bindings/power/reset/sram-reboot-mode.txt | 47 +++++++++++++++ arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts | 36 ++++++++++++ arch/arm64/configs/hikey_defconfig | 3 + drivers/power/reset/Kconfig | 9 +++ drivers/power/reset/Makefile | 1 + drivers/power/reset/sram-reboot-mode.c | 66 ++++++++++++++++++++++ 6 files changed, 162 insertions(+) create mode 100644 Documentation/devicetree/bindings/power/reset/sram-reboot-mode.txt create mode 100644 drivers/power/reset/sram-reboot-mode.c -- 1.9.1