From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f178.google.com (mail-pl1-f178.google.com [209.85.214.178]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5615C136E17 for ; Thu, 4 Jul 2024 17:53:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.178 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720115605; cv=none; b=sFtYllBGbT3zfwAfjQATh3iIv7V5RrhgpuOWhqLOYAwT68A+QzCAZi1fwyHlEzpT2JUEPxOy1ARSkZjS4nVLTEXehYDqUFi7OJl1vFQJ5jYkrHSmbkCDg0FJPMPOM2UJekHriAAmS3VV2J2d7/I7zSKbPR5U9hL9bAaFWWj3B+A= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720115605; c=relaxed/simple; bh=l1uxIG0dbpQ3vxd8XuFSAJOOykrgwtWmdtGyfes2CDE=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=Wj+5QElh628oNe3/0R2IXRqGZIqv0p50Nni5QDAqUovf4tGNQaYVSAyxIhYCK/33R7YX7IAtD0nWtB+FIVAVbJhVP96n3FaReghWOK8gYVKTDYprxo3hLKzGJGRjPHOFf8RMhRbgPeD3sn3Z3XMDqm0DhXd4SCmxzjkr9TIXOv8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=pdp7.com; spf=none smtp.mailfrom=pdp7.com; dkim=pass (2048-bit key) header.d=pdp7-com.20230601.gappssmtp.com header.i=@pdp7-com.20230601.gappssmtp.com header.b=FkKLwZUH; arc=none smtp.client-ip=209.85.214.178 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=pdp7.com Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=pdp7.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=pdp7-com.20230601.gappssmtp.com header.i=@pdp7-com.20230601.gappssmtp.com header.b="FkKLwZUH" Received: by mail-pl1-f178.google.com with SMTP id d9443c01a7336-1fb222a8eaeso6527185ad.3 for ; Thu, 04 Jul 2024 10:53:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=pdp7-com.20230601.gappssmtp.com; s=20230601; t=1720115603; x=1720720403; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=8uryZ79GMK3Hbevhm1GUwmInwdQUDjClI1eiCDdIwGw=; b=FkKLwZUHYrqqT/KTWdrSJ82iX7uPO5AUF2mxxnc+SoMCak83WJg9PUB/UzDytTK+xg N4Q4FC9anjKKv8fkkfN4fn6/oETpAfrlLE2ci5wDeddEsGh4jNpFIDFtGDAOofn1SiIn 9kZT5aeFF2fAhHnrj1gZ6AExEp5Fpc57pf3QJMJu6KhCyO9jZ/tWBfm2pMqvNc8lsqNT ORPQxy+sytZolrHWF83Rd2Zhb4WST+mFb/7oj5/sJyxKfTQ5xqueb3aUbk099PTkHBU1 tfs5KN1qTxgHA7QMVKG688LwtbFko1TMjzkl1oDamGDvfOEf1m8b9rPU/k86Ylre8Nks cMJg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1720115603; x=1720720403; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=8uryZ79GMK3Hbevhm1GUwmInwdQUDjClI1eiCDdIwGw=; b=uVRoAM5pK+RnIiyi+pWOc1QgyDnKjChH9fJ5KoMW3Qzl74UmkGnx+dreVpBdmWqCu5 nC5+Y/IsdeQkS5TPB1D7o4f3bA4N5pwP1xyN6soSAw9OcepeKHfQrn5Fw5k1gqdYKKLv NK+KTeQOwp7ka+wmEldBQK6z/Oth5GHIGnn1R1vTObLqN/QfJcWaO+P7f5OOXYMx2mCR 4yy3x6ZEqq4XRgG7X027Hl1gQrJN9MEkUuGTFjA3OKeqXnA8/n/JADKF90UCMbhuBROX k1qlLnNDohdOAQ4KBXMdhH1aWZpTWZwj2ORoXHAPiDmdDcsitxn8uJ1140j3S1zN5K1w xh2Q== X-Forwarded-Encrypted: i=1; AJvYcCVUIp3JdpKOOzqqYS7yfxqU5dRv1zHYeqZvk6m9L2jqb4gKjrXWzZRoQix3SRMIiVSoTjUH9/yC7nKaTRsvzd61gFeLQIjwdbbM X-Gm-Message-State: AOJu0Ywoc4CdmvyiVWQxXuxZuEvvK7V1nbvlvH7FFxE14YqmNHwUhDrU aJ+p8+c9PVSP85VRTowlQtnvo9lBoxrNMHmmDsgJiwePN0v/utcbVAjEzJ+l7YQ= X-Google-Smtp-Source: AGHT+IHr2Ly4uGxHjqpvelPSNVZYUdi1gXcvcSsc8YvBTQFpKVrUUJwR8TdaP1EKvh1cnQR14j81oQ== X-Received: by 2002:a17:903:192:b0:1fa:cec7:9a9d with SMTP id d9443c01a7336-1fb33e0de4cmr17233465ad.15.1720115603590; Thu, 04 Jul 2024 10:53:23 -0700 (PDT) Received: from x1 ([2601:1c2:1802:170:8756:5e46:4951:e2d]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-1fb3dbd7dcdsm10711545ad.157.2024.07.04.10.53.22 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 04 Jul 2024 10:53:23 -0700 (PDT) Date: Thu, 4 Jul 2024 10:53:21 -0700 From: Drew Fustini To: Drew Fustini Cc: Conor Dooley , Kanak Shilledar , Serge Semin , Mark Brown , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Jisheng Zhang , Guo Ren , Fu Wei , Paul Walmsley , Palmer Dabbelt , Albert Ou , linux-spi@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org Subject: Re: [PATCH v2 3/3] riscv: dts: thead: add basic spi node Message-ID: References: <20240701121355.262259-2-kanakshilledar@gmail.com> <20240701121355.262259-5-kanakshilledar@gmail.com> <20240703-juice-refreeze-62c468a56ea5@spud> Precedence: bulk X-Mailing-List: linux-spi@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Thu, Jul 04, 2024 at 09:59:20AM -0700, Drew Fustini wrote: > On Wed, Jul 03, 2024 at 03:45:37PM +0100, Conor Dooley wrote: > > Kanak, Drew, > > > > On Mon, Jul 01, 2024 at 05:43:54PM +0530, Kanak Shilledar wrote: > > > created spi0 node with fixed clock. the spi0 node > > > uses synopsis designware driver and has the following > > > compatible "snps,dw-apb-ssi". the spi0 node is connected > > > to a SPI NOR flash pad which is left unpopulated on the back > > > side of the board. > > > > > > Signed-off-by: Kanak Shilledar > > > --- > > > Changes in v2: > > > - Separated from a single patch file > > > --- > > > .../boot/dts/thead/th1520-beaglev-ahead.dts | 9 +++++++++ > > > .../boot/dts/thead/th1520-lichee-module-4a.dtsi | 4 ++++ > > > .../riscv/boot/dts/thead/th1520-lichee-pi-4a.dts | 5 +++++ > > > > Didn't you say there was a flash on one of these two boards? > > > > > arch/riscv/boot/dts/thead/th1520.dtsi | 16 ++++++++++++++++ > > > 4 files changed, 34 insertions(+) > > > > > > diff --git a/arch/riscv/boot/dts/thead/th1520-beaglev-ahead.dts b/arch/riscv/boot/dts/thead/th1520-beaglev-ahead.dts > > > index d9b4de9e4757..3103b74e0288 100644 > > > --- a/arch/riscv/boot/dts/thead/th1520-beaglev-ahead.dts > > > +++ b/arch/riscv/boot/dts/thead/th1520-beaglev-ahead.dts > > > @@ -17,6 +17,7 @@ aliases { > > > gpio1 = &gpio1; > > > gpio2 = &gpio2; > > > gpio3 = &gpio3; > > > + spi0 = &spi0; > > > > "spi" would sort after "serial". > > > > > serial0 = &uart0; > > > serial1 = &uart1; > > > serial2 = &uart2; > > > @@ -52,6 +53,10 @@ &sdhci_clk { > > > clock-frequency = <198000000>; > > > }; > > > > > > +&spi_clk { > > > + clock-frequency = <396000000>; > > > +}; > > > > I'm pretty sceptical about adding more of these fixed clocks, rather > > than waiting for the clock driver. Drew, what do you think? Should we > > just add one more to your fixup list or would you rather delay? Guess it > > depends on how long more you think that clock driver is likely to take. > > I think the clk driver [1] is in good shape but it has not been reviewed > by the clk maintainer yet. Thus it is hard to predict any timeline for > it getting merged. > > SPI support doesn't require any driver changes so I'd be inclined to > allow the fixed clock in this case. It will be simple to change it over > to a real clock once the clk driver is upstream. > > Acked-by: Drew Fustini > > Thanks, > Drew Sorry, I meant to include the link to the most recent clk series: https://lore.kernel.org/linux-riscv/20240623-th1520-clk-v2-0-ad8d6432d9fb@tenstorrent.com/ -Drew