Message-ID: <9cfd5bc1-64e9-5250-5a8d-18ac4c205584@arinc9.com>
Date:   Mon, 20 Mar 2023 21:07:33 +0300
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.8.0
Subject: Re: [PATCH 01/10] dt: bindings: clock: add mtmips SoCs clock device
 tree binding documentation
Content-Language: en-US
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        Sergio Paracuellos <sergio.paracuellos@gmail.com>,
        linux-clk@vger.kernel.org
Cc:     linux-mips@vger.kernel.org, tsbogend@alpha.franken.de,
        john@phrozen.org, linux-kernel@vger.kernel.org,
        p.zabel@pengutronix.de, mturquette@baylibre.com, sboyd@kernel.org,
        robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
        matthias.bgg@gmail.com, devicetree@vger.kernel.org
References: <20230320161823.1424278-1-sergio.paracuellos@gmail.com>
 <20230320161823.1424278-2-sergio.paracuellos@gmail.com>
 <5109c01b-48bd-2854-3f42-bf8ef8b4a821@linaro.org>
From:   =?UTF-8?B?QXLEsW7DpyDDnE5BTA==?= <arinc.unal@arinc9.com>
In-Reply-To: <5109c01b-48bd-2854-3f42-bf8ef8b4a821@linaro.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk

On 20.03.2023 21:01, Krzysztof Kozlowski wrote:
> On 20/03/2023 17:18, Sergio Paracuellos wrote:
>> +properties:
>> +  compatible:
>> +    items:
>> +      - enum:
>> +          - ralink,rt2880-sysc
>> +          - ralink,rt3050-sysc
>> +          - ralink,rt3052-sysc
>> +          - ralink,rt3352-sysc
>> +          - ralink,rt3883-sysc
>> +          - ralink,rt5350-sysc
>> +          - ralink,mt7620-sysc
>> +          - ralink,mt7620a-sysc
>> +          - ralink,mt7628-sysc
>> +          - ralink,mt7688-sysc
> 
> One more comment - this and maybe other compatibles - have wrong vendor
> prefix. This is mediatek, not ralink.

This platform was acquired from Ralink by MediaTek. I couldn't change 
some existing ralink compatible strings to mediatek as Rob explained on 
my pinctrl patch series that we don't do that. The compatible strings on 
this patch series here are new but I'd rather keep the compatible 
strings ralink to keep things consistent.

Arınç