Unverified Commit 6f130e04 authored by Xi Pardee's avatar Xi Pardee Committed by Ilpo Järvinen
Browse files

platform/x86:intel/pmc: Use devm for mutex_init 



Use devm_mutex_init() to avoid accidental resource leak in the
future.

Signed-off-by: default avatarXi Pardee <xi.pardee@linux.intel.com>
Link: https://lore.kernel.org/r/20250425195237.493129-4-xi.pardee@linux.intel.com


Reviewed-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
parent b5d46539

drivers/platform/x86/intel/pmc/core.c

+3 −2
Original line number Diff line number Diff line
@@ -1681,7 +1681,6 @@ static void pmc_core_clean_structure(struct platform_device *pdev) 
		pmt_telem_unregister_endpoint(pmcdev->punit_ep);



	platform_set_drvdata(pdev, NULL);

	mutex_destroy(&pmcdev->lock);

}



static int pmc_core_probe(struct platform_device *pdev)
@@ -1726,7 +1725,9 @@ static int pmc_core_probe(struct platform_device *pdev) 
	if (!pmcdev->pkgc_res_cnt)

		return -ENOMEM;



	mutex_init(&pmcdev->lock);

	ret = devm_mutex_init(&pdev->dev, &pmcdev->lock);

	if (ret)

		return ret;



	if (pmc_dev_info->init)

		ret = pmc_dev_info->init(pmcdev, pmc_dev_info);