2024-01-12 13:45:16

by KyuHyuk Lee

[permalink] [raw]
Subject: [PATCH 1/2] dt-bindings: rockchip: Add Hardkernel ODROID-M1S board

Add device tree binding for Hardkernel ODROID-M1S board based on
RK3566 SoC.

Signed-off-by: KyuHyuk Lee <[email protected]>
---
Documentation/devicetree/bindings/arm/rockchip.yaml | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml
index 5cf5cbef2cf5..cc37d2118131 100644
--- a/Documentation/devicetree/bindings/arm/rockchip.yaml
+++ b/Documentation/devicetree/bindings/arm/rockchip.yaml
@@ -566,6 +566,11 @@ properties:
- const: rockchip,rk3568-odroid-m1
- const: rockchip,rk3568

+ - description: Hardkernel Odroid M1S
+ items:
+ - const: rockchip,rk3566-odroid-m1s
+ - const: rockchip,rk3566
+
- description: Hugsun X99 TV Box
items:
- const: hugsun,x99
--
2.34.1



2024-01-12 16:58:18

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH 1/2] dt-bindings: rockchip: Add Hardkernel ODROID-M1S board

On Fri, Jan 12, 2024 at 10:42:30PM +0900, KyuHyuk Lee wrote:
> Add device tree binding for Hardkernel ODROID-M1S board based on
> RK3566 SoC.
>
> Signed-off-by: KyuHyuk Lee <[email protected]>
> ---
> Documentation/devicetree/bindings/arm/rockchip.yaml | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml
> index 5cf5cbef2cf5..cc37d2118131 100644
> --- a/Documentation/devicetree/bindings/arm/rockchip.yaml
> +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml
> @@ -566,6 +566,11 @@ properties:
> - const: rockchip,rk3568-odroid-m1
> - const: rockchip,rk3568
>
> + - description: Hardkernel Odroid M1S

if the vendor for this board is hardkernel...

> + items:
> + - const: rockchip,rk3566-odroid-m1s

...why is the vendor prefix here rockchip?

Thanks,
Conor.

> + - const: rockchip,rk3566
> +
> - description: Hugsun X99 TV Box
> items:
> - const: hugsun,x99
> --
> 2.34.1
>


Attachments:
(No filename) (1.09 kB)
signature.asc (235.00 B)
Download all attachments

2024-01-12 17:02:26

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 1/2] dt-bindings: rockchip: Add Hardkernel ODROID-M1S board

On 12/01/2024 17:58, Conor Dooley wrote:
> On Fri, Jan 12, 2024 at 10:42:30PM +0900, KyuHyuk Lee wrote:
>> Add device tree binding for Hardkernel ODROID-M1S board based on
>> RK3566 SoC.
>>
>> Signed-off-by: KyuHyuk Lee <[email protected]>
>> ---
>> Documentation/devicetree/bindings/arm/rockchip.yaml | 5 +++++
>> 1 file changed, 5 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml
>> index 5cf5cbef2cf5..cc37d2118131 100644
>> --- a/Documentation/devicetree/bindings/arm/rockchip.yaml
>> +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml
>> @@ -566,6 +566,11 @@ properties:
>> - const: rockchip,rk3568-odroid-m1
>> - const: rockchip,rk3568
>>
>> + - description: Hardkernel Odroid M1S
>
> if the vendor for this board is hardkernel...
>
>> + items:
>> + - const: rockchip,rk3566-odroid-m1s
>
> ...why is the vendor prefix here rockchip?

Uh, good catch. I missed it when acking their earlier mistake year ago
:( Would be nice if they fixed that one too.

Best regards,
Krzysztof


2024-01-12 17:26:27

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH 1/2] dt-bindings: rockchip: Add Hardkernel ODROID-M1S board

On Fri, Jan 12, 2024 at 06:02:09PM +0100, Krzysztof Kozlowski wrote:
> On 12/01/2024 17:58, Conor Dooley wrote:
> > On Fri, Jan 12, 2024 at 10:42:30PM +0900, KyuHyuk Lee wrote:
> >> Add device tree binding for Hardkernel ODROID-M1S board based on
> >> RK3566 SoC.
> >>
> >> Signed-off-by: KyuHyuk Lee <[email protected]>
> >> ---
> >> Documentation/devicetree/bindings/arm/rockchip.yaml | 5 +++++
> >> 1 file changed, 5 insertions(+)
> >>
> >> diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml
> >> index 5cf5cbef2cf5..cc37d2118131 100644
> >> --- a/Documentation/devicetree/bindings/arm/rockchip.yaml
> >> +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml
> >> @@ -566,6 +566,11 @@ properties:
> >> - const: rockchip,rk3568-odroid-m1
> >> - const: rockchip,rk3568
> >>
> >> + - description: Hardkernel Odroid M1S
> >
> > if the vendor for this board is hardkernel...
> >
> >> + items:
> >> + - const: rockchip,rk3566-odroid-m1s
> >
> > ...why is the vendor prefix here rockchip?
>
> Uh, good catch. I missed it when acking their earlier mistake year ago
> :( Would be nice if they fixed that one too.

Maybe they will if they got your email, they did not get mine
apparently:
<[email protected]>: host mx02.mail.icloud.com[17.57.155.34] said: 554 5.7.1
[HM08] Message rejected due to local policy. Please visit
https://support.apple.com/en-us/HT204137 (in reply to end of DATA command)


Attachments:
(No filename) (1.52 kB)
signature.asc (235.00 B)
Download all attachments

2024-01-12 17:43:48

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 1/2] dt-bindings: rockchip: Add Hardkernel ODROID-M1S board

On 12/01/2024 18:26, Conor Dooley wrote:
>>> if the vendor for this board is hardkernel...
>>>
>>>> + items:
>>>> + - const: rockchip,rk3566-odroid-m1s
>>>
>>> ...why is the vendor prefix here rockchip?
>>
>> Uh, good catch. I missed it when acking their earlier mistake year ago
>> :( Would be nice if they fixed that one too.
>
> Maybe they will if they got your email, they did not get mine
> apparently:
> <[email protected]>: host mx02.mail.icloud.com[17.57.155.34] said: 554 5.7.1
> [HM08] Message rejected due to local policy. Please visit
> https://support.apple.com/en-us/HT204137 (in reply to end of DATA command)

Uh, what did you do to Apple to be denied by their policy? Admit, you
have an Android phone?

Best regards,
Krzysztof


2024-01-12 17:45:24

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 1/2] dt-bindings: rockchip: Add Hardkernel ODROID-M1S board

On 12/01/2024 14:42, KyuHyuk Lee wrote:
> Add device tree binding for Hardkernel ODROID-M1S board based on
> RK3566 SoC.
>
> Signed-off-by: KyuHyuk Lee <[email protected]>

Your system bounces emails from Conor, so just formality:

Fix the vendor prefix.

Best regards,
Krzysztof


2024-01-13 17:00:21

by KyuHyuk Lee

[permalink] [raw]
Subject: Re: [PATCH 1/2] dt-bindings: rockchip: Add Hardkernel ODROID-M1S board

On Fri, Jan 12, 2024 at 06:44:26PM +0100, Krzysztof Kozlowski wrote:
> On 12/01/2024 14:42, KyuHyuk Lee wrote:
> > Add device tree binding for Hardkernel ODROID-M1S board based on
> > RK3566 SoC.
> >
> > Signed-off-by: KyuHyuk Lee <[email protected]>
>
> Your system bounces emails from Conor, so just formality:
>
> Fix the vendor prefix.
>
> Best regards,
> Krzysztof
>

Thank you for reviewing.

Ok. Will fix it.

Thank you,
KyuHyuk Lee.

2024-01-15 16:45:27

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH 1/2] dt-bindings: rockchip: Add Hardkernel ODROID-M1S board

On Fri, Jan 12, 2024 at 06:43:33PM +0100, Krzysztof Kozlowski wrote:
> On 12/01/2024 18:26, Conor Dooley wrote:
> >>> if the vendor for this board is hardkernel...
> >>>
> >>>> + items:
> >>>> + - const: rockchip,rk3566-odroid-m1s
> >>>
> >>> ...why is the vendor prefix here rockchip?
> >>
> >> Uh, good catch. I missed it when acking their earlier mistake year ago
> >> :( Would be nice if they fixed that one too.
> >
> > Maybe they will if they got your email, they did not get mine
> > apparently:
> > <[email protected]>: host mx02.mail.icloud.com[17.57.155.34] said: 554 5.7.1
> > [HM08] Message rejected due to local policy. Please visit
> > https://support.apple.com/en-us/HT204137 (in reply to end of DATA command)
>
> Uh, what did you do to Apple to be denied by their policy? Admit, you
> have an Android phone?

I do. I guess that puts me on -1 social credit score. I have a macbook,
but I run linux on it, so I guess that is a +1 -0.5 sorta deal, leaving
me negative overall.

Either way, not sure what causes it. Maybe the fact that I sign all my
mail is a contributing factor, but I may not be the only @kernel.org
address that KyuHyuk Lee is not receiving mail from.


Attachments:
(No filename) (1.21 kB)
signature.asc (235.00 B)
Download all attachments