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 94B96C04FFE for ; Tue, 14 May 2024 08:51:44 +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-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-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=Utsi3VwFrSp0UomuAhv4xbQk3Sogtx6oFPdrFoAcmDo=; b=FCv+75aEqayJR42B9IewSgPlBj QF5p+blg0wxL3LFgmZ+P4dShRIVVb21CqflRiprHyuID1BbKajk9KNQqNuedbI6pgBqZD/LXg6XT5 GgJMRTv4ANUCPzQxQY1K83cDanTXhCAq5FvDrFfkb80iE6seDzSOt+RCdRjwCFjbZRcXtjrhbcGeH fo/5IwokMppyKGon4N3XGz9ENsOtDjYUASRytyb6558/YFGW8K7R/gsA8fXtEYL5/iJuE6PB3kTMc 9/qfPxwOyW7LGbF62ZL+DKjrZbSwCqh3fvGA/4sxjxun6Ruia9m2pdA87RyvQBpe8AAWYEC2dmwto FaUEkrig==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1s6nsa-0000000FP1K-0J1L; Tue, 14 May 2024 08:51:24 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1s6nsW-0000000FP0n-3qGd for linux-arm-kernel@lists.infradead.org; Tue, 14 May 2024 08:51:22 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 9CB9E60C8C; Tue, 14 May 2024 08:51:19 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 404FDC4DE12; Tue, 14 May 2024 08:51:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1715676679; bh=dl9xK7NMauuyzjNTNgcvu+mipCW/bKmup0pxO3DFFko=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=W5GvxS3jvoInC+tPo/qJ0nW9Vfq+0oFTT9NxWgMVjQe7Eg7l7j7LhFr/hZoaTzRkb btxK7WRl+TeQ/wXm4no7rFxrNOZUvcB9xjsxbspOar/rmGFNVdxIGt9BSuNOvOdqUv BTrrDdIF+5tzbB7hZ4hiDj4G0abLOyaa4hycmSWCblITved3A0NU6bwl94YL41izYP biDE0e8kAzLsT+Pco/2Ehkf+AV2NtFKpiE+bY/ulT3xBXKQ5UE4ZmZIIZ7XpY+hx6l 77a0v/YTVuEgKPOkr2bN9OY0Gtsp0Ao9SeItld0fCG6kC7Usb59mFyzrZD1rLfECOB Zr1B5vboceM4g== Date: Tue, 14 May 2024 09:51:14 +0100 From: Mark Brown To: Ben Wolsieffer Cc: Leonard =?iso-8859-1?Q?G=F6hrs?= , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com, linux-spi@vger.kernel.org, Pengutronix Kernel Team , Alexandre Torgue , Maxime Coquelin , Alain Volmat Subject: Re: [PATCH v2] spi: stm32: enable controller before asserting CS Message-ID: <32d2be34-07e5-4f99-bd31-12aeb9bdce09@sirena.org.uk> References: <20240424135237.1329001-2-ben.wolsieffer@hefring.com> <39033ed7-3e57-4339-80b4-fc8919e26aa7@pengutronix.de> MIME-Version: 1.0 In-Reply-To: X-Cookie: In the war of wits, he's unarmed. X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240514_015121_016743_CAC6163A X-CRM114-Status: GOOD ( 22.33 ) 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: multipart/mixed; boundary="===============7304064122251796100==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============7304064122251796100== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="Qd3XkgjD6LiVNaV5" Content-Disposition: inline --Qd3XkgjD6LiVNaV5 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, May 13, 2024 at 12:36:57PM -0400, Ben Wolsieffer wrote: > On Mon, May 13, 2024 at 10:29:49AM +0200, Leonard G=F6hrs wrote: > > Reverting this commit fixes the issue for me. It may be some time before > > I get around to investigating the issue in detail, so I thought I should > > ask if anyone else has already noticed this as well. > Sorry about that; it looks like the STM32H7/MP platforms require the > controller to be enabled later. I agree that it should be reverted and > I'll try to think of another approach. Can one of you please send a patch with the revert and a changelog explaining the issue? > The STM32H7/MP devices are significantly different from the F4/7 > devices, which makes it difficult to change shared code without causing > problems like this. I wonder if it might be better to split the F4/7 > support into a separate driver. This would duplicate a bit of code, > namely the initialization in probe, the baud rate divider calculation > and the SPI mode config, but would make testing easier and get rid of > the indirection that handles the different register offsets and field > masks on each platform. The code for actually transcieving data and > handling IRQs is already platform specific. That might make sense. --Qd3XkgjD6LiVNaV5 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmZDJgEACgkQJNaLcl1U h9AzgwgAgs2oYQ4OnaMxz5ePCOZWtN8efc/rTzOMHKcGBE5yg1KaEr5ZsAB+m+nm uE6aHtctR6iZkebjfWUjAPCTCIxSBMOqZZcDvJpAz6Vyg4knJCWuwd9ZJNW+/7xr CeDdED9zn9UC92dYCaP0f6v9P4O/DPsoKf/A4OnhH/4AF+umWPAdRi06BQThEhDM yljfm3v4Odbuf1YadUqsNgb9Kd5iHwL9lhjTWImT+KjNhT4h7xKX9K33c1STBy9F mKgOkzHnTJt+f7whV+hSQkP549BZxql15/PXADZOytyC+D/n66YXx2WZKcWITA/E 61OwRBpdr3KpqxnnzwnzB5LLeCnRQQ== =wxj6 -----END PGP SIGNATURE----- --Qd3XkgjD6LiVNaV5-- --===============7304064122251796100== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============7304064122251796100==--