Re: [PATCH i-g-t] tools: rename power into igt_power

["Ville Syrjälä" <ville.syrjala@linux.intel.com>]

On Tue, Jan 21, 2025 at 04:49:19PM +0100, Kamil Konieczny wrote:
> Some distros installs igt tools in /usr/bin and as long as
> a name of a tool has a prefix of igt_ or intel_ it will not
> cause a problem. However name like 'power' is a little too
> generic, so change it into igt_power.
> 
> Fixes: a886e0ad9783 ("tools/power: Introduce a small power/energy measurement tool")
> Reported-by: Jordan Justen <jljusten@debian.org>
> Closes: https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/issues/168
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Signed-off-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>

Acked-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

> ---
>  tools/{power.c => igt_power.c} | 0
>  tools/meson.build              | 2 +-
>  2 files changed, 1 insertion(+), 1 deletion(-)
>  rename tools/{power.c => igt_power.c} (100%)
> 
> diff --git a/tools/power.c b/tools/igt_power.c
> similarity index 100%
> rename from tools/power.c
> rename to tools/igt_power.c
> diff --git a/tools/meson.build b/tools/meson.build
> index 511aec69e..f091af380 100644
> --- a/tools/meson.build
> +++ b/tools/meson.build
> @@ -12,6 +12,7 @@ foreach prog : tools_progs_noisnt
>  endforeach
>  
>  tools_progs = [
> +	'igt_power',
>  	'igt_stats',
>  	'intel_audio_dump',
>  	'intel_backlight',
> @@ -44,7 +45,6 @@ tools_progs = [
>  	'dpcd_reg',
>  	'lsfacts',
>  	'lsgpu',
> -	'power',
>  ]
>  tool_deps = igt_deps
>  tool_deps += zlib
> -- 
> 2.48.1

-- 
Ville Syrjälä
Intel