Re: [PATCH] dt-bindings: arm64: dts: mediatek: add description for mt8365-evk board

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 9/11/23 15:06, Krzysztof Kozlowski wrote:
	

External email : Please do not click links or open attachments until you have verified the sender or the content.

On 11/09/2023 08:34, Macpaul Lin wrote:
On 9/11/23 14:09, Krzysztof Kozlowski wrote:


External email : Please do not click links or open attachments until you have verified the sender or the content.

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?

I'm not sure if there will be more hardware version released to market.
Current the p1v3 is the final version of mt8365-evk.

Then your code (enum) does not match description. The code is saying:
this is for multiple boards based on mt8365.

Your description is saying: this is only for 350 boards.

Now your email is saying: there is only one 350 board.

Three different versions. Sorry, I am confused.


Okay, this description has been modified to support single board.
Since there is only one type of the hardware is available on the market.


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!


Sorry I'm confused now.
These are independent patches.
This patch is for SOC mt83'6'5 and board mt8365-evk, which is not relate SOC mt83'9'5 and board mt8395-evk.

OK, I see my confusion. mt8365 and mt8395.

Still the description is not matching the code, at least to my limited
understanding of the description.


patch v2 of this change has been send and rebase with that v5 change.
This will generate a different base line for the v2 patch for avoiding confusion. :)


Best regards,
Krzysztof


Thanks
Macpaul Lin



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux