From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH v2] net: hamradio: use eth_broadcast_addr Date: Wed, 20 Jun 2018 07:52:02 +0900 (KST) Message-ID: <20180620.075202.1458380285687445735.davem@davemloft.net> References: <20180617214058.6926-1-stefan@agner.ch> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org To: stefan@agner.ch Return-path: In-Reply-To: <20180617214058.6926-1-stefan@agner.ch> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org From: Stefan Agner Date: Sun, 17 Jun 2018 23:40:53 +0200 > The array bpq_eth_addr is only used to get the size of an > address, whereas the bcast_addr is used to set the broadcast > address. This leads to a warning when using clang: > drivers/net/hamradio/bpqether.c:94:13: warning: variable 'bpq_eth_addr' is not > needed and will not be emitted [-Wunneeded-internal-declaration] > static char bpq_eth_addr[6]; > ^ > > Remove both variables and use the common eth_broadcast_addr > to set the broadcast address. > > Signed-off-by: Stefan Agner Applied.