public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [PATCH] arm: mvebu: Espressobin: Set the maximum slave SPI speed to 40MHz
@ 2021-01-27 16:12 Pali Rohár
  2021-02-02 14:56 ` Stefan Roese
  2021-02-08 11:39 ` Stefan Roese
  0 siblings, 2 replies; 3+ messages in thread
From: Pali Rohár @ 2021-01-27 16:12 UTC (permalink / raw)
  To: u-boot

From: Konstantin Porotchkin <kostap@marvell.com>

While the SPI controller speed is defined by DTS, the maximum
slave speed (connected devices) is limited by the pre-defined
configuration value CONFIG_SF_DEFAULT_SPEED to 1MHz
This patch increases this maximum SPI slave device speed to 40MHz

Change-Id: I0d1239bd8a2061c66725c2c227c1e1f49c92c29e
Signed-off-by: Konstantin Porotchkin <kostap@marvell.com>
Reviewed-on: http://vgitil04.il.marvell.com:8080/59516
Tested-by: iSoC Platform CI <ykjenk@marvell.com>
Reviewed-by: Igal Liberman <igall@marvell.com>
[pali: Set CONFIG_SF_DEFAULT_SPEED via defconfig]
Signed-off-by: Pali Roh?r <pali@kernel.org>
---
 configs/mvebu_espressobin-88f3720_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/configs/mvebu_espressobin-88f3720_defconfig b/configs/mvebu_espressobin-88f3720_defconfig
index 8a859cba2e..4956199ccd 100644
--- a/configs/mvebu_espressobin-88f3720_defconfig
+++ b/configs/mvebu_espressobin-88f3720_defconfig
@@ -54,6 +54,7 @@ CONFIG_MMC_SDHCI_SDMA=y
 CONFIG_MMC_SDHCI_XENON=y
 CONFIG_MTD=y
 CONFIG_SF_DEFAULT_MODE=0
+CONFIG_SF_DEFAULT_SPEED=40000000
 CONFIG_SPI_FLASH_GIGADEVICE=y
 CONFIG_SPI_FLASH_ISSI=y
 CONFIG_SPI_FLASH_MACRONIX=y
-- 
2.20.1

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

* [PATCH] arm: mvebu: Espressobin: Set the maximum slave SPI speed to 40MHz
  2021-01-27 16:12 [PATCH] arm: mvebu: Espressobin: Set the maximum slave SPI speed to 40MHz Pali Rohár
@ 2021-02-02 14:56 ` Stefan Roese
  2021-02-08 11:39 ` Stefan Roese
  1 sibling, 0 replies; 3+ messages in thread
From: Stefan Roese @ 2021-02-02 14:56 UTC (permalink / raw)
  To: u-boot

On 27.01.21 17:12, Pali Roh?r wrote:
> From: Konstantin Porotchkin <kostap@marvell.com>
> 
> While the SPI controller speed is defined by DTS, the maximum
> slave speed (connected devices) is limited by the pre-defined
> configuration value CONFIG_SF_DEFAULT_SPEED to 1MHz
> This patch increases this maximum SPI slave device speed to 40MHz
> 
> Change-Id: I0d1239bd8a2061c66725c2c227c1e1f49c92c29e
> Signed-off-by: Konstantin Porotchkin <kostap@marvell.com>
> Reviewed-on: http://vgitil04.il.marvell.com:8080/59516
> Tested-by: iSoC Platform CI <ykjenk@marvell.com>
> Reviewed-by: Igal Liberman <igall@marvell.com>
> [pali: Set CONFIG_SF_DEFAULT_SPEED via defconfig]
> Signed-off-by: Pali Roh?r <pali@kernel.org>

Reviewed-by: Stefan Roese <sr@denx.de>

Thanks,
Stefan

> ---
>   configs/mvebu_espressobin-88f3720_defconfig | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/configs/mvebu_espressobin-88f3720_defconfig b/configs/mvebu_espressobin-88f3720_defconfig
> index 8a859cba2e..4956199ccd 100644
> --- a/configs/mvebu_espressobin-88f3720_defconfig
> +++ b/configs/mvebu_espressobin-88f3720_defconfig
> @@ -54,6 +54,7 @@ CONFIG_MMC_SDHCI_SDMA=y
>   CONFIG_MMC_SDHCI_XENON=y
>   CONFIG_MTD=y
>   CONFIG_SF_DEFAULT_MODE=0
> +CONFIG_SF_DEFAULT_SPEED=40000000
>   CONFIG_SPI_FLASH_GIGADEVICE=y
>   CONFIG_SPI_FLASH_ISSI=y
>   CONFIG_SPI_FLASH_MACRONIX=y
> 


Viele Gr??e,
Stefan

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr at denx.de

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

* [PATCH] arm: mvebu: Espressobin: Set the maximum slave SPI speed to 40MHz
  2021-01-27 16:12 [PATCH] arm: mvebu: Espressobin: Set the maximum slave SPI speed to 40MHz Pali Rohár
  2021-02-02 14:56 ` Stefan Roese
@ 2021-02-08 11:39 ` Stefan Roese
  1 sibling, 0 replies; 3+ messages in thread
From: Stefan Roese @ 2021-02-08 11:39 UTC (permalink / raw)
  To: u-boot

On 27.01.21 17:12, Pali Roh?r wrote:
> From: Konstantin Porotchkin <kostap@marvell.com>
> 
> While the SPI controller speed is defined by DTS, the maximum
> slave speed (connected devices) is limited by the pre-defined
> configuration value CONFIG_SF_DEFAULT_SPEED to 1MHz
> This patch increases this maximum SPI slave device speed to 40MHz
> 
> Change-Id: I0d1239bd8a2061c66725c2c227c1e1f49c92c29e
> Signed-off-by: Konstantin Porotchkin <kostap@marvell.com>
> Reviewed-on: http://vgitil04.il.marvell.com:8080/59516
> Tested-by: iSoC Platform CI <ykjenk@marvell.com>
> Reviewed-by: Igal Liberman <igall@marvell.com>
> [pali: Set CONFIG_SF_DEFAULT_SPEED via defconfig]
> Signed-off-by: Pali Roh?r <pali@kernel.org>

Applied to u-boot-marvell/master

Thanks,
Stefan

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

end of thread, other threads:[~2021-02-08 11:39 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-01-27 16:12 [PATCH] arm: mvebu: Espressobin: Set the maximum slave SPI speed to 40MHz Pali Rohár
2021-02-02 14:56 ` Stefan Roese
2021-02-08 11:39 ` Stefan Roese

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox