linux-kernel - Re: [PATCH v1 1/3] dt-bindings: arm: samsung: document g0s board binding

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <c39e6058-0649-452f-9a21-d9d5a28ccc06@gmail.com>
Date: Wed, 19 Feb 2025 12:29:59 +0200
From: Ivaylo Ivanov <ivo.ivanov.ivanov1@...il.com>
To: Krzysztof Kozlowski <krzk@...nel.org>, Rob Herring <robh@...nel.org>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, Alim Akhtar <alim.akhtar@...sung.com>
Cc: linux-samsung-soc@...r.kernel.org, devicetree@...r.kernel.org,
 linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1 1/3] dt-bindings: arm: samsung: document g0s board
 binding

On 2/19/25 10:35, Krzysztof Kozlowski wrote:
> On 15/02/2025 14:04, Ivaylo Ivanov wrote:
>> Add binding for the Samsung Galaxy S22+ (SM-S906B) board, which is
>> based on the Samsung Exynos2200 SoC.
>>
>> Signed-off-by: Ivaylo Ivanov <ivo.ivanov.ivanov1@...il.com>
>> ---
>>  .../devicetree/bindings/arm/samsung/samsung-boards.yaml     | 6 ++++++
>>  1 file changed, 6 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/samsung/samsung-boards.yaml b/Documentation/devicetree/bindings/arm/samsung/samsung-boards.yaml
>> index fab29f95d..bb3f6a0e1 100644
>> --- a/Documentation/devicetree/bindings/arm/samsung/samsung-boards.yaml
>> +++ b/Documentation/devicetree/bindings/arm/samsung/samsung-boards.yaml
>> @@ -45,6 +45,12 @@ properties:
>>            - const: samsung,aries
>>            - const: samsung,s5pv210
>>  
>> +      - description: Exynos2200 based boards
>> +        items:
>> +          - enum:
>> +              - samsung,g0s                     # Samsung Galaxy S22+ (SM-S906B)
>
> What does g0s stand for?

Samsung started codenaming their S series like that since ~S20. I'd actually
much rather use something like samsung,s22p but since 990 devices upstream
already use the weird codenames, I've decided to do the same.

I'll mention G0S it in the commit message next revision.

Best regards,
Ivaylo

>  Sound a bit cryptic and you did not use it in
> commit msg to explain the origin.
>
>
> Best regards,
> Krzysztof