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 X-Spam-Level: X-Spam-Status: No, score=-7.3 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6EAE9C433E0 for ; Mon, 25 Jan 2021 03:15:43 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 4597021D79 for ; Mon, 25 Jan 2021 03:15:43 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726958AbhAYDPf (ORCPT ); Sun, 24 Jan 2021 22:15:35 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41058 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726861AbhAYDOq (ORCPT ); Sun, 24 Jan 2021 22:14:46 -0500 Received: from mail-pj1-x1032.google.com (mail-pj1-x1032.google.com [IPv6:2607:f8b0:4864:20::1032]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E098EC061574 for ; Sun, 24 Jan 2021 19:14:05 -0800 (PST) Received: by mail-pj1-x1032.google.com with SMTP id j12so7473587pjy.5 for ; Sun, 24 Jan 2021 19:14:05 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=fv/Vky7VFm1rltfdk/uW9VJjxFj6h7S+3peZRA6VmK4=; b=ONt2ctS2r6+gdTUGauF4KDHMMjNs4o1+EEOOKWV+b1aRsybyGD7LvWmgzB6RMbVZRU Liy7uBpfAvBSvQC4139zEHKth8hSk3A9duRcJedtQLzwB0kvwuprI5q3FCsWVwjqYbGH Sgudy/BHGhLiwAuK8CGrAUhJtJwUbcnoLY2TjDrBoSpZbqcDSTi6+Jw4XpqSj5ZNOrwX PQR/2kTs1O82swWCPcgVJSq3bzg1y2R3trldcKxpIkQ+hMkpdh6yhyh4l5EAqivwUVlB pIaV3A3caecW+YcKxf4HKxYUy6Qrqik0Ji3rximBJ5vKLG4ZQyTqZkyAWg+D1930Q8lg L7iA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=fv/Vky7VFm1rltfdk/uW9VJjxFj6h7S+3peZRA6VmK4=; b=MOvzDuia+FvciPm1WmE6fdP7td3svQRBdZaSxl0PbxPW9b03wryAyjRDmIQgzIF7vy mek5tIefC4LrXrzOR7nmJnPXp24/6Jud01dXUIgYyxKC7SQ69ZxUPAB84h7vrvK2qER0 F9lH/szqt4bwE1m2GKmwFH3+pKdTOEy9yiorh1xQa+yN38ZSbbmkjnU4F29kPxk5ET69 IvSskog4S/hAZFZcaaPqzgAxAWfoQHB9+jTRWy+UezmxIuAdb4C9lTsxFjLmvXh3iCt+ SQloQct8D0A2IPSVGCLe0TO/SH5dxAr/n+8d8NtS0RFyqThHtToXop2GO8ufTayazvGC RRVA== X-Gm-Message-State: AOAM531/XZEYJ7SZowTMEkAyZMlT3HfN9giCoutRfiPe5rvyup5wkeY2 ZHyULseW36HZM8F3eI5UsypZI6A9I7wRNw== X-Google-Smtp-Source: ABdhPJx4sy7iiTC8WR6f/YSHtq6fKmuH/YAA5pa1K04GTf1kXOR1QF1XbdaKvtqwkRuURwGS1g7aOQ== X-Received: by 2002:a17:90a:4088:: with SMTP id l8mr1063445pjg.106.1611544445506; Sun, 24 Jan 2021 19:14:05 -0800 (PST) Received: from localhost ([122.172.59.240]) by smtp.gmail.com with ESMTPSA id a21sm14746508pgd.57.2021.01.24.19.14.04 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sun, 24 Jan 2021 19:14:04 -0800 (PST) Date: Mon, 25 Jan 2021 08:44:02 +0530 From: Viresh Kumar To: Dmitry Osipenko Cc: MyungJoo Ham , Kyungmin Park , Chanwoo Choi , Thierry Reding , Jonathan Hunter , linux-pm@vger.kernel.org, Vincent Guittot , Rafael Wysocki , Stephen Boyd , Nishanth Menon , Sibi Sankar , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-tegra@vger.kernel.org Subject: Re: [PATCH 11/13] devfreq: tegra30: Migrate to dev_pm_opp_set_opp() Message-ID: <20210125031402.aqiwhlsozvakehxd@vireshk-i7> References: <3345fd49f7987d022f4f61edb6c44f230f7354c4.1611227342.git.viresh.kumar@linaro.org> <71451eb2-46b2-1ea0-efcc-0811568159a4@gmail.com> <20210122062659.qss3hef4kltfgciu@vireshk-i7> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20180716-391-311a52 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 22-01-21, 18:28, Dmitry Osipenko wrote: > Either way will work, but maybe keeping the dev_pm_opp_set_bw() is a bit > more straightforward variant for now since it will avoid the code's > changes and it's probably a bit more obvious variant for the OPP users. The problem is it creates unnecessary paths which we need to support. For example, in case of bandwidth itself we may want to update regulator/pm domain/required OPPs and this should all be done for everyone. I really do not want to keep separate paths for such stuff, it makes it hard to maintain.. -- viresh