[coreboot-gerrit] [M] Change in coreboot[master]: drivers/net/r8168: Add support for ACPI DmaProperty

[Subrata Banik (Code Review)]

Subrata Banik has submitted this change. ( https://review.coreboot.org/c/coreboot/+/70016 )

(

6 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
 )Change subject: drivers/net/r8168: Add support for ACPI DmaProperty
......................................................................

drivers/net/r8168: Add support for ACPI DmaProperty

BUG=b:259716145
TEST=Verified SSDT on google/osiris.

Before:
Scope (_SB.PCI0.RP01)
{
    Device (RLTK)
    {
        Name (_HID, "R8168")  // _HID: Hardware ID
        Name (_UID, 0xD0E889DD)  // _UID: Unique ID
        Name (_DDN, "Realtek r8168")  // _DDN: DOS Device Name
        Name (_ADR, 0x00000000)  // _ADR: Address
        Name (_PRW, Package (0x02)  // _PRW: Power Resources for Wake
        {
            0x07,
            0x03
        })
    }
}

After:
Scope (_SB.PCI0.RP01)
{
    Device (RLTK)
    {
        Name (_HID, "R8168")  // _HID: Hardware ID
        Name (_UID, 0xD0E889DD)  // _UID: Unique ID
        Name (_DDN, "Realtek r8168")  // _DDN: DOS Device Name
        Name (_ADR, 0x00000000)  // _ADR: Address
        Name (_PRW, Package (0x02)  // _PRW: Power Resources for Wake
        {
            0x07,
            0x03
        })
        Name (_DSD, Package (0x02)  // _DSD: Device-Specific Data
        {
            ToUUID ("70d24161-6dd5-4c9e-8070-705531292865"),
            Package (0x01)
            {
                Package (0x02)
                {
                    "DmaProperty",
                    One
                }
            }
        })
    }
}

Signed-off-by: Kapil Porwal 
Change-Id: I647230082362b1093b63793a201eba23a6289121
Reviewed-on: https://review.coreboot.org/c/coreboot/+/70016
Reviewed-by: Subrata Banik 
Tested-by: build bot (Jenkins) 
---
M src/drivers/net/chip.h
M src/drivers/net/r8168.c
2 files changed, 69 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Subrata Banik: Looks good to me, approved

diff --git a/src/drivers/net/chip.h b/src/drivers/net/chip.h
index f253cae..203707c 100644
--- a/src/drivers/net/chip.h
+++ b/src/drivers/net/chip.h
@@ -34,6 +34,10 @@

/* Allow kernel driver to enable ASPM L1.2. */
    bool enable_aspm_l1_2;
+
+	/* When set to true, this will add a _DSD which contains a single
+	   property, `DmaProperty`, set to 1, under the ACPI Device. */
+	bool add_acpi_dma_property;
 };

#endif /* __DRIVERS_R8168_CHIP_H__ */
diff --git a/src/drivers/net/r8168.c b/src/drivers/net/r8168.c
index 32e0275..5d19d69 100644
--- a/src/drivers/net/r8168.c
+++ b/src/drivers/net/r8168.c
@@ -400,6 +400,9 @@
    if (config->wake)
    	acpigen_write_PRW(config->wake, 3);

+	if (config->add_acpi_dma_property)
+		acpi_device_add_dma_property(NULL);
+
    acpigen_pop_len(); /* Device */
    acpigen_pop_len(); /* Scope */