From: "Frédéric Danis" <frederic.danis.oss-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> To: robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, marcel-kz+m5ild9QBg9hUCZPvPmw@public.gmane.org, sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, loic.poulain-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, johan-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, lukas-JFq808J9C/izQB+pC5nmwQ@public.gmane.org, hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, rafael-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, greg-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org Cc: linux-bluetooth-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-serial-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, frederic.danis.oss-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org Subject: [PATCH v3 0/2] ACPI serdev support Date: Wed, 11 Oct 2017 10:32:12 +0200 [thread overview] Message-ID: <1507710734-32520-1-git-send-email-frederic.danis.oss@gmail.com> (raw) Add ACPI support for serial attached devices. Currently, serial devices are not set as enumerated during ACPI scan for SPI or i2c buses (but not for UART). This should also be done for UART serial devices. I renamed *spi_i2c_slave* to *serial_bus_slave* to reflect this. This needs Johan Hovold's "serdev: fix registration of second slave" patch. Tested on T100TA with Broadcom BCM2E39. Since v2: - Remove ctrl->serdev set to NULL in acpi_serdev_register_device() in favor of Johan's patch - Fallback to ctrl->serdev check when acpi_walk_namespace() returns an error to prevent memory leak - Remove a change in dev_dbg() call in serdev_controller_add(), this will be done in separate patch Since v1: - Check if a serdev device as been allocated during acpi_walk_namespace() to prevent serdev controller registration instead of the tty-class device. - Reword dev_dbg() strings replacing Serial by serdev - Removing redundant "serdev%d" in dev_dbg() calls in serdev_controller_add() Since RFC: - Add or reword commit messages - Rename *serial_slave* to *serial_bus_slave* - Add specific check for Apple in acpi_is_serial_bus_slave(), thanks to Lukas Wunner - Update comment in acpi_default_enumeration() - Remove patch 3 "Bluetooth: hci_bcm: Add ACPI serdev support for BCM2E39" in favor of patches from Hans de Goede Frédéric Danis (2): serdev: Add ACPI support ACPI / scan: Fix enumeration for special UART devices drivers/acpi/scan.c | 37 ++++++++--------- drivers/tty/serdev/core.c | 100 +++++++++++++++++++++++++++++++++++++++++++--- include/acpi/acpi_bus.h | 2 +- 3 files changed, 113 insertions(+), 26 deletions(-) -- 2.7.4
WARNING: multiple messages have this Message-ID (diff)
From: "Frédéric Danis" <frederic.danis.oss@gmail.com> To: robh@kernel.org, marcel@holtmann.org, sre@kernel.org, loic.poulain@gmail.com, johan@kernel.org, lukas@wunner.de, hdegoede@redhat.com, rafael@kernel.org, greg@kroah.com Cc: linux-bluetooth@vger.kernel.org, linux-serial@vger.kernel.org, linux-acpi@vger.kernel.org, frederic.danis.oss@gmail.com Subject: [PATCH v3 0/2] ACPI serdev support Date: Wed, 11 Oct 2017 10:32:12 +0200 [thread overview] Message-ID: <1507710734-32520-1-git-send-email-frederic.danis.oss@gmail.com> (raw) Add ACPI support for serial attached devices. Currently, serial devices are not set as enumerated during ACPI scan for SPI or i2c buses (but not for UART). This should also be done for UART serial devices. I renamed *spi_i2c_slave* to *serial_bus_slave* to reflect this. This needs Johan Hovold's "serdev: fix registration of second slave" patch. Tested on T100TA with Broadcom BCM2E39. Since v2: - Remove ctrl->serdev set to NULL in acpi_serdev_register_device() in favor of Johan's patch - Fallback to ctrl->serdev check when acpi_walk_namespace() returns an error to prevent memory leak - Remove a change in dev_dbg() call in serdev_controller_add(), this will be done in separate patch Since v1: - Check if a serdev device as been allocated during acpi_walk_namespace() to prevent serdev controller registration instead of the tty-class device. - Reword dev_dbg() strings replacing Serial by serdev - Removing redundant "serdev%d" in dev_dbg() calls in serdev_controller_add() Since RFC: - Add or reword commit messages - Rename *serial_slave* to *serial_bus_slave* - Add specific check for Apple in acpi_is_serial_bus_slave(), thanks to Lukas Wunner - Update comment in acpi_default_enumeration() - Remove patch 3 "Bluetooth: hci_bcm: Add ACPI serdev support for BCM2E39" in favor of patches from Hans de Goede Frédéric Danis (2): serdev: Add ACPI support ACPI / scan: Fix enumeration for special UART devices drivers/acpi/scan.c | 37 ++++++++--------- drivers/tty/serdev/core.c | 100 +++++++++++++++++++++++++++++++++++++++++++--- include/acpi/acpi_bus.h | 2 +- 3 files changed, 113 insertions(+), 26 deletions(-) -- 2.7.4
next reply other threads:[~2017-10-11 8:32 UTC|newest] Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-10-11 8:32 Frédéric Danis [this message] 2017-10-11 8:32 ` [PATCH v3 0/2] ACPI serdev support Frédéric Danis [not found] ` <1507710734-32520-1-git-send-email-frederic.danis.oss-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 2017-10-11 8:32 ` [PATCH v3 1/2] serdev: Add ACPI support Frédéric Danis 2017-10-11 8:32 ` Frédéric Danis 2017-10-11 8:43 ` Johan Hovold 2017-10-11 8:32 ` [PATCH v3 2/2] ACPI / scan: Fix enumeration for special UART devices Frédéric Danis 2017-10-15 9:57 ` Lukas Wunner [not found] ` <1507710734-32520-3-git-send-email-frederic.danis.oss-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 2018-01-31 14:21 ` [v3,2/2] " Graeme Gregory 2018-01-31 14:21 ` Graeme Gregory 2018-02-02 10:03 ` Frédéric Danis 2018-02-02 10:03 ` Frédéric Danis 2018-02-02 15:28 ` Graeme Gregory 2017-10-11 9:03 ` [PATCH v3 0/2] ACPI serdev support Johan Hovold 2017-10-11 13:09 ` Rafael J. Wysocki 2017-10-11 13:09 ` Rafael J. Wysocki 2017-10-11 18:32 ` Marcel Holtmann 2017-10-11 18:32 ` Marcel Holtmann 2017-10-18 14:46 ` Frédéric Danis 2017-10-18 14:56 ` Greg Kroah-Hartman [not found] ` <20171018145608.GB27138-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org> 2017-10-19 14:23 ` Johan Hovold 2017-10-19 14:23 ` Johan Hovold 2017-10-19 14:26 ` Hans de Goede 2017-10-19 14:32 ` Marcel Holtmann 2017-10-19 14:56 ` Hans de Goede 2017-10-19 16:15 ` Loic Poulain [not found] ` <CAMZdPi_ZpNrdjJPnS_rQ9t3oYe6+en9oHQMjmMjx_xHL-WWZKQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2017-10-19 18:50 ` Hans de Goede 2017-10-19 18:50 ` Hans de Goede 2017-10-19 19:00 ` Marcel Holtmann [not found] ` <FFD85CA6-6BD9-45AA-B3C2-898212711C0C-kz+m5ild9QBg9hUCZPvPmw@public.gmane.org> 2017-10-19 19:05 ` Hans de Goede 2017-10-19 19:05 ` Hans de Goede 2017-10-21 9:59 ` Johan Hovold [not found] ` <877ea825-eec5-d982-f962-d67067749009-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2017-10-19 14:32 ` Johan Hovold 2017-10-19 14:32 ` Johan Hovold
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=1507710734-32520-1-git-send-email-frederic.danis.oss@gmail.com \ --to=frederic.danis.oss-re5jqeeqqe8avxtiumwx3w@public.gmane.org \ --cc=greg-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org \ --cc=hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \ --cc=johan-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \ --cc=linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=linux-bluetooth-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=linux-serial-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=loic.poulain-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \ --cc=lukas-JFq808J9C/izQB+pC5nmwQ@public.gmane.org \ --cc=marcel-kz+m5ild9QBg9hUCZPvPmw@public.gmane.org \ --cc=rafael-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \ --cc=robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \ --cc=sre-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.