linux-kernel - Re: [PATCH v3 00/10] update Facebook Bletchley BMC

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: <CACPK8Xfs34gw2_XGeduJ6D=-khN7RtMj3LY8hQ8B96fFxMuE8A@mail.gmail.com>
Date:   Wed, 16 Feb 2022 05:01:56 +0000
From:   Joel Stanley <joel@....id.au>
To:     Potin Lai <potin.lai@...ntatw.com>
Cc:     Rob Herring <robh+dt@...nel.org>, Andrew Jeffery <andrew@...id.au>,
        Patrick Williams <patrick@...cx.xyz>,
        devicetree <devicetree@...r.kernel.org>,
        Linux ARM <linux-arm-kernel@...ts.infradead.org>,
        linux-aspeed <linux-aspeed@...ts.ozlabs.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v3 00/10] update Facebook Bletchley BMC

On Tue, 15 Feb 2022 at 16:32, Potin Lai <potin.lai@...ntatw.com> wrote:
>
> This patch series update Facebook Bletchley BMC devicetree base on EVT HW
> schematioc design, and rebase SLED numbering to 1-based for OpenBMC
> multi-host.
>
> - GPIO:
>   - adding more gpio line names
>   - include interrupt line in io expander for gpio interrupt monitoring
>
> - SPI flash:
>   - adding dual flash BSM module support
>   - switch to spi2-gpio on spi2 due to unstable signal issue
>
> - Hwmon Sensors:
>   - adding INA230 sensors for monitoring
>   - fix ADM1278 shunt-resistor
>
> - MDIO Bus: enable mido3 bus
>
> - RTC: switch to external battery-backed rtc
>
> - OpenBMC: 1-based SLED numbering
>
>
> LINK: [v2] https://lore.kernel.org/all/20220214042538.12132-1-potin.lai@quantatw.com/
> LINK: [v1] https://lore.kernel.org/all/20220211014347.24841-1-potin.lai@quantatw.com/
>
>
> Changes v2 --> v3:
> - update patch name prefix
> - Disable spi2, and create a new node call spi2_gpio

Thanks, the patches look good.

Reviewed-by: Joel Stanley <joel@....id.au>

and given the minor differences from v2, lets add Patrick's review:

Reviewed-by: Patrick Williams <patrick@...cx.xyz>

I've applied these for submission in the v5.18 merge window, and to
the openbmc tree.


>
> Changes v1 --> v2:
> - update the details of new added gpio line names in commit message
> - add battery-backed rtc information in comment and commit message
>
>
> Potin Lai (10):
>   ARM: dts: aspeed: bletchley: switch sled numbering to 1-based
>   ARM: dts: aspeed: bletchley: separate leds into multiple groups
>   ARM: dts: aspeed: bletchley: update gpio-line-names
>   ARM: dts: aspeed: bletchley: update fmc configurations
>   ARM: dts: aspeed: bletchley: switch to spi-gpio for spi2
>   ARM: dts: aspeed: bletchley: add interrupt support for sled io
>     expander
>   ARM: dts: aspeed: bletchley: add shunt-resistor for ADM1278
>   ARM: dts: aspeed: bletchley: add INA230 sensor on each sled
>   ARM: dts: aspeed: bletchley: enable mdio3 bus
>   ARM: dts: aspeed: bletchley: cleanup redundant nodes
>
>  .../dts/aspeed-bmc-facebook-bletchley.dts     | 320 +++++++++++-------
>  1 file changed, 202 insertions(+), 118 deletions(-)
>
> --
> 2.17.1
>