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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id E3600C43219 for ; Wed, 27 Oct 2021 19:42:44 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id C75B360FC0 for ; Wed, 27 Oct 2021 19:42:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S243734AbhJ0TpJ (ORCPT ); Wed, 27 Oct 2021 15:45:09 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59994 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231656AbhJ0TpI (ORCPT ); Wed, 27 Oct 2021 15:45:08 -0400 Received: from mail-lj1-x236.google.com (mail-lj1-x236.google.com [IPv6:2a00:1450:4864:20::236]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2A307C061570; Wed, 27 Oct 2021 12:42:42 -0700 (PDT) Received: by mail-lj1-x236.google.com with SMTP id 17so3423091ljq.0; Wed, 27 Oct 2021 12:42:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=lMrIGDyxX8Zy9n5Qfaorgc1V0f7+eeLTleV6gBWprQs=; b=EpcfosP+IsCdHIOo4g9lQWA3Dt1mvkaa2S1lXNcMJ+wAv70XvIlAza40HBE97tgQSR IkfIOp4xJgtTffYGkAODDYFJUZj25zWXktUjS8WptldLXdmHDvPyX9mlbT0mLS1M3dXX vUj4Del4y/UjFwXSQBiS0m51FgS+KELr7DJbgRJsAD/6braCFNhttH/KC80FJLOUhVUT XfbMBVd+oQ2hJle6n3br/BePBAtqcauPVqovMypm7aya3CAfvvL8qtYTm6PKzR8w5vVX 22lLwx9nfxESrOYG2VjBRQJktOLAAIQCePAjr6ylcuwqDGPz2+ysUeVsBLBoQzGs0iWt nJuQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=lMrIGDyxX8Zy9n5Qfaorgc1V0f7+eeLTleV6gBWprQs=; b=u8oweZ26UHmgHHGh/v81E2HTEa0EJGrShkUbfkEDLE/H8x/L9VCiPjcbj5C+KeCnmN mbIyaSHSr+PaNMftdbtexOz0R2U9Yw67yjDERe1ajmtzXmwsijd0KTfHdXTel3EvHByz 0DXbkg2ysx/mUNWwUAsOiwGtA/7zAqRIrvNmlx2NZRpvZZ2hC/tQc8ZgO1WxowU4he6/ um56cwS5s+JhkKugQ2muHjVnuq506dqlrN38W+Gqd8ncWdrvcvs9X6mO1slURrEQ8Q+G PhbPRPGVJTiN4ti+2jqBQs9hgOsex1b5q3Moh4YKVE5CwOFgUiWk2bMvX6VGWApOsn5D RDcA== X-Gm-Message-State: AOAM532Xx3TFi7QdHOMnFEe4BmN/dfQaqsW151rv/1ztjnd988eIvr0o YjRKZnXeSykQA94b34dUvds= X-Google-Smtp-Source: ABdhPJxS+SO4cZo9cpkl8K0LznK+P9fYpGRd1oswPP0T1GAwpmwlx8un9r9DrfnHA/QNv1WaeAvq9A== X-Received: by 2002:a05:651c:4cf:: with SMTP id e15mr34624456lji.139.1635363760555; Wed, 27 Oct 2021 12:42:40 -0700 (PDT) Received: from [192.168.2.145] (46-138-41-28.dynamic.spd-mgts.ru. [46.138.41.28]) by smtp.googlemail.com with ESMTPSA id o14sm71771ljh.62.2021.10.27.12.42.39 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 27 Oct 2021 12:42:40 -0700 (PDT) Subject: Re: [PATCH v14 00/39] NVIDIA Tegra power management patches for 5.17 To: Ulf Hansson Cc: Thierry Reding , Jonathan Hunter , Viresh Kumar , Stephen Boyd , Peter De Schrijver , Mikko Perttunen , Lee Jones , =?UTF-8?Q?Uwe_Kleine-K=c3=b6nig?= , Nishanth Menon , Adrian Hunter , Michael Turquette , linux-kernel@vger.kernel.org, linux-tegra@vger.kernel.org, linux-pm@vger.kernel.org, linux-pwm@vger.kernel.org, linux-mmc@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-clk@vger.kernel.org, David Heidelberg References: <20211025224032.21012-1-digetx@gmail.com> From: Dmitry Osipenko Message-ID: <98072124-c196-5a5a-7fa6-9c8e465f9878@gmail.com> Date: Wed, 27 Oct 2021 22:42:39 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.14.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-tegra@vger.kernel.org 27.10.2021 19:01, Ulf Hansson пишет: > On Tue, 26 Oct 2021 at 00:45, Dmitry Osipenko wrote: >> >> This series adds runtime PM support to Tegra drivers and enables core >> voltage scaling for Tegra20/30 SoCs, resolving overheating troubles. >> >> All patches in this series are interdependent and should go via Tegra tree >> for simplicity. >> >> Changelog: >> >> v14: - Fixed missing runtime PM syncing on removal of drivers, which was >> spotted by Ulf Hansson in v13. >> >> - clk-device driver now resumes RPM on system suspend instead of >> preparing clock which it backs. This was suggested by Ulf Hansson. >> >> - clk-device driver now syncs power domain performance unconditionally >> during driver's probe time since GENPD API allows to do this now. >> It was spotted by Ulf Hansson. >> >> - Added new "Enable runtime PM during OPP state-syncing" patch, which >> allows drivers to sync state at any time. Previously drivers were >> obligated to take care of enabling RPM at the "right" time. >> >> - Moved runtime PM initialization/uninitialization of DRM drivers that >> use host1x channel to host1x client init/deinit phase. I noticed that >> there is UAF problem because RPM-suspend callback waits until channel >> is idling and channel is already released/freed during driver's removal >> phase. >> >> - Added system suspend support to the new NVDEC DRM driver. >> >> - Added missing pm_runtime_mark_last_busy() to DRM driver. >> >> - Corrected VDE GENPD patch which previously made video decoder clock >> always-enabled by mistake if legacy PD code path was used. It was >> spotted while we were testing VDE on Tegra114 that doesn't support >> GENPD yet. >> >> - Added ack from Peter Chen to the USB patch that he gave to v13. >> >> - Changed OPP table names in accordance to the new naming scheme >> required by the recent core OPP binding. >> >> - Added 500MHz memory OPP entry used by ASUS Transformer tablets. > > Besides those minor nitpicks/questions that I have sent for patch1 and > patch29, the series looks good to me! > > Feel free to add, for the whole series: > > Reviewed-by: Ulf Hansson Thank you, Ulf! I'll wait for more comments and re-spin v15 with minor improvements around the end of the upcoming merge window.