Commit e5e9b7bd authored by Qianfeng Rong's avatar Qianfeng Rong Committed by Rafael J. Wysocki
Browse files

cpufreq: speedstep-lib: Use int type to store negative error codes 



Change the return type of the speedstep_get_freqs() function from unsigned
int to int because it may return negative error codes.  For the same
reason, change the 'ret' variables to int type as well.

No effect on runtime.

Signed-off-by: default avatarQianfeng Rong <rongqianfeng@vivo.com>
Acked-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Link: https://patch.msgid.link/20250902114545.651661-4-rongqianfeng@vivo.com


Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent b49d7084

drivers/cpufreq/speedstep-lib.c

+6 −6
Original line number Diff line number Diff line
@@ -378,16 +378,16 @@ EXPORT_SYMBOL_GPL(speedstep_detect_processor); 
 *                     DETECT SPEEDSTEP SPEEDS                       *

 *********************************************************************/



unsigned int speedstep_get_freqs(enum speedstep_processor processor,

int speedstep_get_freqs(enum speedstep_processor processor,

			unsigned int *low_speed,

			unsigned int *high_speed,

			unsigned int *transition_latency,

			void (*set_state)(unsigned int state))

{

	unsigned int prev_speed;

	unsigned int ret = 0;

	unsigned long flags;

	ktime_t tv1, tv2;

	int ret = 0;



	if ((!processor) || (!low_speed) || (!high_speed) || (!set_state))

		return -EINVAL;

drivers/cpufreq/speedstep-lib.h

+5 −5
Original line number Diff line number Diff line
@@ -41,7 +41,7 @@ extern unsigned int speedstep_get_frequency(enum speedstep_processor processor); 
 * SPEEDSTEP_LOW; the second argument is zero so that no

 * cpufreq_notify_transition calls are initiated.

 */

extern unsigned int speedstep_get_freqs(enum speedstep_processor processor,

extern int speedstep_get_freqs(enum speedstep_processor processor,

			       unsigned int *low_speed,

			       unsigned int *high_speed,

			       unsigned int *transition_latency,