Re: [PATCH] power: Fix style error in Macros

From: Anton Vorontsov
Date: Fri Jul 08 2011 - 09:06:59 EST


On Sun, May 29, 2011 at 06:53:12PM +0100, Ken O'Brien wrote:
> Two Macros in the changed file contained complex expressions which were not
> enclosed by parentheses.
>
> Signed-off-by: Ken O'Brien <kernel@xxxxxxxxxxxxx>
> ---

Appled, thanks.

> drivers/power/apm_power.c | 8 ++++----
> 1 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/power/apm_power.c b/drivers/power/apm_power.c
> index dc628cb..8a612de 100644
> --- a/drivers/power/apm_power.c
> +++ b/drivers/power/apm_power.c
> @@ -14,11 +14,11 @@
> #include <linux/apm-emulation.h>
>
>
> -#define PSY_PROP(psy, prop, val) psy->get_property(psy, \
> - POWER_SUPPLY_PROP_##prop, val)
> +#define PSY_PROP(psy, prop, val) (psy->get_property(psy, \
> + POWER_SUPPLY_PROP_##prop, val))
>
> -#define _MPSY_PROP(prop, val) main_battery->get_property(main_battery, \
> - prop, val)
> +#define _MPSY_PROP(prop, val) (main_battery->get_property(main_battery, \
> + prop, val))
>
> #define MPSY_PROP(prop, val) _MPSY_PROP(POWER_SUPPLY_PROP_##prop, val)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/