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

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

[Macpaul Lin]

Fix the missing description for Mediatek mt8365-evk board.

Fixes: 4f5fc078ac6f ("dt-bindings: arm64: dts: mediatek: Add mt8365-evk board")
Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
---
 Documentation/devicetree/bindings/arm/mediatek.yaml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/arm/mediatek.yaml b/Documentation/devicetree/bindings/arm/mediatek.yaml
index c2935e99626b..3ce39a346fb1 100644
--- a/Documentation/devicetree/bindings/arm/mediatek.yaml
+++ b/Documentation/devicetree/bindings/arm/mediatek.yaml
@@ -248,7 +248,8 @@ properties:
           - enum:
               - mediatek,mt8183-pumpkin
           - const: mediatek,mt8183
-      - items:
+      - description: MediaTek Genio 350 Boards (Genio 350 EVK)
+        items:
           - enum:
               - mediatek,mt8365-evk
           - const: mediatek,mt8365
-- 
2.18.0

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

[Krzysztof Kozlowski]

On 10/09/2023 08:28, Macpaul Lin wrote:
> Fix the missing description for Mediatek mt8365-evk board.
> 
> Fixes: 4f5fc078ac6f ("dt-bindings: arm64: dts: mediatek: Add mt8365-evk board")

There is no bug here. Drop.

> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
> ---
>  Documentation/devicetree/bindings/arm/mediatek.yaml | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/arm/mediatek.yaml b/Documentation/devicetree/bindings/arm/mediatek.yaml
> index c2935e99626b..3ce39a346fb1 100644
> --- a/Documentation/devicetree/bindings/arm/mediatek.yaml
> +++ b/Documentation/devicetree/bindings/arm/mediatek.yaml
> @@ -248,7 +248,8 @@ properties:
>            - enum:
>                - mediatek,mt8183-pumpkin
>            - const: mediatek,mt8183
> -      - items:
> +      - 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

Best regards,
Krzysztof

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

[Macpaul Lin]

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
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.

Thanks
Macpaul Lin

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

[Krzysztof Kozlowski]

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

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

[Macpaul Lin]

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.

>> 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.
Although they are all belongs to Genio EVK's product line.
This patch is not related to that v5 change for mt8395.

This is an independent patch for adding more detail description for 
mt83'6'5 and mt83'6'5-evk.

> Best regards,
> Krzysztof
> 

Thanks
Macpaul Lin

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

[Krzysztof Kozlowski]

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.

> 
>>> 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.


Best regards,
Krzysztof

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

[Macpaul Lin]

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

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

[Krzysztof Kozlowski]

On 11/09/2023 10:16, Macpaul Lin wrote:
> 
> 
> 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.

Then why the code uses enum?



Best regards,
Krzysztof

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

[Krzysztof Kozlowski]

On 10/09/2023 11:09, Krzysztof Kozlowski wrote:
> On 10/09/2023 08:28, Macpaul Lin wrote:
>> Fix the missing description for Mediatek mt8365-evk board.
>>
>> Fixes: 4f5fc078ac6f ("dt-bindings: arm64: dts: mediatek: Add mt8365-evk board")
> 
> There is no bug here. Drop.

So you decided to ignore this one?

Best regards,
Krzysztof