From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.manjaro.org (mail.manjaro.org [116.203.91.91]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 449F438F9A; Thu, 18 Apr 2024 07:18:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=116.203.91.91 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1713424693; cv=none; b=pTLEs9RAutbGmtIfnHoIUo5Q6RLeFmJmIW8SPFx/DTFKlSeOWm1TJUgxqCDVIQZPrsTimW3fBZFwns/ozjDxEbB3J2H8KEJofYCvygLaCEZGuzRrNtxJ8t2iiqgT1EZUXOgNW7TDpene0sXUtuDdF6sjceCcoBPfIFLtGzgqa+k= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1713424693; c=relaxed/simple; bh=goE/qDaO4VHKS7K9FeJgejduaauF9topo8TgMn4Vd4g=; h=MIME-Version:Date:From:To:Cc:Subject:In-Reply-To:References: Message-ID:Content-Type; b=YxdatqokD4Yl4FUHczNNlMc5DoePSyNaO8dr0QjMF/seVoBWbNMRDYrz3hg+PgDmb0VNy7oEOiQvHeclnsdUYBaif++WQXHnI9IUi3VMgnj3YUFT7zRKGWjq9KoL1igSHrt3zEqza4ZyzfPc72TfNT88Am0fM0BtdRBlitt0Xlk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=manjaro.org; spf=pass smtp.mailfrom=manjaro.org; dkim=pass (2048-bit key) header.d=manjaro.org header.i=@manjaro.org header.b=ulnMLQ0y; arc=none smtp.client-ip=116.203.91.91 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=manjaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=manjaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=manjaro.org header.i=@manjaro.org header.b="ulnMLQ0y" Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=manjaro.org; s=2021; t=1713424688; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=kNxJpDgz3ydxdfldvoCq44swMIRMijaZQ8+0vq0di0E=; b=ulnMLQ0yGxKcMqnkHFDU3bPy/kR5S9tGBHsKJsHd6i2aIUt2N7ItMvCjqGiTdYC1HvpW0k X6f6w7o9SdfRVpF88iZScbRcnfTGWJItDyhsjhnZ141zgGSqzrQIMIaeXiU0hc9+fUltqn 9lql3nH0u2pjQxbzlmc+J2wRYKwr9r+H5zrdZjyUyoNtak66VvpV0ZQJKxyYSKvxO9ZtRI Oby02TC9WHTjTfYv+pBq/IhsM9VvbfIpd6Ulzu4wrgw+cJK8KHmc+9Mig7gFshKFIRIIjT /ul+QTMWIBlgDzPzxYHuyeE6BQZFW4lNsvx8bdVx01ZOfCd68J7scQjUmK/h+w== Date: Thu, 18 Apr 2024 09:18:07 +0200 From: Dragan Simic To: FUKAUMI Naoki Cc: Chukun Pan , jonas@kwiboo.se, conor+dt@kernel.org, devicetree@vger.kernel.org, heiko@sntech.de, krzk+dt@kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org, robh@kernel.org Subject: Re: [PATCH 2/2] arm64: dts: rockchip: Add Radxa ROCK3 Model C In-Reply-To: References: <20240417133016.62753-1-amadeus@jmu.edu.cn> <9e915ef29a2b49ce56446ec9d474eb64@manjaro.org> Message-ID: <730e21fc4c74bf42b8a5e54473270899@manjaro.org> X-Sender: dsimic@manjaro.org Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Authentication-Results: ORIGINATING; auth=pass smtp.auth=dsimic@manjaro.org smtp.mailfrom=dsimic@manjaro.org On 2024-04-18 08:33, Dragan Simic wrote: > On 2024-04-18 07:55, Dragan Simic wrote: >> On 2024-04-18 07:52, FUKAUMI Naoki wrote: >>> this is Naoki @ Radxa. >>> >>> thank you for your great work! >>> >>> On 4/18/24 02:02, Dragan Simic wrote: >>>> 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. >>> >>> sorry, wiki.radxa.com is outdated. >>> >>> docs.radxa.com is correct one and maintained. >>> (but it still has some errors, sorry) >>> >>>> 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. >>> >>> we have a document named "Radxa Product Naming Convention". >>> there are "full name" and "short name". "Model" is used in "full >>> name", but it's not used in "short name". both are correct. >>> >>> but, we preferred to use "short name" for "model = " in mainline >>> linux/u-boot. >>> >>> for ROCK 3C, please use >>> >>> model = "Radxa ROCK 3C"; >>> >>> for ROCK 5A/5B and ROCK 3A, >>> >>> model = "Radxa ROCK 5A"; >>> model = "Radxa ROCK 5B"; >>> model = "Radxa ROCK 3A"; >>> >>> are preferred. >>> >>> we need to fix this inconsistency. (include our docs) >> >> Great, thanks for this detailed clarification! > > I went ahead and prepared a patch that cleans up currently present > naming inconsistencies for the Radxa boards. AFAICT, there should > be no issues arising from the changes in the "model" and "compatible" > strings, because they aren't used anywhere in the drivers. There > are no board-specific ALSA UCM2 profiles, which could be affected > by those changes. U-Boot is also fine. > > I'll think a bit more about it, and I'll send the patch over a bit > later. If it goes well, I'll prepare and send one more similar patch, > for some other boards. Actually, a couple of board dts(i) files, listed below, need to be renamed to add dashes into their names. That, together with the other naming cleanups I've already prepared, would make the naming cleanup of the Radxa boards complete. - arch/arm/boot/dts/rockchip/rk3288-rock2-som.dtsi (-> arch/arm/boot/dts/rockchip/rk3288-rock-2-som.dtsi) - arch/arm/boot/dts/rockchip/rk3288-rock2-square.dts (-> arch/arm/boot/dts/rockchip/rk3288-rock-2-square.dts) I see no possible issues in the kernel arising from these two file renames, with the other related changes applied, of course, but there are some issues in U-Boot. Luckily, overcoming those issues shouldn't be too hard, and I've already submitted a small preparatory patch to the U-Boot mailing list. [1] We'll see how it goes. [1] https://lore.kernel.org/u-boot/369151a2b8d73d733ca79eca98d4d5958f90d491.1713423687.git.dsimic@manjaro.org/