From patchwork Tue Jan 14 03:45:22 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Xingyu Chen X-Patchwork-Id: 11331243 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id A85ED139A for ; Tue, 14 Jan 2020 03:45:38 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 8EC12214AF for ; Tue, 14 Jan 2020 03:45:38 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729308AbgANDpe (ORCPT ); Mon, 13 Jan 2020 22:45:34 -0500 Received: from mail-sz.amlogic.com ([211.162.65.117]:56845 "EHLO mail-sz.amlogic.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729335AbgANDpe (ORCPT ); Mon, 13 Jan 2020 22:45:34 -0500 Received: from droid12-sz.software.amlogic (10.28.8.22) by mail-sz.amlogic.com (10.28.11.5) with Microsoft SMTP Server id 15.1.1591.10; Tue, 14 Jan 2020 11:45:58 +0800 From: Xingyu Chen To: Wim Van Sebroeck , Guenter Roeck , Kevin Hilman , Neil Armstrong CC: Xingyu Chen , Rob Herring , Jerome Brunet , Martin Blumenstingl , Qianggui Song , Jianxin Pan , Jian Hu , , , , , Subject: [PATCH v6 0/5] add meson secure watchdog driver Date: Tue, 14 Jan 2020 11:45:22 +0800 Message-ID: <1578973527-4759-1-git-send-email-xingyu.chen@amlogic.com> X-Mailer: git-send-email 2.7.4 MIME-Version: 1.0 X-Originating-IP: [10.28.8.22] Sender: linux-watchdog-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-watchdog@vger.kernel.org The watchdog controller on the Meson-A/C series SoCs is moved to secure world, We have to call SMC instruction to trap the ATF for watchdog operation. These operations are different from previous SoCs, so we introduce a new watchdog driver to support this kind of SoCs. Changes since v5 at [4]: - take the wdt node as child node of secure-monitor - update dt-binding description - invoke of_platform_default_populate() instantiate wdt device according to Rob's suggestion at [5] - find the secure-monitor node through the wdt parent node instead of of_find_compatible_node() - add Guenter's Reviewed-by Changes since v4 at [3]: - add watchdog node in dts, and introduce a new optional property "timeout-sec" - add dt-binding for meson secure watchdog - instantiate wdt device through dts node instead of platform_device_register_simple() Changes since v3 at [2]: - add SM_A1_ prefix for WATCHDOG_OPS - remove phandle to secure-monitor node - remove watchdog node from dts, and register wdt device by platform_device_register_simple() - remove dt-binding for meson secure watchdog - use the msec as unit of timeout parameter which is passed to fw side Changes since v2 at [1]: - remove useless dependency in Kconfig - return zero when getting left time value fails Changes since v1 at [0]: - add a new dependency in Kconfig - simplify/add the return operation - remove useless ping operation when setting the timeout - fix some return values - fix the license statement [0]:https://lore.kernel.org/linux-amlogic/1570874721-36077-1-git-send-email-xingyu.chen@amlogic.com [1]:https://lore.kernel.org/linux-amlogic/1571387622-35132-1-git-send-email-xingyu.chen@amlogic.com [2]:https://lore.kernel.org/linux-amlogic/1571983984-11771-1-git-send-email-xingyu.chen@amlogic.com [3]:https://lore.kernel.org/linux-amlogic/1574685218-31164-1-git-send-email-xingyu.chen@amlogic.com [4]:https://lore.kernel.org/linux-amlogic/1576153187-28378-1-git-send-email-xingyu.chen@amlogic.com [5]:https://patchwork.kernel.org/patch/11288017 Xingyu Chen (5): firmware: meson_sm: add new SMC ID support for accessing secure watchdog firmware: meson_sm: populate platform device based on the child node dt-bindings: watchdog: add new binding for meson secure watchdog watchdog: add meson secure watchdog driver arm64: dts: a1: add secure watchdog controller .../bindings/watchdog/amlogic,meson-sec-wdt.yaml | 40 +++++ arch/arm64/boot/dts/amlogic/meson-a1.dtsi | 6 + drivers/firmware/meson/meson_sm.c | 3 + drivers/watchdog/Kconfig | 16 ++ drivers/watchdog/Makefile | 1 + drivers/watchdog/meson_sec_wdt.c | 185 +++++++++++++++++++++ include/linux/firmware/meson/meson_sm.h | 1 + 7 files changed, 252 insertions(+) create mode 100644 Documentation/devicetree/bindings/watchdog/amlogic,meson-sec-wdt.yaml create mode 100644 drivers/watchdog/meson_sec_wdt.c