2013-03-12 18:53:15

by Alexandru Gheorghiu

Subject: [PATCH] drivers: acpi: Used resource_size function

Used resource_size function instead of explicit computation.

Signed-off-by: Alexandru Gheorghiu <[email protected]>
---
drivers/acpi/osl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/acpi/osl.c b/drivers/acpi/osl.c
index 586e7e9..4d31748 100644
--- a/drivers/acpi/osl.c
+++ b/drivers/acpi/osl.c
@@ -1555,7 +1555,7 @@ int acpi_check_resource_conflict(const struct resource *res)
else
space_id = ACPI_ADR_SPACE_SYSTEM_MEMORY;

- length = res->end - res->start + 1;
+ length = resource_size(res);
if (acpi_enforce_resources != ENFORCE_RESOURCES_NO)
warn = 1;
clash = acpi_check_address_range(space_id, res->start, length, warn);
--
1.7.9.5

2013-03-26 13:21:21

by Rafael J. Wysocki

Subject: Re: [PATCH] drivers: acpi: Used resource_size function

On Tuesday, March 12, 2013 10:53:02 AM Alexandru Gheorghiu wrote:
> Used resource_size function instead of explicit computation.
>
> Signed-off-by: Alexandru Gheorghiu <[email protected]>

Applied to linux-pm.git/linux-next.

Thanks,
Rafael


> ---
> drivers/acpi/osl.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/acpi/osl.c b/drivers/acpi/osl.c
> index 586e7e9..4d31748 100644
> --- a/drivers/acpi/osl.c
> +++ b/drivers/acpi/osl.c
> @@ -1555,7 +1555,7 @@ int acpi_check_resource_conflict(const struct resource *res)
> else
> space_id = ACPI_ADR_SPACE_SYSTEM_MEMORY;
>
> - length = res->end - res->start + 1;
> + length = resource_size(res);
> if (acpi_enforce_resources != ENFORCE_RESOURCES_NO)
> warn = 1;
> clash = acpi_check_address_range(space_id, res->start, length, warn);
>
--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.