mirror of
https://github.com/torvalds/linux.git
synced 2026-11-07 06:17:02 +01:00
mfd: intel-lpss: Remove redundant DebugFS error check
The DebugFS API is designed to handle errors gracefully. Any explicit checking on return values is considered an anti-pattern. This patch removes unnecessary error checking and converts intel_lpss_debugfs_add() into a void function. Signed-off-by: Maria Lisina <sekoohaka.sarisan@gmail.com> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Link: https://patch.msgid.link/20260916-intel-lpss-debugfs-v7-1-eb51d41f55e0@gmail.com Signed-off-by: Lee Jones <lee@kernel.org>
This commit is contained in:
parent
3c5c55d979
commit
b761214ac8
|
|
@ -142,13 +142,11 @@ static void intel_lpss_cache_ltr(struct intel_lpss *lpss)
|
|||
lpss->idle_ltr = readl(lpss->priv + LPSS_PRIV_IDLELTR);
|
||||
}
|
||||
|
||||
static int intel_lpss_debugfs_add(struct intel_lpss *lpss)
|
||||
static void intel_lpss_debugfs_add(struct intel_lpss *lpss)
|
||||
{
|
||||
struct dentry *dir;
|
||||
|
||||
dir = debugfs_create_dir(dev_name(lpss->dev), intel_lpss_debugfs);
|
||||
if (IS_ERR(dir))
|
||||
return PTR_ERR(dir);
|
||||
|
||||
/* Cache the values into lpss structure */
|
||||
intel_lpss_cache_ltr(lpss);
|
||||
|
|
@ -158,7 +156,6 @@ static int intel_lpss_debugfs_add(struct intel_lpss *lpss)
|
|||
debugfs_create_x32("idle_ltr", S_IRUGO, dir, &lpss->idle_ltr);
|
||||
|
||||
lpss->debugfs = dir;
|
||||
return 0;
|
||||
}
|
||||
|
||||
static void intel_lpss_debugfs_remove(struct intel_lpss *lpss)
|
||||
|
|
@ -432,10 +429,7 @@ int intel_lpss_probe(struct device *dev,
|
|||
goto err_clk_register;
|
||||
|
||||
intel_lpss_ltr_expose(lpss);
|
||||
|
||||
ret = intel_lpss_debugfs_add(lpss);
|
||||
if (ret)
|
||||
dev_warn(dev, "Failed to create debugfs entries\n");
|
||||
intel_lpss_debugfs_add(lpss);
|
||||
|
||||
if (intel_lpss_has_idma(lpss)) {
|
||||
ret = mfd_add_devices(dev, lpss->devid, &intel_lpss_idma64_cell,
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user