Angel Pons has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/42127 )
Change subject: MAINTAINERS: Add myself to x4x and pineview northbridges ......................................................................
MAINTAINERS: Add myself to x4x and pineview northbridges
Change-Id: I42b1e8670f4e2ddfea7d473421e86649211eae86 Signed-off-by: Angel Pons th3fanbus@gmail.com --- M MAINTAINERS 1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/42127/1
diff --git a/MAINTAINERS b/MAINTAINERS index 53c36e0..8845851 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -180,6 +180,7 @@
INTEL PINEVIEW CHIPSET M: Damien Zammit damien@zamaudio.com +M: Angel Pons th3fanbus@gmail.com S: Odd Fixes F: src/northbridge/intel/pineview/
@@ -190,6 +191,7 @@
INTEL X4X CHIPSET M: Damien Zammit damien@zamaudio.com +M: Angel Pons th3fanbus@gmail.com S: Odd Fixes F: src/northbridge/intel/x4x/
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42127 )
Change subject: MAINTAINERS: Add myself to x4x and pineview northbridges ......................................................................
Patch Set 1: Code-Review+2
Do you want to change the maintenance ‘level’ too?
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42127 )
Change subject: MAINTAINERS: Add myself to x4x and pineview northbridges ......................................................................
Patch Set 1:
Patch Set 1: Code-Review+2
Do you want to change the maintenance ‘level’ too?
I'm not sure if it can be changed independently so that Damien is "Odd Fixes" and I were "Maintained"
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42127 )
Change subject: MAINTAINERS: Add myself to x4x and pineview northbridges ......................................................................
Patch Set 1:
Patch Set 1:
Patch Set 1: Code-Review+2
Do you want to change the maintenance ‘level’ too?
I'm not sure if it can be changed independently so that Damien is "Odd Fixes" and I were "Maintained"
It would make sense to display the highest common denominator?
Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/42127 )
Change subject: MAINTAINERS: Add myself to x4x and pineview northbridges ......................................................................
MAINTAINERS: Add myself to x4x and pineview northbridges
Change-Id: I42b1e8670f4e2ddfea7d473421e86649211eae86 Signed-off-by: Angel Pons th3fanbus@gmail.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/42127 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Arthur Heymans arthur@aheymans.xyz --- M MAINTAINERS 1 file changed, 2 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Arthur Heymans: Looks good to me, approved
diff --git a/MAINTAINERS b/MAINTAINERS index 53c36e0..8845851 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -180,6 +180,7 @@
INTEL PINEVIEW CHIPSET M: Damien Zammit damien@zamaudio.com +M: Angel Pons th3fanbus@gmail.com S: Odd Fixes F: src/northbridge/intel/pineview/
@@ -190,6 +191,7 @@
INTEL X4X CHIPSET M: Damien Zammit damien@zamaudio.com +M: Angel Pons th3fanbus@gmail.com S: Odd Fixes F: src/northbridge/intel/x4x/
-
Angel Pons (Code Review) -
Arthur Heymans (Code Review)
-
Patrick Georgi (Code Review)