[PATCH 1/6] Revert "spi: zynq_qspi: Add parallel memories support in QSPI driver"

Abbarapu, Venkatesh venkatesh.abbarapu at amd.com
Tue Dec 10 09:32:56 CET 2024



> -----Original Message-----
> From: Marek Vasut <marek.vasut at mailbox.org>
> Sent: Tuesday, December 10, 2024 1:50 PM
> To: Abbarapu, Venkatesh <venkatesh.abbarapu at amd.com>; Tom Rini
> <trini at konsulko.com>; Tudor Ambarus <tudor.ambarus at linaro.org>
> Cc: Jon Humphreys <j-humphreys at ti.com>; Simek, Michal
> <michal.simek at amd.com>; u-boot at lists.denx.de; Andre Przywara
> <andre.przywara at arm.com>; Ashok Reddy Soma <ashok.reddy.soma at amd.com>;
> Jagan Teki <jagan at amarulasolutions.com>; Michael Walle <mwalle at kernel.org>;
> Patrice Chotard <patrice.chotard at foss.st.com>; Patrick Delaunay
> <patrick.delaunay at foss.st.com>; Pratyush Yadav <p.yadav at ti.com>; Quentin
> Schulz <quentin.schulz at cherry.de>; Sean Anderson <seanga2 at gmail.com>;
> Simon Glass <sjg at chromium.org>; Takahiro Kuwano
> <Takahiro.Kuwano at infineon.com>; uboot-stm32 at st-md-mailman.stormreply.com
> Subject: Re: [PATCH 1/6] Revert "spi: zynq_qspi: Add parallel memories support in
> QSPI driver"
> 
> On 12/10/24 4:20 AM, Abbarapu, Venkatesh wrote:
> 
> >>> Hi Humphreys,
> >>> Thank you for confirming that the patch worked and resolved the issue.
> >>
> >> OK, but going back to the patch in question, Tudor has requested
> >> changes. When can we expect a v2 that addresses those, so that Jon
> >> and Marek can also confirm their platforms work? Thanks.
> >
> > Tudor, Do you want me to implement separate
> spi_nor_erase()/spi_nor_read()/spi_nor_write() for the parallel/stacked
> configurations?
> > Marek, any suggestions regarding this implementation?
> 
> What do you refer to ? Is there a discussion I missed ?

This is regarding the comments mentioned by Tudor on this patch 
https://lore.kernel.org/u-boot/2773d157-2b75-4fe5-bf16-9e6927e0f6f4@linaro.org/

Thanks
Venkatesh
> 
> --
> Best regards,
> Marek Vasut


More information about the U-Boot mailing list