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: <13fbb134dda72f979b9a02e6abec12ebac18aef8.camel@linux.intel.com>
Subject: Re: [PATCH 0/7] thermal: processor_thermal: Suport workload hint
From:   srinivas pandruvada <srinivas.pandruvada@linux.intel.com>
To:     "Rafael J. Wysocki" <rafael@kernel.org>
Cc:     rui.zhang@intel.com, daniel.lezcano@linaro.org,
        linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org
Date:   Wed, 21 Jun 2023 08:45:10 -0700
In-Reply-To: <CAJZ5v0jGp_Rsu6S+znmrKhQ+y88Mqf9PLf66Ec-SffdFdyH_4g@mail.gmail.com>
References: <20230620230150.3068704-1-srinivas.pandruvada@linux.intel.com>
         <CAJZ5v0jGp_Rsu6S+znmrKhQ+y88Mqf9PLf66Ec-SffdFdyH_4g@mail.gmail.com>
Content-Type: text/plain; charset="UTF-8"
User-Agent: Evolution 3.42.4 (3.42.4-2.fc35) 
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Precedence: bulk

On Wed, 2023-06-21 at 16:58 +0200, Rafael J. Wysocki wrote:
> On Wed, Jun 21, 2023 at 1:01 AM Srinivas Pandruvada
> <srinivas.pandruvada@linux.intel.com> wrote:
> > 
> > Add support for Meteor Lake workload hints. Before adding this
> > support,
> > some reorganization and clean up is required.
> > First four changes are for clean up and to reorganize code to add
> > support for workload hint. The last patch adds a test program as
> > part
> > of self tests.
> > 
> > Srinivas Pandruvada (7):
> >   thermal: int340x: processor_thermal: Move mailbox code to common
> >     module
> >   thermal: int340x: processor_thermal: Add interrupt configuration
> >   thermal: int340x: processor_thermal: Use non MSI interrupts
> >   thermal/drivers/int340x: Remove PROC_THERMAL_FEATURE_WLT_REQ for
> >     Meteor Lake
> >   thermal: int340x: processor_thermal: Add workload type hint
> >   thermal/drivers/int340x: Support workload hint interrupts
> >   selftests/thermel/intel: Add test to read workload hint
> > 
> >  .../driver-api/thermal/intel_dptf.rst         |  38 +++
> >  .../thermal/intel/int340x_thermal/Makefile    |   2 +
> >  .../processor_thermal_device.c                |  17 +-
> >  .../processor_thermal_device.h                |  21 +-
> >  .../processor_thermal_device_pci.c            |  76 ++++--
> >  .../processor_thermal_device_pci_legacy.c     |   3 +-
> >  .../int340x_thermal/processor_thermal_mbox.c  | 179 ++++---------
> >  .../processor_thermal_wlt_hint.c              | 239
> > ++++++++++++++++++
> >  .../processor_thermal_wlt_req.c               | 137 ++++++++++
> >  .../testing/selftests/thermal/intel/Makefile  |  16 ++
> >  .../thermal/intel/workload_hint_test.c        | 114 +++++++++
> >  11 files changed, 680 insertions(+), 162 deletions(-)
> >  create mode 100644
> > drivers/thermal/intel/int340x_thermal/processor_thermal_wlt_hint.c
> >  create mode 100644
> > drivers/thermal/intel/int340x_thermal/processor_thermal_wlt_req.c
> >  create mode 100644 tools/testing/selftests/thermal/intel/Makefile
> >  create mode 100644
> > tools/testing/selftests/thermal/intel/workload_hint_test.c
> > 
> > --
> 
> Because of the timing of the first posting, I'm going to treat this
> series as 6.6 material.
That is fine. Just review is important, so that it can be back ported
to Chrome kernel.

Thanks,
Srinivas