Received: by 2002:a05:6358:45e:b0:b5:b6eb:e1f9 with SMTP id 30csp836317rwe; Thu, 1 Sep 2022 08:18:42 -0700 (PDT) X-Google-Smtp-Source: AA6agR53IzT+38pELbEZ3mpnZJfkKVxq3gothiX4KENqrjnvAk9vSAC37rX/I6BOipNY+yO/bCfZ X-Received: by 2002:a05:6402:909:b0:435:a8b:5232 with SMTP id g9-20020a056402090900b004350a8b5232mr29164066edz.240.1662045522522; Thu, 01 Sep 2022 08:18:42 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1662045522; cv=none; d=google.com; s=arc-20160816; b=Jpd5iooYbl5JVClMiGqbnYrL9lHMSu5axmQleFa+7erakIKjYP+nxtlEbYdH7FyBuu hI6Rv3STYyRejPH+dMYHUiWErXlZY/e7aehppAfiPklw2euXKcQkdKQDxxHAOKCLfnEy KvLzM0T2IQU/18MrW9Io8B+kbzZcZrgIxDJ89lAkyLMZizOCE9p0PddlFOHsXLk1XA27 Y2aYLoGa4mx7qrXlBGAnzGyDZDaoTqEWC529/mNofOrI3aCCXMEdxtLNX9DrXTFvMoKG 5mpc17T3w0d97EdPaelTiQ5KyzbxCCDdrubYV0iTGMX5RQpqQgZ+2ek33Pic4crm2osb fvOA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:message-id:date:user-agent :references:in-reply-to:subject:cc:to:from:dkim-signature; bh=Qm7z+n9v24oP6zpPgqQul26KMHD50paO2Y9IVMFiRhQ=; b=zD3tiZtx6zIfq7RuHzdIH1HSZioT6PM9L4aKNi1VtYsHw2Ocj+hKND2KpuIouhSsHq 294OmPxdmdA4UiZ8kYzy5oAfscz8wLTnJGyHKHq8jEMn+8rwvVGUWOMKSL9uVvDACfNC xrJk7orLLXnQWQIRFVusd1VD0elwEtwuA2R1pEttU7BFHV9AC455422WXoux7c6Dy6iq bu0aB+2pcZ/r47mhMZM7ESgOecTlX/4My8bl4zpjy2LbcbPHPbx4xuuEGUVrTXPJ82GV Z0j1tK6kJ5KFCLW1O2Lbr1OMmlbg8G9bjKO02UciZWSDUX22avseIqWiSYy+CCkck3I3 tCbA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@bytedance-com.20210112.gappssmtp.com header.s=20210112 header.b=dd+Zsnug; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=bytedance.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id gb23-20020a170907961700b007418e87eea8si11474914ejc.770.2022.09.01.08.18.13; Thu, 01 Sep 2022 08:18:42 -0700 (PDT) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@bytedance-com.20210112.gappssmtp.com header.s=20210112 header.b=dd+Zsnug; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=bytedance.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233263AbiIAOzm (ORCPT + 99 others); Thu, 1 Sep 2022 10:55:42 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39034 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234477AbiIAOzf (ORCPT ); Thu, 1 Sep 2022 10:55:35 -0400 Received: from mail-wm1-x32c.google.com (mail-wm1-x32c.google.com [IPv6:2a00:1450:4864:20::32c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2409137F95 for ; Thu, 1 Sep 2022 07:55:31 -0700 (PDT) Received: by mail-wm1-x32c.google.com with SMTP id k17so9171810wmr.2 for ; Thu, 01 Sep 2022 07:55:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bytedance-com.20210112.gappssmtp.com; s=20210112; h=mime-version:message-id:date:user-agent:references:in-reply-to :subject:cc:to:from:from:to:cc; bh=Qm7z+n9v24oP6zpPgqQul26KMHD50paO2Y9IVMFiRhQ=; b=dd+Zsnugr8tFEBhl38/jbIsJ0BodE3L0d6on+MXP5Gp/QC92iSbt90u+oraLzgx0iP chX8O52Z11VUfusvEYE06r25mQwVDyBw/hwgmJ8scDQb6qlWx8H6L1n+h/IgpHkisROE TvBY9xGjVEDYq8aipQJ54wC0EyalZ3OWVNYKeqQepIPeX1FT+0zLOIAokKToNo2u5qcq PT6NS8ANAUJ1QpZ3tSoBW+gIIU2QUUIOWzG1G56x+KDOddNrzJiyiqtnRqOVnfROLXph YkbuF6dazdnIRYZe6gjQhJX5hi3YiTYVDswYAlYr1KZsuR0QKzH505dJHQvZJPyTXjng fvOg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=mime-version:message-id:date:user-agent:references:in-reply-to :subject:cc:to:from:x-gm-message-state:from:to:cc; bh=Qm7z+n9v24oP6zpPgqQul26KMHD50paO2Y9IVMFiRhQ=; b=yAqMEyTIyllRqA7HEgqFELRLbt+oScjmItEMLFq1DilH4rF5SM6dLgJTFpEgFglCDm 8aPYVE3v4dsQ7+vqFwXyghQ0EAWGPztcPKFQIG6ShBruHjXZY7LaL5XT76o/NJHQzFzr gwd7r86Mc3D8ebRBd7Gwe+h09HgaQEqPrJxvb2SZaPdtOFq+/1IE3RnWob3BnP58kWOl cWyFt+wC/CPGwBISdVS+z7VPkowOkYCnIrL2SNzCRWMVxPuxAyoGrw9bAO6QfHNX39DW N6luEpkbEmwv4nrRLeybinWRdQNHwBRY3sMVxgEvm2s2jN9Y+EiRc6a/RVuhIqSqAtln Uf/A== X-Gm-Message-State: ACgBeo1Gw63WwFw4AR6Dqv50pkuaEM2P/ZrQhj2d0KZmCg8l3+zJGhLg tdoDm04kNhNVTi6o8UoD3bF7fg== X-Received: by 2002:a05:600c:35d0:b0:3a8:3f16:2c38 with SMTP id r16-20020a05600c35d000b003a83f162c38mr5695050wmq.162.1662044129389; Thu, 01 Sep 2022 07:55:29 -0700 (PDT) Received: from localhost ([95.148.15.66]) by smtp.gmail.com with ESMTPSA id e20-20020a05600c4e5400b003a5bd5ea215sm5559533wmq.37.2022.09.01.07.55.28 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 01 Sep 2022 07:55:28 -0700 (PDT) From: Punit Agrawal To: Perry Yuan Cc: , , , , , , , , , , , , Subject: Re: [PATCH v6] cpufreq: amd-pstate: cleanup the unused and duplicated headers declaration In-Reply-To: <20220901062339.2357552-1-Perry.Yuan@amd.com> (Perry Yuan's message of "Thu, 1 Sep 2022 14:23:39 +0800") References: <20220901062339.2357552-1-Perry.Yuan@amd.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) Date: Thu, 01 Sep 2022 15:55:27 +0100 Message-ID: <87y1v3cffk.fsf@stealth> MIME-Version: 1.0 Content-Type: text/plain X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS, T_SCC_BODY_TEXT_LINE autolearn=ham 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: X-Mailing-List: linux-kernel@vger.kernel.org Hi Perry, Perry Yuan writes: > Cleanup the headers declaration which are not used > actually and some duplicated declaration which is declarated in some > other headers already, it will help to simplify the header part. > > Reviewed-by: Huang Rui > Acked-by: Viresh Kumar > Signed-off-by: Perry Yuan > --- > drivers/cpufreq/amd-pstate.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/drivers/cpufreq/amd-pstate.c b/drivers/cpufreq/amd-pstate.c > index 9ac75c1cde9c..9cf7bc162e7c 100644 > --- a/drivers/cpufreq/amd-pstate.c > +++ b/drivers/cpufreq/amd-pstate.c > @@ -31,19 +31,15 @@ > #include > #include > #include > -#include > #include > #include > #include > #include > > -#include > #include > > #include > #include > -#include > -#include > #include "amd-pstate-trace.h" > > #define AMD_PSTATE_TRANSITION_LATENCY 0x20000 The patch has the same issue as mentioned in the previous version. Please don't remove any header includes whose definitions are directly used in amd-pstate.c. Look at example from my previous comment.