diff mbox series

[v2,1/2] dt-bindings: arm: amlogic: add S805Y and Mi TV Stick

Message ID 20250201193044.28856-2-funderscore@postmarketos.org (mailing list archive)
State New
Headers show
Series Add support for Xiaomi Mi TV Stick | expand

Commit Message

Ferass El Hafidi Feb. 1, 2025, 7:30 p.m. UTC
From: Ferass El Hafidi <funderscore@postmarketos.org>

Add support for the Amlogic S805Y SoC and the Xiaomi Mi TV Stick (aka.
xiaomi-aquaman).  The S805Y is very similar to the S805X, with just a
few minor differences.

Signed-off-by: Ferass El Hafidi <funderscore@postmarketos.org>
---
 Documentation/devicetree/bindings/arm/amlogic.yaml | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Krzysztof Kozlowski Feb. 2, 2025, 1:54 p.m. UTC | #1
On Sat, Feb 01, 2025 at 07:30:43PM +0000, Ferass El Hafidi wrote:
> From: Ferass El Hafidi <funderscore@postmarketos.org>
> 
> Add support for the Amlogic S805Y SoC and the Xiaomi Mi TV Stick (aka.
> xiaomi-aquaman).  The S805Y is very similar to the S805X, with just a
> few minor differences.
> 
> Signed-off-by: Ferass El Hafidi <funderscore@postmarketos.org>

Please run checkpatch. You have from/SoB mismatch.


> ---
>  Documentation/devicetree/bindings/arm/amlogic.yaml | 7 +++++++
>  1 file changed, 7 insertions(+)

Best regards,
Krzysztof
Ferass El Hafidi Feb. 2, 2025, 1:55 p.m. UTC | #2
Hi,

On Sun Feb 2, 2025 at 1:54 PM UTC, Krzysztof Kozlowski wrote:
> On Sat, Feb 01, 2025 at 07:30:43PM +0000, Ferass El Hafidi wrote:
> > From: Ferass El Hafidi <funderscore@postmarketos.org>
> > 
> > Add support for the Amlogic S805Y SoC and the Xiaomi Mi TV Stick (aka.
> > xiaomi-aquaman).  The S805Y is very similar to the S805X, with just a
> > few minor differences.
> > 
> > Signed-off-by: Ferass El Hafidi <funderscore@postmarketos.org>
>
> Please run checkpatch. You have from/SoB mismatch.

I don't see any mismatch.  Maybe it complains because of a spurious
non-breaking space?

Regards.
Krzysztof Kozlowski Feb. 2, 2025, 2:07 p.m. UTC | #3
On 02/02/2025 14:55, Ferass El Hafidi wrote:
> Hi,
> 
> On Sun Feb 2, 2025 at 1:54 PM UTC, Krzysztof Kozlowski wrote:
>> On Sat, Feb 01, 2025 at 07:30:43PM +0000, Ferass El Hafidi wrote:
>>> From: Ferass El Hafidi <funderscore@postmarketos.org>
>>>
>>> Add support for the Amlogic S805Y SoC and the Xiaomi Mi TV Stick (aka.
>>> xiaomi-aquaman).  The S805Y is very similar to the S805X, with just a
>>> few minor differences.
>>>
>>> Signed-off-by: Ferass El Hafidi <funderscore@postmarketos.org>
>>
>> Please run checkpatch. You have from/SoB mismatch.
> 
> I don't see any mismatch.  Maybe it complains because of a spurious
> non-breaking space?
> 
Could be, no clue. I copied them to editor and it also sees mismatch -
both entries just do not match.

Best regards,
Krzysztof
Ferass El Hafidi Feb. 2, 2025, 2:07 p.m. UTC | #4
On Sun Feb 2, 2025 at 2:07 PM UTC, Krzysztof Kozlowski wrote:
> On 02/02/2025 14:55, Ferass El Hafidi wrote:
> > Hi,
> > 
> > On Sun Feb 2, 2025 at 1:54 PM UTC, Krzysztof Kozlowski wrote:
> >> On Sat, Feb 01, 2025 at 07:30:43PM +0000, Ferass El Hafidi wrote:
> >>> From: Ferass El Hafidi <funderscore@postmarketos.org>
> >>>
> >>> Add support for the Amlogic S805Y SoC and the Xiaomi Mi TV Stick (aka.
> >>> xiaomi-aquaman).  The S805Y is very similar to the S805X, with just a
> >>> few minor differences.
> >>>
> >>> Signed-off-by: Ferass El Hafidi <funderscore@postmarketos.org>
> >>
> >> Please run checkpatch. You have from/SoB mismatch.
> > 
> > I don't see any mismatch.  Maybe it complains because of a spurious
> > non-breaking space?
> > 
> Could be, no clue. I copied them to editor and it also sees mismatch -
> both entries just do not match.

OK, I'll see what I can do.

>
> Best regards,
> Krzysztof

Cheers.
Ferass El Hafidi Feb. 3, 2025, 9:16 a.m. UTC | #5
Hi Krzysztof,

On Sun Feb 2, 2025 at 2:07 PM UTC, Krzysztof Kozlowski wrote:
> On 02/02/2025 14:55, Ferass El Hafidi wrote:
> > Hi,
> > 
> > On Sun Feb 2, 2025 at 1:54 PM UTC, Krzysztof Kozlowski wrote:
> >> On Sat, Feb 01, 2025 at 07:30:43PM +0000, Ferass El Hafidi wrote:
> >>> From: Ferass El Hafidi <funderscore@postmarketos.org>
> >>>
> >>> Add support for the Amlogic S805Y SoC and the Xiaomi Mi TV Stick (aka.
> >>> xiaomi-aquaman).  The S805Y is very similar to the S805X, with just a
> >>> few minor differences.
> >>>
> >>> Signed-off-by: Ferass El Hafidi <funderscore@postmarketos.org>
> >>
> >> Please run checkpatch. You have from/SoB mismatch.
> > 
> > I don't see any mismatch.  Maybe it complains because of a spurious
> > non-breaking space?
> > 
> Could be, no clue. I copied them to editor and it also sees mismatch -
> both entries just do not match.

The From/SoB mismatch should be fixed in the v3:
https://lore.kernel.org/linux-amlogic/20250203091453.15751-1-funderscore@postmarketos.org/T/#u

Thanks.

>
> Best regards,
> Krzysztof
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/amlogic.yaml b/Documentation/devicetree/bindings/arm/amlogic.yaml
index 0647851ae..3403362e0 100644
--- a/Documentation/devicetree/bindings/arm/amlogic.yaml
+++ b/Documentation/devicetree/bindings/arm/amlogic.yaml
@@ -73,6 +73,13 @@  properties:
           - const: amlogic,s805x
           - const: amlogic,meson-gxl
 
+      - description: Boards with the Amlogic Meson GXL S805Y SoC
+        items:
+          - enum:
+              - xiaomi,aquaman
+          - const: amlogic,s805y
+          - const: amlogic,meson-gxl
+
       - description: Boards with the Amlogic Meson GXL S905W SoC
         items:
           - enum: