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

Wasim Khan posted 3 patches 3 years, 9 months ago
Failed in applying to current master (apply log)
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
[edk2-devel] [PATCH edk2-platforms v5 0/3] Add SerDes Support
Posted by Wasim Khan 3 years, 9 months ago
From: Wasim Khan <wasim.khan@nxp.com>

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

-- 
2.7.4


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#60907): https://edk2.groups.io/g/devel/message/60907
Mute This Topic: https://groups.io/mt/74759293/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH edk2-platforms v5 0/3] Add SerDes Support
Posted by Jonathan Cameron 3 years, 9 months ago
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
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#60934): https://edk2.groups.io/g/devel/message/60934
Mute This Topic: https://groups.io/mt/74759293/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH edk2-platforms v5 0/3] Add SerDes Support
Posted by Leif Lindholm 3 years, 9 months ago
On Tue, Jun 09, 2020 at 00:22:32 +0530, Wasim Khan wrote:
> From: Wasim Khan <wasim.khan@nxp.com>
> 
> 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

For the series:
Reviewed-by: Leif Lindholm <leif@nuviainc.com>
Pushed as 6d4fed696d00..c764551258ee.

> 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
> 
> -- 
> 2.7.4
> 

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#61217): https://edk2.groups.io/g/devel/message/61217
Mute This Topic: https://groups.io/mt/74759293/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-