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 F1B47C3271E for ; Mon, 8 Jul 2024 17:45:11 +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:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From: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=c19fUGD4GZj+lmmNMIl5Gi2qfLFqH+R3Vd8mPm6/uB4=; b=ZyDTMKY6/cDs+6od6mhTW8rcnQ i+qsBMlPaAoZhDOiupGuBAa+dWW+U2WbsrAmaG8P7+DWnw+9CrRAogwFImb9CYUtfmfZHZb16d/LL 3GPAGOsExKwCAehLP9ols0N8hyprs5A4OxLUsOH7Y2Jr3t7CaVHVlUJn0JT7QIqXtsEkwol5m38LL DWw3el52ClowJrnnXc5/T5yuXJNFSJNNun5SQawXgOhoc8FTi4htz1YWmHl93RJPGtBWgEKwSNSza JvziSC/+oe2Aur4hu0zayVGXTyS9rEO/TtvilMioa2kVWTTu0eU7pXcHj6EIAGoSabcxDKWILclvZ ocYfuWAg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sQsQ8-00000004cQ9-1gKg; Mon, 08 Jul 2024 17:45:00 +0000 Received: from sin.source.kernel.org ([145.40.73.55]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sQsPs-00000004cN1-3R2L for linux-arm-kernel@lists.infradead.org; Mon, 08 Jul 2024 17:44:46 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id E7FCCCE0E92; Mon, 8 Jul 2024 17:44:42 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id A914BC32786; Mon, 8 Jul 2024 17:44:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1720460682; bh=fJdijarOdeOiYkJmtENxWG6XvECxSgzYRkmGDVjF3cs=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Cc7i3+N7+FIV0PBx6xtg8oegYLv+kIAV8RfCoFmm+QwC+4f1J3CRSZdu/ERk+R4DQ o8IAP28r2NQ2PuydwzXzOKcCDFbOXqaHRVLcbMI1MYiMGnGJ5c70/Wl4+igc9Afr3u 7xtMe5n38a+l9RdOwbtrEuBG8b+GTSq01oUqX/gReHej9OyznvVQj800WgmYBn7//K qE1SDSZvAco8s5+q/lmFpfV/S9+O5Upg5JmcamNtDpHu6Z4kbPNbwSHtnNO+dRiwjk L6/rHtK9XUhz95O/AW3r00nu47NonicxCj7WOGQ8TeDmL99oh99nO+ca0W6hlwoIU1 AbgUPedE9V6oQ== Date: Mon, 8 Jul 2024 18:44:37 +0100 From: Mark Brown To: Daniel Lezcano Cc: Thomas Gleixner , David Abdurachmanov , Sudeep Holla , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Ross Burton Subject: Re: [PATCH v2] clocksource: sp804: Make user selectable Message-ID: References: <20240529-arm64-vexpress-sp804-v2-1-c542a80af33a@kernel.org> <2b946a68-dcdd-4a1e-b7c3-416725033c2e@linaro.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="B3oMzoTFaOm/yyLw" Content-Disposition: inline In-Reply-To: <2b946a68-dcdd-4a1e-b7c3-416725033c2e@linaro.org> X-Cookie: Many are cold, but few are frozen. X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240708_104445_058565_3EB9DD16 X-CRM114-Status: GOOD ( 22.44 ) 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 --B3oMzoTFaOm/yyLw Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Mon, Jul 08, 2024 at 06:49:38PM +0200, Daniel Lezcano wrote: > On 29/05/2024 21:48, Mark Brown wrote: > > The sp804 is currently only user selectable if COMPILE_TEST, this was > > done by commit dfc82faad725 ("clocksource/drivers/sp804: Add > > COMPILE_TEST to CONFIG_ARM_TIMER_SP804") in order to avoid it being > > spuriously offered on platforms that won't have the hardware since it's > > generally only seen on Arm based platforms. This config is overly > > restrictive, while platforms that rely on the SP804 do select it in > > their Kconfig there are others such as the Arm fast models which have a > > SP804 available but currently unused by Linux. Relax the dependency to > > allow it to be user selectable on arm and arm64 to avoid surprises and > > in case someone comes up with a use for extra timer hardware. > Would it make sense to add the option in the platform so it selects the > timer ? As the commit log says far as I'm aware all the platforms that rely on the sp804 timer already select it (they wouldn't otherwise be able to work unless COMPILE_TEST was enabled). The Arm models and possibly other platforms have the sp804 but it will currently be ignored by Linux and the architected timers used instead so it would be wasteful to force it on for them. --B3oMzoTFaOm/yyLw Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmaMJYUACgkQJNaLcl1U h9D/PQf/ex4DnX3Sz8k/XryILFJMLeyiZmFd2gRAFjcP10e0sUXv8QEuB99JtntF F/U6KIKhGa3lY6LvnY8ZWMs++1VzyCQkGMjcOxudenZpYfEG6tRrsai2UQeNw/CB po9s1i3tzj/6iXSRMetXsZvP0egNNEXBLAMOqJn+DQTOYKYpJKDIMIWUUGgsfKor 8eMpo7gGSuBzB+iJbiSiuhkkeaXzduea3Bl+0LVeuNc8Pn1Wcnw6j/HEvCZNxvcm rxyQ2V6ub2TGraMI+hey/t6i5zwC/7pcw6mDJ77/qbNt84KOLoKrK7J8IVC6geRk zXoiFKUB7kZdS588Ggyx5upcLsMUDw== =QXrh -----END PGP SIGNATURE----- --B3oMzoTFaOm/yyLw--