From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from fhigh-a1-smtp.messagingengine.com (fhigh-a1-smtp.messagingengine.com [103.168.172.152]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5B1793A8732; Thu, 23 Apr 2026 17:47:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=103.168.172.152 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776966437; cv=none; b=huKg8zzk7ziqaH1sbkObrM9hgm+lr7ufwM81AxjQ77N1Z+1MBryvO2/PpbaaITy2ij2xWUU6TDixMfb0zzntoVuKxegqNTNm/ZlpY/r+xn1ryiUJZO1cysy/QM9hUcwG/JVrBaNMt2iGIWlM2tPzwoMj/f1fq1Satwgif5eV/ag= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776966437; c=relaxed/simple; bh=/yyOKutn1AUh3qcy0gcHugV9yT5TCQBCWscV32jNero=; h=MIME-Version:Date:From:To:Cc:Message-Id:In-Reply-To:References: Subject:Content-Type; b=hTdM+ZXSRC06Ct6yRN6FsiQSQ/m/I54HjrfEb6qrnFR81kf+ch+qWZ6nJ65nlrvcjlzYXiHm/5OOACMBz9B3DiFATCU18GkGGyhgev9DZKkQQnslQB7BhXn0MzaP/phP3Vko2vYUyt6ivYHO31fA1WXxN2DClkGOE1F7i4galZo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=kevinmehall.net; spf=pass smtp.mailfrom=kevinmehall.net; dkim=pass (2048-bit key) header.d=kevinmehall.net header.i=@kevinmehall.net header.b=J9hxGh4d; dkim=pass (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b=F0/sK1oG; arc=none smtp.client-ip=103.168.172.152 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=kevinmehall.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=kevinmehall.net Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kevinmehall.net header.i=@kevinmehall.net header.b="J9hxGh4d"; dkim=pass (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b="F0/sK1oG" Received: from phl-compute-04.internal (phl-compute-04.internal [10.202.2.44]) by mailfhigh.phl.internal (Postfix) with ESMTP id B2BBE140016D; Thu, 23 Apr 2026 13:47:15 -0400 (EDT) Received: from phl-imap-02 ([10.202.2.81]) by phl-compute-04.internal (MEProxy); Thu, 23 Apr 2026 13:47:15 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kevinmehall.net; h=cc:cc:content-transfer-encoding:content-type:content-type :date:date:from:from:in-reply-to:in-reply-to:message-id :mime-version:references:reply-to:subject:subject:to:to; s=fm1; t=1776966435; x=1777052835; bh=/yyOKutn1AUh3qcy0gcHugV9yT5TCQBC WscV32jNero=; b=J9hxGh4dzvGSBJ4ZN4TcZNc2mfbhNIJ9Z2eWmHFKGuyngHOO U3sXFV59P3Mgkf08kmSeQgNghcLCFY5RGcUaUYAhRqlOZ3P1MT3jKrsqrG6bhWW/ Y8dJUO9ND6nvkM5fkztdRaTYf1HIzeynkkwSuyO7DjFXTzpNUhAyJDxmbXr3NA/w OHXsch2D/4Ml9agn3f1lia3Ljl0DgYH0Ga30qJ68jBNaQtdcNflCKlBaFCi7NWAE lZdxZe/P+GLWWqUCVvWvvbi/p3rDWBpKEueXWC3thsJb0vW14Tiur52w2QxR8uWq vzHpgoMBuCq6+HCVEwzplRCu2o9psXDXC/Uc1A== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm2; t=1776966435; x= 1777052835; bh=/yyOKutn1AUh3qcy0gcHugV9yT5TCQBCWscV32jNero=; b=F 0/sK1oGT0XgjbemMNXGAaTSHH9002fBNX0cXg1DYebw9VdMmSPzmVlegL/scTwTM zRgt2QUkpvUdYuQ19AHSxZ8JnKWUb+CQ5IWLqHYqN1GHtAX3BPVEXscNvFyr7iVk OrMtIcGXb3x53ulUqs4nM5W/VgSYufZ1rpyrEIWguts9Zg7JfJ3y0M8bdzduUiyY K7FRT/7ODh6gBErQh8tg44UmNVrCuFPm6QCoLUp6I9Y9ihin1XOW/JHJZq2wYBk9 e/OFCVsfeVAHhwz+jbXjJlQa+Vl/joa9yXVdjHIgifCm1KfaJ7R29jVRaJO09w08 SVjf8qva+KjmWH05vWsXw== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeefhedrtddtgdeijeejlecutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpuffrtefokffrpgfnqfghnecuuegr ihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenucfjug hrpefoggffhffvvefkjghfufgtgfesthejredtredttdenucfhrhhomhepfdfmvghvihhn ucfovghhrghllhdfuceokhhmsehkvghvihhnmhgvhhgrlhhlrdhnvghtqeenucggtffrrg htthgvrhhnpeetfedvheeigfevheefkeejkeegleetiedvgfehieelkeekfefgtdffvdfg gfegfeenucffohhmrghinhepkhgvrhhnvghlrdhorhhgnecuvehluhhsthgvrhfuihiivg eptdenucfrrghrrghmpehmrghilhhfrhhomhepkhhmsehkvghvihhnmhgvhhgrlhhlrdhn vghtpdhnsggprhgtphhtthhopedutddpmhhouggvpehsmhhtphhouhhtpdhrtghpthhtoh epjhgvrhhnvghjrdhskhhrrggsvggtsehgmhgrihhlrdgtohhmpdhrtghpthhtohepsghr ohhonhhivgeskhgvrhhnvghlrdhorhhgpdhrtghpthhtohepfigvnhhssehkvghrnhgvlh drohhrghdprhgtphhtthhopehlihhnuhigqdgrrhhmqdhkvghrnhgvlheslhhishhtshdr ihhnfhhrrgguvggrugdrohhrghdprhgtphhtthhopehlihhnuhigqdhsuhhngihisehlih hsthhsrdhlihhnuhigrdguvghvpdhrtghpthhtohepmhhirhhkohdquggvvhiklhhinhhu giesnhgrnhhlrdguvgdprhgtphhtthhopehrshgtsehruhhnthhugidrtghomhdprhgtph htthhopehsrghmuhgvlhesshhhohhllhgrnhgurdhorhhgpdhrtghpthhtoheplhhinhhu gidqkhgvrhhnvghlsehvghgvrhdrkhgvrhhnvghlrdhorhhg X-ME-Proxy: Feedback-ID: i421842c8:Fastmail Received: by mailuser.phl.internal (Postfix, from userid 501) id 1472F700065; Thu, 23 Apr 2026 13:47:15 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface Precedence: bulk X-Mailing-List: linux-spi@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Date: Thu, 23 Apr 2026 11:46:28 -0600 From: "Kevin Mehall" To: "Mark Brown" Cc: "Chen-Yu Tsai" , "Jernej Skrabec" , "Samuel Holland" , "Mirko Vogt" , "Ralf Schlatterbeck" , linux-spi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org Message-Id: <9987fdee-7146-4dfd-a34b-d475a96685eb@app.fastmail.com> In-Reply-To: References: <20260420164755.1131645-1-km@kevinmehall.net> <66909301-ed23-4b36-8955-a69b64eed9a1@sirena.org.uk> Subject: Re: [PATCH] spi: sun6i: Set SPI mode in prepare_message Content-Type: text/plain Content-Transfer-Encoding: 7bit I tested with `spi-cs-setup-delay-ns = <1000000>;` and a hardware CS in the device tree, and confirmed the suspected second bug: without this patch, the first transfer after autosuspend ignores the setup delay. > I do see that the driver uses a reset controller over suspend, are you > sure that setup() will be called again on resume? SUN6I_TFR_CTL_REG is indeed being reset after autosuspend because the above bug reoccurs after a few seconds of inactivity. I am not sure if setup() is running on resume because the datasheet reset value of SUN6I_TFR_CTL_CS_LEVEL is the same as what would be written. Either way, it's moot if we set all CS-related bits in the same register write. I've moved the line that sets SUN6I_TFR_CTL_CS_MANUAL into sun6i_spi_set_cs() rather than into sun6i_spi_prepare_message() as a separated commit in a new patch series: https://lore.kernel.org/linux-spi/20260423174001.2797797-1-km@kevinmehall.net/