From: Leif Lindholm <leif.lindholm@linaro.org>
To: Marcin Wojtas <mw@semihalf.com>
Cc: edk2-devel@lists.01.org, ard.biesheuvel@linaro.org,
nadavh@marvell.com, neta@marvell.com, kostap@marvell.com,
jinghua@marvell.com, jsd@semihalf.com
Subject: Re: [platforms: PATCH v3 00/10] Armada 7k/8k - misc improvements pt.2
Date: Sun, 29 Oct 2017 16:51:13 +0000 [thread overview]
Message-ID: <20171029165113.7ueoun3vgqeh7sfz@bivouac.eciton.net> (raw)
In-Reply-To: <1509121913-12937-1-git-send-email-mw@semihalf.com>
On Fri, Oct 27, 2017 at 06:31:43PM +0200, Marcin Wojtas wrote:
> Hi,
>
> I send v3 with minor fixes, requested in latest review. Details
> can be found in the changelog below.
>
> The patches are available in the github:
> https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commits/misc-upstream-r20171027-2
>
> I'm looking forward to your comments or remarks.
>
> Best regards,
> Marcin
Remaining patches:
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
Series pushed as 5fc9a86fb3..f79bce44ac.
/
Leif
> Changelog:
> v2 -> v3:
> 7/10
> - simplify code by supporting the only real usecase (forcing link up)
>
> 9/10
> - print actual clock value in DumpCapabilityReg and inform about
> its mismatch with original register contents.
> - fix typo in functions' declarations
>
> 1/10, 2/10, 6/10, 8/10
> - add RBs
>
> v1 -> v2:
> 1/10
> - remove unrelated style fix
> - fix style around modified functions calls
>
> 2/10
> - leave original EFI_SUCCESS assignment
>
> 6/10
> - use descriptively named temporary variable for pin index in a loop
>
> 7/10
> - use single flag for link up/down
> - simplify logic
> - correct style
>
> 8/10
> - mention missing SDR25 in a commit message
>
> 9/10
> - use new member of SD_MMC_HC_PRIVATE_DATA
> to set actual input clock speed and use it for the output
> clock configuration
> - rewrite commit message
>
> 3/10, 4/10, 5/10, 10/10
> - add RB's
>
> Ard Biesheuvel (2):
> Marvell/Library: MppLib: Disable the stack protector
> Marvell/Library: MppLib: Take 0xFF placeholders into account
>
> David Greeson (2):
> Marvell/Drivers: MvI2cDxe: Abort transaction immediately upon fail
> Marvell/Drivers: MvI2cDxe: Reduce bus occupation time
>
> Joe Zhou (1):
> Marvell/Library: MppLib: Prevent overwriting PCD values
>
> Marcin Wojtas (5):
> Marvell/Drivers: MvI2cDxe: Fix returning status in MvI2cStartRequest
> Marvell/Drivers: Pp2Dxe: Change settings for the always-up link
> Marvell/Drivers: XenonDxe: Fix UHS signalling mode setting
> Marvell/Drivers: XenonDxe: Allow overriding base clock frequency
> Marvell/Drivers: XenonDxe: Do not modify FIFO default values
>
> Platform/Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.c | 70 ++++++++++++--------
> Platform/Marvell/Drivers/I2c/MvI2cDxe/MvI2cDxe.h | 2 +-
> Platform/Marvell/Drivers/Net/Pp2Dxe/Mvpp2Lib.c | 19 ++++++
> Platform/Marvell/Drivers/Net/Pp2Dxe/Mvpp2Lib.h | 5 ++
> Platform/Marvell/Drivers/Net/Pp2Dxe/Pp2Dxe.c | 6 +-
> Platform/Marvell/Drivers/SdMmc/XenonDxe/EmmcDevice.c | 6 +-
> Platform/Marvell/Drivers/SdMmc/XenonDxe/SdDevice.c | 4 +-
> Platform/Marvell/Drivers/SdMmc/XenonDxe/SdMmcPciHcDxe.c | 15 +++--
> Platform/Marvell/Drivers/SdMmc/XenonDxe/SdMmcPciHcDxe.h | 6 ++
> Platform/Marvell/Drivers/SdMmc/XenonDxe/SdMmcPciHci.c | 31 +++++----
> Platform/Marvell/Drivers/SdMmc/XenonDxe/SdMmcPciHci.h | 16 +++--
> Platform/Marvell/Drivers/SdMmc/XenonDxe/XenonSdhci.c | 16 -----
> Platform/Marvell/Library/MppLib/MppLib.c | 35 +++++-----
> Platform/Marvell/Library/MppLib/MppLib.inf | 3 +
> 14 files changed, 144 insertions(+), 90 deletions(-)
>
> --
> 2.7.4
>
prev parent reply other threads:[~2017-10-29 16:47 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-27 16:31 [platforms: PATCH v3 00/10] Armada 7k/8k - misc improvements pt.2 Marcin Wojtas
2017-10-27 16:31 ` [platforms: PATCH v3 01/10] Marvell/Drivers: MvI2cDxe: Abort transaction immediately upon fail Marcin Wojtas
2017-10-27 16:31 ` [platforms: PATCH v3 02/10] Marvell/Drivers: MvI2cDxe: Fix returning status in MvI2cStartRequest Marcin Wojtas
2017-10-27 16:31 ` [platforms: PATCH v3 03/10] Marvell/Drivers: MvI2cDxe: Reduce bus occupation time Marcin Wojtas
2017-10-27 16:31 ` [platforms: PATCH v3 04/10] Marvell/Library: MppLib: Prevent overwriting PCD values Marcin Wojtas
2017-10-27 16:31 ` [platforms: PATCH v3 05/10] Marvell/Library: MppLib: Disable the stack protector Marcin Wojtas
2017-10-27 16:31 ` [platforms: PATCH v3 06/10] Marvell/Library: MppLib: Take 0xFF placeholders into account Marcin Wojtas
2017-10-27 16:31 ` [platforms: PATCH v3 07/10] Marvell/Drivers: Pp2Dxe: Change settings for the always-up link Marcin Wojtas
2017-10-27 16:31 ` [platforms: PATCH v3 08/10] Marvell/Drivers: XenonDxe: Fix UHS signalling mode setting Marcin Wojtas
2017-10-27 16:31 ` [platforms: PATCH v3 09/10] Marvell/Drivers: XenonDxe: Allow overriding base clock frequency Marcin Wojtas
2017-10-27 16:31 ` [platforms: PATCH v3 10/10] Marvell/Drivers: XenonDxe: Do not modify FIFO default values Marcin Wojtas
2017-10-29 16:51 ` Leif Lindholm [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171029165113.7ueoun3vgqeh7sfz@bivouac.eciton.net \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox