From patchwork Tue Jan 17 01:23:53 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Vladimir Zapolskiy X-Patchwork-Id: 715993 Return-Path: X-Original-To: incoming-imx@patchwork.ozlabs.org Delivered-To: patchwork-incoming-imx@bilbo.ozlabs.org Received: from bombadil.infradead.org (bombadil.infradead.org [IPv6:2001:1868:205::9]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 3v2XSs2VRgz9srY for ; Tue, 17 Jan 2017 12:24:29 +1100 (AEDT) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1cTIVe-0006Wh-O1; Tue, 17 Jan 2017 01:24:26 +0000 Received: from mleia.com ([178.79.152.223] helo=mail.mleia.com) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1cTIVb-0006Db-3N for linux-arm-kernel@lists.infradead.org; Tue, 17 Jan 2017 01:24:25 +0000 Received: from mail.mleia.com (localhost [127.0.0.1]) by mail.mleia.com (Postfix) with ESMTP id 6C18338BF10; Tue, 17 Jan 2017 01:24:02 +0000 (GMT) From: Vladimir Zapolskiy To: Wim Van Sebroeck , Guenter Roeck , Shawn Guo , Fabio Estevam , =?UTF-8?q?Uwe=20Kleine-K=C3=B6nig?= Subject: [PATCH v2] watchdog: imx2: fix hang-up on boot for i.MX21, i.MX27 and i.MX31 SoCs Date: Tue, 17 Jan 2017 03:23:53 +0200 Message-Id: <20170117012353.6948-1-vz@mleia.com> X-Mailer: git-send-email 2.10.2 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-49551924 X-CRM114-CacheID: sfid-20170117_012402_466480_2BBAE861 X-CRM114-Status: GOOD ( 26.45 ) /bin/ln: failed to access 'reaver_cache/texts/20170117_012402_466480_2BBAE861': No such file or directory X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20170117_012402_466480_2BBAE861 X-CRM114-Status: GOOD ( 19.06 ) X-Spam-Score: -5.1 (-----) X-Spam-Report: SpamAssassin version 3.4.1 on bombadil.infradead.org summary: Content analysis details: (-5.1 points) pts rule name description ---- ---------------------- -------------------------------------------------- -3.2 RP_MATCHES_RCVD Envelope sender domain matches handover relay domain -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-arm-kernel@lists.infradead.org, stable@vger.kernel.org, Magnus Lilja , linux-watchdog@vger.kernel.org, Stefan Agner Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org List-Id: linux-imx-kernel.lists.patchwork.ozlabs.org Power down counter enable/disable bit switch is located in WMCR register, but watchdog controllers found on legacy i.MX21, i.MX27 and i.MX31 SoCs don't have this register. As a result of writing data to the non-existing register on driver probe the SoC hangs up, to fix the problem add more OF compatible strings and on this basis get information about availability of the WMCR register. Fixes: 5fe65ce7ccbb ("watchdog: imx2_wdt: Disable power down counter on boot") Reviewed-by: Guenter Roeck Tested-by: Magnus Lilja Reviewed-by: Fabio Estevam Cc: Signed-off-by: Vladimir Zapolskiy --- Changes from v1 to v2: * removed i.MX27 and i.MX31 compatibles from the list, since it is expected that their DTB files has a reference to the listed i.MX21 compatible, many thanks to Uwe Kleine-König for the hint, * start the list of compatibles from an i.MX35 specific one, since it is agreed that i.MX35 precedes i.MX25, thanks to Uwe Kleine-König for review, * added a comment describing any potential changes in future over the list of watchdog compatible values, * added all collected tags to the commit message. Changes from RFC to v1: * replaced private data struct with a macro as suggested by Guenter, * updated the comment in the source code to reflect the change, * rearranged and simplified the logic of detecting WMCR presence, * pulled the fix out from the series with optional proposed DTS changes. drivers/watchdog/imx2_wdt.c | 41 ++++++++++++++++++++++++++++++++++++----- 1 file changed, 36 insertions(+), 5 deletions(-) diff --git a/drivers/watchdog/imx2_wdt.c b/drivers/watchdog/imx2_wdt.c index 4874b0f..e08c367 100644 --- a/drivers/watchdog/imx2_wdt.c +++ b/drivers/watchdog/imx2_wdt.c @@ -30,6 +30,7 @@ #include #include #include +#include #include #include #include @@ -57,6 +58,7 @@ #define IMX2_WDT_WICR_WICT 0xFF /* -> Interrupt Count Timeout */ #define IMX2_WDT_WMCR 0x08 /* Misc Register */ +#define IMX2_WDT_NO_WMCR ((void *)true) /* Indicates unavailable WMCR */ #define IMX2_WDT_MAX_TIME 128 #define IMX2_WDT_DEFAULT_TIME 60 /* in seconds */ @@ -70,6 +72,8 @@ struct imx2_wdt_device { bool ext_reset; }; +static const struct of_device_id imx2_wdt_dt_ids[]; + static bool nowayout = WATCHDOG_NOWAYOUT; module_param(nowayout, bool, 0); MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once started (default=" @@ -242,6 +246,7 @@ static const struct regmap_config imx2_wdt_regmap_config = { static int __init imx2_wdt_probe(struct platform_device *pdev) { + const struct of_device_id *of_id; struct imx2_wdt_device *wdev; struct watchdog_device *wdog; struct resource *res; @@ -310,11 +315,13 @@ static int __init imx2_wdt_probe(struct platform_device *pdev) } /* - * Disable the watchdog power down counter at boot. Otherwise the power - * down counter will pull down the #WDOG interrupt line for one clock - * cycle. + * If watchdog controller has WMCR, disable the watchdog power + * down counter at boot. Otherwise the power down counter will + * pull down the #WDOG interrupt line for one clock cycle. */ - regmap_write(wdev->regmap, IMX2_WDT_WMCR, 0); + of_id = of_match_device(imx2_wdt_dt_ids, &pdev->dev); + if (of_id && !of_id->data) + regmap_write(wdev->regmap, IMX2_WDT_WMCR, 0); ret = watchdog_register_device(wdog); if (ret) { @@ -411,8 +418,32 @@ static int imx2_wdt_resume(struct device *dev) static SIMPLE_DEV_PM_OPS(imx2_wdt_pm_ops, imx2_wdt_suspend, imx2_wdt_resume); +/* + * The list of compatibles is added to achieve compatibility between + * old DTS and new kernel while fixing the incompatibility between + * i.MX21/i.MX27/i.MX31 and i.MX35/i.MX25/etc. watchdog controllers. + * + * Please do *NOT* extend the list by adding a compatible value for + * a controller, which is compatible with one of the already listed, + * almost certainly "fsl,imx35-wdt" compatible serves newer i.MX SoCs. + * + * You may consider to shrink the list at your own risk, but this may + * break the backward compatibility and users may have to update their + * DTB, which is good eventually. + */ static const struct of_device_id imx2_wdt_dt_ids[] = { - { .compatible = "fsl,imx21-wdt", }, + { .compatible = "fsl,imx21-wdt", .data = IMX2_WDT_NO_WMCR }, + { .compatible = "fsl,imx35-wdt", }, + { .compatible = "fsl,imx25-wdt", }, + { .compatible = "fsl,imx50-wdt", }, + { .compatible = "fsl,imx51-wdt", }, + { .compatible = "fsl,imx53-wdt", }, + { .compatible = "fsl,imx6q-wdt", }, + { .compatible = "fsl,imx6sl-wdt", }, + { .compatible = "fsl,imx6sx-wdt", }, + { .compatible = "fsl,imx6ul-wdt", }, + { .compatible = "fsl,imx7d-wdt", }, + { .compatible = "fsl,vf610-wdt", }, { /* sentinel */ } }; MODULE_DEVICE_TABLE(of, imx2_wdt_dt_ids);