platform/x86: Switch back to struct platform_driver::remove()After commit 0edb555a65d1 ("platform: Make platform_driver::remove()return void") .remove() is (again) the right callback to implement
platform/x86: Switch back to struct platform_driver::remove()After commit 0edb555a65d1 ("platform: Make platform_driver::remove()return void") .remove() is (again) the right callback to implement forplatform drivers.Convert all platform drivers below drivers/platform/x86/ to use.remove(), with the eventual goal to drop structplatform_driver::remove_new(). As .remove() and .remove_new() have thesame prototypes, conversion is done by just changing the structuremember name in the driver initializer.While touching these files, make indention of the struct initializerconsistent in several files.Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com>Link: https://lore.kernel.org/r/20241017073802.53235-2-u.kleine-koenig@baylibre.comReviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
show more ...
platform/x86: intel_telemetry: Switch to new Intel CPU model definesNew CPU #defines encode vendor and family as well as model.Signed-off-by: Tony Luck <tony.luck@intel.com>Acked-by: Hans de Goe
platform/x86: intel_telemetry: Switch to new Intel CPU model definesNew CPU #defines encode vendor and family as well as model.Signed-off-by: Tony Luck <tony.luck@intel.com>Acked-by: Hans de Goede <hdegoede@redhat.com>Link: https://lore.kernel.org/r/20240531203706.233365-3-tony.luck@intel.comReviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
platform/x86: intel_telemetry: Fix kernel doc descriptionsLKP found issues with a kernel doc in the driver:core.c:116: warning: Function parameter or member 'ioss_evtconfig' not described in 'tel
platform/x86: intel_telemetry: Fix kernel doc descriptionsLKP found issues with a kernel doc in the driver:core.c:116: warning: Function parameter or member 'ioss_evtconfig' not described in 'telemetry_update_events'core.c:188: warning: Function parameter or member 'ioss_evtconfig' not described in 'telemetry_get_eventconfig'It looks like it were copy'n'paste typos when these descriptionshad been introduced. Fix the typos.Reported-by: kernel test robot <lkp@intel.com>Closes: https://lore.kernel.org/oe-kbuild-all/202310070743.WALmRGSY-lkp@intel.com/Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>Link: https://lore.kernel.org/r/20231120150756.1661425-1-andriy.shevchenko@linux.intel.comReviewed-by: Rajneesh Bhardwaj <irenic.rajneesh@gmail.com>Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
platform/x86: intel: telemetry: pltdrv: Convert to platform remove callback returning voidThe .remove() callback for a platform driver returns an int which makesmany driver authors wrongly assume
platform/x86: intel: telemetry: pltdrv: Convert to platform remove callback returning voidThe .remove() callback for a platform driver returns an int which makesmany driver authors wrongly assume it's possible to do error handling byreturning an error code. However the value returned is (mostly) ignoredand this typically results in resource leaks. To improve here there is aquest to make the remove callback return void. In the first step of thisquest all drivers are converted to .remove_new() which already returnsvoid.Trivially convert this driver from always returning zero in the removecallback to the void returning variant.Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>Link: https://lore.kernel.org/r/20230302144732.1903781-25-u.kleine-koenig@pengutronix.deReviewed-by: Hans de Goede <hdegoede@redhat.com>Signed-off-by: Hans de Goede <hdegoede@redhat.com>
platform/x86: intel_telemetry: Move to intel sub-directoryMove Intel telemetry driver to intel sub-directory to improve readability.While at it, spell APL fully in the Kconfig.Signed-off-by: Ka
platform/x86: intel_telemetry: Move to intel sub-directoryMove Intel telemetry driver to intel sub-directory to improve readability.While at it, spell APL fully in the Kconfig.Signed-off-by: Kate Hsuan <hpa@redhat.com>Reviewed-by: Hans de Goede <hdegoede@redhat.com>Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>Reviewed-by: Rajneesh Bhardwaj <irenic.rajneesh@gmail.com>Link: https://lore.kernel.org/r/20210820110458.73018-9-andriy.shevchenko@linux.intel.comSigned-off-by: Hans de Goede <hdegoede@redhat.com>