km/arm: set SPI NOR Flash default parameters

PATCH New Valentin Longchamp 2 participants 5 messages Aug 14, 2012 – Sep 3, 2012

Share & tools

Valentin Longchamp

author

2012-08-14 11:39 UTC #0 lore raw

PATCH 1/1 km/arm: set SPI NOR Flash default parameters

These parameters are used by the the sf probe command that are used by our update script and they therefore need to be set for all of our boards.

The timing is the same as for the ENV SPI NOR Flash (since it's the same physical device) and takes the boco2 delay on the bus into account.

Signed-off-by Valentin Longchamp <[email protected]>

cc Holger Brunck <[email protected]>

---
 include/configs/km/km_arm.h |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

include/configs/km/km_arm.h +4 -0

index 2d573e4..055d946 100644
@@ -57,6 +57,10 @@
 #define CONFIG_CMD_SF
 #define CONFIG_SOFT_I2C		/* I2C bit-banged	*/
 
+/* SPI NOR Flash default params, used by sf commands */
+#define CONFIG_SF_DEFAULT_SPEED		8100000
+#define CONFIG_SF_DEFAULT_MODE		SPI_MODE_3
+
 #if defined CONFIG_KM_ENV_IS_IN_SPI_NOR
 #define CONFIG_ENV_SPI_BUS		0
 #define CONFIG_ENV_SPI_CS		0

Signature · 1 line

1.7.1

Prafulla Wadaskar

2012-08-14 13:17 UTC #1 lore raw

PATCH 1/1 km/arm: set SPI NOR Flash default parameters

Quoted 9 lines

-----Original Message-----
From: Valentin Longchamp [mailto:valentin.longchamp at keymile.com]
Sent: 14 August 2012 17:09
To: Prafulla Wadaskar
Cc: u-boot at lists.denx.de; Valentin Longchamp; Holger Brunck
Subject: [PATCH] km/arm: set SPI NOR Flash default parameters

Quoted 6 lines

These parameters are used by the the sf probe command that are used by our update script and they therefore need to be set for all of our boards.
The timing is the same as for the ENV SPI NOR Flash (since it's the same physical device) and takes the boco2 delay on the bus into

What is boco2? Is it typo mistake?

Quoted 14 lines

account.
Signed-off-by Valentin Longchamp <[email protected]>
cc Holger Brunck <[email protected]>
---
 include/configs/km/km_arm.h |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)
include/configs/km/km_arm.h +0 -0
index 2d573e4..055d946 100644
@@ -57,6 +57,10 @@
 #define CONFIG_CMD_SF

include/configs/km/km_arm.h

Quoted 10 lines

include/configs/km/km_arm.h +4 -1

 #define CONFIG_SOFT_I2C		/* I2C bit-banged	*/
 
+/* SPI NOR Flash default params, used by sf commands */
+#define CONFIG_SF_DEFAULT_SPEED		8100000
+#define CONFIG_SF_DEFAULT_MODE		SPI_MODE_3
+
 #if defined CONFIG_KM_ENV_IS_IN_SPI_NOR
 #define CONFIG_ENV_SPI_BUS		0
 #define CONFIG_ENV_SPI_CS		0
--

Regards...
Prafulla . . .

Valentin Longchamp

author

2012-08-15 15:16 UTC #2 lore raw

PATCH 1/1 km/arm: set SPI NOR Flash default parameters

Hi Prafulla,

Quoted 9 lines · Prafulla Wadaskar

Quoted 9 lines
-----Original Message-----
From: Valentin Longchamp [mailto:valentin.longchamp at keymile.com]
Sent: 14 August 2012 17:09
To: Prafulla Wadaskar
Cc: u-boot at lists.denx.de; Valentin Longchamp; Holger Brunck
Subject: [PATCH] km/arm: set SPI NOR Flash default parameters

Quoted 8 lines

Quoted 6 lines
These parameters are used by the the sf probe command that are used by our update script and they therefore need to be set for all of our boards.
The timing is the same as for the ENV SPI NOR Flash (since it's the same physical device) and takes the boco2 delay on the bus into
What is boco2? Is it typo mistake?

No typo mistake. It's the name of a chip developped at Keymile that is the the SPI bus as well.

Quoted 29 lines

Quoted 15 lines
account.
Signed-off-by Valentin Longchamp <[email protected]>
cc Holger Brunck <[email protected]>
---
 include/configs/km/km_arm.h |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)
include/configs/km/km_arm.h +0 -0
index 2d573e4..055d946 100644
@@ -57,6 +57,10 @@
 #define CONFIG_CMD_SF
include/configs/km/km_arm.h
Quoted 10 lines
include/configs/km/km_arm.h +4 -1
 #define CONFIG_SOFT_I2C		/* I2C bit-banged	*/
 
+/* SPI NOR Flash default params, used by sf commands */
+#define CONFIG_SF_DEFAULT_SPEED		8100000
+#define CONFIG_SF_DEFAULT_MODE		SPI_MODE_3
+
 #if defined CONFIG_KM_ENV_IS_IN_SPI_NOR
 #define CONFIG_ENV_SPI_BUS		0
 #define CONFIG_ENV_SPI_CS		0
--
Regards...
Prafulla . . .

Signature · 13 lines

Valentin Longchamp
Embedded Software Engineer
Hardware and Chip Integration
______________________________________
KEYMILE AG
Schwarzenburgstr. 73
CH-3097 Liebefeld
Phone +41 31 377 1318
Fax   +41 31 377 1212
valentin.longchamp at keymile.com
www.keymile.com
______________________________________
KEYMILE: A Specialist as a Partner

Prafulla Wadaskar

2012-08-16 04:15 UTC #3 lore raw

PATCH 1/1 km/arm: set SPI NOR Flash default parameters

Quoted 26 lines

-----Original Message-----
From: Valentin Longchamp [mailto:valentin.longchamp at keymile.com]
Sent: 15 August 2012 20:46
To: Prafulla Wadaskar
Cc: u-boot at lists.denx.de; Holger Brunck
Subject: Re: [PATCH] km/arm: set SPI NOR Flash default parameters
Hi Prafulla,
Quoted 10 lines · Prafulla Wadaskar
Quoted 10 lines
-----Original Message-----
From: Valentin Longchamp [mailto:valentin.longchamp at keymile.com]
Sent: 14 August 2012 17:09
To: Prafulla Wadaskar
Cc: u-boot at lists.denx.de; Valentin Longchamp; Holger Brunck
Subject: [PATCH] km/arm: set SPI NOR Flash default parameters
These parameters are used by the the sf probe command that are used
by
Quoted 3 lines
Quoted 3 lines
our update script and they therefore need to be set for all of our boards.

Quoted 8 lines

Quoted 4 lines
Quoted 2 lines
The timing is the same as for the ENV SPI NOR Flash (since it's the same physical device) and takes the boco2 delay on the bus into
What is boco2? Is it typo mistake?
No typo mistake. It's the name of a chip developped at Keymile that is the the
SPI bus as well.

Thanks for clarification.

Acked-by Prafulla Wadaskar <[email protected]>

Will pull this patch soon.

Regards...
Prafulla . . .

Prafulla Wadaskar

2012-09-03 08:21 UTC #4 lore raw

PATCH 1/1 km/arm: set SPI NOR Flash default parameters

In reply to Valentin Longchamp

Quoted 44 lines

-----Original Message-----
From: Prafulla Wadaskar
Sent: 16 August 2012 09:46
To: 'Valentin Longchamp'
Cc: u-boot at lists.denx.de; Holger Brunck
Subject: RE: [PATCH] km/arm: set SPI NOR Flash default parameters
Quoted 7 lines
-----Original Message-----
From: Valentin Longchamp [mailto:valentin.longchamp at keymile.com]
Sent: 15 August 2012 20:46
To: Prafulla Wadaskar
Cc: u-boot at lists.denx.de; Holger Brunck
Subject: Re: [PATCH] km/arm: set SPI NOR Flash default parameters
Quoted 13 lines
Hi Prafulla,
Quoted 10 lines · Prafulla Wadaskar
Quoted 10 lines
-----Original Message-----
From: Valentin Longchamp [mailto:valentin.longchamp at keymile.com]
Sent: 14 August 2012 17:09
To: Prafulla Wadaskar
Cc: u-boot at lists.denx.de; Valentin Longchamp; Holger Brunck
Subject: [PATCH] km/arm: set SPI NOR Flash default parameters
These parameters are used by the the sf probe command that are
used
Quoted 2 lines
by
Quoted 1 line
Quoted 1 line
our update script and they therefore need to be set for all of
our
Quoted 3 lines
Quoted 3 lines
Quoted 3 lines
boards.
The timing is the same as for the ENV SPI NOR Flash (since it's
the
Quoted 5 lines
Quoted 3 lines
Quoted 1 line
same physical device) and takes the boco2 delay on the bus into
What is boco2? Is it typo mistake?
No typo mistake. It's the name of a chip developped at Keymile that

Quoted 8 lines

is
Quoted 2 lines
the the
SPI bus as well.
Thanks for clarification.
Acked-by Prafulla Wadaskar <[email protected]>
Will pull this patch soon.

Applied to u-boot-marvell.git master branch

Regards...
Prafulla . . .