Re: [PATCH v2] cpupower: Change the var type of the 'monitor' subcommand display mode

From: Shuah Khan
Date: Thu Jun 20 2024 - 13:23:33 EST


On 6/20/24 09:57, Roman Storozhenko wrote:
There is a type 'enum operation_mode_e' contains the display modes of
the 'monitor' subcommand. This type isn't used though, instead the
variable 'mode' is of a simple 'int' type.
Change 'mode' variable type from 'int' to 'enum operation_mode_e' in
order to improve compiler type checking.
Built and tested this with different monitor cmdline params. Everything
works as expected, that is nothing changed and no regressions encountered.

Signed-off-by: Roman Storozhenko <romeusmeister@xxxxxxxxx>
---
Changes in v2:
- Moved cover letter content to the change log
- Link to v1: https://lore.kernel.org/r/20240619-change-mode-type-v1-1-31e7e45028f0@xxxxxxxxx

Applied for Linux 6.11-rc1
https://git.kernel.org/pub/scm/linux/kernel/git/shuah/linux.git/log/?h=cpupower

thanks,
-- Shuah