diff mbox series

[RESEND,v4] net: pfe_eth: Use spi_flash_read API to access flash memory

Message ID 1590646373-8795-1-git-send-email-kuldeep.singh@nxp.com
State Accepted
Commit e0152dbed683ed02af9294551fd4f03823ef7a5a
Headers show
Series [RESEND,v4] net: pfe_eth: Use spi_flash_read API to access flash memory | expand

Commit Message

Kuldeep Singh May 28, 2020, 6:12 a.m. UTC
Current PFE firmware access spi-nor memory directly. New spi-mem
framework does not support direct memory access. So, let's use
spi_flash_read API to access memory instead of directly using it.

Signed-off-by: Kuldeep Singh <kuldeep.singh at nxp.com>
Reviewed-by: Frieder Schrempf <frieder.schrempf at kontron.de>
---
v4:
-Rebase to top
-Use complete firmware size
-Return -ENODEV if flash probe fails
v3:
-Replace ret with 0 if flash probe fails
v2:
-Add return error code
-Changes in displayed error log

 drivers/net/pfe_eth/pfe_firmware.c | 45 +++++++++++++++++++++++++++++++++++++-
 include/configs/ls1012a_common.h   |  5 ++++-
 2 files changed, 48 insertions(+), 2 deletions(-)

Comments

Frieder Schrempf May 28, 2020, 8:16 a.m. UTC | #1
On 28.05.20 08:12, Kuldeep Singh wrote:
> Current PFE firmware access spi-nor memory directly. New spi-mem
> framework does not support direct memory access. So, let's use
> spi_flash_read API to access memory instead of directly using it.
> 
> Signed-off-by: Kuldeep Singh <kuldeep.singh at nxp.com>
> Reviewed-by: Frieder Schrempf <frieder.schrempf at kontron.de>

So this patch has been floating around for about half a year now with 
almost no attention from the maintainers! Several pings have been sent 
by the author without response.

In fact this patch was blocking 91afd36f3802 ("spi: Transform the FSL 
QuadSPI driver to use the SPI MEM API") that has finally been merged, 
ignoring the fact that it will break ls1012a without this being applied too.

I don't want to blame anyone, but I want to voice my disappointment and 
raise attention that something is obviously wrong here.

> ---
> v4:
> -Rebase to top
> -Use complete firmware size
> -Return -ENODEV if flash probe fails
> v3:
> -Replace ret with 0 if flash probe fails
> v2:
> -Add return error code
> -Changes in displayed error log
> 
>   drivers/net/pfe_eth/pfe_firmware.c | 45 +++++++++++++++++++++++++++++++++++++-
>   include/configs/ls1012a_common.h   |  5 ++++-
>   2 files changed, 48 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/pfe_eth/pfe_firmware.c b/drivers/net/pfe_eth/pfe_firmware.c
> index 0493cfe..55e661c 100644
> --- a/drivers/net/pfe_eth/pfe_firmware.c
> +++ b/drivers/net/pfe_eth/pfe_firmware.c
> @@ -16,13 +16,14 @@
>   #include <linux/bitops.h>
>   #include <net/pfe_eth/pfe_eth.h>
>   #include <net/pfe_eth/pfe_firmware.h>
> +#include <spi_flash.h>
>   #ifdef CONFIG_CHAIN_OF_TRUST
>   #include <fsl_validate.h>
>   #endif
>   
>   #define PFE_FIRMWARE_FIT_CNF_NAME	"config at 1"
>   
> -static const void *pfe_fit_addr = (void *)CONFIG_SYS_LS_PFE_FW_ADDR;
> +static const void *pfe_fit_addr;
>   
>   /*
>    * PFE elf firmware loader.
> @@ -163,6 +164,44 @@ static int pfe_fit_check(void)
>   	return ret;
>   }
>   
> +int pfe_spi_flash_init(void)
> +{
> +	struct spi_flash *pfe_flash;
> +	int ret = 0;
> +	void *addr = malloc(CONFIG_SYS_QE_FMAN_FW_LENGTH);
> +
> +#ifdef CONFIG_DM_SPI_FLASH
> +	struct udevice *new;
> +
> +	/* speed and mode will be read from DT */
> +	ret = spi_flash_probe_bus_cs(CONFIG_ENV_SPI_BUS,
> +				     CONFIG_ENV_SPI_CS, 0, 0, &new);
> +
> +	pfe_flash = dev_get_uclass_priv(new);
> +#else
> +	pfe_flash = spi_flash_probe(CONFIG_ENV_SPI_BUS,
> +				    CONFIG_ENV_SPI_CS,
> +				    CONFIG_ENV_SPI_MAX_HZ,
> +				    CONFIG_ENV_SPI_MODE);
> +#endif
> +	if (!pfe_flash) {
> +		printf("SF: probe for pfe failed\n");
> +		return -ENODEV;
> +	}
> +
> +	ret = spi_flash_read(pfe_flash,
> +			     CONFIG_SYS_LS_PFE_FW_ADDR,
> +			     CONFIG_SYS_QE_FMAN_FW_LENGTH,
> +			     addr);
> +	if (ret)
> +		printf("SF: read for pfe failed\n");
> +
> +	pfe_fit_addr = addr;
> +	spi_flash_free(pfe_flash);
> +
> +	return ret;
> +}
> +
>   /*
>    * PFE firmware initialization.
>    * Loads different firmware files from FIT image.
> @@ -187,6 +226,10 @@ int pfe_firmware_init(void)
>   	int ret = 0;
>   	int fw_count;
>   
> +	ret = pfe_spi_flash_init();
> +	if (ret)
> +		goto err;
> +
>   	ret = pfe_fit_check();
>   	if (ret)
>   		goto err;
> diff --git a/include/configs/ls1012a_common.h b/include/configs/ls1012a_common.h
> index 3bea9a9..06af8bf 100644
> --- a/include/configs/ls1012a_common.h
> +++ b/include/configs/ls1012a_common.h
> @@ -36,8 +36,11 @@
>   /* Size of malloc() pool */
>   #define CONFIG_SYS_MALLOC_LEN		(CONFIG_ENV_SIZE + 1024 * 1024)
>   
> -/*SPI device */
> +/* PFE */
>   #define CONFIG_SYS_FMAN_FW_ADDR		0x400d0000
> +#define CONFIG_SYS_QE_FMAN_FW_LENGTH	0x300000
> +
> +/*SPI device */
>   #define CONFIG_SYS_FSL_QSPI_BASE	0x40000000
>   
>   /* SATA */
>
diff mbox series

Patch

diff --git a/drivers/net/pfe_eth/pfe_firmware.c b/drivers/net/pfe_eth/pfe_firmware.c
index 0493cfe..55e661c 100644
--- a/drivers/net/pfe_eth/pfe_firmware.c
+++ b/drivers/net/pfe_eth/pfe_firmware.c
@@ -16,13 +16,14 @@ 
 #include <linux/bitops.h>
 #include <net/pfe_eth/pfe_eth.h>
 #include <net/pfe_eth/pfe_firmware.h>
+#include <spi_flash.h>
 #ifdef CONFIG_CHAIN_OF_TRUST
 #include <fsl_validate.h>
 #endif
 
 #define PFE_FIRMWARE_FIT_CNF_NAME	"config at 1"
 
-static const void *pfe_fit_addr = (void *)CONFIG_SYS_LS_PFE_FW_ADDR;
+static const void *pfe_fit_addr;
 
 /*
  * PFE elf firmware loader.
@@ -163,6 +164,44 @@  static int pfe_fit_check(void)
 	return ret;
 }
 
+int pfe_spi_flash_init(void)
+{
+	struct spi_flash *pfe_flash;
+	int ret = 0;
+	void *addr = malloc(CONFIG_SYS_QE_FMAN_FW_LENGTH);
+
+#ifdef CONFIG_DM_SPI_FLASH
+	struct udevice *new;
+
+	/* speed and mode will be read from DT */
+	ret = spi_flash_probe_bus_cs(CONFIG_ENV_SPI_BUS,
+				     CONFIG_ENV_SPI_CS, 0, 0, &new);
+
+	pfe_flash = dev_get_uclass_priv(new);
+#else
+	pfe_flash = spi_flash_probe(CONFIG_ENV_SPI_BUS,
+				    CONFIG_ENV_SPI_CS,
+				    CONFIG_ENV_SPI_MAX_HZ,
+				    CONFIG_ENV_SPI_MODE);
+#endif
+	if (!pfe_flash) {
+		printf("SF: probe for pfe failed\n");
+		return -ENODEV;
+	}
+
+	ret = spi_flash_read(pfe_flash,
+			     CONFIG_SYS_LS_PFE_FW_ADDR,
+			     CONFIG_SYS_QE_FMAN_FW_LENGTH,
+			     addr);
+	if (ret)
+		printf("SF: read for pfe failed\n");
+
+	pfe_fit_addr = addr;
+	spi_flash_free(pfe_flash);
+
+	return ret;
+}
+
 /*
  * PFE firmware initialization.
  * Loads different firmware files from FIT image.
@@ -187,6 +226,10 @@  int pfe_firmware_init(void)
 	int ret = 0;
 	int fw_count;
 
+	ret = pfe_spi_flash_init();
+	if (ret)
+		goto err;
+
 	ret = pfe_fit_check();
 	if (ret)
 		goto err;
diff --git a/include/configs/ls1012a_common.h b/include/configs/ls1012a_common.h
index 3bea9a9..06af8bf 100644
--- a/include/configs/ls1012a_common.h
+++ b/include/configs/ls1012a_common.h
@@ -36,8 +36,11 @@ 
 /* Size of malloc() pool */
 #define CONFIG_SYS_MALLOC_LEN		(CONFIG_ENV_SIZE + 1024 * 1024)
 
-/*SPI device */
+/* PFE */
 #define CONFIG_SYS_FMAN_FW_ADDR		0x400d0000
+#define CONFIG_SYS_QE_FMAN_FW_LENGTH	0x300000
+
+/*SPI device */
 #define CONFIG_SYS_FSL_QSPI_BASE	0x40000000
 
 /* SATA */