Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <569eae9e-d14c-6736-3cee-9e7ba75da0b7@nvidia.com>
Date:   Fri, 24 Jun 2022 11:32:10 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.9.1
Subject: Re: [PATCH 21/31] soc/tegra: Remove the call to
 devm_pm_opp_set_clkname()
Content-Language: en-US
To:     Viresh Kumar <viresh.kumar@linaro.org>,
        Dmitry Osipenko <digetx@gmail.com>
Cc:     Thierry Reding <thierry.reding@gmail.com>,
        linux-pm@vger.kernel.org,
        Vincent Guittot <vincent.guittot@linaro.org>,
        Rafael Wysocki <rjw@rjwysocki.net>,
        Stephen Boyd <sboyd@kernel.org>, Nishanth Menon <nm@ti.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org
References: <cover.1653564321.git.viresh.kumar@linaro.org>
 <1e88b248352afe03cd3bf0e887b1f2be86b5afb5.1653564321.git.viresh.kumar@linaro.org>
 <12c085af-1202-95cf-e9ad-ddcfbdadf0d6@nvidia.com>
 <20220624002805.anv62ufihdrncwus@vireshk-i7>
 <20220624005913.ytsskavsaxt4rzzm@vireshk-i7>
From:   Jon Hunter <jonathanh@nvidia.com>
In-Reply-To: <20220624005913.ytsskavsaxt4rzzm@vireshk-i7>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?NVN0QjR6eVNhc2hVVHZvOS9XenRqZE16UGZyWm5OTzNGaHArWkZvMThSVEZx?=
 =?utf-8?B?U3NUalcwMldCYmUrSkF2a3ZzbDFJazJDV1huV293UHZDNGdSUFhYdGdUZUln?=
 =?utf-8?B?cElOSU5zd1BiWUVYMHJzbmhoRWRkcGtoWHI2MzFJNnE4TW95SEg4T3Q3eW5u?=
 =?utf-8?B?R3orQzlXZStDdjFiTi9hOWFncXZOOUlOdC9yYzNHQ3NVeGM2dTFvd1c0QWdx?=
 =?utf-8?B?TFNTMjVzZG14TkRNS05XRzA5Z29KWkFndFg0Ny9xMWtEeC9EWkI0SUtGekha?=
 =?utf-8?B?V2xXekFub2lGMXVESHRnUjBYWW54QzRnMGFsRmlCeVFoWGc2THpNdkJHOGZr?=
 =?utf-8?B?YVNkaEJGbGhCV2k0RVNkbitpUkQyL0lORGoxL25ON1UzT2JFR1NTUXJtUkxM?=
 =?utf-8?B?WXRHK1VyOGxrZ2pOUnZYRXozT3FBbHpsNUJVZEN3ZHVmZUowZFJyYWtmdXhR?=
 =?utf-8?B?TVpqU0xvdDRTaFB2bFN4ZFROMXJZSE9qRzdEUndrSEZ4TXhhQzM5VHlBbXRq?=
 =?utf-8?B?NHBQdXRFdktsZ1Axa3ZrR2hkZ20vR3hwcEpHUk9BWFF3VHkzZEhvTVptd09R?=
 =?utf-8?B?U0tiVTF4akNDSzAxNjhMMnhXQjRPejlZR3psNTNhdVhhejZrbEJxZ1RVQ0U2?=
 =?utf-8?B?eTg2WXlxK1lRZHM3SCtQcDVWcVRybVFBVUxaMkR6SGIzY082VHAwU2xYMVMv?=
 =?utf-8?B?TDBNeWJ0Wi90MURLaTVqZVpPVDNBejFJSEl6RUNaK0cySHdVVGdiRDR1VDVW?=
 =?utf-8?B?UVUxL0tnRC94Nm9ER3E1T1JkVWlnajZCamRKaXpxeUZqMmxvTlJLeFI5cDgz?=
 =?utf-8?B?ZG9jdmIrRUJwNzdBSWIyZXk0Vzd5eWVNaVN2K0J4YmdKdzZISkRHcFZJekUz?=
 =?utf-8?B?Q0RwMGVWVzV2S2NQVVkydmdPd1liakpIVUl4b3BsWXdFcUN1aW5SeTBnQ0I0?=
 =?utf-8?B?R0FMWmxkSG0xVGd5RnJiWVBqNzBCZ01LY3NDMGNQZ2lseXJWbG0zNE1rVzAy?=
 =?utf-8?B?TjFZRWhtSVdGaTZ0Q2E1L09Ja1ByTE1FUlpwSGRVVy9taW85QWNoSFdSTFNL?=
 =?utf-8?B?MHY0VDhMQy9sV2JFeFJzenpEdU5iZE5kZXlEa1kwa05hSzlQQ1pOVVVLdlBt?=
 =?utf-8?B?SGF6MUtncVIweXlTNWt4UDVHZ1IyRVFoNlZNOG1ab2ZRRkU4TTgxUjkrdSt0?=
 =?utf-8?B?UmxhQVBaMHE0NHN0M3JwYkRyQWxJc1BYcU1BYkFBS0FMSWVOTHQxd3gvUWVa?=
 =?utf-8?B?UEZJWDB3NmNJZ0ppdjY3bTVzREloMGZqTDdESjA0TmxzOVpueGdLanMwWmxY?=
 =?utf-8?B?UkxtVWd4QWE4K2hROXViajRuTlFiWVpqN2tMM2NvajRVV05qODdzNUQ2cnpV?=
 =?utf-8?B?NWZhQkVDaDVjM24wNjJqdmhOYVVJWVIxVStwRXlnd2ZVNHRUemM2OVNFMFZL?=
 =?utf-8?B?dHdYcDB5aVQ5aDNnS0tWUmFXTUFoR0tETi91M2JMem5GclpmU1RSUWJqTmFZ?=
 =?utf-8?B?R3ZpSTFROVVmTTk2VSt4ZDM5QTBDbzJhK21rdGpmVGlLNnpHQkNpejNVTkVx?=
 =?utf-8?B?ODZzdFQ4YTdHUkhzbkZ1Y1lpQklBcms4Vmg3NzFtenJ3d0JJUkFiQXpYVHB2?=
 =?utf-8?B?cE9ZdE5oeHQrMnFSazdDTTcvRThFbWVkV3pXcTZRc1g2WjZFNDdkUzBvazRk?=
 =?utf-8?B?UFdScFFRK3ZJMDFVcWY5ZmNVaHo0UjY2YnBQbm14RTgzWHpudmNWTVNtVnRG?=
 =?utf-8?B?dzB5bEs3bWtLbjBVL1FKb1U1a21EWTZSdFk2Y0RLTlpzWXpnNmpMdWpQYzBO?=
 =?utf-8?B?aXh4a1dyZUh6NnZPN1VwVEFjaVkvbVhSQlJ4aS9UUC9YY0Zzajh1dkdiTzQx?=
 =?utf-8?B?L2hpZ0dQakJKM3YvOXJLWW5sNWpLWVJSRG9mZ0dyeFFTUG9aUHlrM2xzeUp6?=
 =?utf-8?B?SUhub0U1NElCVE9ueHVHQUZpVzhGbDEyTlpIdElEMEp2QXlsaXhEMFdoN3ZX?=
 =?utf-8?B?eEJCZ0lJMS9JY1YyN2pHNDh6QkNpdHlrVzZ6dUkzYld2UGQ2VzBPNXlUSWRu?=
 =?utf-8?B?bDU5SEE0MHAwMEF1eitMek0ySnhhaW9DbWVIMnpTU1ZqRTRUajlnMGtBaHE4?=
 =?utf-8?B?MTRDSFUrL1Y2ODVvVEJod1ZZTEpEYmwzNE9SVjE4RFZOMENhbnNJczV6QmZT?=
 =?utf-8?B?SWc9PQ==?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b10cb127-71b8-4656-18a5-08da55ccd010
X-MS-Exchange-CrossTenant-AuthSource: CO6PR12MB5444.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Jun 2022 10:32:18.4375
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: LUMuauZzvwK3SSyQnAtRmApSJwGwGQ7IEUomeetlYFOikQQYrseU5uvNhEeKGvmmhA2nQwIaufRFb3aWBhnLvw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR1201MB2544
X-Spam-Status: No, score=-1.7 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        NICE_REPLY_A,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,
        SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 24/06/2022 01:59, Viresh Kumar wrote:
> On 24-06-22, 05:58, Viresh Kumar wrote:
>> Though there will still be a problem here with my changes, we don't
>> accept NULL clkname anymore for the set-clkname API. And tegra does
>> this to pick the first clock available in DT (at index 0) I think.
>> Other drivers (mostly qcom) who need such dummy OPP table, provide a
>> real clock name instead. Will it be possible to pass that here somehow
>> ?
> 
> Jon,
> 
> Okay, I was able to handle it without making any further updates to
> the OPP core. Nothing else required from your side on this.
> 
> I have pushed the updates (to this patch and 22/31) to opp/linux-next
> branch. You can try it now and it should just work. I have only build
> tested it though.


Thanks. I have reverted the previous versions of 21 and 22, and pulled 
in the latest and these are working.

Cheers
Jon

-- 
nvpublic