-----Original Message----- From: Tom Rini trini@konsulko.com Sent: Friday, June 19, 2020 8:07 PM To: Priyanka Jain priyanka.jain@nxp.com Cc: Kuldeep Singh kuldeep.singh@nxp.com; Joe Hershberger joe.hershberger@ni.com; u-boot@lists.denx.de; Schrempf Frieder frieder.schrempf@kontron.de Subject: Re: [EXT] Re: [RESEND][Patch v4] net: pfe_eth: Use spi_flash_read API to access flash memory
On Fri, Jun 19, 2020 at 02:13:37PM +0000, Priyanka Jain wrote:
-----Original Message----- From: Kuldeep Singh kuldeep.singh@nxp.com Sent: Friday, June 19, 2020 3:40 PM To: Joe Hershberger joe.hershberger@ni.com; u-boot@lists.denx.de; Priyanka Jain priyanka.jain@nxp.com Cc: Tom Rini trini@konsulko.com; Schrempf Frieder frieder.schrempf@kontron.de Subject: RE: [EXT] Re: [RESEND][Patch v4] net: pfe_eth: Use spi_flash_read API to access flash memory
-----Original Message----- From: Schrempf Frieder frieder.schrempf@kontron.de Sent: Thursday, May 28, 2020 1:46 PM To: Kuldeep Singh kuldeep.singh@nxp.com; Joe Hershberger joe.hershberger@ni.com; u-boot@lists.denx.de Cc: Priyanka Jain priyanka.jain@nxp.com; Tom Rini trini@konsulko.com Subject: [EXT] Re: [RESEND][Patch v4] net: pfe_eth: Use spi_flash_read API to access flash memory
Caution: EXT Email
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@nxp.com Reviewed-by: Frieder Schrempf frieder.schrempf@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.
Hi Priyanka,
Could you please help in reviewing this patch. The idea/motivation of these changes are taken from fm driver. The said changes in fm are already accepted in commit 382c53f94631
("net:
fm: add TFABOOT support"). You may take a look at code here[1], line: 380.
Thanks Kuldeep [1] https://gitlab.denx.de/u-boot/u-boot/-/blob/master/drivers/net/fm/fm. c
Reviewed-by: Priyanka Jain priyanka.jain@nxp.com
Joe,
Kindly help to pick this patch.
This is something that should go via the NXP trees, thanks!
Thanks Tom for mentioning. If this is so Priyanka, could you please pick this up in next PR.