Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37;
From:   Inochi Amaoto <inochiama@outlook.com>
To:     Conor Dooley <conor+dt@kernel.org>
Cc:     Jisheng Zhang <jszhang@kernel.org>,
        Thomas Gleixner <tglx@linutronix.de>,
        Marc Zyngier <maz@kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Palmer Dabbelt <palmer@dabbelt.com>,
        Paul Walmsley <paul.walmsley@sifive.com>,
        Albert Ou <aou@eecs.berkeley.edu>,
        Daniel Lezcano <daniel.lezcano@linaro.org>,
        Anup Patel <anup@brainfault.org>, linux-kernel@vger.kernel.org,
        devicetree@vger.kernel.org, linux-riscv@lists.infradead.org,
        chao.wei@sophgo.com, xiaoguang.xing@sophgo.com,
        Inochi Amaoto <inochiama@outlook.com>
Subject: Re: [PATCH 4/5] riscv: dts: sophgo: add initial CV1800B SoC device tree
Date:   Sun,  1 Oct 2023 20:19:02 +0800
Message-ID: <IA1PR20MB4953D58BA3ECFD487918E3A4BBC6A@IA1PR20MB4953.namprd20.prod.outlook.com>
In-Reply-To: <20231001-abruptly-giddily-ef340b924774@spud>
References: <20231001-abruptly-giddily-ef340b924774@spud>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
X-MS-PublicTrafficType: Email
X-MS-TrafficTypeDiagnostic: IA1PR20MB4953:EE_|PH0PR20MB6018:EE_
X-MS-Office365-Filtering-Correlation-Id: 7de4266e-2439-4021-56d3-08dbc27898ad
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: GZ4ZVZ0l64JYPrUfCNm/dEl7BWFRBG2/ms+BdFLoQ1vG+T8TEpN7IZwE8e12eqLFkW6a7yhTP6ev17olXcQg5C1kZpBErleoSrc95WgCKb3A8M+Fpe/QvYfxDcTHsX5o2fOOKYX7GW+nI0Ma8ZrjO1wf8QkZQRkGDfE3urquQ3FfYhIFYqs0/zk2+djdIBjEcZ0oyDpY8MrlNMhSs0AmsAVQTIC+3xOZAVyBwrls51kKBK3ShZ8QFy/jSgOW9W965/Lgk65HxlTNvcAJvfiVpPVmqzguKNvUrJgTc6GFkYZpOtwSU6ypUqGxD3eftW2mLFFyWfb7LVtJFoguCaqlanI2maLSp12XppVJv26G8xM8gY9h8mkmKwoC1pXWHbIf7PLTeM5uD+dCB2Om389thf47KDiipJ0q4cceNHSkUc4cyWopu+MQZuNxL9Az0Gii5+sy/c/l05rshqnu6HVjIpbJ9tnS+8WA+SIYRnjNrq+SwDvTBEKot9Yh1WNFzE3sSuai0EX5XCymnhIakxacw2RlVTul3QlBlRrIdmj4NmRMz/8QhEBl9oo3ngXs5MDJtrdQwB6yhhkn2zbkZ82YpiDBYHP78UjyrGEIu+I5tWrRxZhwX6xP8gMlmRr9PoGM
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?sCYHH2gQky47lTnWPCPZaLVqi2gdka1hLnOxz5sg9kkUb5kyoAHGQttSFvc+?=
 =?us-ascii?Q?uvcRAANsWjei3VY3MT30sSRgJ95UANopMBZgk4bvDUwufypkoEJCYYPusw+e?=
 =?us-ascii?Q?wzNNg8BVuwjg6fJxytQuEwZriGvNdkUqdKawAXs1vuGf8qbbJfTVnH+17oXj?=
 =?us-ascii?Q?g1KyPwPaeXSYkq68Z7d7+H37YA6VvZkitj/NRUovTzvmH1RfdHvcbhybdTS4?=
 =?us-ascii?Q?HSLKSe8kpThG/TDQM9iGa3INPOSRZQSTrX+6iaLKV1O8Fau3b8C/Szep6lf3?=
 =?us-ascii?Q?zpemH8aA3/mXzlzXrIkg+DrO863cnSsLExSBGDuPJn4MroTkylPnU9JR02TM?=
 =?us-ascii?Q?0FZfL7hjWP4AIiArxfMypdgjaXp8PJAzTMvb83l+GzwM9z63aLEdkhqb6roI?=
 =?us-ascii?Q?TWaCwAUQsa01h/Qk7HayiLXvq5KDHf7B9eymbb4r2re7unSfpqXziGvTQQSe?=
 =?us-ascii?Q?TajV4cFKpYcCay1bnS27bgKaBLe09J8CE9DrpYDbFGUdU1bXuoP+WSVtM3Bp?=
 =?us-ascii?Q?Caa9fRuHdqCBab0/K0hC1FPzYINacwGXQRequy7DlwDHpzxKNjJYSAb6k0Os?=
 =?us-ascii?Q?W/dB0Gpd6CvShfKv1n18lZZQl03jGxHD4UYMkiuaOrLrhl67a9BIZc+t/+mB?=
 =?us-ascii?Q?duuVS0FFDbVhafRQD4BOu1LSiO3IZKQXsJU1fmNXJqSwAGXy6kA4bzBJmtFx?=
 =?us-ascii?Q?32gSJ8v1q+7M05PeFhoSmc04Use4Tza9GpG1Diun6gHzLLSA+f8cPHKkI3j0?=
 =?us-ascii?Q?tP4831voGxScHi2qN0iFceuf/xUoB5xiROA15QIJYvYfx0dZX/kxpQdcgrqV?=
 =?us-ascii?Q?6PTFt2+EclFURVvlpcBEWkJnB1aU7w/PsYnPbQFCzV8EPxZ75MEcmniHz5d+?=
 =?us-ascii?Q?stFS+tXkgvr6efbPhqGRcmlH8DB+UX6Xv5kilUSLyTulPNcfHuUaaaYyxYc6?=
 =?us-ascii?Q?RTUFC8+8vwUK/fFeT1yVqT7Z5rp+GrRpNkxOStWmtLsfw05tOQ3VdL/6PpVa?=
 =?us-ascii?Q?NO20CZ14L2J8MXA9/KIdlHgBPBmQNdt6hC9+ZZnS7yYCRT9CCPb2wh+VCpjM?=
 =?us-ascii?Q?Y/qeMjrE2VxXzpKegDMc7RCRlzN5acSPweVlVxQfecx5J3t48gTDwQEj5Le7?=
 =?us-ascii?Q?sfPr4STLVJyglAGkJHLCWsaWzVRTUmuM3z+o9gBGm6t56FUhBFFlETsl21Ty?=
 =?us-ascii?Q?oDa+OQRB/4K8ibCqPDY0REUKIVz4dCH6vanwgwOaz6SroA93k6huDbsyymU?=
 =?us-ascii?Q?=3D?=
X-OriginatorOrg: outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 7de4266e-2439-4021-56d3-08dbc27898ad
X-MS-Exchange-CrossTenant-AuthSource: IA1PR20MB4953.namprd20.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Oct 2023 12:19:01.9721
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa
X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR20MB6018
X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Sun, 01 Oct 2023 05:19:11 -0700 (PDT)

>On Sun, Oct 01, 2023 at 06:34:21AM +0800, Inochi Amaoto wrote:
>> Hi, Jisheng
>
>>> Add initial device tree for the CV1800B RISC-V SoC by SOPHGO.
>
>>
>> You add the clint dt-bindings of CV1800B clint, but I don't see the clint
>> node in this dt. The SBI needs this clint node to provide timer for linux.
>> AFAIK, the dt of SBI comes from the linux or the bootloader, and bootloader
>> may load the linux dt and pass it to the SBI. I think it is better to add
>> the clint node.
>
>> In addition, please separate the peripheral node to a different file, which
>> can be reused by both the CV1800 series and CV1810 series.
>
>How do these SoCs differ?

AFAIK, the most peripheral of CV1800 and CV1810 are the same. there are
only a few difference between CV1800 and CV1810:
1. CV1810 have mmc interrupt, but CV1800 have none
2. CV1810 have more RAM and a more powerful TPU.
3. Some models of CV1810 support I2S.

Also is some you have already mentioned, the video capabilities (including
encoding, output steam number, input steam number) are different.

The only board with a CV1800 soc is Huashan Pi (CV1812H).

>Documentation seems rather lacking, but I was able to find something on
>github that suggests there is also a cv180zb. The difference between the
>three seems to, from a quick look, be their video encoding capabilities.
>Is that correct?
>

Yes. it is correct.
It seems like you have forgot a chip called CV1801B, which has 128MB
RAM. But I see no board with this soc, so at now it is not necessary to
care it.