Re: [edk2-devel] [PATCH v3] MinPlatformPkg: Remove _ADR from MinDsdt.asl

2022-10-04 Thread Chiu, Chasel


Patch has been merged:
https://github.com/tianocore/edk2-platforms/commit/ae75c51f27e21036b6ee021a2d5b9f365f951413

Thanks,
Chasel


> -Original Message-
> From: Chen, Aryeh 
> Sent: Sunday, September 11, 2022 10:18 PM
> To: devel@edk2.groups.io
> Cc: Chen, Aryeh ; Chiu, Chasel
> ; Desimone, Nathaniel L
> ; Oram, Isaac W ;
> Sinha, Ankit ; Gao, Liming
> ; Dong, Eric 
> Subject: [PATCH v3] MinPlatformPkg: Remove _ADR from MinDsdt.asl
> 
> From: Aryeh Chen 
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4057
> 
> According to ASL Coding Guidelines - Device Identifiers "A Device should 
> contain
> either an _ADR or a _HID object, never both."
> , so remove _ADR due to _HID exist.
> 
> Signed-off-by: Aryeh Chen 
> Cc: Chasel Chiu 
> Cc: Nate DeSimone 
> Cc: Isaac Oram 
> Cc: Ankit Sinha 
> Cc: Liming Gao 
> Cc: Eric Dong 
> ---
>  Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> b/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> index 0d94472450..4efb8709ac 100644
> --- a/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> +++ b/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> @@ -27,7 +27,6 @@ DefinitionBlock (
>Name(_HID, EISAID("PNP0A08")) // Indicates PCI Express/PCI-X Mode2 host
> hierarchy   Name(_CID, EISAID("PNP0A03")) // To support legacy OS that
> doesn't understand the new HID   Name(_SEG, 0)-  Name(_ADR, 
> 0x)
> Method(^BN00, 0){ return(0x) }  // Returns default Bus number for Peer PCI
> busses. Name can be overriden with control method placed directly under
> Device scope   Method(_BBN, 0){ return(BN00()) } // Bus number, optional 
> for
> the Root PCI Bus   Name(_UID, 0x)  // Unique Bus ID, optional--
> 2.26.2.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94721): https://edk2.groups.io/g/devel/message/94721
Mute This Topic: https://groups.io/mt/93627173/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




Re: [edk2-devel] [PATCH v3] MinPlatformPkg: Remove _ADR from MinDsdt.asl

2022-09-28 Thread Ankit Sinha
Hi Aryeh,

Thank you for testing.

Reviewed-by: Ankit Sinha 

> -Original Message-
> From: Chen, Aryeh 
> Sent: Wednesday, September 21, 2022 12:34 AM
> To: Sinha, Ankit ; devel@edk2.groups.io
> Cc: Chiu, Chasel ; Desimone, Nathaniel L
> ; Oram, Isaac W
> ; Gao, Liming ;
> Dong, Eric 
> Subject: RE: [PATCH v3] MinPlatformPkg: Remove _ADR from MinDsdt.asl
> 
> Hi Ankit,
> 
> Add a change to remove Name(_ADR, 0x) from edk2-
> platforms\Platform\Intel\MinPlatformPkg\Acpi\MinDsdt\MinDsdt.asl
> It can boot to OS success on Win10 21H2 and Ubuntu 22.4.0 on TGLU
> openboard bios.
> 
> === Acpidump ===
> Signature "DSDT"
> Length0x00FD (253)
> Revision  0x01 (1)
> Checksum  0x33 (51)
> OEM ID"INTEL "
> OEM Table ID  "MIN "
> OEM Revision  0x (0)
> Creator ID"INTL"
> Creator Revision  0x20210930 (539035952)
> DefinitionBlock ("DSDT.AML", "DSDT", 0x01, "INTEL ", "MIN ", 0x)
> {
>   Scope(_SB)
>   {
>   Device(PCI0)
>   {
>   Name(_HID, EISAID("PNP0A08"))
>   Name(_CID, EISAID("PNP0A03"))
>   Name(_SEG, Zero)
>   Method(^BN00, 0, NotSerialized)
>   {
>   Return(Zero)
>   }
> 
> === Original mindsdt.asl ===
>   Scope(\_SB) {
>   
> //---
>   // Begin PCI tree object scope
>   
> //---
> Device(PCI0) { // PCI Bridge "Host Bridge"
>   Name(_HID, EISAID("PNP0A08")) // Indicates PCI Express/PCI-X Mode2
> host hierarchy
>   Name(_CID, EISAID("PNP0A03")) // To support legacy OS that doesn't
> understand the new HID
>   Name(_SEG, 0)
>   Name(_ADR, 0x)
>   Method(^BN00, 0){ return(0x) }  // Returns default Bus number for
> Peer PCI busses. Name can be overriden with control method placed directly
> under Device scope
>   Method(_BBN, 0){ return(BN00()) } // Bus number, optional for the Root
> PCI Bus
> 
> Thanks,
> Aryeh
> 
> -Original Message-
> From: Sinha, Ankit 
> Sent: Tuesday, September 13, 2022 4:54 AM
> To: Chen, Aryeh ; devel@edk2.groups.io
> Cc: Chiu, Chasel ; Desimone, Nathaniel L
> ; Oram, Isaac W
> ; Gao, Liming ;
> Dong, Eric 
> Subject: RE: [PATCH v3] MinPlatformPkg: Remove _ADR from MinDsdt.asl
> 
> Hi Aryeh,
> 
> Can you please post the test results for this change. It would be good to note
> that there is no change in behavior in the OS.
> 
> Thank you,
> Ankit
> 
> > -Original Message-
> > From: Chen, Aryeh 
> > Sent: Sunday, September 11, 2022 10:18 PM
> > To: devel@edk2.groups.io
> > Cc: Chen, Aryeh ; Chiu, Chasel
> > ; Desimone, Nathaniel L
> > ; Oram, Isaac W
> > ; Sinha, Ankit ; Gao,
> > Liming ; Dong, Eric 
> > Subject: [PATCH v3] MinPlatformPkg: Remove _ADR from MinDsdt.asl
> >
> > From: Aryeh Chen 
> >
> > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4057
> >
> > According to ASL Coding Guidelines - Device Identifiers "A Device
> > should contain either an _ADR or a _HID object, never both."
> > , so remove _ADR due to _HID exist.
> >
> > Signed-off-by: Aryeh Chen 
> > Cc: Chasel Chiu 
> > Cc: Nate DeSimone 
> > Cc: Isaac Oram 
> > Cc: Ankit Sinha 
> > Cc: Liming Gao 
> > Cc: Eric Dong 
> > ---
> >  Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> > b/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> > index 0d94472450..4efb8709ac 100644
> > --- a/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> > +++ b/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> > @@ -27,7 +27,6 @@ DefinitionBlock (
> >Name(_HID, EISAID("PNP0A08")) // Indicates PCI Express/PCI-X Mode2
> > host hierarchy   Name(_CID, EISAID("PNP0A03")) // To support legacy OS
> > that doesn't understand the new HID   Name(_SEG, 0)-  Name(_ADR,
> > 0x)   Method(^BN00, 0){ return(0x) }  // Returns default Bus
> > number for Peer PCI busses. Name can be overriden with control method
> > placed directly under Device scope   Method(_BBN, 0){ return(BN00()) }
> //
> > Bus number, optional for the Root PCI Bus   Name(_UID, 0x)  //
> Unique
> > Bus ID, optional--
> > 2.26.2.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94486): https://edk2.groups.io/g/devel/message/94486
Mute This Topic: https://groups.io/mt/93627173/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




Re: [edk2-devel] [PATCH v3] MinPlatformPkg: Remove _ADR from MinDsdt.asl

2022-09-21 Thread Chen, Aryeh
Hi Ankit,

Add a change to remove Name(_ADR, 0x) from 
edk2-platforms\Platform\Intel\MinPlatformPkg\Acpi\MinDsdt\MinDsdt.asl
It can boot to OS success on Win10 21H2 and Ubuntu 22.4.0 on TGLU openboard 
bios.

=== Acpidump ===
Signature   "DSDT"
Length  0x00FD (253)
Revision0x01 (1)
Checksum0x33 (51)
OEM ID  "INTEL "
OEM Table ID"MIN "
OEM Revision0x (0)
Creator ID  "INTL"
Creator Revision0x20210930 (539035952)
DefinitionBlock ("DSDT.AML", "DSDT", 0x01, "INTEL ", "MIN ", 0x)
{
Scope(_SB)
{
Device(PCI0)
{
Name(_HID, EISAID("PNP0A08"))
Name(_CID, EISAID("PNP0A03"))
Name(_SEG, Zero)
Method(^BN00, 0, NotSerialized)
{
Return(Zero)
}

=== Original mindsdt.asl ===
  Scope(\_SB) {
  //---
  // Begin PCI tree object scope
  //---
Device(PCI0) { // PCI Bridge "Host Bridge"
  Name(_HID, EISAID("PNP0A08")) // Indicates PCI Express/PCI-X Mode2 host 
hierarchy
  Name(_CID, EISAID("PNP0A03")) // To support legacy OS that doesn't 
understand the new HID
  Name(_SEG, 0)
  Name(_ADR, 0x)
  Method(^BN00, 0){ return(0x) }  // Returns default Bus number for 
Peer PCI busses. Name can be overriden with control method placed directly 
under Device scope
  Method(_BBN, 0){ return(BN00()) } // Bus number, optional for the Root 
PCI Bus

Thanks,
Aryeh

-Original Message-
From: Sinha, Ankit  
Sent: Tuesday, September 13, 2022 4:54 AM
To: Chen, Aryeh ; devel@edk2.groups.io
Cc: Chiu, Chasel ; Desimone, Nathaniel L 
; Oram, Isaac W ; Gao, 
Liming ; Dong, Eric 
Subject: RE: [PATCH v3] MinPlatformPkg: Remove _ADR from MinDsdt.asl

Hi Aryeh,

Can you please post the test results for this change. It would be good to note 
that there is no change in behavior in the OS.

Thank you,
Ankit

> -Original Message-
> From: Chen, Aryeh 
> Sent: Sunday, September 11, 2022 10:18 PM
> To: devel@edk2.groups.io
> Cc: Chen, Aryeh ; Chiu, Chasel 
> ; Desimone, Nathaniel L 
> ; Oram, Isaac W 
> ; Sinha, Ankit ; Gao, 
> Liming ; Dong, Eric 
> Subject: [PATCH v3] MinPlatformPkg: Remove _ADR from MinDsdt.asl
> 
> From: Aryeh Chen 
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4057
> 
> According to ASL Coding Guidelines - Device Identifiers "A Device 
> should contain either an _ADR or a _HID object, never both."
> , so remove _ADR due to _HID exist.
> 
> Signed-off-by: Aryeh Chen 
> Cc: Chasel Chiu 
> Cc: Nate DeSimone 
> Cc: Isaac Oram 
> Cc: Ankit Sinha 
> Cc: Liming Gao 
> Cc: Eric Dong 
> ---
>  Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> b/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> index 0d94472450..4efb8709ac 100644
> --- a/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> +++ b/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> @@ -27,7 +27,6 @@ DefinitionBlock (
>Name(_HID, EISAID("PNP0A08")) // Indicates PCI Express/PCI-X Mode2
> host hierarchy   Name(_CID, EISAID("PNP0A03")) // To support legacy OS
> that doesn't understand the new HID   Name(_SEG, 0)-  Name(_ADR,
> 0x)   Method(^BN00, 0){ return(0x) }  // Returns default Bus
> number for Peer PCI busses. Name can be overriden with control method
> placed directly under Device scope   Method(_BBN, 0){ return(BN00()) } //
> Bus number, optional for the Root PCI Bus   Name(_UID, 0x)  // Unique
> Bus ID, optional--
> 2.26.2.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94052): https://edk2.groups.io/g/devel/message/94052
Mute This Topic: https://groups.io/mt/93627173/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




Re: [edk2-devel] [PATCH v3] MinPlatformPkg: Remove _ADR from MinDsdt.asl

2022-09-12 Thread Ankit Sinha
Hi Aryeh,

Can you please post the test results for this change. It would be good to note 
that there is no change in behavior in the OS.

Thank you,
Ankit

> -Original Message-
> From: Chen, Aryeh 
> Sent: Sunday, September 11, 2022 10:18 PM
> To: devel@edk2.groups.io
> Cc: Chen, Aryeh ; Chiu, Chasel
> ; Desimone, Nathaniel L
> ; Oram, Isaac W
> ; Sinha, Ankit ; Gao,
> Liming ; Dong, Eric 
> Subject: [PATCH v3] MinPlatformPkg: Remove _ADR from MinDsdt.asl
> 
> From: Aryeh Chen 
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4057
> 
> According to ASL Coding Guidelines - Device Identifiers "A Device should
> contain either an _ADR or a _HID object, never both."
> , so remove _ADR due to _HID exist.
> 
> Signed-off-by: Aryeh Chen 
> Cc: Chasel Chiu 
> Cc: Nate DeSimone 
> Cc: Isaac Oram 
> Cc: Ankit Sinha 
> Cc: Liming Gao 
> Cc: Eric Dong 
> ---
>  Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> b/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> index 0d94472450..4efb8709ac 100644
> --- a/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> +++ b/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> @@ -27,7 +27,6 @@ DefinitionBlock (
>Name(_HID, EISAID("PNP0A08")) // Indicates PCI Express/PCI-X Mode2
> host hierarchy   Name(_CID, EISAID("PNP0A03")) // To support legacy OS
> that doesn't understand the new HID   Name(_SEG, 0)-  Name(_ADR,
> 0x)   Method(^BN00, 0){ return(0x) }  // Returns default Bus
> number for Peer PCI busses. Name can be overriden with control method
> placed directly under Device scope   Method(_BBN, 0){ return(BN00()) } //
> Bus number, optional for the Root PCI Bus   Name(_UID, 0x)  // Unique
> Bus ID, optional--
> 2.26.2.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#93663): https://edk2.groups.io/g/devel/message/93663
Mute This Topic: https://groups.io/mt/93627173/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




Re: [edk2-devel] [PATCH v3] MinPlatformPkg: Remove _ADR from MinDsdt.asl

2022-09-12 Thread Ankit Sinha
Hi Igor,

Section 6.1  of the ACPI specification describes this (Last line of the 
paragraph below states what Aryeh was saying):

For any device that is on a non-enumerable type of bus (for example, an ISA 
bus), OSPM enumerates the devices' identifier(s) and the ACPI system firmware 
must supply an _HID object (plus one or more optional objects such as _CID, 
_CLS, _HRV, _SUB) for each device to enable OSPM to do that. For devices on an 
enumerable type of bus, such as a PCI bus, the ACPI system must identify which 
device on the enumerable bus is identified by a particular address; the ACPI 
system firmware must supply an _ADR object for each device to enable this. A 
device object must contain either an _HID object or an _ADR object, but must 
not contain both.

Thank you,
Ankit

> -Original Message-
> From: devel@edk2.groups.io  On Behalf Of Igor
> Mammedov
> Sent: Monday, September 12, 2022 1:33 AM
> To: Chen, Aryeh 
> Cc: devel@edk2.groups.io; Chiu, Chasel ; Desimone,
> Nathaniel L ; Oram, Isaac W
> ; Sinha, Ankit ; Gao,
> Liming ; Dong, Eric 
> Subject: Re: [edk2-devel] [PATCH v3] MinPlatformPkg: Remove _ADR from
> MinDsdt.asl
> 
> On Mon, 12 Sep 2022 13:17:45 +0800
> aryeh.c...@intel.com wrote:
> 
> > From: Aryeh Chen 
> >
> > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4057
> >
> > According to ASL Coding Guidelines - Device Identifiers "A Device
> > should contain either an _ADR or a _HID object, never both."
> > , so remove _ADR due to _HID exist.
> 
> I'm curious where exactly in ACPI spec it's said...
> 
> > Signed-off-by: Aryeh Chen 
> > Cc: Chasel Chiu 
> > Cc: Nate DeSimone 
> > Cc: Isaac Oram 
> > Cc: Ankit Sinha 
> > Cc: Liming Gao 
> > Cc: Eric Dong 
> > ---
> >  Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> > b/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> > index 0d94472450..4efb8709ac 100644
> > --- a/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> > +++ b/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> > @@ -27,7 +27,6 @@ DefinitionBlock (
> >Name(_HID, EISAID("PNP0A08")) // Indicates PCI Express/PCI-X Mode2
> host hierarchy
> >Name(_CID, EISAID("PNP0A03")) // To support legacy OS that doesn't
> understand the new HID
> >Name(_SEG, 0)
> > -  Name(_ADR, 0x)
> Spec also says
> v6.3
> 6.1.1 _ADR (Address)
> An _ADR object must be used when specifying the address of any device on
> a bus that has a standard enumeration algorithm
> 
> >Method(^BN00, 0){ return(0x) }  // Returns default Bus number for
> Peer PCI busses. Name can be overriden with control method placed directly
> under Device scope
> >Method(_BBN, 0){ return(BN00()) } // Bus number, optional for the
> Root PCI Bus
> >Name(_UID, 0x)  // Unique Bus ID, optional
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#93662): https://edk2.groups.io/g/devel/message/93662
Mute This Topic: https://groups.io/mt/93627173/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




Re: [edk2-devel] [PATCH v3] MinPlatformPkg: Remove _ADR from MinDsdt.asl

2022-09-12 Thread Igor Mammedov
On Mon, 12 Sep 2022 13:17:45 +0800
aryeh.c...@intel.com wrote:

> From: Aryeh Chen 
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4057
> 
> According to ASL Coding Guidelines - Device Identifiers
> "A Device should contain either an _ADR or a _HID object, never both."
> , so remove _ADR due to _HID exist.

I'm curious where exactly in ACPI spec it's said...

> Signed-off-by: Aryeh Chen 
> Cc: Chasel Chiu 
> Cc: Nate DeSimone 
> Cc: Isaac Oram 
> Cc: Ankit Sinha 
> Cc: Liming Gao 
> Cc: Eric Dong 
> ---
>  Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl 
> b/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> index 0d94472450..4efb8709ac 100644
> --- a/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> +++ b/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
> @@ -27,7 +27,6 @@ DefinitionBlock (
>Name(_HID, EISAID("PNP0A08")) // Indicates PCI Express/PCI-X Mode2 
> host hierarchy
>Name(_CID, EISAID("PNP0A03")) // To support legacy OS that doesn't 
> understand the new HID
>Name(_SEG, 0)
> -  Name(_ADR, 0x)
Spec also says
v6.3
6.1.1 _ADR (Address)
An _ADR object must be used when specifying the address of any device on a bus 
that has a standard enumeration algorithm

>Method(^BN00, 0){ return(0x) }  // Returns default Bus number for 
> Peer PCI busses. Name can be overriden with control method placed directly 
> under Device scope
>Method(_BBN, 0){ return(BN00()) } // Bus number, optional for the Root 
> PCI Bus
>Name(_UID, 0x)  // Unique Bus ID, optional



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#93642): https://edk2.groups.io/g/devel/message/93642
Mute This Topic: https://groups.io/mt/93627173/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




[edk2-devel] [PATCH v3] MinPlatformPkg: Remove _ADR from MinDsdt.asl

2022-09-11 Thread aryeh . chen
From: Aryeh Chen 

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4057

According to ASL Coding Guidelines - Device Identifiers
"A Device should contain either an _ADR or a _HID object, never both."
, so remove _ADR due to _HID exist.

Signed-off-by: Aryeh Chen 
Cc: Chasel Chiu 
Cc: Nate DeSimone 
Cc: Isaac Oram 
Cc: Ankit Sinha 
Cc: Liming Gao 
Cc: Eric Dong 
---
 Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl | 1 -
 1 file changed, 1 deletion(-)

diff --git a/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl 
b/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
index 0d94472450..4efb8709ac 100644
--- a/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
+++ b/Platform/Intel/MinPlatformPkg/Acpi/MinDsdt/MinDsdt.asl
@@ -27,7 +27,6 @@ DefinitionBlock (
   Name(_HID, EISAID("PNP0A08")) // Indicates PCI Express/PCI-X Mode2 host 
hierarchy
   Name(_CID, EISAID("PNP0A03")) // To support legacy OS that doesn't 
understand the new HID
   Name(_SEG, 0)
-  Name(_ADR, 0x)
   Method(^BN00, 0){ return(0x) }  // Returns default Bus number for 
Peer PCI busses. Name can be overriden with control method placed directly 
under Device scope
   Method(_BBN, 0){ return(BN00()) } // Bus number, optional for the Root 
PCI Bus
   Name(_UID, 0x)  // Unique Bus ID, optional
-- 
2.26.2.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#93640): https://edk2.groups.io/g/devel/message/93640
Mute This Topic: https://groups.io/mt/93627173/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-