[GIT PULL]: tools/power/x86/intel-speed-select pull request for 5.20-rc1

[GIT PULL]: tools/power/x86/intel-speed-select pull request for 5.20-rc1

[Pandruvada, Srinivas]

Hi Hans,

There are two minor fixes:
- One for just removing unneeded semicolon
- Another for array range check

These changes are on top of
https://kernel.googlesource.com/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86/+/refs/heads/for-next

The following changes since commit
7a4a04f4e90d746928501a6d551de87d02c28323:

  platform/surface: tabletsw: Fix __le32 integer access (2022-07-17
16:20:35 +0200)

are available in the Git repository at:

  https://github.com/spandruvada/linux-kernel.git intel-sst

for you to fetch changes up to
7d565542f8bdc097d611030572ecf8d9706f14dc:

  tools/power/x86/intel-speed-select:Unneeded semicolon (2022-07-18
09:51:09 -0700)

----------------------------------------------------------------
Dan Carpenter (1):
      tools/power/x86/intel-speed-select: Fix off by one check

Xin Gao (1):
      tools/power/x86/intel-speed-select:Unneeded semicolon

 tools/power/x86/intel-speed-select/hfi-events.c  | 2 +-
 tools/power/x86/intel-speed-select/isst-daemon.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Thanks,
Srinivas

Re: [GIT PULL]: tools/power/x86/intel-speed-select pull request for 5.20-rc1

[Hans de Goede]

Hi,

On 7/18/22 19:05, Pandruvada, Srinivas wrote:
> Hi Hans,
> 
> There are two minor fixes:
> - One for just removing unneeded semicolon
> - Another for array range check
> 
> These changes are on top of
> https://kernel.googlesource.com/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86/+/refs/heads/for-next
> 
> The following changes since commit
> 7a4a04f4e90d746928501a6d551de87d02c28323:
> 
>   platform/surface: tabletsw: Fix __le32 integer access (2022-07-17
> 16:20:35 +0200)
> 
> are available in the Git repository at:
> 
>   https://github.com/spandruvada/linux-kernel.git intel-sst
> 
> for you to fetch changes up to
> 7d565542f8bdc097d611030572ecf8d9706f14dc:
> 
>   tools/power/x86/intel-speed-select:Unneeded semicolon (2022-07-18
> 09:51:09 -0700)

Thanks. Both commits did not include your Signed-off-by though,
if you apply patches of others you must at your S-o-b since you
are part of the chain how they got upstream.

After pulling I ended up with 2 commits where the S-o-b of
the committer of the patch was not present.

Instead I've picked both patches directly from the list,
now setting me as the committer (and added my own S-o-b).

So this is resolved now and both patches are in my review-hans
branch now. But the next time when applying patches from others
for a pull-req please don't forget to add your own S-o-b.

Regards,

Hans




> 
> ----------------------------------------------------------------
> Dan Carpenter (1):
>       tools/power/x86/intel-speed-select: Fix off by one check
> 
> Xin Gao (1):
>       tools/power/x86/intel-speed-select:Unneeded semicolon
> 
>  tools/power/x86/intel-speed-select/hfi-events.c  | 2 +-
>  tools/power/x86/intel-speed-select/isst-daemon.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> Thanks,
> Srinivas

Re: [GIT PULL]: tools/power/x86/intel-speed-select pull request for 5.20-rc1

[Pandruvada, Srinivas]

On Thu, 2022-07-28 at 16:20 +0200, Hans de Goede wrote:
> Hi,
> 
> On 7/18/22 19:05, Pandruvada, Srinivas wrote:
> > Hi Hans,
> > 
> > There are two minor fixes:
> > - One for just removing unneeded semicolon
> > - Another for array range check
> > 
> > These changes are on top of
> > https://kernel.googlesource.com/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86/+/refs/heads/for-next
> > 
> > The following changes since commit
> > 7a4a04f4e90d746928501a6d551de87d02c28323:
> > 
> >   platform/surface: tabletsw: Fix __le32 integer access (2022-07-17
> > 16:20:35 +0200)
> > 
> > are available in the Git repository at:
> > 
> >   https://github.com/spandruvada/linux-kernel.git intel-sst
> > 
> > for you to fetch changes up to
> > 7d565542f8bdc097d611030572ecf8d9706f14dc:
> > 
> >   tools/power/x86/intel-speed-select:Unneeded semicolon (2022-07-18
> > 09:51:09 -0700)
> 
> Thanks. Both commits did not include your Signed-off-by though,
> if you apply patches of others you must at your S-o-b since you
> are part of the chain how they got upstream.
> 
> After pulling I ended up with 2 commits where the S-o-b of
> the committer of the patch was not present.
> 
> Instead I've picked both patches directly from the list,
> now setting me as the committer (and added my own S-o-b).
> 
> So this is resolved now and both patches are in my review-hans
> branch now. But the next time when applying patches from others
> for a pull-req please don't forget to add your own S-o-b.
Sorry about that. 

Thanks,
Srinivas

> 
> Regards,
> 
> Hans
> 
> 
> 
> 
> > 
> > ----------------------------------------------------------------
> > Dan Carpenter (1):
> >       tools/power/x86/intel-speed-select: Fix off by one check
> > 
> > Xin Gao (1):
> >       tools/power/x86/intel-speed-select:Unneeded semicolon
> > 
> >  tools/power/x86/intel-speed-select/hfi-events.c  | 2 +-
> >  tools/power/x86/intel-speed-select/isst-daemon.c | 2 +-
> >  2 files changed, 2 insertions(+), 2 deletions(-)
> > 
> > Thanks,
> > Srinivas
> 

[GIT PULL]: tools/power/x86/intel-speed-select pull request for 6.1-rc1

[Pandruvada, Srinivas]

Hi Hans,

This pull request contains:
- One fix for invalid display of cpu count for TDP level
- Removal of unused code and data
- consolidate package/die/cpu in a struct and use this
instead of passing all and recompute package and die ids

These changes are on top of
https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git
for-next


The following changes since commit
76fba1221e3045570478381c3436c8c2a2656376:

  Merge tag 'backlight-detect-refactor-1' into review-hans (2022-09-05
11:01:56 +0200)

are available in the Git repository at:

  https://github.com/spandruvada/linux-kernel.git intel-sst

for you to fetch changes up to
97ec890d070158f15fe8ef2419a5ff677bdbbc20:

  tools/power/x86/intel-speed-select: Release v1.13 (2022-09-15
11:16:06 -0700)

----------------------------------------------------------------
Srinivas Pandruvada (1):
      tools/power/x86/intel-speed-select: Release v1.13

Zhang Rui (13):
      tools/power/x86/intel-speed-select: Fix cpu count for TDP level
display
      tools/power/x86/intel-speed-select: Remove dead code
      tools/power/x86/intel-speed-select: Remove unused core_mask array
      tools/power/x86/intel-speed-select: Introduce struct isst_id
      tools/power/x86/intel-speed-select: Add pkg and die in isst_id
      tools/power/x86/intel-speed-select: Convert more function to use
isst_id
      tools/power/x86/intel-speed-select: Cleanup get_physical_id usage
      tools/power/x86/intel-speed-select: Introduce
is_cpu_in_power_domain helper
      tools/power/x86/intel-speed-select: Do not export get_physical_id
      tools/power/x86/intel-speed-select: Enforce isst_id value
      tools/power/x86/intel-speed-select: Remove unused struct
clos_config fields
      tools/power/x86/intel-speed-select: Utilize cpu_map to get
physical id
      tools/power/x86/intel-speed-select: Optimize CPU initialization

 tools/power/x86/intel-speed-select/hfi-events.c   |   5 +-
 tools/power/x86/intel-speed-select/isst-config.c  | 511
++++++++++++++++++++++++++++++++++++-----------------------------------
--
 tools/power/x86/intel-speed-select/isst-core.c    | 313
+++++++++++++++++++++------------------------
 tools/power/x86/intel-speed-select/isst-daemon.c  |  36 +++---
 tools/power/x86/intel-speed-select/isst-display.c |  76 +++++------
 tools/power/x86/intel-speed-select/isst.h         |  88 ++++++-------
 6 files changed, 496 insertions(+), 533 deletions(-)

Thanks,
Srinivas

Re: [GIT PULL]: tools/power/x86/intel-speed-select pull request for 6.1-rc1

[Hans de Goede]

Hi,

On 9/15/22 19:31, Pandruvada, Srinivas wrote:
> Hi Hans,
> 
> This pull request contains:
> - One fix for invalid display of cpu count for TDP level
> - Removal of unused code and data
> - consolidate package/die/cpu in a struct and use this
> instead of passing all and recompute package and die ids
> 
> These changes are on top of
> https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git
> for-next
> 
> 
> The following changes since commit
> 76fba1221e3045570478381c3436c8c2a2656376:
> 
>   Merge tag 'backlight-detect-refactor-1' into review-hans (2022-09-05
> 11:01:56 +0200)
> 
> are available in the Git repository at:
> 
>   https://github.com/spandruvada/linux-kernel.git intel-sst
> 
> for you to fetch changes up to
> 97ec890d070158f15fe8ef2419a5ff677bdbbc20:
> 
>   tools/power/x86/intel-speed-select: Release v1.13 (2022-09-15
> 11:16:06 -0700)

Thanks, I have merged this into my review-hans (soon to be for-next)
branch now.

Regards,

Hans


> 
> ----------------------------------------------------------------
> Srinivas Pandruvada (1):
>       tools/power/x86/intel-speed-select: Release v1.13
> 
> Zhang Rui (13):
>       tools/power/x86/intel-speed-select: Fix cpu count for TDP level
> display
>       tools/power/x86/intel-speed-select: Remove dead code
>       tools/power/x86/intel-speed-select: Remove unused core_mask array
>       tools/power/x86/intel-speed-select: Introduce struct isst_id
>       tools/power/x86/intel-speed-select: Add pkg and die in isst_id
>       tools/power/x86/intel-speed-select: Convert more function to use
> isst_id
>       tools/power/x86/intel-speed-select: Cleanup get_physical_id usage
>       tools/power/x86/intel-speed-select: Introduce
> is_cpu_in_power_domain helper
>       tools/power/x86/intel-speed-select: Do not export get_physical_id
>       tools/power/x86/intel-speed-select: Enforce isst_id value
>       tools/power/x86/intel-speed-select: Remove unused struct
> clos_config fields
>       tools/power/x86/intel-speed-select: Utilize cpu_map to get
> physical id
>       tools/power/x86/intel-speed-select: Optimize CPU initialization
> 
>  tools/power/x86/intel-speed-select/hfi-events.c   |   5 +-
>  tools/power/x86/intel-speed-select/isst-config.c  | 511
> ++++++++++++++++++++++++++++++++++++-----------------------------------
> --
>  tools/power/x86/intel-speed-select/isst-core.c    | 313
> +++++++++++++++++++++------------------------
>  tools/power/x86/intel-speed-select/isst-daemon.c  |  36 +++---
>  tools/power/x86/intel-speed-select/isst-display.c |  76 +++++------
>  tools/power/x86/intel-speed-select/isst.h         |  88 ++++++-------
>  6 files changed, 496 insertions(+), 533 deletions(-)
> 
> Thanks,
> Srinivas
> 
>