diff for duplicates of <20120309151527.GH4497@localhost> diff --git a/a/1.txt b/N1/1.txt index 6abc9f4..668383a 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -18,18 +18,18 @@ On Fri, Mar 09, 2012 at 03:35:46PM +0100, David Herrmann wrote: > >> > --- a/drivers/bluetooth/hci_ldisc.c > >> > +++ b/drivers/bluetooth/hci_ldisc.c > >> > @@ -310,11 +310,11 @@ static void hci_uart_tty_close(struct tty_struct *tty) -> >> > hci_uart_close(hdev); +> >> > � � � � � � � � � � � �hci_uart_close(hdev); > >> > -> >> > if (test_and_clear_bit(HCI_UART_PROTO_SET, &hu->flags)) { -> >> > - hu->proto->close(hu); -> >> > if (hdev) { -> >> > hci_unregister_dev(hdev); -> >> > hci_free_dev(hdev); -> >> > } -> >> > + hu->proto->close(hu); -> >> > } -> >> > } -> >> > } +> >> > � � � � � � � �if (test_and_clear_bit(HCI_UART_PROTO_SET, &hu->flags)) { +> >> > - � � � � � � � � � � � hu->proto->close(hu); +> >> > � � � � � � � � � � � �if (hdev) { +> >> > � � � � � � � � � � � � � � � �hci_unregister_dev(hdev); +> >> > � � � � � � � � � � � � � � � �hci_free_dev(hdev); +> >> > � � � � � � � � � � � �} +> >> > + � � � � � � � � � � � hu->proto->close(hu); +> >> > � � � � � � � �} +> >> > � � � �} +> >> > �} > >> > >> I can confirm this. hci_uart_set_proto() opens the proto before it > >> registers the hci device. Hence, we should also unregister the hci diff --git a/a/content_digest b/N1/content_digest index 74c1b27..33b2f5b 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -36,18 +36,18 @@ "> >> > --- a/drivers/bluetooth/hci_ldisc.c\n" "> >> > +++ b/drivers/bluetooth/hci_ldisc.c\n" "> >> > @@ -310,11 +310,11 @@ static void hci_uart_tty_close(struct tty_struct *tty)\n" - "> >> > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240hci_uart_close(hdev);\n" + "> >> > \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275hci_uart_close(hdev);\n" "> >> >\n" - "> >> > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240if (test_and_clear_bit(HCI_UART_PROTO_SET, &hu->flags)) {\n" - "> >> > - \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 hu->proto->close(hu);\n" - "> >> > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240if (hdev) {\n" - "> >> > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240hci_unregister_dev(hdev);\n" - "> >> > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240hci_free_dev(hdev);\n" - "> >> > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240}\n" - "> >> > + \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 hu->proto->close(hu);\n" - "> >> > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240}\n" - "> >> > \302\240 \302\240 \302\240 \302\240}\n" - "> >> > \302\240}\n" + "> >> > \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275if (test_and_clear_bit(HCI_UART_PROTO_SET, &hu->flags)) {\n" + "> >> > - \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 hu->proto->close(hu);\n" + "> >> > \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275if (hdev) {\n" + "> >> > \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275hci_unregister_dev(hdev);\n" + "> >> > \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275hci_free_dev(hdev);\n" + "> >> > \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275}\n" + "> >> > + \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 hu->proto->close(hu);\n" + "> >> > \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275}\n" + "> >> > \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275 \303\257\302\277\302\275}\n" + "> >> > \303\257\302\277\302\275}\n" "> >>\n" "> >> I can confirm this. hci_uart_set_proto() opens the proto before it\n" "> >> registers the hci device. Hence, we should also unregister the hci\n" @@ -89,4 +89,4 @@ "Thanks,\n" Johan -b9492ca8f42235367d874263f49f6e6d9a84683aa1e277dc6ca98bb56653189c +0097f4affd8c0af14f7e77bc46c0954656419efe8bc9057e9d590dbd8d6849cd
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.