[LTP] [PATCH v8] cpufreq.c: add new test for cpufreq sysfs interface validation

Cyril Hrubis chrubis@suse.cz
Wed Jun 3 14:27:26 CEST 2026


Hi!
> +		SAFE_FILE_SCANF(path, "%255s", contents);
> +		tst_res(TDEBUG, "Checking whether %s is \"intel_pstate\"", path);

I slightly prefer single quotes inside double quotes e.g. " ...
'intel_pstate'" since they does not need to be escaped. And maybe the
run() could have been split into functions so that it's not that long.
But both of these are very minor.

Reviewed-by: Cyril Hrubis <chrubis@suse.cz>

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list