Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966091AbbLQBQF (ORCPT ); Wed, 16 Dec 2015 20:16:05 -0500 Received: from mail-yk0-f182.google.com ([209.85.160.182]:33987 "EHLO mail-yk0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755175AbbLQBQD convert rfc822-to-8bit (ORCPT ); Wed, 16 Dec 2015 20:16:03 -0500 MIME-Version: 1.0 In-Reply-To: <566E9A47.8010004@rock-chips.com> References: <1447840044-19689-1-git-send-email-andy.yan@rock-chips.com> <45323116.BWNAaZT5p4@diego> <566E9A47.8010004@rock-chips.com> Date: Wed, 16 Dec 2015 17:16:01 -0800 Message-ID: Subject: Re: [PATCH v3 0/5] Add reboot notifier driver for rockchip platform From: John Stultz To: Andy Yan Cc: =?UTF-8?Q?Heiko_St=C3=BCbner?= , Arnd Bergmann , Mark Rutland , devicetree@vger.kernel.org, Kevin Hilman , Russell King - ARM Linux , wxt@rock-chips.com, =?UTF-8?Q?Pawe=C5=82_Moll?= , Ian Campbell , sjg@chromium.org, lkml , linux-rockchip@lists.infradead.org, Rob Herring , Kumar Gala , treding@nvidia.com, "linux-arm-kernel@lists.infradead.org" , benchan@google.com 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 Content-Length: 2357 Lines: 56 On Mon, Dec 14, 2015 at 2:30 AM, Andy Yan wrote: > On 2015年12月12日 05:29, Heiko Stübner wrote: >> Am Mittwoch, 18. November 2015, 17:47:24 schrieb Andy Yan: >>> >>> rockchip platform have a protocol to pass the kernel reboot >>> mode to bootloader by some special registers when system reboot. >>> By this way the bootloader can take different action according >>> to the different kernel reboot mode, for example, command >>> "reboot loader" will reboot the board to rockusb mode, this is >>> a very convenient way to get the board enter download mode. >>> And Android system also use this protocol to pass "recovery"、 >>> “fastboot” reboot mode to bootloader. In upstream land, We found >>> tegra platform also use this mechanism. >>> >>> Before this version, I have sent two Series, which can be found >>> at [0] [1] >>> [0] https://patchwork.kernel.org/patch/7140751/ >>> [1] https://patchwork.kernel.org/patch/7153531/ >> >> just so it doesn't get lost, this ties into a slightly more generic >> approach, >> John Stultz is doing for the android reboot reasons: >> >> http://thread.gmane.org/gmane.linux.kernel/2103447 >> >> The only difference is that we'd need to use the pmu syscon for it. >> >> > Very glad to see your work on this. And your work is more generic than > my. > So hope we can share the code. As you can see, the rockchip platform > use > a GRF or PMU register to store the reboot reason. So maybe we need a > phandle > in the DT node to describe it. Yea. I need to figure out the best way to represent these various methods in DT. Apologies for not being super responsive here. Been a bit heads down trying to finish up some items before holiday break. >From some of Arnd's comments, it sounds like we may need a larger generic subsystem for kernel to firmware communications here. So more then likely I'll not have a chance to take another swing at this after the new year. So if in the meantime you want to try to take my code and rework it, or extend your code in a similar way, I'll not take any offence. :) thanks -john -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/