From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 9C6D8C36008 for ; Wed, 26 Mar 2025 09:15:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Type:MIME-Version: References:In-Reply-To:Subject:Cc:To:From:Message-ID:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=BvUs9M0NfiRe+VuLU02CEA5G9rv30LFGHZ/0m4kb5Ic=; b=o2a+e1MCizveioCjuhEjtWGgIp wEsmjH5aXAfymZXGiLLuIBYRieD5ltdO9dYgnYTrUexse+8MLNNFpon+y6gfcZqESin+etTjTXQWy JMpqPSuzXWX5XaVTL6DMjMpzpNiP41/TbfhMv6ABswHvcKdF5TXHXACvcDWkT8WQRXdgnIO0tYIqw jHJID90CeLpNxwE7mWZoukUndyNn3vKeDcTkCGZ9P5fqa/NN7bHIoYnynZmWmLqdGhzQY3IlJnOq2 tRFQAtfzce0w/25jI72NQQkN+fsPA4sHTs93yXs6mUBImksUTPnV9XSSDOf4+Vz1VIdalaMWD4mDW sbv56keA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.1 #2 (Red Hat Linux)) id 1txMqW-0000000808Q-47Mo; Wed, 26 Mar 2025 09:14:48 +0000 Received: from nyc.source.kernel.org ([147.75.193.91]) by bombadil.infradead.org with esmtps (Exim 4.98.1 #2 (Red Hat Linux)) id 1txMoo-0000000802K-2xEL for linux-arm-kernel@lists.infradead.org; Wed, 26 Mar 2025 09:13:03 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by nyc.source.kernel.org (Postfix) with ESMTP id 02F80A40E28; Wed, 26 Mar 2025 09:07:33 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id A119EC4CEE2; Wed, 26 Mar 2025 09:13:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1742980381; bh=aN+g0071I7Vt/PENJP2xKw1ev/MOlfvCtYEPMiSMFoo=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=KO4GxINV6olNF2UQXuKkU8PE7klKPjHGZdg2MCcNmhwRPQWiGhojXXelfplsWVCi+ AM9jbm6mLHrSoRMiDVMT7lMk/u4nAD0eRkb32dhblMEll6b/F0/n1KumF+aeY3y0s3 OneulYbDcLOrIjrwP+JJRIpN7Sn7xkLXqM/3DvbVzq4hEe0XgcesrU3psE3OUe6wBP pm/l64u3LGqEdzkZ0p0+LieV00X7THRVODBGW+vRK6jzmvs/sonuyGcUgEjsQBParA d7RxtjIu8b5r9gvi+O4JYO7lDvRoLkNFN4HJMXm05vh42ep74Rj00961kpa78jfY14 ExyrBH2W51E/g== Received: from sofa.misterjones.org ([185.219.108.64] helo=goblin-girl.misterjones.org) by disco-boy.misterjones.org with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.95) (envelope-from ) id 1txMol-00HEBK-5u; Wed, 26 Mar 2025 09:12:59 +0000 Date: Wed, 26 Mar 2025 09:12:58 +0000 Message-ID: <861pukm9yd.wl-maz@kernel.org> From: Marc Zyngier To: Peter Chen Cc: soc@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, catalin.marinas@arm.com, will@kernel.org, arnd@arndb.de, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, cix-kernel-upstream@cixtech.com, marcin@juszkiewicz.com.pl, kajetan.puchalski@arm.com, Krzysztof Kozlowski , Fugang Duan Subject: Re: [PATCH v5 5/6] arm64: dts: cix: add initial CIX P1(SKY1) dts support In-Reply-To: References: <20250324062420.360289-1-peter.chen@cixtech.com> <20250324062420.360289-6-peter.chen@cixtech.com> <865xjxmlgl.wl-maz@kernel.org> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM-LB/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL-LB/10.8 EasyPG/1.0.0 Emacs/29.4 (aarch64-unknown-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII X-SA-Exim-Connect-IP: 185.219.108.64 X-SA-Exim-Rcpt-To: peter.chen@cixtech.com, soc@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, catalin.marinas@arm.com, will@kernel.org, arnd@arndb.de, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, cix-kernel-upstream@cixtech.com, marcin@juszkiewicz.com.pl, kajetan.puchalski@arm.com, krzysztof.kozlowski@linaro.org, fugang.duan@cixtech.com X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250326_021302_881442_1E45A9A8 X-CRM114-Status: GOOD ( 25.74 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, 26 Mar 2025 03:26:08 +0000, Peter Chen wrote: > > On 25-03-25 10:52:10, Marc Zyngier wrote: > > > + timer { > > > + compatible = "arm,armv8-timer"; > > > + interrupt-names = "sec-phys", "phys", "virt", "hyp-phys", "hyp-virt"; > > > + interrupts = , > > > + , > > > + , > > > + , > > > + ; > > > + }; > > > +}; > > > > I don't think there is anything wrong here, but it is also a pretty > > useless DT. There isn't even a UART to interact with the machine and > > find out whether it has actually booted. > > > > UEFI uses the same UART, so we could see all kernel boot logs until > switch to use kernel UART driver for printk. If you would like boot > to the console at initramfs, just add uart node like patchset v1. What's the point in upstreaming something that requires extra changes just to boot it? It only outlines these patches are not useful as they stand. > > > I reckon this should be part of the initial DT, as this otherwise > > serves little purpose. > > > > Without this initial support, we can't add some base drivers, like > mailbox. The dt_binding_check will report warnings/errors [1]. Of course you can. You just add additional patches to this series, making it something that is actually useful. So far, this series only serves as marketing material. > Full UART support depends on clock, clock control needs mailbox > to talk with FW using SCMI protocol. Then do it. You obviously have existing DT support for it already. > There is no any support for CIX SoC, so we had to add one small step by > step. No, you are deliberately choosing to make this platform useless. That's a bit sad, and a waste of everybody's time. M. -- Without deviation from the norm, progress is not possible.