diff for duplicates of <20120309142939.GD4497@localhost> diff --git a/a/1.txt b/N1/1.txt index 15da2d6..e522fe9 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -18,18 +18,18 @@ On Fri, Mar 09, 2012 at 02:44:30PM +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 ceca894..83c369a 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -34,18 +34,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" @@ -74,4 +74,4 @@ "Thanks,\n" Johan -ff1e357b426b395cd3c0b9a450911ac02835ffdf1125a0e50a01c87379552708 +cd6be3483bb33b90279c72e82af7bf1296ab1d23d1170d18be5bcb2ae1f405fe
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.