From patchwork Tue Dec 15 01:17:56 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kuninori Morimoto X-Patchwork-Id: 7850381 X-Patchwork-Delegate: geert@linux-m68k.org Return-Path: X-Original-To: patchwork-linux-sh@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 5260ABEEE1 for ; Tue, 15 Dec 2015 01:18:43 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 8283120361 for ; Tue, 15 Dec 2015 01:18:42 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 8E58820357 for ; Tue, 15 Dec 2015 01:18:41 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933126AbbLOBSF (ORCPT ); Mon, 14 Dec 2015 20:18:05 -0500 Received: from relmlor3.renesas.com ([210.160.252.173]:42410 "EHLO relmlie2.idc.renesas.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932994AbbLOBSB (ORCPT ); Mon, 14 Dec 2015 20:18:01 -0500 Received: from unknown (HELO relmlir1.idc.renesas.com) ([10.200.68.151]) by relmlie2.idc.renesas.com with ESMTP; 15 Dec 2015 10:18:00 +0900 Received: from relmlac4.idc.renesas.com (relmlac4.idc.renesas.com [10.200.69.24]) by relmlir1.idc.renesas.com (Postfix) with ESMTP id 2963D4E1E1; Tue, 15 Dec 2015 10:18:00 +0900 (JST) Received: by relmlac4.idc.renesas.com (Postfix, from userid 0) id 24CFD480A3; Tue, 15 Dec 2015 10:18:00 +0900 (JST) Received: from relmlac4.idc.renesas.com (localhost [127.0.0.1]) by relmlac4.idc.renesas.com (Postfix) with ESMTP id 1E25C48014; Tue, 15 Dec 2015 10:18:00 +0900 (JST) Received: from relmlii2.idc.renesas.com [10.200.68.66] by relmlac4.idc.renesas.com with ESMTP id LAB09888; Tue, 15 Dec 2015 10:18:00 +0900 X-IronPort-AV: E=Sophos;i="5.20,429,1444662000"; d="scan'";a="201809412" Received: from mail-hk2apc01lp0211.outbound.protection.outlook.com (HELO APC01-HK2-obe.outbound.protection.outlook.com) ([65.55.88.211]) by relmlii2.idc.renesas.com with ESMTP/TLS/AES256-SHA; 15 Dec 2015 10:17:58 +0900 Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=kuninori.morimoto.gx@renesas.com; Received: from morimoto-PC.renesas.com (211.11.155.144) by SIXPR06MB1023.apcprd06.prod.outlook.com (10.160.240.28) with Microsoft SMTP Server (TLS) id 15.1.355.16; Tue, 15 Dec 2015 01:17:56 +0000 Message-ID: <878u4wqzq9.wl%kuninori.morimoto.gx@renesas.com> From: Kuninori Morimoto Subject: [PATCH 3/8 v5] thermal: rcar: check irq possibility in rcar_thermal_irq_xxx() User-Agent: Wanderlust/2.15.9 Emacs/24.3 Mule/6.0 MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") To: Simon , Zhang Rui , Eduardo Valentin CC: Geert Uytterhoeven , Magnus , , , , "devicetree@vger.kernel.org" In-Reply-To: <87d1u8qzsy.wl%kuninori.morimoto.gx@renesas.com> References: <87d1u8qzsy.wl%kuninori.morimoto.gx@renesas.com> Date: Tue, 15 Dec 2015 01:17:56 +0000 X-Originating-IP: [211.11.155.144] X-ClientProxiedBy: HKXPR02CA0057.apcprd02.prod.outlook.com (25.161.48.42) To SIXPR06MB1023.apcprd06.prod.outlook.com (25.160.240.28) X-Microsoft-Exchange-Diagnostics: 1; SIXPR06MB1023; 2:c3PujABTjLlPn5K8+wSKo+5PXggr2bpSRz1nwRjG729+hfHWWxM2+Z5m0sSLHk4NeED+kzF6SC4kdjW/6WNE1OsPhWloYt36iao/9Lt+ioxaoW1aklobmVrvJ09PhY15AJhTLa3eATIskiN8Q+I4ug==; 3:NFi9CrdfNkPrVTI8I6FfYbkOUv9pH0FxTwuIB86JwL/pY/x24an7tSRLsLzDZOajAggtgRDg9Sb2aDX3eK9VLgB1BjL7F6VuEQcSwGhIhUY7zH3Mdb+whxSTMYLRfnBc; 25:vumIoaQwZCLkmjDkATeXTV2vj1/abMxVgGp4mONzy0BS7O/Bld4LUE9XFYuM90iu+TuTj+/r1YPoQTo8udRF4TfnlaFFuBeax/DBlyNg7/Tq3UTIVoowXvv99quSEvZuNzkiQj1wYZfm/3lu0ji8UxaBA4gvBX5b7cdV0rEYX6PKzLvscnUt5JAbs8GdDDuo6omdoAsD7/kYLU3kdJpaPuprNhfKo99NH34k4TXk8O3Hks0NzP6vi3tx3VoBwrd88RE5DoXdvXTQ0rpijuUOig== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:SIXPR06MB1023; X-Microsoft-Exchange-Diagnostics: 1; SIXPR06MB1023; 20:P7EaEXYAyeYwIwGKMz4Wlg04kg88eWUi//Kk2NxT72JHf5C0f2XmaPGbvaab0kyGlbDm1p+x+RNBlqigkTANT9bIMLbx/TJMbpxynWujxmeJDhNH9HrSbofJIrKbzzMvWPOY2/2h8Bp7gVtRvZLtAjYOw9nD1wrr4zVDE61fKnJ4cgKS+bbYfasIxCc+MFph+IPfVwocZ8SsMwiUQdQ4evw/uJamJYqNjqvqiFiOvUPziqQS4Cxh8zq4iufXJrqsa2K9ApHCP65TDvEYvdZBG9C6VtuEDev0a9E/J4YY35RSyb/5gEEqd3zjf4ZHBzjKajde+zZ/0sZYNGRCuN7MA+51uE4uEUj2pKADVX5/6ZzBCLzaEdDzAYoX5IUqf8cJg7TJtwGxbmQq2jypKPNt5WAc7YAvSW4AntEk8+Z2gPbcK8wZ7mjWO7sdlQ/ukUKFbN3k07VeR1ennJUkMaClNTinN5UQ3F83Cdk4MoXEyCsdpR1QE8bWzlCze8Rl3Du6; 4:7IWG5HyVO2x3RBss+UEkMFYub11ww/LkT0E5goX8PChTMYbfdpYZEGyYQTLehOQEE3YxoWK1AANjR6Lpd8I2J5F7Ucy4UzwnKLEWB61lbJOT8B5mihZoLRzKrAJ089sfRzlEdujtY21IsS1lqkmArTgOLvcCQhkIyvWdUFsoS+wGv3AOJBB0cmiH29jOQimOjEUrO278lg+0Kw2WqTnhS1+1heOe8BU8Z1yySY0yxW5fArDHwGMTpkUuSkRhTZgVw4a0r27f1HOn12OVFl6yF50Kxi8rTaIsUX9/+d4VEdo22zfMvNDO1aN0ckQDsMzOxFCc14ol4wOww2FtYmzIYIl0wIRJ0hHBvm2/VaT1vHqxLcpJkNVbINyvRtPLbU1zukTJRK67q1P1u4KoWXikDHp3mdec1BVZflhGMv uOyV8mJfZ/AIYs2mBHvchoWO0W X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(85106069007906); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(601004)(2401047)(8121501046)(520078)(5005006)(3002001)(10201501046); SRVR:SIXPR06MB1023; BCL:0; PCL:0; RULEID:; SRVR:SIXPR06MB1023; X-Forefront-PRVS: 07915F544A X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10019020)(6009001)(199003)(189002)(5001770100001)(66066001)(92566002)(54356999)(46406003)(5008740100001)(36756003)(87976001)(47776003)(586003)(1096002)(6116002)(122386002)(3846002)(40100003)(23726003)(50986999)(19580395003)(105586002)(69596002)(81156007)(50466002)(106356001)(53416004)(5001960100002)(189998001)(77096005)(229853001)(2950100001)(5004730100002)(19580405001)(83506001)(76176999)(4001350100001)(42186005)(86362001)(33646002)(97736004)(101416001); DIR:OUT; SFP:1102; SCL:1; SRVR:SIXPR06MB1023; H:morimoto-PC.renesas.com; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; SIXPR06MB1023; 23:2POxNwS0go/Gpl1ejlG1eAS1UKk4cadxH8pGYJN+B?= =?us-ascii?Q?3FjpNmle3Klfd9ZxTWZbtCL0mw54zUgVQey8qMnrabvvo0oTN3est/TPxfEy?= =?us-ascii?Q?gn4fgGu3WRUiU9pGXL5yna5/jCZgMZi4m2qYEC35wQXy9eyskpBZ2IeQki+N?= =?us-ascii?Q?jAOsLGqAlUB6LvyZ30XFl81ZGzJrvvyuyva4lqr2SJPszuhWqIWcpd8lkvLo?= =?us-ascii?Q?NlvwmvKMBZTUpwasi7TPoiMHXzoznht90iwlMW06/ebnVvDMVsVFQlPTrCEy?= =?us-ascii?Q?53mtdFg0QmgicbM1F876QZ2rByNHngK6cVyx2w/oUG9n1KFlCYAhhR6ZKTol?= =?us-ascii?Q?66Q/M8tekl8P5aZW0bXJf1Wyr48YyO2qrDv9GbQZllw2vMzesL0S3ilEuua6?= =?us-ascii?Q?5X0UinBuQQk0SfC/4DuRlZJ5G4ibh3gZ3DQHF9w5OOQQYRWLgadvT+s7C/Og?= =?us-ascii?Q?rNvi4Qj3mQYWnGkeHvy/RScoChZM7n0KM5Zs87EXHcbTZ6I6LSg5N3JT11Xh?= =?us-ascii?Q?CZR+1ZedGpp23L43nvi4NJHdA6UX4ZrG3b73FKwwYumAONXNNz6K4tosIS8k?= =?us-ascii?Q?CEv0LDCpn7lRapYF0bF9+/AbZLY9njqbOOe4G/h3UUWaCiVhkEiFZjDHu9p8?= =?us-ascii?Q?5x+4TtisK896nrbhVWZV5PJERya9ukeUV/doTJRnqkPI6Jjkk2JQ+QPlsHj6?= =?us-ascii?Q?6LkDLjRKoh+cNeK61bLp4KDTmqmSNXssixG/myjzHwG/K4A8zCDCzI+2CN3e?= =?us-ascii?Q?TKsNMLcfOFqWmYm0RJew4Qtw1YJDnceTWemsQ+fT2pz6DGOraNQAmy1KJIHm?= =?us-ascii?Q?2aDCDMzoEQK5BGfxA05hLnvjEQZfXyt65UWM59lIHa9wOFAohW6yrhtzk4t/?= =?us-ascii?Q?bW8/eAXwXc3wlnDXN+tMpvU4uzGYahQUqjbW5dUYiloNLo/Kqh5j8AxJgTeT?= =?us-ascii?Q?gnwED09nlR5qFTj2bIJlvs2+iJhmsTRlgXuZUo+jX05IyD2cLJsHJGY07e/l?= =?us-ascii?Q?3/TpT/HY6VN4+W/WUB0Yo+AOyNYvzwOaPe3MOe7eHZ8Ioc+WD9UW44NWgTW7?= =?us-ascii?Q?SYlF6H2zziwoApWLHhCRfUa2qaxx41Cxll3XjxLgLuvD4ojl9NnRotMmum/k?= =?us-ascii?Q?kQEbcukP/Q=3D?= X-Microsoft-Exchange-Diagnostics: 1; SIXPR06MB1023; 5:rA2mFuAcumT1QwTIwaNp7CXWqhQ4KYStYNJOyngxyoUIy35EBMraG5I0mYwDqaFNAMzO5IN7cG28kdzv6xu+KYrOTCk2/Ki+BKgOkxYkrHBGi56dD9THZ1ds2AmLmf1cSplqYApENI9CCW3cemNKqw==; 24:meMIHXK8cBmDbQPPa9hKC+AFGq8yjO+xLCxK+Oj59FJiMAgIODKlzLLCtSScq7AF/2iQWKqZd71lN7+XjhGMFypbuo4UhoO1mZ/ZP5k/rfw=; 20:rLpKggJUtiBawbdJffCNkuFM09331d2mkHN5k8mQAE12W+kO2xQ7iyaLDwWvkgOVrerFdHoCaOltVGbEmKAqg0PC3vnKn9lERX35fhXjPsdQeLpkMPhZY+vc3S2cM52z2PGFmE3LT+t7rWUVu9jf/4N+oMT94uRRllC4GABEXvc= SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: renesas.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Dec 2015 01:17:56.8869 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: SIXPR06MB1023 Sender: linux-sh-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-sh@vger.kernel.org X-Spam-Status: No, score=-6.9 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, T_RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Kuninori Morimoto Current rcar thermal driver sometimes checks irq possibility when it calls rcar_thermal_irq_enable/disable(), but sometimes not. This patch checks it inside rcar_thermal_irq_enable/disable(). Signed-off-by: Kuninori Morimoto --- v4 -> v5 - no change drivers/thermal/rcar_thermal.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/thermal/rcar_thermal.c b/drivers/thermal/rcar_thermal.c index 0b19743..e9e4e39 100644 --- a/drivers/thermal/rcar_thermal.c +++ b/drivers/thermal/rcar_thermal.c @@ -290,6 +290,9 @@ static void _rcar_thermal_irq_ctrl(struct rcar_thermal_priv *priv, int enable) unsigned long flags; u32 mask = 0x3 << rcar_id_to_shift(priv); /* enable Rising/Falling */ + if (!rcar_has_irq_support(priv)) + return; + spin_lock_irqsave(&common->lock, flags); rcar_thermal_common_bset(common, INTMSK, mask, enable ? 0 : mask); @@ -441,8 +444,7 @@ static int rcar_thermal_probe(struct platform_device *pdev) goto error_unregister; } - if (rcar_has_irq_support(priv)) - rcar_thermal_irq_enable(priv); + rcar_thermal_irq_enable(priv); list_move_tail(&priv->list, &common->head); @@ -488,8 +490,7 @@ static int rcar_thermal_remove(struct platform_device *pdev) struct rcar_thermal_priv *priv; rcar_thermal_for_each_priv(priv, common) { - if (rcar_has_irq_support(priv)) - rcar_thermal_irq_disable(priv); + rcar_thermal_irq_disable(priv); thermal_zone_device_unregister(priv->zone); }