linux-kernel - Re: [PATCH v2 2/4] dt-bindings: bus: Add Baikal-T1 APB-bus binding

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]

Date:   Thu, 14 May 2020 22:13:59 -0500
From:   Rob Herring <robh@...nel.org>
To:     Serge Semin <Sergey.Semin@...kalelectronics.ru>
Cc:     Paul Burton <paulburton@...nel.org>, soc@...nel.org,
        Alexey Malahov <Alexey.Malahov@...kalelectronics.ru>,
        Linus Walleij <linus.walleij@...aro.org>,
        Ralf Baechle <ralf@...ux-mips.org>,
        linux-kernel@...r.kernel.org, Jeffrey Hugo <jhugo@...eaurora.org>,
        linux-mips@...r.kernel.org, Serge Semin <fancer.lancer@...il.com>,
        Tero Kristo <t-kristo@...com>, Arnd Bergmann <arnd@...db.de>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Thomas Bogendoerfer <tsbogend@...ha.franken.de>,
        Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>,
        Olof Johansson <olof@...om.net>, devicetree@...r.kernel.org,
        Tony Lindgren <tony@...mide.com>,
        Rob Herring <robh+dt@...nel.org>
Subject: Re: [PATCH v2 2/4] dt-bindings: bus: Add Baikal-T1 APB-bus binding

On Fri, 8 May 2020 01:41:14 +0300, Serge Semin wrote:
> Baikal-T1 CPU or DMAC MMIO requests are handled by the AMBA 3 AXI
> Interconnect which routes them to the AXI-APB bridge, which in turn
> serializes accesses and routes them to the corresponding APB slave device.
> This binding describes the AXI-APB bridge considered as the APB-bus. It is
> supposed to be compatible with "be,bt1-apb" and "simple-bus" drivers,
> should be equipped with EHB MMIO region and a region with no slave device
> mapped, interrupts line number, APB reference clock and domain reset line.
> 
> Signed-off-by: Serge Semin <Sergey.Semin@...kalelectronics.ru>
> Cc: Alexey Malahov <Alexey.Malahov@...kalelectronics.ru>
> Cc: Paul Burton <paulburton@...nel.org>
> Cc: Ralf Baechle <ralf@...ux-mips.org>
> Cc: Tony Lindgren <tony@...mide.com>
> Cc: Tero Kristo <t-kristo@...com>
> Cc: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
> Cc: Jeffrey Hugo <jhugo@...eaurora.org>
> Cc: Linus Walleij <linus.walleij@...aro.org>
> Cc: Olof Johansson <olof@...om.net>
> Cc: linux-mips@...r.kernel.org
> Cc: soc@...nel.org
> 
> ---
> 
> Rob, I had to remove your Reviewed-by tag, since new changes had been
> introduced.
> 
> Changelog v2:
> - Move driver to the bus subsystem.
> - Don't use a multi-arg clock phandle reference in the examples dt-bindings
>   property. Thus redundant include statement can be removed.
> - Use dual GPL/BSD license.
> - Use single lined copyright header.
> - Lowercase the unit-address.
> - Convert a dedicated EHB block binding to the Baikal-T1 APB-bus one.
> - Add APB reference clock and reset support.
> - Replace "additionalProperties: false" property with
>   "unevaluatedProperties: false".
> - Add reg-names property.
> ---
>  .../bindings/bus/baikal,bt1-apb.yaml          | 90 +++++++++++++++++++
>  1 file changed, 90 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/bus/baikal,bt1-apb.yaml
> 

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