tools/power/x86/intel-speed-select: clean all the produced files

[PATCH] tools/power/x86/intel-speed-select: clean all the produced files

Posted by Jiri Slaby (SUSE) 2 weeks, 3 days ago

1) The commit 7d440da009b6 ("tools/power/x86/intel-speed-select: HFI
   support") added the thermal.h link, but did not remove it up on
   cleanup. Do so.
2) .*.cmd files were never deleted, so were left dangling. Clean them up
   too.
3) The "*.d" pattern was updated:
   a) the backslash was superfluous,
   b) it was made more explicit by specifying ".o" in ".*.o.d".

Signed-off-by: Jiri Slaby (SUSE) <jirislaby@kernel.org>
Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
---
Cc: platform-driver-x86@vger.kernel.org
---
 tools/power/x86/intel-speed-select/Makefile | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/power/x86/intel-speed-select/Makefile b/tools/power/x86/intel-speed-select/Makefile
index 6b299aae2ded..6f896f40c1a8 100644
--- a/tools/power/x86/intel-speed-select/Makefile
+++ b/tools/power/x86/intel-speed-select/Makefile
@@ -52,8 +52,8 @@ $(OUTPUT)intel-speed-select: $(ISST_IN)
 
 clean:
 	rm -f $(ALL_PROGRAMS)
-	rm -rf $(OUTPUT)include/linux/isst_if.h
-	find $(or $(OUTPUT),.) -name '*.o' -delete -o -name '\.*.d' -delete
+	rm -rf $(OUTPUT)include/linux/isst_if.h $(OUTPUT)include/linux/thermal.h
+	find $(or $(OUTPUT),.) -name '*.o' -delete -o -name '.*.o.d' -delete -o -name '.*.o.cmd' -delete
 
 install: $(ALL_PROGRAMS)
 	install -d -m 755 $(DESTDIR)$(bindir);		\
-- 
2.55.0

Re: [PATCH] tools/power/x86/intel-speed-select: clean all the produced files

Posted by srinivas pandruvada 2 weeks, 1 day ago

On Tue, 2026-09-08 at 08:50 +0200, Jiri Slaby (SUSE) wrote:
> 1) The commit 7d440da009b6 ("tools/power/x86/intel-speed-select: HFI
>    support") added the thermal.h link, but did not remove it up on
>    cleanup. Do so.
> 2) .*.cmd files were never deleted, so were left dangling. Clean them
> up
>    too.
> 3) The "*.d" pattern was updated:
>    a) the backslash was superfluous,
>    b) it was made more explicit by specifying ".o" in ".*.o.d".
> 
> Signed-off-by: Jiri Slaby (SUSE) <jirislaby@kernel.org>
> Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>


Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/spandruvada/linux.git/log/?h=for-sst-v7.4.rc1


Thanks,
Srinivas

> ---
> Cc: platform-driver-x86@vger.kernel.org
> ---
>  tools/power/x86/intel-speed-select/Makefile | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tools/power/x86/intel-speed-select/Makefile
> b/tools/power/x86/intel-speed-select/Makefile
> index 6b299aae2ded..6f896f40c1a8 100644
> --- a/tools/power/x86/intel-speed-select/Makefile
> +++ b/tools/power/x86/intel-speed-select/Makefile
> @@ -52,8 +52,8 @@ $(OUTPUT)intel-speed-select: $(ISST_IN)
>  
>  clean:
>  	rm -f $(ALL_PROGRAMS)
> -	rm -rf $(OUTPUT)include/linux/isst_if.h
> -	find $(or $(OUTPUT),.) -name '*.o' -delete -o -name '\.*.d'
> -delete
> +	rm -rf $(OUTPUT)include/linux/isst_if.h
> $(OUTPUT)include/linux/thermal.h
> +	find $(or $(OUTPUT),.) -name '*.o' -delete -o -name '.*.o.d'
> -delete -o -name '.*.o.cmd' -delete
>  
>  install: $(ALL_PROGRAMS)
>  	install -d -m 755 $(DESTDIR)$(bindir);		\