mbox series

[v7,0/2] 8250: Add driver for Broadcom UART

Message ID 20210325185256.16156-1-alcooperx@gmail.com
Headers show
Series 8250: Add driver for Broadcom UART | expand

Message

Alan Cooper March 25, 2021, 6:52 p.m. UTC 
v7 - Change Kconfig for SERIAL_8250_BCM7271 from "bool" to "tristate"
     so the driver can be built as a module.

v6 - A few more changes based on requests from Greg Kroah-Hartman
     - Don't save the debugfs file handle in priv because it was
       never used.
     - Change all dev_info() messages to dev_dbg() messages.
   - Fix warnings from the kernel test robot
   - Use --base with "git format-patch"
   - Fixes in the .yaml file based on requests from Rob Herring
     - Change "oneOf" to "minItems: 1".
     - Reference serial.yaml instead of using "$nodname".
     - Add missing newline.

v5 - Changes based on requests from Greg Kroah-Hartman
     - Move sysfs stats to debugfs.
     - Merge the .h file into the .c file.
     - Convert "flags" with only one flag to bool
     - Remove most dev_dbg() messages. Some can be replaced by using
       ftrace and some would cause a crash if the driver was used
       for the console.

v4 - Fix MAINTAINERS
   - Make all changes requested by Jira Slaby. The only functional
     change was to handle errors returned by brcmuart_arbitration()
     in brcmuart_resume().

v3 - remove "disable_dma" module param because it can be done
     by modifying the device tree node instead. Reduce size by
     removing some debug fuctionality that was no longer used.
   - Fix error from yaml compiler in bindings

v2 - remove the patch that modified 8250_of.c to keep it from
     registering before this driver when this driver was deferred
     as it was getting it's "clocks". This was fixed by changing
     the Device Tree entry to remove "clock-frequency". This results
     in both drivers getting "clocks" and getting same the deferral.


Al Cooper (2):
  dt-bindings: Add support for the Broadcom UART driver
  serial: 8250: Add new 8250-core based Broadcom STB driver

 .../bindings/serial/brcm,bcm7271-uart.yaml    |   95 ++
 MAINTAINERS                                   |    8 +
 drivers/tty/serial/8250/8250_bcm7271.c        | 1202 +++++++++++++++++
 drivers/tty/serial/8250/Kconfig               |   10 +
 drivers/tty/serial/8250/Makefile              |    1 +
 5 files changed, 1316 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/serial/brcm,bcm7271-uart.yaml
 create mode 100644 drivers/tty/serial/8250/8250_bcm7271.c


base-commit: e138138003eb3b3d06cc91cf2e8c5dec77e2a31e

Comments

Greg Kroah-Hartman March 26, 2021, 2:18 p.m. UTC | #1 
On Thu, Mar 25, 2021 at 02:52:54PM -0400, Al Cooper wrote:
> v7 - Change Kconfig for SERIAL_8250_BCM7271 from "bool" to "tristate"
>      so the driver can be built as a module.

Much better, thanks for sticking with this, now queued up.

greg k-h