From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0A6A5C00140 for ; Fri, 5 Aug 2022 16:37:29 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240873AbiHEQh0 (ORCPT ); Fri, 5 Aug 2022 12:37:26 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38450 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S240504AbiHEQh0 (ORCPT ); Fri, 5 Aug 2022 12:37:26 -0400 Received: from mail-wr1-x42f.google.com (mail-wr1-x42f.google.com [IPv6:2a00:1450:4864:20::42f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DACD36D9F5 for ; Fri, 5 Aug 2022 09:37:23 -0700 (PDT) Received: by mail-wr1-x42f.google.com with SMTP id bv3so3832798wrb.5 for ; Fri, 05 Aug 2022 09:37:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc; bh=Jfp/EtpEG6qo6OMuAe522j8e+zV5R8oPt8iblzqNd7c=; b=y2imUW//ZeRpPSmlTXLHDXgNAORsIqksqVSBuH+RhwuIlSjr1Kla2tBW4afOpEEoZk PIOA5q/PfeQWfgZXvwEh27bVxYwZTFghX9DLkBUe+IL/ipypUrAAzfjhIodqlyKTrNQz BKDOFUxEHPDYtdNYcVBXWDSc4A88Mutrb79Kmo6GkE6UzQEjctQMcogwUeyNYEnbro7c cyfSlj+G4z9+nMcVjenpoIjqFMIJyxa0jbaPrjoyujN7fewi7pFnwG+neoMDppor3TTw JebM1SKhf9kcWZRI9WfrVG5UWyUejEqwN26wDBBghgRuBwi40R8U8YVqSyvWj7FTVfBL bR0w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc; bh=Jfp/EtpEG6qo6OMuAe522j8e+zV5R8oPt8iblzqNd7c=; b=kLJwpKTANtMXIYXdLKm63wO09cDBnb4YQuGlO7CHN/F77HSRd8Lz+c7mXIf90OI7rh pYeqjWdwlzLVp1bcdJ37VZ+mV9HG2qsqbMuSYa3oDBdOVv0/HONeyMMbmO/fL1olfOZN +EOxpcya5xodcAfO9VuEW1N+3BCQORcDKDJZ3YxwMVSAtlX8pC5J/e3EaL+iO/d70Ofw 4qPSWD55pXf39edCWruoUTrpFgoZyfQhmQGZ548lMX+kIhdcqXwrpcWzp4R91NwXXOLD 3G/FyfEs+n16X8jlUZOlaFgwaUH4GvPKBMv+BdfUgGDWF3TR11EFDRTKtcZAMpkgKixz 6/5A== X-Gm-Message-State: ACgBeo35UhE9JZIwxlQIIPGzXpw6APuMBFgyTTMXrvQRnOa9zSl/Af5F +r84EtKfXcAP4D2AJslVCTS+/4yxZFatgA== X-Google-Smtp-Source: AA6agR5L3BarDWIfJaXyZ+9r08uJ94DcLrt53xqiX23qwNDpiHhqcfaOj2HIAkloYYbR2z0UMOoXMw== X-Received: by 2002:a5d:5848:0:b0:221:6d80:ced8 with SMTP id i8-20020a5d5848000000b002216d80ced8mr2405988wrf.705.1659717442338; Fri, 05 Aug 2022 09:37:22 -0700 (PDT) Received: from ?IPV6:2a05:6e02:1041:c10:aef0:8606:da6b:79ef? ([2a05:6e02:1041:c10:aef0:8606:da6b:79ef]) by smtp.googlemail.com with ESMTPSA id j42-20020a05600c1c2a00b003a30c3d0c9csm10528223wms.8.2022.08.05.09.37.21 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 05 Aug 2022 09:37:22 -0700 (PDT) Message-ID: Date: Fri, 5 Aug 2022 18:37:21 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 Subject: Re: [PATCH 1/5] thermal/core: Rearm the monitoring only one time Content-Language: en-US To: "Rafael J. Wysocki" Cc: "Zhang, Rui" , Linux PM , Linux Kernel Mailing List , Amit Kucheria References: <20220805153834.2510142-1-daniel.lezcano@linaro.org> From: Daniel Lezcano In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org Hi Rafael, On 05/08/2022 18:29, Rafael J. Wysocki wrote: > On Fri, Aug 5, 2022 at 5:38 PM Daniel Lezcano wrote: >> >> The current code calls monitor_thermal_zone() inside the >> handle_thermal_trip() function. But this one is called in a loop for >> each trip point which means the monitoring is rearmed several times >> for nothing (assuming there could be several passive and active trip >> points). >> >> Move the monitor_thermal_zone() function out of the >> handle_thermal_trip() function and after the thermal trip loop, so the >> timer will be disabled or rearmed one time. >> >> Signed-off-by: Daniel Lezcano > > Does this series depend on any other? > > You've been sending quite a lot of material lately and it is not > always easy to tell what the dependencies between the different patch > series are. Yes I understand. This series does not depend on any other. It is not related to any pending changes. It may have trivial conflicts with the other 26 patches series but it would be the case for any submissions posted by someone else anyway. -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog