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 7A360C77B7C for ; Wed, 24 May 2023 23:11:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=7Ac3TGMicqOp5Akog4jjRcWHtJo94kJb/OG+nWR0wzY=; b=Ft83Sg9PTWoK9j DMSI587WvhNyR+JehCcKkj/wB1eQgTXl2BlZrplcLiyiOo+ZhmON3oANLLJ+i2O0/6/QQywKZRi4U PosnBKkAduYdYxvARzcaSRg0AVWYGXK08j7W3c1XJv60c9dqWCuX8VAHFrLFhs3PWF5eaHoDThuKp DP10cqOa+kP1ZtF2Qj+jidmfMiJm3/BLA6oC5wVfJgEMu1aVLdffahX5HLSkHAGNMsg67lCOIK6SS RPJ0b8zN62K03fu45A9lK41Tz16t59BkkGhaucTJkkBHxxayU4xzZduzFDViPECfASrk4DA18//Xt KhN0biPoVWjerDmIubAg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1q1xdA-00Etzh-1T; Wed, 24 May 2023 23:10:56 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1q1xd7-00Etyx-16 for linux-arm-kernel@lists.infradead.org; Wed, 24 May 2023 23:10:54 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 7FF7561473; Wed, 24 May 2023 23:10:51 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 3D3F6C433EF; Wed, 24 May 2023 23:10:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1684969850; bh=xc+vGxKn9dNhSNFBqyxkDcXu5G90NN+jy0FXT2D5lJc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=bNCxkDf30oJ1kKyfE+xjNQve5pKHvBgLJyNpJEwXmUaJmx+qDiUw4b1RA5aPvUTYz +/m0nRmrRInm2gm7G7VqYsM97j1Dz9i5hy9FKkrpawGcTi1DhX2bhZhypUDYXPSKEU 6PgDx6MeRqaUWQOn5LNFVrJPUxR9J2vWXLW/h7BJQlW8mE99HUUpRiUCvMHEoEfWWK wFO16PBd/6FKFTThEO6pKk3Mtq3B8l5678bRbOr4jtlI/BtNLjNwiGbdIk18mwroYA gxAqhrEyStMMlqzn1EZSPUmkZFkRonZkwiMMzP0Q1TiXplIf5ffplR5dvBPbE2JZNj BWJNhmNbgaoWQ== Date: Wed, 24 May 2023 16:14:41 -0700 From: Bjorn Andersson To: Arnd Bergmann Cc: Dmitry Baryshkov , Konrad Dybcio , Vladimir Zapolskiy , Bjorn Andersson , Will Deacon , Krzysztof Kozlowski , linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org Subject: Re: [PATCH 3/5] arm64: defconfig: Build interconnect driver for QCM2290 Message-ID: <20230524231441.5zcdp2vms6s7giz7@ripper> References: <20230523165411.1136102-1-vladimir.zapolskiy@linaro.org> <20230523165411.1136102-4-vladimir.zapolskiy@linaro.org> <21baf481-347f-d4b1-87df-833ed16cb729@linaro.org> <179b8c7e-d355-4051-beb4-ac44808bdaaf@app.fastmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <179b8c7e-d355-4051-beb4-ac44808bdaaf@app.fastmail.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230524_161053_442670_F5EA09D1 X-CRM114-Status: GOOD ( 24.36 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, May 24, 2023 at 08:36:20AM +0200, Arnd Bergmann wrote: > On Tue, May 23, 2023, at 23:11, Dmitry Baryshkov wrote: > > On 23/05/2023 20:30, Konrad Dybcio wrote: > >> > >> > >> On 23.05.2023 18:54, Vladimir Zapolskiy wrote: > >>> Build Qualcomm QCM2290 interconnect driver. > >>> > >>> Signed-off-by: Vladimir Zapolskiy > >>> --- > >> Do we know why some are =y and some are =m? > >> > >> I'm for =y either way, if we can. > > > > I think this might change from platform to platform. What is the > > condition for selecting 'y' or 'm' for the core drivers? Is it 'should > > boot to rootfs without modules' or 'should boot to UART and load initrd > > without modules' ? > > I think we are a bit inconsistent here. I'm generally fine taking > changes that are required to boot into rootfs without initramfs, > but would like everything else to be loadable modules. > > Android tends to want everything as modules and require an initramfs > for loading them, so I think platforms that are mostly Android > specific lean towards that even for core drivers. > I (any many other in the Qcom community) do most of my (our) testing and development using a ramdisk-only approach and if systemd isn't provided a valid /dev/console at launch, things doesn't work as expected... So the inconsistency here relates to the fact that not all platforms has the UART defined to depend on interconnect, once this is populated in DT, we tend to be forced to move them to =y, together with GCC and pinctrl-msm. Looking to storage etc on a modern Qualcomm platform, the dependency tree is growing quite a bit (regulators, genpd providers etc), and we have UFS, SDHCI and NVME to support. So I think it's reasonable to have the generic defconfig require that you use a initramfs to get further (on the Qualcomm platforms). Regards, Bjorn _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel