Re: [PATCH v2 0/4] Add SWIG Bindings to libcpupower

From: Shuah Khan
Date: Wed Sep 04 2024 - 11:56:57 EST


On 9/4/24 08:13, John B. Wyatt IV wrote:
On Wed, Sep 04, 2024 at 06:41:14AM -0600, Shuah Khan wrote:
On 8/27/24 00:24, John B. Wyatt IV wrote:
[snipped]

Couple of things to address:

1. I noticed none of the patches have the subsystem prefix:
pm:cpupower is the right prefix for patch subject for all
the patches except the MAINTAINERS file

I will pull the fix "Implement dummy function for SWIG to accept
the full library" Patch 2 in your series.

I want this subject changed to just fix as it is a problem irrespective
of SWIG - we have a missing function. Subject would be as follows:

""pm:cpupower: Add missing powercap_set_enabled() stub function"

Make this the first patch in the series. I will send this up for
Linux 6.11-rc7 or Linux 6.12-rc1

Understood.

Depending on how the timelines for merge window work, expect the
series to land in Linux 6.13-rc1. I would prefer to delay it anyway
so we can get some soaking in next.

Thank you, I appreciate it.

What kind of soaking are you looking for? Is there anything I can do to
help?


It is more the timing than anything you can do. We are at rc6. Please do
send the series with the changes I requested.

thanks,
-- Shuah