diff mbox series

[1/2] dt-bindings: rockchip: Add Hardkernel ODROID-M1S board

Message ID 20240112134230.28055-1-lee@kyuhyuk.kr (mailing list archive)
State New
Headers show
Series [1/2] dt-bindings: rockchip: Add Hardkernel ODROID-M1S board | expand

Commit Message

KyuHyuk Lee Jan. 12, 2024, 1:42 p.m. UTC
Add device tree binding for Hardkernel ODROID-M1S board based on
RK3566 SoC.

Signed-off-by: KyuHyuk Lee <lee@kyuhyuk.kr>
---
 Documentation/devicetree/bindings/arm/rockchip.yaml | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Conor Dooley Jan. 12, 2024, 4:58 p.m. UTC | #1
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 <lee@kyuhyuk.kr>
> ---
>  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
>
Krzysztof Kozlowski Jan. 12, 2024, 5:02 p.m. UTC | #2
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 <lee@kyuhyuk.kr>
>> ---
>>  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
Conor Dooley Jan. 12, 2024, 5:26 p.m. UTC | #3
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 <lee@kyuhyuk.kr>
> >> ---
> >>  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:
<lee@kyuhyuk.kr>: 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)
Krzysztof Kozlowski Jan. 12, 2024, 5:43 p.m. UTC | #4
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:
> <lee@kyuhyuk.kr>: 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
Krzysztof Kozlowski Jan. 12, 2024, 5:44 p.m. UTC | #5
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 <lee@kyuhyuk.kr>

Your system bounces emails from Conor, so just formality:

Fix the vendor prefix.

Best regards,
Krzysztof
KyuHyuk Lee Jan. 13, 2024, 4:59 p.m. UTC | #6
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 <lee@kyuhyuk.kr>
> 
> 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.
Conor Dooley Jan. 15, 2024, 4:45 p.m. UTC | #7
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:
> > <lee@kyuhyuk.kr>: 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.
diff mbox series

Patch

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