On Mon, Sep 25, 2017 at 03:51:49AM +0200, Marcin Wojtas wrote: > Hi, > > Finally I found time to respin SPI patchset. As agreed, I extracted > style fixes from dynamic flash detection patch, but its main part > is kept aside for now until FlashId table (and additionally both SPI > protocols land in the edk2 mainline). A lot of minor fixes were > implemented, details can be found in the commit logs and changelog > below. > > Patches are available in the github: > https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commits/spi-upstream-r20170925 > > I'm looking forward to the comments or remarks.
If the two minor changes I have suggested are acceptable - for the series: Reviewed-by: Leif Lindholm <[email protected]> As mentioned, I can fold these in before pushing. Let me know. / Leif > Best regards, > Marcin > > Changelog > v1 -> v2 > Slightly improve commit log prefixes. > > 1/10 > * Move documentation to Silicon/Marvell > * Update PortingGuide with new paths in edk2-platforms repository > > 2/10 > * Correct error print > > 4/10 > * Sort PCDs > > 7/10 > * Simplify change and remove additional local variable > > 8/10 > * Correct FileSize type and let it really compile for ARM and AARCH64 > > 9/10 > * modify macros -> s/SPI_ERASE_SIZE_/SIZE_/ > > 10/10 > * New patch > > 3/10, 5/10, 6/10 > * Add Reviewed-by's > > Ard Biesheuvel (3): > Marvell/Applications/SpiTool: Fix bug in error test > Marvell/Applications/FirmwareUpdate: Fix 32-bit issues > Marvell/Applications/SpiTool: Fix 32-bit issues > > Joe Zhou (1): > Marvell/Drivers: MvSpiDxe: Fix write bug > > Marcin Wojtas (4): > Marvell/Applications/SpiTool: Enable configurable CS and SCLK mode > Platform/Marvell/Armada70x0: set CS and SCLK Mode for SPI flash > Marvell/Drivers: MvSpiFlash: Fix usage of erase size parameter > Marvell/Drivers: MvSpiFlash: Minor style fix > > Nir Erez (1): > Silicon/Marvell: Refactor Documentation > > Piotr Król (1): > Marvell/Drivers: MvSpiDxe: Log and return correct error > > Platform/Marvell/Applications/FirmwareUpdate/FUpdate.c | 6 +- > Platform/Marvell/Applications/SpiTool/SpiFlashCmd.c | 13 +- > Platform/Marvell/Applications/SpiTool/SpiFlashCmd.inf | 2 + > Platform/Marvell/Armada/Armada70x0.dsc | 2 + > Platform/Marvell/Documentation/Drivers/EepromDriver.txt | 96 ----- > Platform/Marvell/Documentation/Drivers/I2cDriver.txt | 64 ---- > Platform/Marvell/Documentation/Drivers/SpiDriver.txt | 116 ------ > Platform/Marvell/Documentation/PortingGuide/ComPhy.txt | 45 --- > Platform/Marvell/Documentation/PortingGuide/I2c.txt | 20 -- > Platform/Marvell/Documentation/PortingGuide/Mdio.txt | 7 - > Platform/Marvell/Documentation/PortingGuide/Mpp.txt | 48 --- > Platform/Marvell/Documentation/PortingGuide/PciEmulation.txt | 31 -- > Platform/Marvell/Documentation/PortingGuide/Phy.txt | 45 --- > Platform/Marvell/Documentation/PortingGuide/Pp2.txt | 35 -- > Platform/Marvell/Documentation/PortingGuide/Reset.txt | 7 - > Platform/Marvell/Documentation/PortingGuide/Spi.txt | 16 - > Platform/Marvell/Documentation/PortingGuide/SpiFlash.txt | 23 -- > Platform/Marvell/Documentation/PortingGuide/Utmi.txt | 35 -- > Platform/Marvell/Drivers/Spi/Devices/MvSpiFlash.c | 36 +- > Platform/Marvell/Drivers/Spi/Devices/MvSpiFlash.h | 6 + > Platform/Marvell/Drivers/Spi/Devices/MvSpiFlash.inf | 1 + > Platform/Marvell/Drivers/Spi/MvSpiDxe.c | 6 +- > Platform/Marvell/Marvell.dec | 3 + > Silicon/Marvell/Documentation/Drivers/EepromDriver.txt | 96 +++++ > Silicon/Marvell/Documentation/Drivers/I2cDriver.txt | 64 ++++ > Silicon/Marvell/Documentation/Drivers/SpiDriver.txt | 116 ++++++ > Silicon/Marvell/Documentation/PortingGuide.txt | 377 > ++++++++++++++++++++ > 27 files changed, 707 insertions(+), 609 deletions(-) > delete mode 100644 Platform/Marvell/Documentation/Drivers/EepromDriver.txt > delete mode 100644 Platform/Marvell/Documentation/Drivers/I2cDriver.txt > delete mode 100644 Platform/Marvell/Documentation/Drivers/SpiDriver.txt > delete mode 100644 Platform/Marvell/Documentation/PortingGuide/ComPhy.txt > delete mode 100644 Platform/Marvell/Documentation/PortingGuide/I2c.txt > delete mode 100644 Platform/Marvell/Documentation/PortingGuide/Mdio.txt > delete mode 100644 Platform/Marvell/Documentation/PortingGuide/Mpp.txt > delete mode 100644 > Platform/Marvell/Documentation/PortingGuide/PciEmulation.txt > delete mode 100644 Platform/Marvell/Documentation/PortingGuide/Phy.txt > delete mode 100644 Platform/Marvell/Documentation/PortingGuide/Pp2.txt > delete mode 100644 Platform/Marvell/Documentation/PortingGuide/Reset.txt > delete mode 100644 Platform/Marvell/Documentation/PortingGuide/Spi.txt > delete mode 100644 Platform/Marvell/Documentation/PortingGuide/SpiFlash.txt > delete mode 100644 Platform/Marvell/Documentation/PortingGuide/Utmi.txt > create mode 100644 Silicon/Marvell/Documentation/Drivers/EepromDriver.txt > create mode 100644 Silicon/Marvell/Documentation/Drivers/I2cDriver.txt > create mode 100644 Silicon/Marvell/Documentation/Drivers/SpiDriver.txt > create mode 100644 Silicon/Marvell/Documentation/PortingGuide.txt > > -- > 1.8.3.1 > _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel

