linux-kernel - Re: [PATCH v2 2/2] serial: 8250: Add new 8250-core based Broadcom STB driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAGh=XABU=jdLWo8AfSBZPwn+_gDfCNN07cg9a58nxknjjyEfxg@mail.gmail.com>
Date:   Mon, 18 Jan 2021 15:32:57 -0500
From:   Al Cooper <al.cooper@...adcom.com>
To:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc:     Al Cooper <alcooperx@...il.com>, linux-kernel@...r.kernel.org,
        BCM Kernel Feedback <bcm-kernel-feedback-list@...adcom.com>,
        "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
        <devicetree@...r.kernel.org>, Jiri Slaby <jirislaby@...nel.org>,
        linux-serial@...r.kernel.org,
        "open list:USB SUBSYSTEM" <linux-usb@...r.kernel.org>,
        Masahiro Yamada <yamada.masahiro@...ionext.com>,
        Rob Herring <robh+dt@...nel.org>
Subject: Re: [PATCH v2 2/2] serial: 8250: Add new 8250-core based Broadcom STB driver

On Mon, Jan 18, 2021 at 12:45 PM Greg Kroah-Hartman
<gregkh@...uxfoundation.org> wrote:
>
> On Fri, Jan 15, 2021 at 04:15:43PM -0500, Al Cooper wrote:
> > Add a UART driver for the new Broadcom 8250 based STB UART. The new
> > UART is backward compatible with the standard 8250, but has some
> > additional features. The new features include a high accuracy baud
> > rate clock system and DMA support.
> >
> > The driver will use the new optional BAUD MUX clock to select the best
> > one of the four master clocks (81MHz, 108MHz, 64MHz and 48MHz) to feed
> > the baud rate selection logic for any requested baud rate.  This allows
> > for more accurate BAUD rates when high speed baud rates are selected.
> >
> > The driver will use the new UART DMA hardware if the UART DMA registers
> > are specified in Device Tree "reg" property. The DMA functionality can
> > be disabled on kernel boot with the argument:
> > "8250_bcm7271.disable_dma=Y".
>
> Shouldn't that be on a per-device basis, and not a per-driver basis?

There is only one instance of the UART DMA hardware and it gets muxed
to just one of the possible UARTS.

>
> And why would you want to disable this, if you have support for this in
> the DT?  Why not just rely on the DT setting?

The DMA feature is used when the UART is connected to a Bluetooth
controller and the BAUD rate is typically 2-3Mbs. The ability to
easily disable DMA is very useful when debugging BT communication
problems in the field. DT settings could also be used to disable DMA,
but knowing the correct modifications to the "reg" and "reg-names"
properties is a lot more complicated.

Thanks
Al

>
> thanks,
>
> greg k-h

Download attachment "smime.p7s" of type "application/pkcs7-signature" (4157 bytes)