Attention is currently required from: Patrick Rudolph, Angel Pons.
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55472 )
Change subject: configs: Build test x86_64 on Sandy Bridge
......................................................................
Patch Set 10:
(1 comment)
File configs/config.hp_compaq_8200_elite_sff_pc.x86_64:
https://review.coreboot.org/c/coreboot/+/55472/comment/143bb196_7b077813
PS10, Line 4: CONFIG_NO_GFX_INIT=y
> Why? I'd appreciate if you could add a comment with the reasoning (I imagine it's related to build errors).
I suspect this is not needed anymore after: https://review.coreboot.org/c/coreboot/+/55549
--
To view, visit https://review.coreboot.org/c/coreboot/+/55472
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I2c18af8bfa87636c68741e4759059276c287d052
Gerrit-Change-Number: 55472
Gerrit-PatchSet: 10
Gerrit-Owner: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Comment-Date: Fri, 02 Jul 2021 11:38:16 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-MessageType: comment
Attention is currently required from: Varshit B Pandya, Bora Guvendik, Anil Kumar K, Rizwan Qureshi, Subrata Banik, Bernardo Perez Priego.
Hello Bora Guvendik, build bot (Jenkins), Anil Kumar K, Maulik V Vaghela, Rizwan Qureshi, Subrata Banik, Bernardo Perez Priego,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/55733
to look at the new patch set (#5).
Change subject: mb/intel/adlrvp_m: Remove ASL code and enable dynamic SSDT creation for camera ACPI
......................................................................
mb/intel/adlrvp_m: Remove ASL code and enable dynamic SSDT creation for camera ACPI
This change updates device tree to enable SSDT generation for
world facing camera and user facing camera for ADLRVP.
Also reverts DSDT changes related to both camera.
TEST=Build and Boot aldrvp check i2c enumeration and output of media-ctl
Signed-off-by: Varshit B Pandya <varshit.b.pandya(a)intel.com>
Change-Id: I39f82dc9eb91496d80479ae3f59ca5e03402a599
---
M src/mainboard/intel/adlrvp/Kconfig.name
M src/mainboard/intel/adlrvp/devicetree_m.cb
2 files changed, 104 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/33/55733/5
--
To view, visit https://review.coreboot.org/c/coreboot/+/55733
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I39f82dc9eb91496d80479ae3f59ca5e03402a599
Gerrit-Change-Number: 55733
Gerrit-PatchSet: 5
Gerrit-Owner: Varshit B Pandya <varshit.b.pandya(a)intel.com>
Gerrit-Reviewer: Anil Kumar K <anil.kumar.k(a)intel.com>
Gerrit-Reviewer: Bernardo Perez Priego <bernardo.perez.priego(a)intel.com>
Gerrit-Reviewer: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-Reviewer: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Selma Bensaid <selma.bensaid(a)intel.com>
Gerrit-Attention: Varshit B Pandya <varshit.b.pandya(a)intel.com>
Gerrit-Attention: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-Attention: Anil Kumar K <anil.kumar.k(a)intel.com>
Gerrit-Attention: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Attention: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Attention: Bernardo Perez Priego <bernardo.perez.priego(a)intel.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Varshit B Pandya, Bora Guvendik, Anil Kumar K, Rizwan Qureshi, Sugnan Prabhu S, Subrata Banik, Bernardo Perez Priego, Ronak Kanabar.
Hello Bora Guvendik, build bot (Jenkins), Anil Kumar K, Maulik V Vaghela, Rizwan Qureshi, Sugnan Prabhu S, Subrata Banik, Bernardo Perez Priego, Ronak Kanabar,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/55526
to look at the new patch set (#17).
Change subject: mb/intel/adlrvp: Remove ASL code and enable dynamic SSDT creation for camera ACPI
......................................................................
mb/intel/adlrvp: Remove ASL code and enable dynamic SSDT creation for camera ACPI
This change updates device tree to enable SSDT generation for
world facing camera and user facing camera for ADLRVP.
Also reverts DSDT changes related to both camera.
TEST=Build and Boot aldrvp check i2c enumeration and output of media-ctl
Compared SSDT with this patch against DSDT without this patch, they are same
Signed-off-by: Varshit B Pandya <varshit.b.pandya(a)intel.com>
Change-Id: I08834bbcf80dc46737de07f69a2402ed6bf93d4f
---
M src/mainboard/intel/adlrvp/Kconfig.name
D src/mainboard/intel/adlrvp/acpi/mipi_camera.asl
M src/mainboard/intel/adlrvp/devicetree.cb
M src/mainboard/intel/adlrvp/dsdt.asl
4 files changed, 104 insertions(+), 506 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/26/55526/17
--
To view, visit https://review.coreboot.org/c/coreboot/+/55526
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I08834bbcf80dc46737de07f69a2402ed6bf93d4f
Gerrit-Change-Number: 55526
Gerrit-PatchSet: 17
Gerrit-Owner: Varshit B Pandya <varshit.b.pandya(a)intel.com>
Gerrit-Reviewer: Anil Kumar K <anil.kumar.k(a)intel.com>
Gerrit-Reviewer: Bernardo Perez Priego <bernardo.perez.priego(a)intel.com>
Gerrit-Reviewer: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-Reviewer: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Reviewer: Ronak Kanabar <ronak.kanabar(a)intel.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: Sugnan Prabhu S <sugnan.prabhu.s(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Selma Bensaid <selma.bensaid(a)intel.com>
Gerrit-Attention: Varshit B Pandya <varshit.b.pandya(a)intel.com>
Gerrit-Attention: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-Attention: Anil Kumar K <anil.kumar.k(a)intel.com>
Gerrit-Attention: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Attention: Sugnan Prabhu S <sugnan.prabhu.s(a)intel.com>
Gerrit-Attention: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Attention: Bernardo Perez Priego <bernardo.perez.priego(a)intel.com>
Gerrit-Attention: Ronak Kanabar <ronak.kanabar(a)intel.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Patrick Rudolph.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55472 )
Change subject: configs: Build test x86_64 on Sandy Bridge
......................................................................
Patch Set 10: Code-Review+1
(1 comment)
File configs/config.hp_compaq_8200_elite_sff_pc.x86_64:
https://review.coreboot.org/c/coreboot/+/55472/comment/0b95b9af_f9fb3640
PS10, Line 4: CONFIG_NO_GFX_INIT=y
Why? I'd appreciate if you could add a comment with the reasoning (I imagine it's related to build errors).
--
To view, visit https://review.coreboot.org/c/coreboot/+/55472
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I2c18af8bfa87636c68741e4759059276c287d052
Gerrit-Change-Number: 55472
Gerrit-PatchSet: 10
Gerrit-Owner: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Comment-Date: Fri, 02 Jul 2021 11:03:46 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Damien Zammit, Arthur Heymans, Patrick Rudolph.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56022 )
Change subject: nb/intel/x4x: Expose x86_64 support
......................................................................
Patch Set 3: Code-Review+1
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/56022/comment/43b11742_a85294ca
PS3, Line 9: TESTED on foxconn g41m.
Please add a config file so that Jenkins can build-test the x86_64 codepath
--
To view, visit https://review.coreboot.org/c/coreboot/+/56022
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I2d5d5dfbd76a84aa400e44b4bc2ac4f3b5b6f739
Gerrit-Change-Number: 56022
Gerrit-PatchSet: 3
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Damien Zammit
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Damien Zammit
Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Comment-Date: Fri, 02 Jul 2021 10:56:53 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Arthur Heymans, Patrick Rudolph.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56021 )
Change subject: cpu/intel/car/p4: Add x86_64 support
......................................................................
Patch Set 3: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/56021
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I77516e3cd5f0d3b7442be660c005a65b00454343
Gerrit-Change-Number: 56021
Gerrit-PatchSet: 3
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Comment-Date: Fri, 02 Jul 2021 10:55:36 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment