diff options
| author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2023-11-24 22:31:24 +0300 |
|---|---|---|
| committer | Lee Jones <lee@kernel.org> | 2023-12-07 16:50:29 +0300 |
| commit | 3b6dba220e67eda42f1263403fbbba64992da3ae (patch) | |
| tree | 985fc9af9d6baee6f26ec9155ace9f8d5a85f573 /drivers/mfd/intel-lpss-acpi.c | |
| parent | d19e5510c84db2cfb60aa187b1cb1a25cb23df52 (diff) | |
| download | linux-3b6dba220e67eda42f1263403fbbba64992da3ae.tar.xz | |
mfd: intel-lpss: Revert "Add missing check for platform_get_resource"
This reverts commit d918e0d5824495a75d00b879118b098fcab36fdb.
The commit in question does not fix anything and only introduces
a duplication in the code. The main intel_lpss_probe() performs
all necessary checks.
While at it and in order of avoiding similar patches to come, add
a comment.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20231124200258.3682979-2-andriy.shevchenko@linux.intel.com
Signed-off-by: Lee Jones <lee@kernel.org>
Diffstat (limited to 'drivers/mfd/intel-lpss-acpi.c')
| -rw-r--r-- | drivers/mfd/intel-lpss-acpi.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/mfd/intel-lpss-acpi.c b/drivers/mfd/intel-lpss-acpi.c index 5184fd1a5df1..52ffc0d9c23f 100644 --- a/drivers/mfd/intel-lpss-acpi.c +++ b/drivers/mfd/intel-lpss-acpi.c @@ -182,10 +182,8 @@ static int intel_lpss_acpi_probe(struct platform_device *pdev) if (!info) return -ENOMEM; + /* No need to check mem and irq here as intel_lpss_probe() does it for us */ info->mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); - if (!info->mem) - return -ENODEV; - info->irq = platform_get_irq(pdev, 0); ret = intel_lpss_probe(&pdev->dev, info); |