linux-kernel - Re: [PATCH v3 1/2] dt-bindings: nvmem: mediatek: add support for MediaTek mt8192 SoC

Open Source and information security mailing list archives

Message-ID: <8a6f9493-a17a-e679-bada-1dd355c9ce8d@gmail.com>
Date:   Sun, 31 Jan 2021 14:57:01 +0100
From:   Matthias Brugger <matthias.bgg@...il.com>
To:     mtk23264 <Yz.Wu@...iatek.com>, Rob Herring <robh@...nel.org>
Cc:     devicetree@...r.kernel.org,
        Andrew-CT Chen <andrew-ct.chen@...iatek.com>,
        Seiya Wang <seiya.wang@...iatek.com>,
        linux-kernel@...r.kernel.org, linux-mediatek@...ts.infradead.org,
        Srinivas Kandagatla <srinivas.kandagatla@...aro.org>,
        linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v3 1/2] dt-bindings: nvmem: mediatek: add support for
 MediaTek mt8192 SoC

On 22/01/2021 07:28, mtk23264 wrote:
> On Sun, 2021-01-03 at 09:25 -0700, Rob Herring wrote:
>> On Mon, Dec 21, 2020 at 02:10:19PM +0800, Yz.Wu@...iatek.com wrote:
>>> From: Ryan Wu <Yz.Wu@...iatek.com>
>>>
>>> This updates dt-binding documentation for MediaTek mt8192
>>>
>>> Signed-off-by: Ryan Wu <Yz.Wu@...iatek.com>
>>> ---
>>> This patch is based on v5.10-rc7.
>>> ---
>>>  Documentation/devicetree/bindings/nvmem/mtk-efuse.txt | 1 +
>>>  1 file changed, 1 insertion(+)
>>>
>>> diff --git a/Documentation/devicetree/bindings/nvmem/mtk-efuse.txt b/Documentation/devicetree/bindings/nvmem/mtk-efuse.txt
>>> index 0668c45a156d..e2f0c0f34d10 100644
>>> --- a/Documentation/devicetree/bindings/nvmem/mtk-efuse.txt
>>> +++ b/Documentation/devicetree/bindings/nvmem/mtk-efuse.txt
>>> @@ -7,6 +7,7 @@ Required properties:
>>>  	      "mediatek,mt7622-efuse", "mediatek,efuse": for MT7622
>>>  	      "mediatek,mt7623-efuse", "mediatek,efuse": for MT7623
>>>  	      "mediatek,mt8173-efuse" or "mediatek,efuse": for MT8173
>>> +	      "mediatek,mt8192-efuse" or "mediatek,efuse": for MT8192
>>
>> No, "mediatek,efuse" by itself is only for MT8173.
> Is it should be modify from "mediatek,mt8192-efuse" or "mediatek,efuse"
> to "mediatek,mt8192-efuse", "mediatek,efuse" ?
> 

Yes, as you can see "mediatek,mt8192-efuse" is not defined in the driver (and
should not as long as the HW has no difference from the already implemented driver).

Regards,
Matthias

> Regards,
> Yz
>>
>>>  - reg: Should contain registers location and length
>>>  
>>>  = Data cells =
>>> -- 
>>> 2.18.0
>>>
>>
>> _______________________________________________
>> Linux-mediatek mailing list
>> Linux-mediatek@...ts.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-mediatek
> 

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