On 10-04-24, 06:22, Lizhe wrote: > For the exit() callback function returning an int type value. > this leads many driver authors mistakenly believing that error > handling can be performed by returning an error code. However. > the returned value is ignore, and to improve this situation. > it is proposed to modify the return type of the exit() callback > function to void > > Signed-off-by: Lizhe <sensor1010@xxxxxxx> > --- > drivers/cpufreq/acpi-cpufreq.c | 4 +--- > drivers/cpufreq/amd-pstate.c | 7 ++----- > drivers/cpufreq/apple-soc-cpufreq.c | 4 +--- > drivers/cpufreq/bmips-cpufreq.c | 4 +--- > drivers/cpufreq/cppc_cpufreq.c | 3 +-- > drivers/cpufreq/cpufreq-dt.c | 3 +-- > drivers/cpufreq/e_powersaver.c | 3 +-- > drivers/cpufreq/intel_pstate.c | 4 +--- > drivers/cpufreq/mediatek-cpufreq-hw.c | 4 +--- > drivers/cpufreq/mediatek-cpufreq.c | 4 +--- > drivers/cpufreq/omap-cpufreq.c | 3 +-- > drivers/cpufreq/pasemi-cpufreq.c | 6 ++---- > drivers/cpufreq/powernow-k6.c | 3 +-- > drivers/cpufreq/powernow-k7.c | 3 +-- > drivers/cpufreq/powernow-k8.c | 4 +--- > drivers/cpufreq/powernv-cpufreq.c | 4 +--- > drivers/cpufreq/ppc_cbe_cpufreq.c | 3 +-- > drivers/cpufreq/qcom-cpufreq-hw.c | 4 +--- > drivers/cpufreq/qoriq-cpufreq.c | 4 +--- > drivers/cpufreq/scmi-cpufreq.c | 4 +--- > drivers/cpufreq/scpi-cpufreq.c | 4 +--- > drivers/cpufreq/sh-cpufreq.c | 4 +--- > drivers/cpufreq/sparc-us2e-cpufreq.c | 3 +-- > drivers/cpufreq/sparc-us3-cpufreq.c | 3 +-- > drivers/cpufreq/speedstep-centrino.c | 4 +--- > drivers/cpufreq/tegra194-cpufreq.c | 4 +--- > drivers/cpufreq/vexpress-spc-cpufreq.c | 3 +-- > 27 files changed, 29 insertions(+), 74 deletions(-) I have discarded all emails with following subject line: "cpufreq: Convert to exit callback returning void". While you have sent decent patches for removing the empty exit callbacks, the way you have handled these changes is not correct. Don't send any patches for now and please wait and understand what's being asked from you. This change you are trying to make is okay and sensible, but you can not send random patches to the list just like that. You are wasting everyone's time here including yourself. Now what we expect here is a single commit (with version history), which changes all the users of the exit() function (each and every cpufreq driver) and cpufreq.h and cpufreq.c. That change should compile fine and break none of the platforms compilation. Please don't send more of these patches unless this is done. -- viresh
