[edk2-devel][edk2-staging] DeviceSimPkg branch creation

2023-11-13 Thread Maciej Czajkowski
Hello, I would like to announce DeviceSimPkg branch creation in edk2-staging repository. Branch will be used to develop DeviceSimPkg, a package for developing OS executable tests for device focused code. Branch owner: Maciej Czajkowski , GitHub ID: 81293748 (https://github.com/mczaj) Signed

[edk2-devel] Request for access for edk2-staging branch creation

2023-10-19 Thread Maciej Czajkowski
Hello, I would like to request for a write access to edk2-staging repository to create a new branch for DeviceSimPkg. Its goal is to provide an environment to write OS-executable tests for code that interacts directly with devices. Regards, Maciej Czajkowski

[edk2-devel] [PATCH v1] MdeModulePkg/AhciPei: Fix MMIO base assignment

2022-09-09 Thread Maciej Czajkowski
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4041 There is a mistake in getting MMIO base using PciDevicePpi that can lead to the data corruption. Cc: Hao A Wu Cc: Ray Ni Signed-off-by: Maciej Czajkowski --- MdeModulePkg/Bus/Ata/AhciPei/AhciPei.c | 5 +++-- 1 file changed, 3

Re: [edk2-devel] [PATCH v2 2/2] MdeModulePkg/AhciPei: Use PCI_DEVICE_PPI to manage AHCI device

2022-08-01 Thread Maciej Czajkowski
is change modifies AhciPei library to allow usage both > EDKII_PCI_DEVICE_PPI and EDKII_PEI_ATA_AHCI_HOST_CONTROLLER_PPI > to manage ATA HDD working under AHCI mode. > > Cc: Hao A Wu > Cc: Ray Ni > Cc: Liming Gao > Signed-off-by: Maciej Czajkowski > --- > > Notes: > v2 changes: >

[edk2-devel] [PATCH v3 0/2] Add EDKII_PCI_DEVICE_PPI support to EDK2

2022-08-01 Thread Maciej Czajkowski
: - collected Acked-by tag for no.1 commit - followed-up with change suggestions in no.2 commit v3 changes: - no.1 commit: picked up reviewed-by tag - no.2 commit: a few fixes based on v2 review Maciej Czajkowski (2): MdeModulePkg: Add EDKII_PCI_DEVICE_PPI definition MdeModulePkg/AhciPei: Use

[edk2-devel] [PATCH v3 2/2] MdeModulePkg/AhciPei: Use PCI_DEVICE_PPI to manage AHCI device

2022-08-01 Thread Maciej Czajkowski
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3907 This change modifies AhciPei library to allow usage both EDKII_PCI_DEVICE_PPI and EDKII_PEI_ATA_AHCI_HOST_CONTROLLER_PPI to manage ATA HDD working under AHCI mode. Cc: Hao A Wu Cc: Ray Ni Cc: Liming Gao Signed-off-by: Maciej Czajkowski

[edk2-devel] [PATCH v3 1/2] MdeModulePkg: Add EDKII_PCI_DEVICE_PPI definition

2022-08-01 Thread Maciej Czajkowski
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3907 This commit introduces EDKII_PCI_DEVICE_PPI. The purpose of this PPI is to provide a way of accessing PCI devices to drvice drivers such as NvmExpressPei or AhciPei. Cc: Hao A Wu Cc: Ray Ni Cc: Liming Gao Signed-off-by: Maciej

[edk2-devel] [PATCH v2 0/2] Add EDKII_PCI_DEVICE_PPI support to EDK2

2022-07-27 Thread Maciej Czajkowski
with change suggestions in no.2 commit Maciej Czajkowski (2): MdeModulePkg: Add EDKII_PCI_DEVICE_PPI definition MdeModulePkg/AhciPei: Use PCI_DEVICE_PPI to manage AHCI device MdeModulePkg/Bus/Ata/AhciPei/AhciPei.c| 585 ++-- MdeModulePkg/Bus/Ata/AhciPei/DevicePath.c | 44

[edk2-devel] [PATCH v2 2/2] MdeModulePkg/AhciPei: Use PCI_DEVICE_PPI to manage AHCI device

2022-07-27 Thread Maciej Czajkowski
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3907 This change modifies AhciPei library to allow usage both EDKII_PCI_DEVICE_PPI and EDKII_PEI_ATA_AHCI_HOST_CONTROLLER_PPI to manage ATA HDD working under AHCI mode. Cc: Hao A Wu Cc: Ray Ni Cc: Liming Gao Signed-off-by: Maciej Czajkowski

[edk2-devel] [PATCH v2 1/2] MdeModulePkg: Add EDKII_PCI_DEVICE_PPI definition

2022-07-27 Thread Maciej Czajkowski
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3907 This commit introduces EDKII_PCI_DEVICE_PPI. The purpose of this PPI is to provide a way of accessing PCI devices to drvice drivers such as NvmExpressPei or AhciPei. Cc: Hao A Wu Cc: Ray Ni Cc: Liming Gao Signed-off-by: Maciej

Re: [edk2-devel][PATCH v1 0/2] Add EDKII_PCI_DEVICE_PPI support to EDK2

2022-06-13 Thread Maciej Czajkowski
t that performs the enumeration will be implemented in > the silicon code. > Sample code can be seen here: https://github.com/mczaj/edk2- > platforms/commit/d443062e58f9fba228869b54f2546d9735b3b506 > > Cc: Hao A Wu > Cc: Ray Ni > Cc: Liming Gao > > Maciej Czajkowski (2):

Re: [edk2-devel][PATCH v1 2/2] MdeModulePkg/AhciPei: Use PCI_DEVICE_PPI to manage AHCI device

2022-06-13 Thread Maciej Czajkowski
h > > EDKII_PCI_DEVICE_PPI and EDKII_PEI_ATA_AHCI_HOST_CONTROLLER_PPI to > > manage ATA HDD working under AHCI mode. > > > > Cc: Hao A Wu > > Cc: Ray Ni > > Signed-off-by: Maciej Czajkowski > > --- > > MdeModulePkg/Bus/Ata/AhciPei/AhciPei.c

[edk2-devel][PATCH v1 1/2] MdeModulePkg: Add EDKII_PCI_DEVICE_PPI definition

2022-06-06 Thread Maciej Czajkowski
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3907 This commit introduces EDKII_PCI_DEVICE_PPI. The purpose of this PPI is to provide a way of accessing PCI devices to drvice drivers such as NvmExpressPei or AhciPei. Cc: Hao A Wu Cc: Ray Ni Cc: Liming Gao Signed-off-by: Maciej

[edk2-devel][PATCH v1 2/2] MdeModulePkg/AhciPei: Use PCI_DEVICE_PPI to manage AHCI device

2022-06-06 Thread Maciej Czajkowski
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3907 This change modifies AhciPei library to allow usage both EDKII_PCI_DEVICE_PPI and EDKII_PEI_ATA_AHCI_HOST_CONTROLLER_PPI to manage ATA HDD working under AHCI mode. Cc: Hao A Wu Cc: Ray Ni Signed-off-by: Maciej Czajkowski

[edk2-devel][PATCH v1 0/2] Add EDKII_PCI_DEVICE_PPI support to EDK2

2022-06-06 Thread Maciej Czajkowski
that performs the enumeration will be implemented in the silicon code. Sample code can be seen here: https://github.com/mczaj/edk2-platforms/commit/d443062e58f9fba228869b54f2546d9735b3b506 Cc: Hao A Wu Cc: Ray Ni Cc: Liming Gao Maciej Czajkowski (2): MdeModulePkg: Add EDKII_PCI_DEVICE_PPI definition

[edk2-devel] [edk2platforms][PATCH v4 0/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-12-07 Thread Maciej Czajkowski
to interface - removed GUID definitons from .h file - corrected descriptions - made PPI/protocol GUIDs unique v2 changes: - added interface description - added die specific GUIDs into .dec file Cc: Ray Ni Cc: Rangasai V Chaganty Maciej Czajkowski (1): IntelSiliconPkg: Add IntelDieInfoProtocol

[edk2-devel] [edk2platforms][PATCH v4 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-12-07 Thread Maciej Czajkowski
Added IntelDieInfo header into IntelSiliconPkg tree. The purpose is to have generic and unified interface for getting information about dies installed in the system. It will be implemented by silicon code. Cc: Ray Ni Cc: Rangasai V Chaganty Signed-off-by: Maciej Czajkowski --- Notes: v4

Re: [edk2-devel] [edk2platforms][PATCH v3 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-12-03 Thread Maciej Czajkowski
gt; Cc: Ray Ni > Cc: Rangasai V Chaganty > Signed-off-by: Maciej Czajkowski > --- > > Notes: > v3: > - added EDKII_ prefix to interface > - removed GUID definitons from .h file > - corrected descriptions > - made PPI/protocol GUIDs unique > >

[edk2-devel] [edk2platforms][PATCH v3 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-11-18 Thread Maciej Czajkowski
Added IntelDieInfo header into IntelSiliconPkg tree. The purpose is to have generic and unified interface for getting information about dies installed in the system. It will be implemented by silicon code. Cc: Ray Ni Cc: Rangasai V Chaganty Signed-off-by: Maciej Czajkowski --- Notes: v3

[edk2-devel] [edk2platforms][PATCH v3 0/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-11-18 Thread Maciej Czajkowski
from .h file - corrected descriptions - made PPI/protocol GUIDs unique v2 changes: - added interface description - added die specific GUIDs into .dec file Cc: Ray Ni Cc: Rangasai V Chaganty Maciej Czajkowski (1): IntelSiliconPkg: Add IntelDieInfoProtocol Silicon/Intel/IntelSiliconPkg

Re: [edk2-devel] [edk2platforms][PATCH v2 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-10-26 Thread Maciej Czajkowski
Hi Ray, Sai, Could you take a look on this patch? Pasting below questions and answers from previous patch for a reference. What's the difference between SOC and CPU? SOC die differs from typical CPU die in Intel's chiplet design. What the purpose of "DieIndex"? We want to have an indication

[edk2-devel] [edk2platforms][PATCH v2 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-10-18 Thread Maciej Czajkowski
Chaganty Signed-off-by: Maciej Czajkowski --- Notes: v2: - added interface description - added die specific GUIDs into .dec file Silicon/Intel/IntelSiliconPkg/Include/Protocol/IntelDieInfoProtocol.h | 117 Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec

[edk2-devel] [edk2platforms][PATCH v2 0/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-10-18 Thread Maciej Czajkowski
into .dec file Cc: Ray Ni Cc: Rangasai V Chaganty Maciej Czajkowski (1): IntelSiliconPkg: Add IntelDieInfoProtocol Silicon/Intel/IntelSiliconPkg/Include/Protocol/IntelDieInfoProtocol.h | 117 Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec | 9 ++ 2

Re: [edk2-devel] [edk2platforms][PATCH v1 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-10-15 Thread Maciej Czajkowski
Can you add some details on how a producer of this interface would know the values to be assigned here? Does this contain any sort of encoding of generation and Die specific ID in a single UINT64? Yes, in order to have a generic field regardless of the die type and per-generation changes it

Re: [edk2-devel] [edk2platforms][PATCH v1 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-10-15 Thread Maciej Czajkowski
Can you add some details on how a producer of this interface would know the values to be assigned here? Does this contain any sort of encoding of generation and Die specific ID in a single UINT64? Yes, in order to have a generic field regardless of the die type and per-generation changes it

Re: [edk2-devel] [edk2platforms][PATCH v1 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-10-15 Thread Maciej Czajkowski
What's the difference between SOC and CPU? SOC die differs from typical CPU die in Intel's chiplet design. What the purpose of "DieIndex"? We want to have an indication to which exactly die given protocol instance is referring to (in example in multi socket/multi PCH designs). Have you

[edk2-devel] [edk2platforms][PATCH v1 1/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-10-07 Thread Maciej Czajkowski
Chaganty Signed-off-by: Maciej Czajkowski --- Silicon/Intel/IntelSiliconPkg/Include/Protocol/IntelDieInfoProtocol.h | 109 Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec | 2 + 2 files changed, 111 insertions(+) diff --git a/Silicon/Intel

[edk2-devel] [edk2platforms][PATCH v1 0/1] IntelSiliconPkg: Add IntelDieInfoProtocol

2021-10-07 Thread Maciej Czajkowski
This commit will add a protocol/PPI definition which will provide a generic and unified way to get information about dies installed in the system(PCH, SoC, CPU). It will be implemented per generation in silicon packages. Cc: Ray Ni Cc: Rangasai V Chaganty Maciej Czajkowski (1