From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtpout-02.galae.net (smtpout-02.galae.net [185.246.84.56]) (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 5B6AD2F533E; Tue, 20 Jan 2026 15:05:43 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.246.84.56 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768921545; cv=none; b=mw9eMWO810NzisIoK481rA+GgRvc0i4fXFJuuRbZ27YQ6K1CAI9kmwGqyfdszxB6C1lDQSbZk0Z6woD1kThhZx+KR6hQ5W9tC+W9UEf3Uo5UpVvXPASRFkUNvipM7bsks0fSPz9ACCM6zNp/TP9gdMXxskz5mv4wCR8Kqn30p5Y= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1768921545; c=relaxed/simple; bh=0ler8iCtqk2EiGfm/2ocxMEnXXml/ihrwsqRs5L067M=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=s6a7M/K9uTkyExHTJHbG5rAMMlmErIkn0j+FHJAUDkipJEmrijDmCmJzJieHIDAUedvbFj2vnbkUm/UGfvUhmRykchIt16HbuBElijq0vn28LJJ787iUW4+5kJ5dRwqM09vo7wKhszeX2/7QpF1+jW66YW6C16Jx0cGgM6JKekw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=oaYWcXbs; arc=none smtp.client-ip=185.246.84.56 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="oaYWcXbs" Received: from smtpout-01.galae.net (smtpout-01.galae.net [212.83.139.233]) by smtpout-02.galae.net (Postfix) with ESMTPS id 8EEE91A2973; Tue, 20 Jan 2026 15:05:41 +0000 (UTC) Received: from mail.galae.net (mail.galae.net [212.83.136.155]) by smtpout-01.galae.net (Postfix) with ESMTPS id 62337606AB; Tue, 20 Jan 2026 15:05:41 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 9FF8B10B6943D; Tue, 20 Jan 2026 16:05:36 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=dkim; t=1768921540; h=from:subject:date:message-id:to:cc:mime-version:content-type: content-transfer-encoding:in-reply-to:references; bh=g12cAMZNPPtUUPqHd1kwBTs8XQb3e9bnrWcT2IaKQyg=; b=oaYWcXbsd4xRk588Tj9Z0c2I8AIf1b2pBiPrk3C+vTgsUfvrKVkgx7S3SyYVauN66Vi1uK rp3hkKpJv0D7y7S9sWd6KXau6mev7uMm6590rS0t25cBNpvJsbOqhMNHyO8FlycPakn8cp jv6J03DfnHP/5lycRu4WZ0UAieIPF2XGyjYv96FjHRxVJbNHmVIg3HGMMMUJC9gNoeW7av tFj3Tqly0nfBaqDI6jdtn0WMneBbuF+upADv+1Z01BojfDQTNdi5Uu0FStQv/YH9qRO6sX lEy+nDaqk1cljv+VjOJH2IOHjtZ6Dbx8MoMQJsPvebjwW386W8xERbkJiThmbA== From: Miquel Raynal To: Santhosh Kumar K Cc: "Mark Brown" , Rob Herring , "Krzysztof Kozlowski" , Conor Dooley , "Geert Uytterhoeven" , Magnus Damm , Vaishnav Achath , Thomas Petazzoni , =?utf-8?Q?Herv=C3=A9?= Codina , Wolfram Sang , Vignesh Raghavendra , Pratyush Yadav , Pascal Eberhard , , , , Subject: Re: [PATCH v2 00/13] spi: cadence-qspi: Add Renesas RZ/N1 support In-Reply-To: <80e7a578-4636-48bd-b92b-54fa33cc076d@ti.com> (Santhosh Kumar K.'s message of "Tue, 20 Jan 2026 14:52:38 +0530") References: <20260115-schneider-6-19-rc1-qspi-v2-0-7e6a06e1e17b@bootlin.com> <80e7a578-4636-48bd-b92b-54fa33cc076d@ti.com> User-Agent: mu4e 1.12.7; emacs 30.2 Date: Tue, 20 Jan 2026 16:05:35 +0100 Message-ID: <87cy34s4sg.fsf@bootlin.com> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Last-TLS-Session-Version: TLSv1.3 Hi Santhosh, On 20/01/2026 at 14:52:38 +0530, Santhosh Kumar K wrote: > Hello Miquel, > > On 15/01/26 14:54, Miquel Raynal (Schneider Electric) wrote: >> Hello, >> This series adds support for the QSPI controller available on Renesas >> RZ/N1S and RZ/N1D SoC. It has been tested with a custom board (see last >> SPI patch for details). >> Adding support for this SoC required a few adaptations in the Cadence >> QSPI driver. The bulk of the work is in the few last patches. Everything >> else is just misc style fixes and improvements which bothered me while I >> was wandering. >> In order to support all constraints, I sometimes used a new quirk (for >> the write protection feature and the "no indirect mode"), and sometimes >> used the compatible directly. The ones I thought might not be RZ/N1 >> specific have been implemented under the form of a quirk, in order to >> ease their reuse. The other adaptations, which I believe are more >> Renesas specific, have been handled using the compatible. This is all >> very arbitrary, and can be discussed. >> Thanks, >> Miqu=C3=A8l >> Signed-off-by: Miquel Raynal (Schneider Electric) >> > > Thank you for the series! Tested it on TI's AM62A SK with > OSPI NAND (Winbond's W35N01JW). > > Controller fails to probe with the following message: > > [ 1.868863] cadence-qspi fc40000.spi: Cannot claim mandatory QSPI ref > clock. Strange, I was nonetheless careful not to change the existing behaviour on other SoCs. I have that board, I will give it a try. Thanks a lot for the feedback. Miqu=C3=A8l