diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2022-03-04 16:19:25 +0300 |
|---|---|---|
| committer | Hans de Goede <hdegoede@redhat.com> | 2022-03-08 18:33:23 +0300 |
| commit | f2a6c7e7474001842a4adaf042d12206c27fc391 (patch) | |
| tree | 02071ff85116c54a23843f53a6320e764d503386 /net/wireless/wext-spy.c | |
| parent | c91a5b1c221a58d008485cf7d02ccce73108b119 (diff) | |
| download | linux-f2a6c7e7474001842a4adaf042d12206c27fc391.tar.xz | |
platform/x86: intel-uncore-freq: fix uncore_freq_common_init() error codes
Currently the uncore_freq_common_init() return one on success and
zero on failure. There is only one caller and it has a "forgot to set
the error code" bug. Change uncore_freq_common_init() to return
negative error codes which makes the code simpler and avoids this kind
of bug in the future.
Fixes: dbce412a7733 ("platform/x86/intel-uncore-freq: Split common and enumeration part")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Link: https://lore.kernel.org/r/20220304131925.GG28739@kili
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Diffstat (limited to 'net/wireless/wext-spy.c')
0 files changed, 0 insertions, 0 deletions