mbox series

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

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

Message

Alan Cooper March 18, 2021, 7:11 p.m. UTC
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: bf152b0b41dc141c8d32eb6e974408f5804f4d00

Comments

Rob Herring (Arm) March 23, 2021, 10:36 p.m. UTC | #1
On Thu, 18 Mar 2021 15:11:30 -0400, Al Cooper wrote:
> Add DT bindings for the Broadcom 8250 based UART driver. This

> UART is based on an 8250 but adds additional functionality. The

> additional features include the ability to use DMA for transfers and

> a baud rate clock system that is more accurate at high baud rates.

> This UART is backward compatible with the standard 8250 UART.

> 

> Signed-off-by: Al Cooper <alcooperx@gmail.com>

> ---

>  .../bindings/serial/brcm,bcm7271-uart.yaml    | 95 +++++++++++++++++++

>  1 file changed, 95 insertions(+)

>  create mode 100644 Documentation/devicetree/bindings/serial/brcm,bcm7271-uart.yaml

> 


Reviewed-by: Rob Herring <robh@kernel.org>