[PATCH 5/5] staging: brcm80211: fix checkpatch macro errors.

Joe Perches joe at perches.com
Wed Oct 6 10:27:52 PDT 2010


On Wed, 2010-10-06 at 10:08 -0400, Jason Cooper wrote:
> Signed-off-by: Jason Cooper <jason at lakedaemon.net>
> diff --git a/drivers/staging/brcm80211/brcmfmac/dhd_linux.c b/drivers/staging/brcm80211/brcmfmac/dhd_linux.c
> index 42e2f29..3536011 100644
> --- a/drivers/staging/brcm80211/brcmfmac/dhd_linux.c
> +++ b/drivers/staging/brcm80211/brcmfmac/dhd_linux.c
> @@ -337,9 +337,12 @@ uint dhd_radio_up = 1;
>  char iface_name[IFNAMSIZ];
>  module_param_string(iface_name, iface_name, IFNAMSIZ, 0);
>  
> -#define DAEMONIZE(a) daemonize(a); \
> -	allow_signal(SIGKILL); \
> -	allow_signal(SIGTERM);
> +#define DAEMONIZE(a) \
> +	do { \
> +		daemonize(a); \
> +		allow_signal(SIGKILL); \
> +		allow_signal(SIGTERM); \
> +	} while (0);

You should not have a trailing ";" after a while (0)

> diff --git a/drivers/staging/brcm80211/brcmfmac/wl_iw.c b/drivers/staging/brcm80211/brcmfmac/wl_iw.c
> index b2514d1..d9124e2 100644
> --- a/drivers/staging/brcm80211/brcmfmac/wl_iw.c
> +++ b/drivers/staging/brcm80211/brcmfmac/wl_iw.c
> @@ -107,9 +107,12 @@ static wlc_ssid_t g_specific_ssid;
>  
>  static wlc_ssid_t g_ssid;
>  
> -#define DAEMONIZE(a) daemonize(a); \
> -	allow_signal(SIGKILL); \
> -	allow_signal(SIGTERM);
> +#define DAEMONIZE(a) \
> +	do { \
> +		daemonize(a); \
> +		allow_signal(SIGKILL); \
> +		allow_signal(SIGTERM); \
> +	} while (0);

Here too, but maybe there should be shared
code for this.




More information about the devel mailing list