[U-Boot] [PATCH] km/arm: set SPI NOR Flash default parameters

[U-Boot] [PATCH] km/arm: set SPI NOR Flash default parameters

[Valentin Longchamp]

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 <valentin.longchamp@keymile.com>
cc: Holger Brunck <holger.brunck@keymile.com>
---
 include/configs/km/km_arm.h |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/include/configs/km/km_arm.h b/include/configs/km/km_arm.h
index 2d573e4..055d946 100644
--- a/include/configs/km/km_arm.h
+++ b/include/configs/km/km_arm.h
@@ -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
-- 
1.7.1

[U-Boot] [PATCH] km/arm: set SPI NOR Flash default parameters

[Prafulla Wadaskar]

> -----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
> 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?

> account.
> 
> Signed-off-by: Valentin Longchamp <valentin.longchamp@keymile.com>
> cc: Holger Brunck <holger.brunck@keymile.com>
> ---
>  include/configs/km/km_arm.h |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/include/configs/km/km_arm.h b/include/configs/km/km_arm.h
> index 2d573e4..055d946 100644
> --- a/include/configs/km/km_arm.h
> +++ b/include/configs/km/km_arm.h
> @@ -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
> --

Regards...
Prafulla . . .

[U-Boot] [PATCH] km/arm: set SPI NOR Flash default parameters

[Valentin Longchamp]

Hi Prafulla,

On 08/14/2012 03:17 PM, Prafulla Wadaskar wrote:
> 
> 
>> -----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
>> 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.

> 
>> account.
>>
>> Signed-off-by: Valentin Longchamp <valentin.longchamp@keymile.com>
>> cc: Holger Brunck <holger.brunck@keymile.com>
>> ---
>>  include/configs/km/km_arm.h |    4 ++++
>>  1 files changed, 4 insertions(+), 0 deletions(-)
>>
>> diff --git a/include/configs/km/km_arm.h b/include/configs/km/km_arm.h
>> index 2d573e4..055d946 100644
>> --- a/include/configs/km/km_arm.h
>> +++ b/include/configs/km/km_arm.h
>> @@ -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
>> --
> 
> Regards...
> Prafulla . . .
> 


-- 
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

[U-Boot] [PATCH] km/arm: set SPI NOR Flash default parameters

[Prafulla Wadaskar]

> -----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,
> 
> On 08/14/2012 03:17 PM, Prafulla Wadaskar wrote:
> >
> >
> >> -----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
> >> 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.

Thanks for clarification.

Acked-by: Prafulla Wadaskar <Prafulla@marvell.com>
Will pull this patch soon.

Regards...
Prafulla . . .

[U-Boot] [PATCH] km/arm: set SPI NOR Flash default parameters

[Prafulla Wadaskar]

> -----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
> 
> 
> 
> > -----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,
> >
> > On 08/14/2012 03:17 PM, Prafulla Wadaskar wrote:
> > >
> > >
> > >> -----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
> > >> 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.
> 
> Thanks for clarification.
> 
> Acked-by: Prafulla Wadaskar <Prafulla@marvell.com>
> Will pull this patch soon.

Applied to u-boot-marvell.git master branch

Regards...
Prafulla . . .