Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47235 )
Change subject: drivers/i2c/rx6110sa: Add basic ACPI support
......................................................................
Patch Set 7:
Patch Set 7:
What the heck is going on here?!?!?! Gyus, stop this, please!
Can somebody in Coreboot community read the ACPI spec (in particular chapter 6.1.5) and forbid such changes?
This MUST be reverted!
Could you please calm down and explain *why* you think this change is wrong?
--
To view, visit
https://review.coreboot.org/c/coreboot/+/47235
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9b319e3088e6511592075b055f8fa3e2aedaa209
Gerrit-Change-Number: 47235
Gerrit-PatchSet: 7
Gerrit-Owner: Werner Zeh
werner.zeh@siemens.com
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Johannes Hahn
johannes-hahn@siemens.com
Gerrit-Reviewer: Mario Scheithauer
mario.scheithauer@siemens.com
Gerrit-Reviewer: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Reviewer: Werner Zeh
werner.zeh@siemens.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Reviewer: siemens-bot
Gerrit-CC: 9elements QA
hardwaretestrobot@gmail.com
Gerrit-CC: Andy Shevchenko
andy.shevchenko@gmail.com
Gerrit-CC: Patrick Georgi
pgeorgi@google.com
Gerrit-Comment-Date: Tue, 17 Nov 2020 12:05:50 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment