linux-kernel - Re: [PATCH] dt-bindings: mtd: add new compatible strings for mtk nor controller

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, 3 Aug 2017 12:09:44 +0200
From:   Matthias Brugger <matthias.bgg@...il.com>
To:     Guochun Mao <guochun.mao@...iatek.com>,
        Cyrille Pitchen <cyrille.pitchen@...ev4u.fr>
Cc:     Boris Brezillon <boris.brezillon@...e-electrons.com>,
        Marek Vasut <marek.vasut@...il.com>,
        Cyrille Pitchen <cyrille.pitchen@...el.com>,
        Mark Rutland <mark.rutland@....com>,
        devicetree@...r.kernel.org, Richard Weinberger <richard@....at>,
        Russell King <linux@...linux.org.uk>,
        linux-kernel@...r.kernel.org, Rob Herring <robh+dt@...nel.org>,
        linux-mtd@...ts.infradead.org, linux-mediatek@...ts.infradead.org,
        David Woodhouse <dwmw2@...radead.org>,
        linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH] dt-bindings: mtd: add new compatible strings for mtk nor
 controller



On 08/03/2017 04:16 AM, Guochun Mao wrote:
> Hi Cyrille,
> 
> On Wed, 2017-08-02 at 17:31 +0200, Cyrille Pitchen wrote:
>> Hi Guochun,
>>
>> Le 02/08/2017 à 10:36, Boris Brezillon a écrit :
>>> Hi Matthias,
>>>
>>> On Tue, 1 Aug 2017 14:33:22 +0200
>>> Matthias Brugger <matthias.bgg@...il.com> wrote:
>>>
>>>> On 07/25/2017 04:10 AM, Guochun Mao wrote:
>>>>> Add "mediatek,mt2712-nor" and "mediatek,mt7622-nor"
>>>>> for nor flash node's compatible strings.
>>>>>
>>>>> Reviewed-by: Matthias Brugger <matthias.bgg@...il.com>
>>>>> Signed-off-by: Guochun Mao <guochun.mao@...iatek.com>
>>>>> ---
>>>>>    .../devicetree/bindings/mtd/mtk-quadspi.txt        |    2 ++
>>>>>    1 file changed, 2 insertions(+)
>>>>>
>>>>> diff --git a/Documentation/devicetree/bindings/mtd/mtk-quadspi.txt b/Documentation/devicetree/bindings/mtd/mtk-quadspi.txt
>>>>> index 5ded66a..9845ff4 100644
>>>>> --- a/Documentation/devicetree/bindings/mtd/mtk-quadspi.txt
>>>>> +++ b/Documentation/devicetree/bindings/mtd/mtk-quadspi.txt
>>>>> @@ -3,6 +3,8 @@
>>>>>    Required properties:
>>>>>    - compatible: 	  The possible values are:
>>>>>    		  "mediatek,mt2701-nor"
>>>>> +		  "mediatek,mt2712-nor"
>>>>> +		  "mediatek,mt7622-nor"
>>>>>    		  "mediatek,mt7623-nor"
>>>>>    		  "mediatek,mt8173-nor"
>>>>>    		  For mt8173, compatible should be "mediatek,mt8173-nor".
>>>>>
>>
>> There should be another patch to update the mtk_nor_of_ids[] array in
>> the mtk-quadspi.c driver, shouldn't it?
>>
>> Currently, I only see the "mediatek,mt8173-nor" compatible string at the
>> driver side. The driver and its documentation should be aligned and both
>> patches should be applied at the same time.
> 
>> Best regards,
>>
>> Cyrille
> 
> 
>  From my designer's info, these IPs' nor controller could use same
> driver, and I had verified it. I think there's no need to update
> mtk_nor_of_ids[], unless we find difference between these IPs(exactly
> compared with mt8173).
> How do you think?
>


Right, mt8173-nor is the fallback binding which will be used by the driver. As 
the documentation states, the compatible string should be something like:
"mediatek,mt7622-nor", "mediatek,mt8173-nor"

So the mt7622-nor binding is not matched in the driver, as the driver is the 
same on all SoCs. Nonetheless if we realize in the future that mt7622 IP has 
some special feature/bug, we can use the binding to implement that in the driver.

AFAIK this is the standard way for bindings. Cyrille, is this ok for you, or do 
want all bindings to be reflected in the driver even though they don't change 
any functionality?

Regards,
Matthias

> And, we will add dts patches for new SOCs, are these patches should
> applied with this documentation?
> 
> Best regards,
> Guochun
>