Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp5012266pxu; Thu, 10 Dec 2020 10:40:32 -0800 (PST) X-Google-Smtp-Source: ABdhPJyYONyNCLybwmipYyEoMTlN819Fnw/F1m/YtaQluICIJ1eYk8PMBPwBESAbyDMuoq1Fy4sZ X-Received: by 2002:a17:906:7243:: with SMTP id n3mr7479956ejk.246.1607625632450; Thu, 10 Dec 2020 10:40:32 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1607625632; cv=none; d=google.com; s=arc-20160816; b=GEzfAEPO2MVDl7fIWSSZJZs4YYOQdHsP1EwCmtNWEgReEC6shBkltN8fZR5UXAK8Ec USZfiWBf1E5kC4Rphz/vdY3qqkrJD3MauAua79lwhN+3P31JFvEqTSNAlHIBJOoayXir 18zPa7MAtlppGp9D2CN9Sjc2DIXlSU6P+c8KKzhh6/FXF2EIim7Fdu1rXVWiESP7ZwRt qhBoRTlOcY06xQFNk4fKRgDm8GmIUVNpnjhsNkmEVWrlhXdS1akdFOrsZwxFuIvOw5nz X4J7FeKjqx7e1Qae1zxn+Hi9DpVCv3u4GcgYd0WtyHkqPqbDmYgmzqi3Kp4uKPGCuEub /5Og== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version; bh=k62Ve7mlBGGKYnLuZVHB8mMB87DGHB1By9AKQqAliVw=; b=QW3szvYt303fhV0fs9S/3hCGzvnp0HH26WqhEf23UsmQV15KSqs1L+OOVCFJ/FqTnG ojtuU+xkkjM1izViUqfCRGarcWERFDi0BTg82GhG8g5WydPX62zOUWDfKVGJnf2uqO7j Czn2XBOCNQ3DgR3Q5uIOMZptCpxGPBJUEs0a7nNDCBMc91lF5DPzFB1Vn4hrVqdG8c+i HP27yLY46j07vtOvkCOmwcydxDjJ2FDpYgAMECjiFLaKgBNn9GcSDYTaZA3Lqkj/0cpy Nke31a8ZRssbfvTYw0cSSBg6KEltIy8NQFAjCdBrBd/aB1s5N3/61VO7WYNjQUcsNcWB 6brg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id v16si2943852eja.188.2020.12.10.10.40.07; Thu, 10 Dec 2020 10:40:32 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2392565AbgLJSgf (ORCPT + 99 others); Thu, 10 Dec 2020 13:36:35 -0500 Received: from mail-oo1-f65.google.com ([209.85.161.65]:42039 "EHLO mail-oo1-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1733295AbgLJOdB (ORCPT ); Thu, 10 Dec 2020 09:33:01 -0500 Received: by mail-oo1-f65.google.com with SMTP id x203so1315217ooa.9; Thu, 10 Dec 2020 06:32:45 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=k62Ve7mlBGGKYnLuZVHB8mMB87DGHB1By9AKQqAliVw=; b=TS2XoujjUkgV1LYJ4nbdQV0XfPv+mJcZ/uGt7DU4uM4ozF1FnhyKGsQvN0nCG15JgF lCoNNHzXKzZ/ia6ZBk1z+P2PziGAANbx5RBw5F+FZsazJuuSIsyDem7ps4I5Z3dTJW4l 02PMjKj6903sUetAlQfDReaAHU50+jcYNNg2sYXW8xALP9kWghymnkiEFL3+lOLGdcb4 50DXf2W4jMf+qvHlBJdalOGnCjezPzRkUS8bQO28xWIWjmW2oc2lCV8dnxfH6G349UWO JJhso9kEZoV0raQPLZFOIRITbSS8/yRK+G2+gsoJApERC8VzDv5ZI5VA3fgC/hEKLDNW eu/A== X-Gm-Message-State: AOAM5310CcSl5riaffpNtK0yw5WBIUOalwllsrLf+HJfyZ4Vx722CDzC 7yEslr8pPfNBUqLV1x3m7A67BGAXjokGi/q/RIY= X-Received: by 2002:a4a:aac4:: with SMTP id e4mr6089606oon.2.1607610740318; Thu, 10 Dec 2020 06:32:20 -0800 (PST) MIME-Version: 1.0 References: <20201210142139.20490-1-yousaf.kaukab@suse.com> In-Reply-To: <20201210142139.20490-1-yousaf.kaukab@suse.com> From: "Rafael J. Wysocki" Date: Thu, 10 Dec 2020 15:32:09 +0100 Message-ID: Subject: Re: [PATCH 1/2] acpi: cppc: add cpufreq device To: Mian Yousaf Kaukab Cc: "Rafael J. Wysocki" , Len Brown , Viresh Kumar , Ionela Voinescu , ACPI Devel Maling List , Linux Kernel Mailing List , Linux PM , Mian Yousaf Kaukab , Petr Cervinka Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Dec 10, 2020 at 3:23 PM Mian Yousaf Kaukab wrote: > > From: Mian Yousaf Kaukab > > Since commit 28f06f770454 ("cppc_cpufreq: replace per-cpu structures with > lists"), cppc-cpufreq driver doesn't check availability of PSD data before > registering with cpufreq core. As a result on a ThunderX2 platform when > CPPC is disabled from BIOS, kernel log is spammed with following messages: > > [ 180.974166] CPPC Cpufreq: Error in acquiring _CPC/_PSD data for CPUxx > > acpi_cppc_processor_probe() never succeed in this case because > acpi_evaluate_object_typed("_CPC") always returns AE_NOT_FOUND. When > cpufreq core calls cppc_cpufreq_cpu_init(), driver fails to obtain PSD data > and print error messages. > > Convert cppc-cpufreq driver to a platform driver (done in a separate patch) > and add cppc-cpufreq device when acpi_cppc_processor_probe() succeeds. Honestly, I prefer to drop 28f06f770454 (along with its follower) instead of making this change. > Fixes: 28f06f770454 ("cppc_cpufreq: replace per-cpu structures with lists") Thanks!