linux-kernel - Re: [PATCH v2 2/2] arm64: dts: mediatek: add mt8173 elm and hana 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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <df0824f0-b546-89fb-e219-a47ad83a0c62@gmail.com>
Date:   Wed, 15 Jan 2020 17:14:09 +0100
From:   Matthias Brugger <matthias.bgg@...il.com>
To:     Hsin-Yi Wang <hsinyi@...omium.org>
Cc:     "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" 
        <linux-arm-kernel@...ts.infradead.org>,
        Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        Devicetree List <devicetree@...r.kernel.org>,
        lkml <linux-kernel@...r.kernel.org>,
        linux-mediatek@...ts.infradead.org,
        Nicolas Boichat <drinkcat@...omium.org>,
        Enric Balletbo i Serra <enric.balletbo@...labora.com>,
        Daniel Kurtz <djkurtz@...omium.org>
Subject: Re: [PATCH v2 2/2] arm64: dts: mediatek: add mt8173 elm and hana
 board



On 15/01/2020 09:53, Hsin-Yi Wang wrote:
> On Tue, Jan 14, 2020 at 11:10 PM Matthias Brugger
> <matthias.bgg@...il.com> wrote:
>>
>>
>>
>> On 13/01/2020 19:01, Hsin-Yi Wang wrote:
>>> On Fri, Jan 10, 2020 at 9:42 PM Matthias Brugger <matthias.bgg@...il.com> wrote:
>>>>
>>>>
>>> Thanks, I will address the comments and send next version
>>>>
>>>> On 10/01/2020 08:37, Hsin-Yi Wang wrote:
>>>>> Elm is Acer Chromebook R13. Hana is Lenovo Chromebook. Both uses mt8173
>>>>> SoC.
>>>>>
>>>>> Signed-off-by: Hsin-Yi Wang <hsinyi@...omium.org>
>>>>> ---
>>>>> Changes in v2:
>>>>> - remove downstream nodes and unused nodes
>>>>> - use GPIO_ACTIVE_LOW for ps8640 gpios
>>>>> - move trackpad to hana
>>>>> ---
>>>>>  arch/arm64/boot/dts/mediatek/Makefile         |    3 +
>>>>>  .../dts/mediatek/mt8173-elm-hana-rev7.dts     |   27 +
>>>>>  .../boot/dts/mediatek/mt8173-elm-hana.dts     |   16 +
>>>>>  .../boot/dts/mediatek/mt8173-elm-hana.dtsi    |   60 +
>>>>>  arch/arm64/boot/dts/mediatek/mt8173-elm.dts   |   15 +
>>>>>  arch/arm64/boot/dts/mediatek/mt8173-elm.dtsi  | 1040 +++++++++++++++++
>>>>>  6 files changed, 1161 insertions(+)
>>>>>  create mode 100644 arch/arm64/boot/dts/mediatek/mt8173-elm-hana-rev7.dts
>>>>>  create mode 100644 arch/arm64/boot/dts/mediatek/mt8173-elm-hana.dts
>>>>>  create mode 100644 arch/arm64/boot/dts/mediatek/mt8173-elm-hana.dtsi
>>>>>  create mode 100644 arch/arm64/boot/dts/mediatek/mt8173-elm.dts
>>>>>  create mode 100644 arch/arm64/boot/dts/mediatek/mt8173-elm.dtsi
>>>>>

[...]

>>>>> +                     compatible = "mediatek,mt6397-rtc";
>>>>> +             };
>>>>> +
>>>>> +             syscfg_pctl_pmic: syscfg_pctl_pmic@...0 {
>>>>> +                     compatible = "mediatek,mt6397-pctl-pmic-syscfg",
>>>>> +                                  "syscon";
>>>>> +                     reg = <0 0x0000c000 0 0x0108>;
>>>>
>>>> not an upstream compatible, please delete the node.
>>>>
>>
>> I think my comment was not correct. I'll have to dig more to understand why we
>> have a compatible without a binding description.
>>
> Here's a similar syscon node that is in a binding document's example:
> 
> syscfg_pctl_a: syscfg-pctl-a@...05000 {
>         compatible = "mediatek,mt8135-pctl-a-syscfg", "syscon";
>         reg = <0 0x10005000 0 0x1000>;
> };
> 
> https://elixir.bootlin.com/linux/latest/source/Documentation/devicetree/bindings/pinctrl/pinctrl-mt65xx.txt#L81
> 

Yes I know, that's why I said that my first comment was not correct. Just leave
it in.

Regards,
Matthias