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: <f62874b9-4588-52d8-b321-49d2d57fb6c3@infradead.org>
Date:   Wed, 31 Aug 2022 17:00:53 -0700
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.2.0
Subject: Re: configuration x86/Kconfig bug? or AMD feature request
Content-Language: en-US
To:     Steven J Abner <pheonix.sja@att.net>, linux-kernel@vger.kernel.org,
        Hans de Goede <hdegoede@redhat.com>,
        "open list:X86 PLATFORM DRIVERS" 
        <platform-driver-x86@vger.kernel.org>
References: <VQ7IHR.9PMZBSNKURXU1.ref@att.net> <VQ7IHR.9PMZBSNKURXU1@att.net>
From:   Randy Dunlap <rdunlap@infradead.org>
In-Reply-To: <VQ7IHR.9PMZBSNKURXU1@att.net>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Precedence: bulk

Hi--

On 8/31/22 16:54, Steven J Abner wrote:
> Hi
> I am tailoring a kernel for AMD 2400g on ASRock B450 using 5.18.12 as base.
> I found a feature I don't believe I need, and the system seems to work just
> fine without it. A quick glance at the code seems like I'll be just fine, too.
> In /drivers/platform/x86/Kconfig:1159  I altered the text to:
> 
> config PMC_ATOM
>      bool "Intel Atom SOC Power Management Controller Driver"
>      default y
>      depends on PCI
>      select COMMON_CLK

I can't tell what you changed it to.

> Please have a look and decide if x86 must include this code to function properly.
> Nothing but being x86 selects this. 'default y' was chosen only because forced y before.
> Steve
> 
> I wish to be personally CC'ed the answers/comments posted to the list
> in response to my posting, please. I not part of the list.


That kconfig option has been deleted:
commit 3cd8cc98d634
Author: Hans de Goede <hdegoede@redhat.com>
Date:   Tue May 3 16:02:07 2022 +0200

    platform/x86: Drop the PMC_ATOM Kconfig option
    
    The def_bool y PMC_ATOM Kconfig option provides a couple of symbols used
    by the code enabled by the X86_INTEL_LPSS option and it registers some
    clocks. These clocks are only registered on Bay Trail, Cherry Trail and
    Brasswell Intel SoCs and kernels targeting these SoCs must always have
    the X86_INTEL_LPSS option enabled otherwise many things will not work.
    
    Building the PMC_ATOM code on kernels which are not targeting the
    mentioned SoCs and which do not have the X86_INTEL_LPSS enabled is
    not useful.
    
    This means that we can simplify things by replacing the PMC_ATOM Kconfig
    option in Makefiles with X86_INTEL_LPSS and then drop the option.


Maybe that commit should be backported?

-- 
~Randy