* [U-Boot] [PATCH] km/arm: set SPI NOR Flash default parameters
@ 2012-08-14 11:39 Valentin Longchamp
2012-08-14 13:17 ` Prafulla Wadaskar
0 siblings, 1 reply; 5+ messages in thread
From: Valentin Longchamp @ 2012-08-14 11:39 UTC (permalink / raw)
To: u-boot
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
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] km/arm: set SPI NOR Flash default parameters
2012-08-14 11:39 [U-Boot] [PATCH] km/arm: set SPI NOR Flash default parameters Valentin Longchamp
@ 2012-08-14 13:17 ` Prafulla Wadaskar
2012-08-15 15:16 ` Valentin Longchamp
0 siblings, 1 reply; 5+ messages in thread
From: Prafulla Wadaskar @ 2012-08-14 13:17 UTC (permalink / raw)
To: u-boot
> -----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 . . .
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] km/arm: set SPI NOR Flash default parameters
2012-08-14 13:17 ` Prafulla Wadaskar
@ 2012-08-15 15:16 ` Valentin Longchamp
2012-08-16 4:15 ` Prafulla Wadaskar
2012-09-03 8:21 ` Prafulla Wadaskar
0 siblings, 2 replies; 5+ messages in thread
From: Valentin Longchamp @ 2012-08-15 15:16 UTC (permalink / raw)
To: u-boot
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] km/arm: set SPI NOR Flash default parameters
2012-08-15 15:16 ` Valentin Longchamp
@ 2012-08-16 4:15 ` Prafulla Wadaskar
2012-09-03 8:21 ` Prafulla Wadaskar
1 sibling, 0 replies; 5+ messages in thread
From: Prafulla Wadaskar @ 2012-08-16 4:15 UTC (permalink / raw)
To: u-boot
> -----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 . . .
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] km/arm: set SPI NOR Flash default parameters
2012-08-15 15:16 ` Valentin Longchamp
2012-08-16 4:15 ` Prafulla Wadaskar
@ 2012-09-03 8:21 ` Prafulla Wadaskar
1 sibling, 0 replies; 5+ messages in thread
From: Prafulla Wadaskar @ 2012-09-03 8:21 UTC (permalink / raw)
To: u-boot
> -----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 . . .
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-09-03 8:21 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-08-14 11:39 [U-Boot] [PATCH] km/arm: set SPI NOR Flash default parameters Valentin Longchamp
2012-08-14 13:17 ` Prafulla Wadaskar
2012-08-15 15:16 ` Valentin Longchamp
2012-08-16 4:15 ` Prafulla Wadaskar
2012-09-03 8:21 ` Prafulla Wadaskar
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox