devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: "Macpaul Lin" <macpaul.lin@mediatek.com>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Matthias Brugger" <matthias.bgg@gmail.com>,
	"AngeloGioacchino Del Regno"
	<angelogioacchino.delregno@collabora.com>,
	"Bernhard Rosenkränzer" <bero@baylibre.com>,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-mediatek@lists.infradead.org
Cc: Bear Wang <bear.wang@mediatek.com>,
	Pablo Sun <pablo.sun@mediatek.com>,
	Macpaul Lin <macpaul@gmail.com>
Subject: Re: [PATCH] dt-bindings: arm64: dts: mediatek: add description for mt8365-evk board
Date: Mon, 11 Sep 2023 08:09:10 +0200	[thread overview]
Message-ID: <ef81697f-473e-898e-ec13-4dd255e2e3d7@linaro.org> (raw)
In-Reply-To: <12bdd17a-98f4-5551-4253-229dc36f0c77@mediatek.com>

On 10/09/2023 14:07, Macpaul Lin wrote:
> 
> 
> On 9/10/23 17:09, Krzysztof Kozlowski wrote:
>>> +      - description: MediaTek Genio 350 Boards (Genio 350 EVK)
>>
>> That's a bit confusing. So all of the boards here will be Genio 350 EVK?
>> I don't get what is the point of this description.
>>
>>
>>> +        items:
>>>            - enum:
>>>                - mediatek,mt8365-evk
>>>            - const: mediatek,mt8365
> 
> Thes boards with compatible 'mediatek,mt8365-evk' is indeed used by

Boards?

> Genio 350-EVK, which Baylibre helped to do upstream tasks. The only 
> production version of hardware will be available on the market. The 
> derived version of customer's hardware is not available yet.
> 
> The separate patch for renaming mt8365-evk.dts file should be send next 
> week for aligning the naming rules.

So the description is not correct.

Anyway, this patch does not make much sense to me considering nothing
like this was merged. You just sent v5 doing the same!

Best regards,
Krzysztof


  reply	other threads:[~2023-09-11  6:09 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-09-10  6:28 [PATCH] dt-bindings: arm64: dts: mediatek: add description for mt8365-evk board Macpaul Lin
2023-09-10  9:09 ` Krzysztof Kozlowski
2023-09-10 12:07   ` Macpaul Lin
2023-09-11  6:09     ` Krzysztof Kozlowski [this message]
2023-09-11  6:34       ` Macpaul Lin
2023-09-11  7:06         ` Krzysztof Kozlowski
2023-09-11  8:16           ` Macpaul Lin
2023-09-11  9:47             ` Krzysztof Kozlowski
2023-09-11  9:49   ` Krzysztof Kozlowski

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ef81697f-473e-898e-ec13-4dd255e2e3d7@linaro.org \
    --to=krzysztof.kozlowski@linaro.org \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=bear.wang@mediatek.com \
    --cc=bero@baylibre.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=macpaul.lin@mediatek.com \
    --cc=macpaul@gmail.com \
    --cc=matthias.bgg@gmail.com \
    --cc=pablo.sun@mediatek.com \
    --cc=robh+dt@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).