[edk2] [platforms: PATCH v3 00/10] Armada 7k/8k - misc improvements pt.2

Marcin Wojtas posted 10 patches 6 years, 5 months ago
Failed in applying to current master (apply log)
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(-)
[edk2] [platforms: PATCH v3 00/10] Armada 7k/8k - misc improvements pt.2
Posted by Marcin Wojtas 6 years, 5 months ago
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

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

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [platforms: PATCH v3 00/10] Armada 7k/8k - misc improvements pt.2
Posted by Leif Lindholm 6 years, 4 months ago
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
> 
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel