Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1256229 > unrolled thread

[PATCH] Staging: wilc1000: Fix warning prefer eth_broadcast_addr over memset()

Started byPunit Vara <punitvara@gmail.com>
First post2015-10-26 21:00 +0100
Last post2015-10-27 06:40 +0100
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] Staging: wilc1000: Fix warning prefer eth_broadcast_addr over memset() Punit Vara <punitvara@gmail.com> - 2015-10-26 21:00 +0100
    Re: [PATCH] Staging: wilc1000: Fix warning prefer eth_broadcast_addr  over memset() Greg KH <gregkh@linuxfoundation.org> - 2015-10-27 06:40 +0100

#1256229 — [PATCH] Staging: wilc1000: Fix warning prefer eth_broadcast_addr over memset()

FromPunit Vara <punitvara@gmail.com>
Date2015-10-26 21:00 +0100
Subject[PATCH] Staging: wilc1000: Fix warning prefer eth_broadcast_addr over memset()
Message-ID<qnW5Y-3Uv-19@gated-at.bofh.it>
This patch is to the host_interface.c that fixes up following
warning by checkpatch.pl:

-Prefer eth_broadcast_addr() over memset()

Signed-off-by: Punit Vara <punitvara@gmail.com>
---
 drivers/staging/wilc1000/host_interface.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c
index 09a6c98..87b4eb8 100644
--- a/drivers/staging/wilc1000/host_interface.c
+++ b/drivers/staging/wilc1000/host_interface.c
@@ -4688,7 +4688,7 @@ s32 host_int_del_station(struct host_if_drv *hif_drv, const u8 *pu8MacAddr)
 	msg.drv = hif_drv;
 
 	if (pu8MacAddr == NULL)
-		memset(pstrDelStationMsg->mac_addr, 255, ETH_ALEN);
+		eth_broadcast_addr(pstrDelStationMsg->mac_addr);
 	else
 		memcpy(pstrDelStationMsg->mac_addr, pu8MacAddr, ETH_ALEN);
 
-- 
2.5.3

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1256467 — Re: [PATCH] Staging: wilc1000: Fix warning prefer eth_broadcast_addr over memset()

FromGreg KH <gregkh@linuxfoundation.org>
Date2015-10-27 06:40 +0100
SubjectRe: [PATCH] Staging: wilc1000: Fix warning prefer eth_broadcast_addr over memset()
Message-ID<qo59f-1gU-7@gated-at.bofh.it>
In reply to#1256229
On Tue, Oct 27, 2015 at 01:21:46AM +0530, Punit Vara wrote:
> This patch is to the host_interface.c that fixes up following
> warning by checkpatch.pl:
> 
> -Prefer eth_broadcast_addr() over memset()
> 
> Signed-off-by: Punit Vara <punitvara@gmail.com>
> ---
>  drivers/staging/wilc1000/host_interface.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c
> index 09a6c98..87b4eb8 100644
> --- a/drivers/staging/wilc1000/host_interface.c
> +++ b/drivers/staging/wilc1000/host_interface.c
> @@ -4688,7 +4688,7 @@ s32 host_int_del_station(struct host_if_drv *hif_drv, const u8 *pu8MacAddr)
>  	msg.drv = hif_drv;
>  
>  	if (pu8MacAddr == NULL)
> -		memset(pstrDelStationMsg->mac_addr, 255, ETH_ALEN);
> +		eth_broadcast_addr(pstrDelStationMsg->mac_addr);
>  	else
>  		memcpy(pstrDelStationMsg->mac_addr, pu8MacAddr, ETH_ALEN);
>  

Doesn't apply to my staging.git tree at all :(
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web