Elyes Haouas has submitted this change. ( https://review.coreboot.org/c/coreboot/+/86130?usp=email )
Change subject: drivers/intel/dptf: Suppress unnecessary static function ......................................................................
drivers/intel/dptf: Suppress unnecessary static function
This commit eliminates the superfluous get_dptf_platform_info() static function.
Change-Id: I0b9d150bab8486cb7e437d5e2b3caa880e14f886 Signed-off-by: Jeremy Compostella jeremy.compostella@intel.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/86130 Reviewed-by: Jamie Ryu jamie.m.ryu@intel.com Reviewed-by: Elyes Haouas ehaouas@noos.fr Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Bora Guvendik bora.guvendik@intel.com --- M src/drivers/intel/dptf/dptf.c 1 file changed, 1 insertion(+), 6 deletions(-)
Approvals: Elyes Haouas: Looks good to me, approved build bot (Jenkins): Verified Bora Guvendik: Looks good to me, approved Jamie Ryu: Looks good to me, approved
diff --git a/src/drivers/intel/dptf/dptf.c b/src/drivers/intel/dptf/dptf.c index b6f5005..0b60dcf 100644 --- a/src/drivers/intel/dptf/dptf.c +++ b/src/drivers/intel/dptf/dptf.c @@ -496,15 +496,10 @@ #endif };
-static const struct dptf_platform_info *get_dptf_platform_info(void) -{ - return &generic_dptf_platform_info; -} - /* Add minimal definitions of DPTF devices into the SSDT */ static void write_device_definitions(const struct device *dev) { - const struct dptf_platform_info *platform_info = get_dptf_platform_info(); + const struct dptf_platform_info *platform_info = &generic_dptf_platform_info; const struct drivers_intel_dptf_config *config; struct device *parent; enum dptf_participant p;