From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kalle Valo Subject: Re: [PATCH v2 00/10] rt2x00: rt2x00: improve calling conventions for register accessors Date: Fri, 19 May 2017 10:12:36 +0300 Message-ID: <87inkxwc9n.fsf@kamboji.qca.qualcomm.com> References: <20170517144702.1381136-1-arnd@arndb.de> <87r2zl8lvi.fsf@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain Cc: Tom Psyborg , linux-wireless , Stanislaw Gruszka , David Miller , Helmut Schaa , Daniel Golle , Mathias Kresin , Johannes Berg , Serge Vasilugin , Roman Yeryomin , Networking , Linux Kernel Mailing List , Jes Sorensen To: Arnd Bergmann Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:35722 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751428AbdESHMp (ORCPT ); Fri, 19 May 2017 03:12:45 -0400 In-Reply-To: (Arnd Bergmann's message of "Fri, 19 May 2017 08:55:21 +0200") Sender: netdev-owner@vger.kernel.org List-ID: Arnd Bergmann writes: > On Fri, May 19, 2017 at 8:44 AM, Tom Psyborg wrote: >> warning: 'rt2800_bbp_read' used but never defined >> static u8 rt2800_bbp_read(struct rt2x00_dev *rt2x00dev, >> ^ >> /home/ubuntu/Music/openwrt/build_dir/target-mipsel_24kc_musl-1.1.16/linux-ramips_mt7620/compat-wireless-2016-05-12/drivers/net/wireless/ralink/rt2x00/rt2800lib.h:262:13: >> warning: 'rt2800_bbp_write' used but never defined >> static void rt2800_bbp_write(struct rt2x00_dev *rt2x00dev, >> ^ >> CC [M] >> /home/ubuntu/Music/openwrt/build_dir/target-mipsel_24kc_musl-1.1.16/linux-ramips_mt7620/compat-wireless-2016-05-12/drivers/net/wireless/ralink/rt2x00/rt2800pci.o >> In file included from >> /home/ubuntu/Music/openwrt/build_dir/target-mipsel_24kc_musl-1.1.16/linux-ramips_mt7620/compat-wireless-2016-05-12/drivers/net/wireless/ralink/rt2x00/rt2800pci.c:43:0: >> /home/ubuntu/Music/openwrt/build_dir/target-mipsel_24kc_musl-1.1.16/linux-ramips_mt7620/compat-wireless-2016-05-12/drivers/net/wireless/ralink/rt2x00/rt2800lib.h:259:11: >> warning: 'rt2800_bbp_read' declared 'static' but never defined >> [-Wunused-function] >> static u8 rt2800_bbp_read(struct rt2x00_dev *rt2x00dev, >> ^ > > On which base version did you apply my patches? There may be a conflict > against patches that are in your tree but not yet in linux-next, as I don't see > the warning and also see no reference to rt2800_bbp_read in rt2800lib.h I did a test build with current wireless-drivers-next and I also don't see any warnings. -- Kalle Valo