Re: [edk2-devel] [PATCH edk2-platforms v5 0/3] Add SerDes Support

["Jonathan Cameron" <jonathan.cameron@huawei.com>]

On Tue, 9 Jun 2020 00:22:32 +0530
Wasim Khan <wasim.khan@oss.nxp.com> wrote:

> From: Wasim Khan <wasim.khan@nxp.com>

Hi Wasim,

Minor moan, please put a prefix on the patch description so those
of us who scan read EDK2 lists don't have to open the thread
to find out what SoC this applies to.

Silicon/NXP: 

as done for the patches would work well but anything similar to 
ensure the title of the cover letter isn't generic would be fine.

Thanks,

Jonathan

> 
> NXP SoCs supports different Serdes protocols using reset
> configuration word (RCW). 
> Based on Serdes protocol value in reset configuration word (RCW)
> different IP blocks gets enabled in HW.
> 
> This patch series implements SerDesHelperLib and provide SoC specific
> serdes configuration for LS1043A, which can be used by different IPs
> (Ex: PCIe) to know the enabled interfaces and perform the required
> initialization.
> 
> V1 Series can be referred here:
> https://edk2.groups.io/g/devel/message/60542
> 
> V2 Series can be referred here:
> https://edk2.groups.io/g/devel/message/60836
> 
> V3 Series can be referred here:
> https://edk2.groups.io/g/devel/message/60901
> 
> Changes in V4:
> - Included SocSerDes.h , which got missed in V3
> 
> Changes in V5:
> - Corrected enum value and CheckPatch fix
> 
> Wasim Khan (3):
>   Silicon/NXP/Library: Implement SerDesHelperLib
>   Silicon/NXP: LS1043A: Add SerDes Support
>   Silicon/NXP: PciHostBridgeLib: Initialize only enabled PCIe
>     controllers
> 
>  Silicon/NXP/NxpQoriqLs.dec                                |   1 +
>  Silicon/NXP/LS1043A/LS1043A.dsc.inc                       |   2 +
>  Silicon/NXP/LS1043A/Library/SocLib/SocLib.inf             |   6 +
>  Silicon/NXP/Library/PciHostBridgeLib/PciHostBridgeLib.inf |   1 +
>  Silicon/NXP/Library/SerDesHelperLib/SerDesHelperLib.inf   |  28 ++++
>  Silicon/NXP/Include/Library/SerDes.h                      |  28 ++++
>  Silicon/NXP/Include/Library/SerDesHelperLib.h             |  64 ++++++++
>  Silicon/NXP/LS1043A/Include/SocSerDes.h                   |  33 ++++
>  Silicon/NXP/LS1043A/Library/SocLib/SerDes.c               | 119 ++++++++++++++
>  Silicon/NXP/Library/PciHostBridgeLib/PciHostBridgeLib.c   |  35 ++++-
>  Silicon/NXP/Library/SerDesHelperLib/SerDesHelperLib.c     | 164 ++++++++++++++++++++
>  11 files changed, 480 insertions(+), 1 deletion(-)
>  create mode 100644 Silicon/NXP/Library/SerDesHelperLib/SerDesHelperLib.inf
>  create mode 100644 Silicon/NXP/Include/Library/SerDes.h
>  create mode 100644 Silicon/NXP/Include/Library/SerDesHelperLib.h
>  create mode 100644 Silicon/NXP/LS1043A/Include/SocSerDes.h
>  create mode 100644 Silicon/NXP/LS1043A/Library/SocLib/SerDes.c
>  create mode 100644 Silicon/NXP/Library/SerDesHelperLib/SerDesHelperLib.c
>