Return-Path: MIME-Version: 1.0 In-Reply-To: <20120919013113.GG24689@joana> References: <20120919013113.GG24689@joana> From: "Luis R. Rodriguez" Date: Fri, 21 Sep 2012 18:06:42 -0700 Message-ID: Subject: Re: pull request: bluetooth-next 2012-09-18 To: Gustavo Padovan , linville@tuxdriver.com, linux-wireless@vger.kernel.org, linux-bluetooth@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: On Tue, Sep 18, 2012 at 6:31 PM, Gustavo Padovan wrote: > Johan Hedberg (2): > Bluetooth: mgmt: Implement support for passkey notification Too late now... but why did we allow this a stable fix? I get its a feature that is important and likely overlooked / someone had a brain fart, but from what I gather this is not on v3.5.4 and yet may make it to v3.5.5. At least for backporting this was a bitch: /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3290:52: error: dereferencing pointer to incomplete type /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3294:6: error: ‘struct hci_conn’ has no member named ‘passkey_notify’ /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3294:25: error: dereferencing pointer to incomplete type /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3295:6: error: ‘struct hci_conn’ has no member named ‘passkey_entered’ /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3298:3: error: implicit declaration of function ‘mgmt_user_passkey_notify’ [-Werror=implicit-function-declaration] /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3299:27: error: ‘struct hci_conn’ has no member named ‘passkey_notify’ /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3300:11: error: ‘struct hci_conn’ has no member named ‘passkey_entered’ /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c: In function ‘hci_keypress_notify_evt’: /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3310:52: error: dereferencing pointer to incomplete type /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3314:12: error: dereferencing pointer to incomplete type /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3315:7: error: ‘HCI_KEYPRESS_STARTED’ undeclared (first use in this function) /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3315:7: note: each undeclared identifier is reported only once for each function it appears in /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3316:7: error: ‘struct hci_conn’ has no member named ‘passkey_entered’ /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3319:7: error: ‘HCI_KEYPRESS_ENTERED’ undeclared (first use in this function) /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3320:7: error: ‘struct hci_conn’ has no member named ‘passkey_entered’ /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3323:7: error: ‘HCI_KEYPRESS_ERASED’ undeclared (first use in this function) /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3324:7: error: ‘struct hci_conn’ has no member named ‘passkey_entered’ /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3327:7: error: ‘HCI_KEYPRESS_CLEARED’ undeclared (first use in this function) /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3328:7: error: ‘struct hci_conn’ has no member named ‘passkey_entered’ /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3331:7: error: ‘HCI_KEYPRESS_COMPLETED’ undeclared (first use in this function) /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3337:27: error: ‘struct hci_conn’ has no member named ‘passkey_notify’ /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3338:11: error: ‘struct hci_conn’ has no member named ‘passkey_entered’ /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c: In function ‘hci_event_packet’: /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3701:7: error: ‘HCI_EV_USER_PASSKEY_NOTIFY’ undeclared (first use in this function) /home/mcgrof/staging/v3.5.4/compat-wireless-3.5.3-2-snpc/net/bluetooth/hci_event.c:3705:7: error: ‘HCI_EV_KEYPRESS_NOTIFY’ undeclared (first use in this function) I'd much prefer to see things like these identified as 'important' but not 'fuck, this fixes that crash'. The fact that kernel maintainers may be anal about these sorts of patches is *good* for many reasons and figuring out a way to categorize, identify and allow users to reap benefits of backporting suck fixes / cherry picking a full solution is what something that should not interfere and hopefully be transparent to Linux upstream development. Here's one way I dealt with these type of patches: https://backports.wiki.kernel.org/index.php/Documentation/compat-drivers/additional-patches If you look at the patch 'Bluetooth: mgmt: Implement support for passkey' its a clear "feature" implementation, not a fucking fix. Luis