Message ID | 20220831140327.79149-3-andriy.shevchenko@linux.intel.com |
---|---|
State | Accepted |
Commit | 895a4d6ce17b377316ebe7e7090efc1ce9ffd1fe |
Headers | show |
Series | [v2,1/4] ACPI: platform: Get rid of redundant 'else' | expand |
diff --git a/drivers/acpi/acpi_platform.c b/drivers/acpi/acpi_platform.c index 042f80588c18..0d812fe248d4 100644 --- a/drivers/acpi/acpi_platform.c +++ b/drivers/acpi/acpi_platform.c @@ -117,8 +117,7 @@ struct platform_device *acpi_create_platform_device(struct acpi_device *adev, if (count < 0) return NULL; if (count > 0) { - resources = kcalloc(count, sizeof(struct resource), - GFP_KERNEL); + resources = kcalloc(count, sizeof(*resources), GFP_KERNEL); if (!resources) { acpi_dev_free_resource_list(&resource_list); return ERR_PTR(-ENOMEM);
It is preferred to use sizeof(*pointer) instead of sizeof(type). The type of the variable can change and one needs not change the former (unlike the latter). No functional change intended. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> --- v2: no changes drivers/acpi/acpi_platform.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)