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 B5F5DC25B75 for ; Wed, 29 May 2024 10:13:36 +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=lE3xPes3s5i15Ri7AhWnMlMzvDGt61vuQeWmYlZskr0=; b=a1mFFO+NcXu/w4 duPSgw7i/Wl+9waHGj4d7s47KUcfPTaLjPb/sKLWQAjQPoZGbh1MsUm75wjKonjbffaa4LR37//Jz OimURYW+k0t3ISVofkED/rcW3jd8tbbiqTXwn6ATgIAiSbRgrDXdWwrl5a5qrJhVGbBVFcR9PE10q PShYkJfE2PZEsBSgVcUqbdYH2q5GbU7JxE2RmmNju2rY9rnVb2+FrIaqY0OD12WSq1e9r7he3lqPM 8nqojWGtVgiSRXy6bTWZr+UQL3huBFTqvVY7aKUp/oxuIctrPibWbO/nAZggWBrJQmb99fpMyDTaM qjOJGQYcx2g4XD4aLJAQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sCGJ9-00000003lUI-0z8r; Wed, 29 May 2024 10:13:23 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sCGJ5-00000003lS6-25n8 for linux-arm-kernel@lists.infradead.org; Wed, 29 May 2024 10:13:21 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 2BC71339; Wed, 29 May 2024 03:13:39 -0700 (PDT) Received: from pluto (usa-sjc-mx-foss1.foss.arm.com [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 32FD23F792; Wed, 29 May 2024 03:13:14 -0700 (PDT) Date: Wed, 29 May 2024 11:13:11 +0100 From: Cristian Marussi To: Geert Uytterhoeven Cc: Sudeep Holla , Jassi Brar , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] mailbox: ARM_MHU_V3 should depend on ARM64 Message-ID: References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240529_031319_611949_5D30BCBA X-CRM114-Status: GOOD ( 21.25 ) 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 29, 2024 at 09:30:45AM +0200, Geert Uytterhoeven wrote: > The ARM MHUv3 controller is only present on ARM64 SoCs. Hence add a > dependency on ARM64, to prevent asking the user about this driver when > configuring a kernel for a different architecture than ARM64. > Hi, the ARM64 dependency was dropped on purpose after a few iterations of this series since, despite this being an ARM IP, it has really no technical dependency on ARM arch, not even the usual one on ARM AMBA bus, being this a platform driver, so it seemed an uneeded artificial restriction to impose... ...having said that, surely my live testing were performed only on arm64 models as of now. So, I am not saying that I am against this proposed fix but what is the issue that is trying to solve, have you seen any compilation error ? or is it just to avoid the user-prompting ? Thanks, Cristian > Fixes: ca1a8680b134b5e6 ("mailbox: arm_mhuv3: Add driver") > Signed-off-by: Geert Uytterhoeven > --- > The MHUv3 documentation is unclear about whether this can be present > only on ARM64, or also on ARM. > Googling 'site:arm.com MHUv3 "aarch32"' shows no results. > Googling 'site:arm.com MHUv3 "aarch64"' does show results. > --- > drivers/mailbox/Kconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/mailbox/Kconfig b/drivers/mailbox/Kconfig > index 3b8842c4a3401579..8d4d1cbb1d4ca625 100644 > --- a/drivers/mailbox/Kconfig > +++ b/drivers/mailbox/Kconfig > @@ -25,6 +25,7 @@ config ARM_MHU_V2 > > config ARM_MHU_V3 > tristate "ARM MHUv3 Mailbox" > + depends on ARM64 || COMPILE_TEST > depends on HAS_IOMEM || COMPILE_TEST > depends on OF > help > -- > 2.34.1 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel