On Fri, Nov 10, 2017 at 02:21:07PM +0000, Ard Biesheuvel wrote:
> From: Pipat Methavanitpong <methavanitpong.pi...@socionext.com>
> 
> This imports the driver sources provided by Socionext for the FIP006
> SPI NOR flash device found on SynQuacer SoCs. It has been slightly
> tweaked to bring it up to date with the changes made on the EDK2 side
> since it was forked.
> 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Pipat Methavanitpong <methavanitpong.pi...@socionext.com>
> 
> [various tweaks and bugfixes]
> 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Ard Biesheuvel <ard.biesheu...@linaro.org>
> ---
> v4: incorporate feedback from Leif, with the exception of subjective feedback
>     (i.e., 'this function is too long'), addressing of which would make this
>     driver deviate more from the driver it as based on in ArmPlatformPkg,
>     making it less likely we can ever merge these into a unified NOR flash
>     driver.

I accept this, and there is logic to what you say, but I also cannot
with good conscience claim that I have properly (even by my standards)
reviewed the driver. So...
Acked-by: Leif Lindholm <leif.lindh...@linaro.org>
?

(This version is certainly a major improvement on the original
submission.)

/
    Leif
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to