linux-kernel - Re: [PATCH] ARM: dts: vf610: Add ZII SPB4 board

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 for Android: free password hash cracker in your pocket

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

Message-ID: <CAOMZO5DwA0jgfwOgvgWUq1j9qFQUTHz+SVcD3oPdUhkwAfG=GQ@mail.gmail.com>
Date:   Tue, 12 Mar 2019 10:20:10 -0300
From:   Fabio Estevam <festevam@...il.com>
To:     Andrey Smirnov <andrew.smirnov@...il.com>
Cc:     Shawn Guo <shawnguo@...nel.org>, Chris Healy <cphealy@...il.com>,
        Andrew Lunn <andrew@...n.ch>,
        Heiner Kallweit <hkallweit1@...il.com>,
        "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" 
        <linux-arm-kernel@...ts.infradead.org>,
        linux-kernel <linux-kernel@...r.kernel.org>,
        NXP Linux Team <linux-imx@....com>
Subject: Re: [PATCH] ARM: dts: vf610: Add ZII SPB4 board

Hi Andrey,

On Mon, Mar 11, 2019 at 3:49 PM Andrey Smirnov <andrew.smirnov@...il.com> wrote:

> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index f4f5aeaf3298..035ad9fc49f3 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -607,7 +607,8 @@ dtb-$(CONFIG_SOC_VF610) += \
>         vf610-zii-dev-rev-c.dtb \
>         vf610-zii-scu4-aib.dtb \
>         vf610-zii-ssmb-dtu.dtb \
> -       vf610-zii-ssmb-spu3.dtb
> +       vf610-zii-ssmb-spu3.dtb \
> +       vf610-zii-spb4.dtb

Please keep it in alphabetical order.

> +       gpio-leds {
> +               compatible = "gpio-leds";
> +               pinctrl-0 = <&pinctrl_leds_debug>;
> +               pinctrl-names = "default";
> +
> +               led-debug {
> +                       label = "zii:green:debug1";
> +                       gpios = <&gpio2 18 GPIO_ACTIVE_HIGH>;
> +                       linux,default-trigger = "heartbeat";
> +                       max-brightness = <1>;

This is an invalid property for gpio-leds. It is used for leds pwm
only. Please remove it.

> +               };
> +       };
> +
> +       reg_vcc_3v3_mcu: regulator {

reg_vcc_3v3_mcu: regulator-vcc-3v3-mcu  {

> +&dspi1 {
> +       bus-num = <1>;
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&pinctrl_dspi1>;
> +       status = "okay";
> +
> +       m25p128@0 {

Node names should be generic:

spi-nor@0

> +&i2c0 {
> +       clock-frequency = <100000>;
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&pinctrl_i2c0>;
> +       status = "okay";
> +
> +       gpio6: pca9505@22 {

io-expander@22

> +               compatible = "nxp,pca9554";
> +               reg = <0x22>;
> +               gpio-controller;
> +               #gpio-cells = <2>;
> +       };
> +
> +       at24c04@50 {

eeprom@50

> +               compatible = "atmel,24c04";
> +               reg = <0x50>;
> +               label = "nameplate";
> +       };
> +
> +       at24c04@52 {

eeprom@52