Christian Walter has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/33250
Change subject: src/superio/aspeed/common: Add ACPI Template for Autogen Feature ......................................................................
src/superio/aspeed/common: Add ACPI Template for Autogen Feature
Add ACPI Template for Autogen Feature.
Change-Id: Ic356af269e381bd5310d9be057bc16f9d5186934 Signed-off-by: Christian Walter christian.walter@9elements.com --- A src/superio/aspeed/common/acpi/superio.asl 1 file changed, 21 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/33250/1
diff --git a/src/superio/aspeed/common/acpi/superio.asl b/src/superio/aspeed/common/acpi/superio.asl new file mode 100644 index 0000000..6b8b8f6 --- /dev/null +++ b/src/superio/aspeed/common/acpi/superio.asl @@ -0,0 +1,21 @@ +## +## This file is part of the coreboot project. +## +## +## This program is free software; you can redistribute it and/or modify +## it under the terms of the GNU General Public License as published by +## the Free Software Foundation; either version 2 of the License, or +## (at your option) any later version. +## +## This program is distributed in the hope that it will be useful, +## but WITHOUT ANY WARRANTY; without even the implied warranty of +## MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +## GNU General Public License for more details. +## + +Device (SIO0) +{ + Name (_HID, EisaId ("PNP0C02") /* PNP Motherboard Resources */) // _HID: Hardware ID + Name (_UID, Zero) // _UID: Unique ID + +} \ No newline at end of file
Philipp Deppenwiese has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33250 )
Change subject: src/superio/aspeed/common: Add ACPI Template for Autogen Feature ......................................................................
Patch Set 1: Code-Review+2
Hello Philipp Deppenwiese, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/33250
to look at the new patch set (#3).
Change subject: src/superio/aspeed/common: Add ACPI Template for Autogen Feature ......................................................................
src/superio/aspeed/common: Add ACPI Template for Autogen Feature
Add ACPI Template for Autogen Feature.
Change-Id: Ic356af269e381bd5310d9be057bc16f9d5186934 Signed-off-by: Christian Walter christian.walter@9elements.com --- A src/superio/aspeed/common/acpi/superio.asl 1 file changed, 21 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/33250/3
Hello Philipp Deppenwiese, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/33250
to look at the new patch set (#4).
Change subject: src/superio/aspeed/common: Add ACPI Template for Autogen Feature ......................................................................
src/superio/aspeed/common: Add ACPI Template for Autogen Feature
Add ACPI Template for Autogen Feature.
Change-Id: Ic356af269e381bd5310d9be057bc16f9d5186934 Signed-off-by: Christian Walter christian.walter@9elements.com --- A src/superio/aspeed/common/acpi/superio.asl 1 file changed, 21 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/33250/4
Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33250 )
Change subject: src/superio/aspeed/common: Add ACPI Template for Autogen Feature ......................................................................
Patch Set 5: Code-Review+1
(1 comment)
https://review.coreboot.org/#/c/33250/5/src/superio/aspeed/common/acpi/super... File src/superio/aspeed/common/acpi/superio.asl:
https://review.coreboot.org/#/c/33250/5/src/superio/aspeed/common/acpi/super... PS5, Line 18: Name (_HID, EisaId ("PNP0C02") /* PNP Motherboard Resources */) // _HID: Hardware ID this is a bit odd mix of comment styles
Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33250 )
Change subject: src/superio/aspeed/common: Add ACPI Template for Autogen Feature ......................................................................
Patch Set 6:
Are you still planning to address the mixed comment style in one line issue? That's the only reason why I'm here a bit hesitant to +2 the patch; otherwise it looks good to me
HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33250 )
Change subject: src/superio/aspeed/common: Add ACPI Template for Autogen Feature ......................................................................
Patch Set 8:
(1 comment)
https://review.coreboot.org/c/coreboot/+/33250/8/src/superio/aspeed/common/a... File src/superio/aspeed/common/acpi/superio.asl:
https://review.coreboot.org/c/coreboot/+/33250/8/src/superio/aspeed/common/a... PS8, Line 18: /* PNP Motherboard Resources */ remove ?
Hello Felix Held, Patrick Rudolph, Philipp Deppenwiese, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/33250
to look at the new patch set (#9).
Change subject: src/superio/aspeed/common: Add ACPI Template for Autogen Feature ......................................................................
src/superio/aspeed/common: Add ACPI Template for Autogen Feature
Add ACPI Template for Autogen Feature.
Change-Id: Ic356af269e381bd5310d9be057bc16f9d5186934 Signed-off-by: Christian Walter christian.walter@9elements.com --- A src/superio/aspeed/common/acpi/superio.asl 1 file changed, 21 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/33250/9
Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33250 )
Change subject: src/superio/aspeed/common: Add ACPI Template for Autogen Feature ......................................................................
Patch Set 9: Code-Review+2
(2 comments)
Patch Set 8:
(1 comment)
https://review.coreboot.org/c/coreboot/+/33250/5/src/superio/aspeed/common/a... File src/superio/aspeed/common/acpi/superio.asl:
https://review.coreboot.org/c/coreboot/+/33250/5/src/superio/aspeed/common/a... PS5, Line 18: Name (_HID, EisaId ("PNP0C02") /* PNP Motherboard Resources */) // _HID: Hardware ID
this is a bit odd mix of comment styles
Done
https://review.coreboot.org/c/coreboot/+/33250/8/src/superio/aspeed/common/a... File src/superio/aspeed/common/acpi/superio.asl:
https://review.coreboot.org/c/coreboot/+/33250/8/src/superio/aspeed/common/a... PS8, Line 18: /* PNP Motherboard Resources */
remove ?
Done
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33250 )
Change subject: src/superio/aspeed/common: Add ACPI Template for Autogen Feature ......................................................................
Patch Set 10:
(4 comments)
https://review.coreboot.org/c/coreboot/+/33250/10//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/33250/10//COMMIT_MSG@9 PS10, Line 9: Autogen Feature autogen feeater
https://review.coreboot.org/c/coreboot/+/33250/10//COMMIT_MSG@9 PS10, Line 9: Template template
https://review.coreboot.org/c/coreboot/+/33250/10//COMMIT_MSG@10 PS10, Line 10: Hmm, that is the commit message summary repeated. What is the autogen feature?
https://review.coreboot.org/c/coreboot/+/33250/10/src/superio/aspeed/common/... File src/superio/aspeed/common/acpi/superio.asl:
https://review.coreboot.org/c/coreboot/+/33250/10/src/superio/aspeed/common/... PS10, Line 3: * One line? Missing author names?
Hello Felix Held, Patrick Rudolph, Philipp Deppenwiese, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/33250
to look at the new patch set (#12).
Change subject: src/superio/aspeed/common: Add ACPI Template for Autogen Feature ......................................................................
src/superio/aspeed/common: Add ACPI Template for Autogen Feature
Add ACPI template for autogen feature.
Change-Id: Ic356af269e381bd5310d9be057bc16f9d5186934 Signed-off-by: Christian Walter christian.walter@9elements.com --- A src/superio/aspeed/common/acpi/superio.asl 1 file changed, 21 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/33250/12
Christian Walter has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33250 )
Change subject: src/superio/aspeed/common: Add ACPI Template for Autogen Feature ......................................................................
Patch Set 12:
(4 comments)
https://review.coreboot.org/c/coreboot/+/33250/10//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/33250/10//COMMIT_MSG@9 PS10, Line 9: Autogen Feature
autogen feeater
Done
https://review.coreboot.org/c/coreboot/+/33250/10//COMMIT_MSG@9 PS10, Line 9: Template
template
Done
https://review.coreboot.org/c/coreboot/+/33250/10//COMMIT_MSG@10 PS10, Line 10:
Hmm, that is the commit message summary repeated. […]
Done
https://review.coreboot.org/c/coreboot/+/33250/10/src/superio/aspeed/common/... File src/superio/aspeed/common/acpi/superio.asl:
https://review.coreboot.org/c/coreboot/+/33250/10/src/superio/aspeed/common/... PS10, Line 3: *
One line? Missing author names?
Done
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33250 )
Change subject: src/superio/aspeed/common: Add ACPI Template for Autogen Feature ......................................................................
Patch Set 12:
(2 comments)
https://review.coreboot.org/c/coreboot/+/33250/12//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/33250/12//COMMIT_MSG@7 PS12, Line 7: src/superio/aspeed/common: Add ACPI Template for Autogen Feature Please remove `src` at the beginning. The description also not very understandable in my opinion.
superio/aspeed: Add ACPI declarations
With the minimal ASL file, coreboot will autogenerate the necessary ACPI entries(?).
Please also add how you tested this.
https://review.coreboot.org/c/coreboot/+/33250/12/src/superio/aspeed/common/... File src/superio/aspeed/common/acpi/superio.asl:
https://review.coreboot.org/c/coreboot/+/33250/12/src/superio/aspeed/common/... PS12, Line 18: Name (_HID, EisaId ("PNP0C02") ) // _HID: Hardware ID That’s the ID for the environment controller?
Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33250 )
Change subject: src/superio/aspeed/common: Add ACPI Template for Autogen Feature ......................................................................
Patch Set 12:
No longer needed with #33033
Christian Walter has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/33250 )
Change subject: src/superio/aspeed/common: Add ACPI Template for Autogen Feature ......................................................................
Abandoned
No longer needed with #33033