> -----Original Message-----
> From: Tom Rini <tr...@konsulko.com>
> Sent: Friday, June 19, 2020 8:07 PM
> To: Priyanka Jain <priyanka.j...@nxp.com>
> Cc: Kuldeep Singh <kuldeep.si...@nxp.com>; Joe Hershberger
> <joe.hershber...@ni.com>; u-boot@lists.denx.de; Schrempf Frieder
> <frieder.schre...@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.si...@nxp.com>
> > >Sent: Friday, June 19, 2020 3:40 PM
> > >To: Joe Hershberger <joe.hershber...@ni.com>; u-boot@lists.denx.de;
> > >Priyanka Jain <priyanka.j...@nxp.com>
> > >Cc: Tom Rini <tr...@konsulko.com>; Schrempf Frieder
> > ><frieder.schre...@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.schre...@kontron.de>
> > >> Sent: Thursday, May 28, 2020 1:46 PM
> > >> To: Kuldeep Singh <kuldeep.si...@nxp.com>; Joe Hershberger
> > >> <joe.hershber...@ni.com>; u-boot@lists.denx.de
> > >> Cc: Priyanka Jain <priyanka.j...@nxp.com>; Tom Rini
> > >> <tr...@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.si...@nxp.com>
> > >> > Reviewed-by: Frieder Schrempf <frieder.schre...@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.j...@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.

Reply via email to