linux-kernel - Re: [PATCH 2/2] arm64: dts: rockchip: Add Radxa ROCK3 Model C

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: <a144c052fcc2460a615a754a64a8f739@manjaro.org>
Date: Wed, 17 Apr 2024 19:02:02 +0200
From: Dragan Simic <dsimic@...jaro.org>
To: Chukun Pan <amadeus@....edu.cn>
Cc: jonas@...boo.se, conor+dt@...nel.org, devicetree@...r.kernel.org,
 heiko@...ech.de, krzk+dt@...nel.org, linux-arm-kernel@...ts.infradead.org,
 linux-kernel@...r.kernel.org, linux-rockchip@...ts.infradead.org,
 robh@...nel.org
Subject: Re: [PATCH 2/2] arm64: dts: rockchip: Add Radxa ROCK3 Model C

Hello Chukun,

On 2024-04-17 15:30, Chukun Pan wrote:
> Hi Jonas,
>>> +	model = "Radxa ROCK3 Model C";
>> 
>> The marketing name seems to be "Radxa ROCK 3C" according to the 
>> product
>> page at [1].
>> 
>> [1] https://radxa.com/products/rock3/3c
> 
> According to https://wiki.radxa.com/Rock3/3c , it should be called
> "Radxa ROCK 3 Model C". I copied rock3a here without paying attention.

If I may interject, here's the result of a quick qrep:

/rk3399pro-rock-pi-n10.dts:	model = "Radxa ROCK Pi N10";
/rk3399-rock-pi-4a-plus.dts:	model = "Radxa ROCK Pi 4A+";
/rk3588s-rock-5a.dts:	model = "Radxa ROCK 5 Model A";
/rk3588-rock-5b.dts:	model = "Radxa ROCK 5 Model B";
/rk3399-rock-4c-plus.dts:	model = "Radxa ROCK 4C+";
/rk3399-rock-pi-4b-plus.dts:	model = "Radxa ROCK Pi 4B+";
/rk3399-rock-pi-4b.dts:	model = "Radxa ROCK Pi 4B";
/rk3399-rock-pi-4c.dts:	model = "Radxa ROCK Pi 4C";
/rk3308-rock-pi-s.dts:	model = "Radxa ROCK Pi S";
/rk3399-rock-pi-4a.dts:	model = "Radxa ROCK Pi 4A";
/rk3399-rock-4se.dts:	model = "Radxa ROCK 4SE";
/rk3328-rock-pi-e.dts:	model = "Radxa ROCK Pi E";
/rk3568-rock-3a.dts:	model = "Radxa ROCK3 Model A";

Based on that, I think that "Radxa ROCK 3 Model C" would actually
be the preferred name...  Perhaps?

If we end up following that approach, the last board dts on the list
above should also be fixed to read "Radxa ROCK 3 Model A".

Either that, or all "Model " strings should be stripped out from
all board dts files that currently contain it.