From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kuninori Morimoto Date: Tue, 15 Dec 2015 01:19:53 +0000 Subject: [PATCH 8/8 v5] thermal: trip_point_temp_store() calls thermal_zone_device_update() Message-Id: <871taoqzn3.wl%kuninori.morimoto.gx@renesas.com> List-Id: References: <87d1u8qzsy.wl%kuninori.morimoto.gx@renesas.com> In-Reply-To: <87d1u8qzsy.wl%kuninori.morimoto.gx@renesas.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Simon , Zhang Rui , Eduardo Valentin Cc: Geert Uytterhoeven , Magnus , linux-sh@vger.kernel.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, "devicetree@vger.kernel.org" From: Kuninori Morimoto trip_point_temp_store() updates trip temperature. It should call thermal_zone_device_update() immediately. Signed-off-by: Kuninori Morimoto --- v4 -> v5 - implemented on thermal_core.c drivers/thermal/thermal_core.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c index d9e525c..768fb10 100644 --- a/drivers/thermal/thermal_core.c +++ b/drivers/thermal/thermal_core.c @@ -676,8 +676,12 @@ trip_point_temp_store(struct device *dev, struct device_attribute *attr, return -EINVAL; ret = tz->ops->set_trip_temp(tz, trip, temperature); + if (ret) + return ret; - return ret ? ret : count; + thermal_zone_device_update(tz); + + return count; } static ssize_t -- 1.9.1 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933190AbbLOBUB (ORCPT ); Mon, 14 Dec 2015 20:20:01 -0500 Received: from relmlor2.renesas.com ([210.160.252.172]:63923 "EHLO relmlie1.idc.renesas.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932824AbbLOBT6 (ORCPT ); Mon, 14 Dec 2015 20:19:58 -0500 X-IronPort-AV: E=Sophos;i="5.20,429,1444662000"; d="scan'";a="201809544" Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=kuninori.morimoto.gx@renesas.com; Message-ID: <871taoqzn3.wl%kuninori.morimoto.gx@renesas.com> From: Kuninori Morimoto Subject: [PATCH 8/8 v5] thermal: trip_point_temp_store() calls thermal_zone_device_update() 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> Content-Type: text/plain; charset="US-ASCII" Date: Tue, 15 Dec 2015 01:19:53 +0000 X-Originating-IP: [211.11.155.144] X-ClientProxiedBy: OS1PR01CA0001.jpnprd01.prod.outlook.com (25.161.225.139) To HK2PR06MB1010.apcprd06.prod.outlook.com (25.161.187.27) X-Microsoft-Exchange-Diagnostics: 1;HK2PR06MB1010;2:HfMCCIYHf9WCukg1Aj7af8tO6OfPry5JNZJBUmywHVkRFqUd0TeRJHWWZIDLFU2rlRKuGxgQaQeb5iMRBelQJ5f+HAJxJZxeLMxgOEXnkoivHaKYi4TVRQ+IPliorolkBiGGKZmw0qP/KDhmcnuENQ==;3:Of/A6bHpH7C3YEueTK2lxR8iyX5MyNMHziIIlr5B9OFnydgHTpwoN5sfCj+IRGDFlYqOLJfW/FgN3+p3lYmm6P8X5z6zGmaeN2mc4+u3BF6FfWrtXFFhqWZRKoAo4G/b;25:wBGZ1Ezxhgc/95zkIQRyMQdLFpYznM7YAsDqDUQGychgTDMFBE6avRVOA3phIhlMaAfarDDoDEgYuiHfTTfYcSAF7zHWK39g9/ucUCF2qeky0OgOigTCyI2gkMpu8ruHdGzT84eQt9UoY+Mlk/UjguACnKi61QtNj55P21oWNdbZvfJhazmuCNoBOv3jm42xOKYMGWkpET9NvXKE7Tm26gS7NLVjw7MLPIqWdMkOdi6LjLi/yxXuRPqQfADj/zDRUfRl1ORw86SszkI4XDM9Vw== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:HK2PR06MB1010; X-Microsoft-Exchange-Diagnostics: 1;HK2PR06MB1010;20:Lg5Nzva3ZPo27gX5dUqW1+mhDNgp3OBsEPAE0W5Agrb9/pgknEzljbVuYDb+D8k3xkZ/co0g9shyP8V8gZ5Ys0wb/4hlDY64IxvGrxrwz8Y4Fjs+0kjdKw1l4foUMYGQGogNNmPCd8ztjSa5UN3AT93wXgX1dwiswoKufDa1OraaHGHlOOyw+wSG1wfH5SDvW4fup+hyD8ChljsXuyFwbDbbqUQVc99pt0fyU7vEneay6MVbpR3CQKn4GTbJb8fE3cU1zvnVBJhyK5fCIPGgbsrm+jQArRGA5+ZwJCtVFDY+9qREiKc2GnaCWc5uFp8iHCftvDMSNAtF1xJY4cPDULGORNo825CJaIrrQ1IB0jht8pZHtZ3X0FgYkcjsDbSsF0nzPZU+OWay8Gk1mb/l+4RUFv8N4PFnlTURYr6x/9Buoe5hpQN2rt34dWfoOTRhqzB+ctO6MV8peezknuCDwQCIWQJL18ucm5mzfiEbajRP76dfJsDn8SAlB2rYK7aj;4:sVQuHHML1oxoGkKRWj9Xs0yaHsjm2nbcd6g5JGtyAgMvtokLpaHrNdox/oc28F4oqJ9v3YOOlPMfViQ/W4uDdHO2IQwMrd9KJ0dCM3TZGqvns83J7j6urQTMoPrToj8pApcNL/qQ1xBGyWihgOQN5XPEzImoPaJkL7YP330TnKDjG18rFXcM5OzADGSAucWUGGPC07+qFkSLiVaRaKrFu044/SjvSDcVLqJwcyDXyAIyAZUPQ7VySS5i1q5ygCX0riEDXl26xEjCcu9s6Nf4b+avcxVqRMhRhjt0zlFDP6VveohZTQni89qI8oQEBzELEoJCeT47Egxy4/RIfV6HX+uHXvpWSrRc+Tox7MRCRU9/F3MGfe3cxJAqAFmc6KWmulvjtRzzcXmLIH2/3W88BdIB514DbvVUWsmcIuVpDvGzesWrullqH+OvsvmbWajc 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:HK2PR06MB1010;BCL:0;PCL:0;RULEID:;SRVR:HK2PR06MB1010; X-Forefront-PRVS: 07915F544A X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10019020)(6009001)(189002)(199003)(54356999)(5001960100002)(46406003)(23726003)(1096002)(50466002)(87976001)(92566002)(586003)(86362001)(2950100001)(33646002)(101416001)(77096005)(5008740100001)(229853001)(50986999)(189998001)(106356001)(3846002)(36756003)(81156007)(40100003)(83506001)(42186005)(76176999)(19580405001)(6116002)(66066001)(69596002)(47776003)(105586002)(53416004)(97736004)(19580395003)(5001770100001)(5004730100002)(4001350100001)(122386002);DIR:OUT;SFP:1102;SCL:1;SRVR:HK2PR06MB1010;H:morimoto-PC.renesas.com;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;HK2PR06MB1010;23:cfMdnfVTfRBIynte3Sfn0YlXWzidGZ9vdCZMvk+5l?= =?us-ascii?Q?Qg4PUSRS2CR6KVLnWzudXMm7Rp4RWnUltg3NDGr2pErb0GCYp3l3tqkovGqa?= =?us-ascii?Q?51cLgXG8MznTl6pp8i9hgiBKBHEJpLldI1BS6smlQ7OQIRAEVl1SvyV1BEXe?= =?us-ascii?Q?54BaAXWXYu+6Y3TsdizmTCTEBDThab3BPt/fcmmJFH/VNu0OPminPmuj7hWY?= =?us-ascii?Q?7lqfm/PvVKjmReOnXTp0O2hHysgo89EkxlQzBdkva8E+6Ff9UIDknwav4kmi?= =?us-ascii?Q?kjC3Dvref3Dvx8uIaCds9n+s7M67zy0QTOB/pfcmozlfSTQzTgN1zMCwd3J1?= =?us-ascii?Q?YAxY0FTp9PDiRjGQdtxQXvfNgisO4Rh53ajj6frrai/Ah1UWFY0KU5jzOAko?= =?us-ascii?Q?zV0yEAyoATmbMWMaCv1Dtq9O+sQ4hjLTDuIujq2rHOo78YuRCddkayj5Yv/v?= =?us-ascii?Q?FXvYLPikVq40pJ2aTEr4qXm6PprYRZoFYs0emJCs1onhp5kXm0j1GztDV1Gh?= =?us-ascii?Q?6EeLiJynQh5ht8VdU31E+1uxDDfIOtOFPUelJ5Up0A3mK8IctahJxsIlA1DQ?= =?us-ascii?Q?2voQoF2/j6DjtNXaFyI0fDqXD1emDTYcMFAmKnGNqFwS4fVjidKPR5Evi3la?= =?us-ascii?Q?Cp0Z08HhQBqEAiHcoGZ2FxN5P1jiFTcPKaJxxsHbV/EotL+5CuSqIoJkIF3h?= =?us-ascii?Q?FD8HnO1jM2dPRioWaFcq6KphbtjYZTetOk7Q4xCImb8af9rUWvLF7hS1bS6v?= =?us-ascii?Q?91epPcVeWmWRHLgQBWAnSoHxD5H8HELerRWq0tjrv2z5MDV49JByUzqWSQRw?= =?us-ascii?Q?yDvl+rV2MMCOw8/K3ztiHhu7jLEnXmWYS1hpe3FH3TDLl0aHtn24+UBH2mE+?= =?us-ascii?Q?hzgQ1DnUY1NCVsh+I94ynawbzidGjAzQWuDNLaPPxQVZ+oWpxqZzaek0SgbQ?= =?us-ascii?Q?NynOrcLscAKiuMMR4vOhEA6exDZ5gYXDp+mttH7I908gi7DW4ciwSFOuGxJe?= =?us-ascii?Q?mAJ38zQS4OacclvYWIkrm+zcTfOcZdXJTHiYRdxgdXH+rHlAva3+GALIbgkg?= =?us-ascii?Q?UhfOfO7C3O0ie2yGboiJig6GsDylSXsfGdGKMLpWcx+G6GW0hjCEpclcFZF3?= =?us-ascii?Q?usq4ejqO7g=3D?= X-Microsoft-Exchange-Diagnostics: 1;HK2PR06MB1010;5:BHUBCDSVdseF2vpWCSoDZF9E4PBrYQf8Zdn8z3Q64uaYp9a5+C4EUwt7IkZB0Oec5tDkR1Bcmd3Z3dgufx6ghxez6+EX7AvUuhPxXgw3AWs/yjQlTchTTM/HznEdjbyCcMpWLtkyeuC36P1kdeIG0Q==;24:SZ11AN0KtU/YI6EqoFj+iYHt5D++SbHm7FsTespJ8F8ndlkkUjmFKQAG6f/yNSbBKtUGto5UNmGXr9GkYZwDYrIQp1KCJZ7qJB3ZvslCsQ4=;20:0hhq24zGyn7w6MRNGG/n6t03PLmejKuAPVHKoLhBN2nsmiQ7wHRNkCS+qQ2tC+XCig5boDSTtVa4I/VzN5vgqACVJQBp3sciB+cyCTzOeruyd1IZ/Dd+v/x0YjT3PyblwhUWOvbkNt2AO666sbQMoYdvp2l+u7InMt3gnzJo8Z8= SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: renesas.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Dec 2015 01:19:53.3783 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: HK2PR06MB1010 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Kuninori Morimoto trip_point_temp_store() updates trip temperature. It should call thermal_zone_device_update() immediately. Signed-off-by: Kuninori Morimoto --- v4 -> v5 - implemented on thermal_core.c drivers/thermal/thermal_core.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c index d9e525c..768fb10 100644 --- a/drivers/thermal/thermal_core.c +++ b/drivers/thermal/thermal_core.c @@ -676,8 +676,12 @@ trip_point_temp_store(struct device *dev, struct device_attribute *attr, return -EINVAL; ret = tz->ops->set_trip_temp(tz, trip, temperature); + if (ret) + return ret; - return ret ? ret : count; + thermal_zone_device_update(tz); + + return count; } static ssize_t -- 1.9.1