linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/4] Add 4-bit SPI bus width on target devices
@ 2025-06-24  6:52 yankei.fong
  2025-06-24  6:52 ` [PATCH 1/4] arm64: dts: socfpga: n5x: Add 4-bit SPI bus width yankei.fong
                   ` (4 more replies)
  0 siblings, 5 replies; 9+ messages in thread
From: yankei.fong @ 2025-06-24  6:52 UTC (permalink / raw)
  To: Dinh Nguyen, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	open list, Matthew Gerlach

From: "Fong, Yan Kei" <yan.kei.fong@altera.com>

Add SPI bus width properties to correctly describe the hardware on the following devices:
 - Stratix10
 - Agilex
 - Agilex5
 - N5X

Fong, Yan Kei (4):
  arm64: dts: socfpga: n5x: Add 4-bit SPI bus width
  arm64: dts: socfpga: stratix10: Add 4-bit SPI bus width
  arm64: dts: socfpga: agilex: Add 4-bit SPI bus width
  arm64: dts: socfpga: agilex5: Add 4-bit SPI bus width

 arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts | 2 ++
 arch/arm64/boot/dts/intel/socfpga_agilex5_socdk.dts    | 2 ++
 arch/arm64/boot/dts/intel/socfpga_agilex_socdk.dts     | 2 ++
 arch/arm64/boot/dts/intel/socfpga_n5x_socdk.dts        | 2 ++
 4 files changed, 8 insertions(+)

-- 
2.25.1


^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH 1/4] arm64: dts: socfpga: n5x: Add 4-bit SPI bus width
  2025-06-24  6:52 [PATCH 0/4] Add 4-bit SPI bus width on target devices yankei.fong
@ 2025-06-24  6:52 ` yankei.fong
  2025-08-31  1:40   ` Dinh Nguyen
  2025-06-24  6:52 ` [PATCH 2/4] arm64: dts: socfpga: stratix10: " yankei.fong
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 9+ messages in thread
From: yankei.fong @ 2025-06-24  6:52 UTC (permalink / raw)
  To: Dinh Nguyen, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	open list, Matthew Gerlach

From: "Fong, Yan Kei" <yan.kei.fong@altera.com>

Add spi-tx-bus-width and spi-rx-bus-width properties with
value 4 to the n5x device tree.
This update configures the SPI controller to use a 4-bit
bus width for both transmission and reception,
potentially improving SPI throughput and
matching the hardware capabilities more closely.

Signed-off-by: Fong, Yan Kei <yan.kei.fong@altera.com>
Reviewed-by: Matthew Gerlach <matthew.gerlach@altera.com>
---
 arch/arm64/boot/dts/intel/socfpga_n5x_socdk.dts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm64/boot/dts/intel/socfpga_n5x_socdk.dts b/arch/arm64/boot/dts/intel/socfpga_n5x_socdk.dts
index 7952c7f47cc2..0034a4897220 100644
--- a/arch/arm64/boot/dts/intel/socfpga_n5x_socdk.dts
+++ b/arch/arm64/boot/dts/intel/socfpga_n5x_socdk.dts
@@ -93,6 +93,8 @@ flash@0 {
 		cdns,tsd2d-ns = <50>;
 		cdns,tchsh-ns = <4>;
 		cdns,tslch-ns = <4>;
+		spi-tx-bus-width = <4>;
+		spi-rx-bus-width = <4>;
 
 		partitions {
 			compatible = "fixed-partitions";
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [PATCH 2/4] arm64: dts: socfpga: stratix10: Add 4-bit SPI bus width
  2025-06-24  6:52 [PATCH 0/4] Add 4-bit SPI bus width on target devices yankei.fong
  2025-06-24  6:52 ` [PATCH 1/4] arm64: dts: socfpga: n5x: Add 4-bit SPI bus width yankei.fong
@ 2025-06-24  6:52 ` yankei.fong
  2025-06-24  6:52 ` [PATCH 3/4] arm64: dts: socfpga: agilex: " yankei.fong
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 9+ messages in thread
From: yankei.fong @ 2025-06-24  6:52 UTC (permalink / raw)
  To: Dinh Nguyen, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	open list, Matthew Gerlach

From: "Fong, Yan Kei" <yan.kei.fong@altera.com>

Add spi-tx-bus-width and spi-rx-bus-width properties with
value 4 to the stratix10 device tree.
This update configures the SPI controller to use a 4-bit
bus width for both transmission and reception,
potentially improving SPI throughput and
matching the hardware capabilities more closely.

Signed-off-by: Fong, Yan Kei <yan.kei.fong@altera.com>
Reviewed-by: Matthew Gerlach <matthew.gerlach@altera.com>
---
 arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts b/arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts
index 4eee777ef1a1..6ff6ea0c6b2d 100644
--- a/arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts
+++ b/arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts
@@ -190,6 +190,8 @@ flash@0 {
 		cdns,tsd2d-ns = <50>;
 		cdns,tchsh-ns = <4>;
 		cdns,tslch-ns = <4>;
+		spi-tx-bus-width = <4>;
+		spi-rx-bus-width = <4>;
 
 		partitions {
 			compatible = "fixed-partitions";
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [PATCH 3/4] arm64: dts: socfpga: agilex: Add 4-bit SPI bus width
  2025-06-24  6:52 [PATCH 0/4] Add 4-bit SPI bus width on target devices yankei.fong
  2025-06-24  6:52 ` [PATCH 1/4] arm64: dts: socfpga: n5x: Add 4-bit SPI bus width yankei.fong
  2025-06-24  6:52 ` [PATCH 2/4] arm64: dts: socfpga: stratix10: " yankei.fong
@ 2025-06-24  6:52 ` yankei.fong
  2025-06-24  6:52 ` [PATCH 4/4] arm64: dts: socfpga: agilex5: " yankei.fong
  2025-06-24 17:32 ` [PATCH 0/4] Add 4-bit SPI bus width on target devices Dinh Nguyen
  4 siblings, 0 replies; 9+ messages in thread
From: yankei.fong @ 2025-06-24  6:52 UTC (permalink / raw)
  To: Dinh Nguyen, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	open list, Matthew Gerlach

From: "Fong, Yan Kei" <yan.kei.fong@altera.com>

Add spi-tx-bus-width and spi-rx-bus-width properties with
value 4 to the agilex device tree.
This update configures the SPI controller to use a 4-bit
bus width for both transmission and reception,
potentially improving SPI throughput and
matching the hardware capabilities more closely.

Signed-off-by: Fong, Yan Kei <yan.kei.fong@altera.com>
Reviewed-by: Matthew Gerlach <matthew.gerlach@altera.com>
---
 arch/arm64/boot/dts/intel/socfpga_agilex_socdk.dts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm64/boot/dts/intel/socfpga_agilex_socdk.dts b/arch/arm64/boot/dts/intel/socfpga_agilex_socdk.dts
index b31cfa6b802d..9ee312bae8d2 100644
--- a/arch/arm64/boot/dts/intel/socfpga_agilex_socdk.dts
+++ b/arch/arm64/boot/dts/intel/socfpga_agilex_socdk.dts
@@ -116,6 +116,8 @@ flash@0 {
 		cdns,tsd2d-ns = <50>;
 		cdns,tchsh-ns = <4>;
 		cdns,tslch-ns = <4>;
+		spi-tx-bus-width = <4>;
+		spi-rx-bus-width = <4>;
 
 		partitions {
 			compatible = "fixed-partitions";
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* [PATCH 4/4] arm64: dts: socfpga: agilex5: Add 4-bit SPI bus width
  2025-06-24  6:52 [PATCH 0/4] Add 4-bit SPI bus width on target devices yankei.fong
                   ` (2 preceding siblings ...)
  2025-06-24  6:52 ` [PATCH 3/4] arm64: dts: socfpga: agilex: " yankei.fong
@ 2025-06-24  6:52 ` yankei.fong
  2025-06-24 17:32 ` [PATCH 0/4] Add 4-bit SPI bus width on target devices Dinh Nguyen
  4 siblings, 0 replies; 9+ messages in thread
From: yankei.fong @ 2025-06-24  6:52 UTC (permalink / raw)
  To: Dinh Nguyen, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	open list, Matthew Gerlach

From: "Fong, Yan Kei" <yan.kei.fong@altera.com>

Add spi-tx-bus-width and spi-rx-bus-width properties with
value 4 to the agilex5 device tree.
This update configures the SPI controller to use a 4-bit
bus width for both transmission and reception,
potentially improving SPI throughput and
matching the hardware capabilities more closely.

Signed-off-by: Fong, Yan Kei <yan.kei.fong@altera.com>
Reviewed-by: Matthew Gerlach <matthew.gerlach@altera.com>
---
 arch/arm64/boot/dts/intel/socfpga_agilex5_socdk.dts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm64/boot/dts/intel/socfpga_agilex5_socdk.dts b/arch/arm64/boot/dts/intel/socfpga_agilex5_socdk.dts
index d3b913b7902c..853e260c3976 100644
--- a/arch/arm64/boot/dts/intel/socfpga_agilex5_socdk.dts
+++ b/arch/arm64/boot/dts/intel/socfpga_agilex5_socdk.dts
@@ -57,6 +57,8 @@ flash@0 {
 		cdns,tsd2d-ns = <50>;
 		cdns,tchsh-ns = <4>;
 		cdns,tslch-ns = <4>;
+		spi-tx-bus-width = <4>;
+		spi-rx-bus-width = <4>;
 
 		partitions {
 			compatible = "fixed-partitions";
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH 0/4] Add 4-bit SPI bus width on target devices
  2025-06-24  6:52 [PATCH 0/4] Add 4-bit SPI bus width on target devices yankei.fong
                   ` (3 preceding siblings ...)
  2025-06-24  6:52 ` [PATCH 4/4] arm64: dts: socfpga: agilex5: " yankei.fong
@ 2025-06-24 17:32 ` Dinh Nguyen
  2025-06-30  2:44   ` yankei.fong
  2025-07-09  7:00   ` Fong, Yan Kei
  4 siblings, 2 replies; 9+ messages in thread
From: Dinh Nguyen @ 2025-06-24 17:32 UTC (permalink / raw)
  To: yankei.fong, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	open list, Matthew Gerlach

On 6/24/25 01:52, yankei.fong@altera.com wrote:
> From: "Fong, Yan Kei" <yan.kei.fong@altera.com>
> 
> Add SPI bus width properties to correctly describe the hardware on the following devices:
>   - Stratix10
>   - Agilex
>   - Agilex5
>   - N5X
> 
> Fong, Yan Kei (4):
>    arm64: dts: socfpga: n5x: Add 4-bit SPI bus width
>    arm64: dts: socfpga: stratix10: Add 4-bit SPI bus width
>    arm64: dts: socfpga: agilex: Add 4-bit SPI bus width
>    arm64: dts: socfpga: agilex5: Add 4-bit SPI bus width
> 
>   arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts | 2 ++
>   arch/arm64/boot/dts/intel/socfpga_agilex5_socdk.dts    | 2 ++
>   arch/arm64/boot/dts/intel/socfpga_agilex_socdk.dts     | 2 ++
>   arch/arm64/boot/dts/intel/socfpga_n5x_socdk.dts        | 2 ++
>   4 files changed, 8 insertions(+)
> 

This is for the QSPI driver right? I don't even see the driver using 
this property. So how would this help?

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [PATCH 0/4] Add 4-bit SPI bus width on target devices
  2025-06-24 17:32 ` [PATCH 0/4] Add 4-bit SPI bus width on target devices Dinh Nguyen
@ 2025-06-30  2:44   ` yankei.fong
  2025-07-09  7:00   ` Fong, Yan Kei
  1 sibling, 0 replies; 9+ messages in thread
From: yankei.fong @ 2025-06-30  2:44 UTC (permalink / raw)
  To: dinguyen
  Cc: conor+dt, devicetree, krzk+dt, linux-kernel, matthew.gerlach,
	robh, yankei.fong

From: Fong, Yan Kei <yan.kei.fong@altera.com>

The changes required for the QSPI subsystem. With this implementation, the
read performance will be greater compare to single bus width when trying
to read the QSPI flash chips. Below is the test results:

$cat /sys/kernel/debug/spi-nor/spi0.0/params
...
...
opcodes
read 0x6c  -> from micron QSPI spec, 6c indicates quad output fast read
...
...
protocols
read 1S-1S-4S

^ permalink raw reply	[flat|nested] 9+ messages in thread

* RE: [PATCH 0/4] Add 4-bit SPI bus width on target devices
  2025-06-24 17:32 ` [PATCH 0/4] Add 4-bit SPI bus width on target devices Dinh Nguyen
  2025-06-30  2:44   ` yankei.fong
@ 2025-07-09  7:00   ` Fong, Yan Kei
  1 sibling, 0 replies; 9+ messages in thread
From: Fong, Yan Kei @ 2025-07-09  7:00 UTC (permalink / raw)
  To: Dinh Nguyen, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	open list, Gerlach, Matthew

> On 6/24/25 01:52, yankei.fong@altera.com wrote:
> > From: "Fong, Yan Kei" <yan.kei.fong@altera.com>
> >
> > Add SPI bus width properties to correctly describe the hardware on the
> following devices:
> >   - Stratix10
> >   - Agilex
> >   - Agilex5
> >   - N5X
> >
> > Fong, Yan Kei (4):
> >    arm64: dts: socfpga: n5x: Add 4-bit SPI bus width
> >    arm64: dts: socfpga: stratix10: Add 4-bit SPI bus width
> >    arm64: dts: socfpga: agilex: Add 4-bit SPI bus width
> >    arm64: dts: socfpga: agilex5: Add 4-bit SPI bus width
> >
> >   arch/arm64/boot/dts/altera/socfpga_stratix10_socdk.dts | 2 ++
> >   arch/arm64/boot/dts/intel/socfpga_agilex5_socdk.dts    | 2 ++
> >   arch/arm64/boot/dts/intel/socfpga_agilex_socdk.dts     | 2 ++
> >   arch/arm64/boot/dts/intel/socfpga_n5x_socdk.dts        | 2 ++
> >   4 files changed, 8 insertions(+)
> >
> 
> This is for the QSPI driver right? I don't even see the driver using this property.
> So how would this help?

Yes, it's for QSPI driver. The bindings (spi-tx-bus-width and spi-rx-bus-width) are defined in Documentation/devicetree/bindings/spi/spi-peripheral-props.yaml. They are read in of_spi_parse_dt() which is ultimately called from spi_register_controller().

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH 1/4] arm64: dts: socfpga: n5x: Add 4-bit SPI bus width
  2025-06-24  6:52 ` [PATCH 1/4] arm64: dts: socfpga: n5x: Add 4-bit SPI bus width yankei.fong
@ 2025-08-31  1:40   ` Dinh Nguyen
  0 siblings, 0 replies; 9+ messages in thread
From: Dinh Nguyen @ 2025-08-31  1:40 UTC (permalink / raw)
  To: yankei.fong, Dinh Nguyen, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	open list, Matthew Gerlach



On 6/24/25 01:52, yankei.fong@altera.com wrote:
> From: "Fong, Yan Kei" <yan.kei.fong@altera.com>
> 
> Add spi-tx-bus-width and spi-rx-bus-width properties with
> value 4 to the n5x device tree.
> This update configures the SPI controller to use a 4-bit
> bus width for both transmission and reception,
> potentially improving SPI throughput and
> matching the hardware capabilities more closely.

Please use 80 columns for your commit message! Like this:

Add spi-tx-bus-width and spi-rx-bus-width properties with value 4 to the
n5x device tree. This update configures the SPI controller to use a
4-bit bus width for both transmission and reception, potentially 
improving SPI throughput and matching the hardware capabilities more 
closely.

Doesn't the above make it much easier to read? Please do this fall your 
patches from now on!

Dinh

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2025-08-31  1:40 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-06-24  6:52 [PATCH 0/4] Add 4-bit SPI bus width on target devices yankei.fong
2025-06-24  6:52 ` [PATCH 1/4] arm64: dts: socfpga: n5x: Add 4-bit SPI bus width yankei.fong
2025-08-31  1:40   ` Dinh Nguyen
2025-06-24  6:52 ` [PATCH 2/4] arm64: dts: socfpga: stratix10: " yankei.fong
2025-06-24  6:52 ` [PATCH 3/4] arm64: dts: socfpga: agilex: " yankei.fong
2025-06-24  6:52 ` [PATCH 4/4] arm64: dts: socfpga: agilex5: " yankei.fong
2025-06-24 17:32 ` [PATCH 0/4] Add 4-bit SPI bus width on target devices Dinh Nguyen
2025-06-30  2:44   ` yankei.fong
2025-07-09  7:00   ` Fong, Yan Kei

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).