Hi Marcel, On Tue, Mar 28, 2017 at 4:37 PM, Marcel Holtmann <marcel@xxxxxxxxxxxx> wrote: > Hi Arnd, > >> A bugfix accidentally changed one line to return the peer_addr >> from setup_header, causing it to become unused: >> >> net/bluetooth/6lowpan.c: In function 'setup_header': >> net/bluetooth/6lowpan.c:402:14: error: parameter 'peer_addr' set but not used [-Werror=unused-but-set-parameter] >> >> The only user of the variable is a subsequent printk(), and >> it is not otherwise initialized, so reverting the changed line >> looks like the right fix. >> >> Fixes: fb6f2f606ce8 ("6lowpan: Fix IID format for Bluetooth") >> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx> >> --- >> net/bluetooth/6lowpan.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > > I took Colin’s patch since it showed up first in my inbox. Thanks, I might be missing something since the error doesn't show up for me: CC [M] net/bluetooth/6lowpan.o LD [M] net/bluetooth/bluetooth_6lowpan.o -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html