Hello Patrick Rudolph, Subrata Banik, Balaji Manigandan, Sooraj Govindan, Aamir Bohra, Justin TerAvest, Rizwan Qureshi, build bot (Jenkins), Patrick Georgi, Furquan Shaikh, Karthik Ramasubramanian, Meera Ravindranath, Ronak Kanabar, Usha P, Tim Wawrzynczak, Martin Roth,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/39111
to look at the new patch set (#11).
Change subject: soc/intel/tigerlake: Add Jasper lake GPIO support
......................................................................
soc/intel/tigerlake: Add Jasper lake GPIO support
Add gpio definition for Jasper Lake gpio controller.
Also created a separate file for JSL and TGL gpio keeping common asl file.
gpio_soc_defs.h must pass correct information/macro values to asl file
for code to work.
GPIO controller includes 4 gpio community and 10 groups. Patch adds
definition for all gpio within community and groups
Updated IRQ mapping for all gpios
TEST=Check if jslrvp and tglrvp code is compiling
Change-Id: Iae4e694ecb30658e43c5ed99e5436579fd7d2ed2
Signed-off-by: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Signed-off-by: Ronak Kanabar <ronak.kanabar(a)intel.com>
Signed-off-by: Usha P <usha.p(a)intel.com>
---
A src/soc/intel/common/acpi/gpio_op.asl
M src/soc/intel/tigerlake/Makefile.inc
M src/soc/intel/tigerlake/acpi/gpio.asl
M src/soc/intel/tigerlake/chip.h
A src/soc/intel/tigerlake/gpio_jsl.c
R src/soc/intel/tigerlake/gpio_tgl.c
M src/soc/intel/tigerlake/include/soc/gpio.h
M src/soc/intel/tigerlake/include/soc/gpio_defs.h
A src/soc/intel/tigerlake/include/soc/gpio_defs_jsl.h
A src/soc/intel/tigerlake/include/soc/gpio_defs_tgl.h
M src/soc/intel/tigerlake/include/soc/gpio_soc_defs.h
A src/soc/intel/tigerlake/include/soc/gpio_soc_defs_jsl.h
M src/soc/intel/tigerlake/include/soc/pmc.h
13 files changed, 1,424 insertions(+), 475 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/11/39111/11
--
To view, visit https://review.coreboot.org/c/coreboot/+/39111
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iae4e694ecb30658e43c5ed99e5436579fd7d2ed2
Gerrit-Change-Number: 39111
Gerrit-PatchSet: 11
Gerrit-Owner: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Reviewer: Aamir Bohra <aamir.bohra(a)intel.com>
Gerrit-Reviewer: Balaji Manigandan <balaji.manigandan(a)intel.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Justin TerAvest <teravest(a)chromium.org>
Gerrit-Reviewer: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Reviewer: Meera Ravindranath <meera.ravindranath(a)intel.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Reviewer: Ronak Kanabar <ronak.kanabar(a)intel.com>
Gerrit-Reviewer: Sooraj Govindan <sooraj.govindan(a)intel.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: Usha P <usha.p(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: newpatchset
Maulik V Vaghela has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39111 )
Change subject: soc/intel/tigerlake: Add Jasper lake GPIO support
......................................................................
Patch Set 10:
> Patch Set 7:
>
> @Maulik, please sit on tree and don't break the tree as we are seeing rebasing issue
Sure Subrata..I will take care of it.
--
To view, visit https://review.coreboot.org/c/coreboot/+/39111
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iae4e694ecb30658e43c5ed99e5436579fd7d2ed2
Gerrit-Change-Number: 39111
Gerrit-PatchSet: 10
Gerrit-Owner: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Reviewer: Aamir Bohra <aamir.bohra(a)intel.com>
Gerrit-Reviewer: Balaji Manigandan <balaji.manigandan(a)intel.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Justin TerAvest <teravest(a)chromium.org>
Gerrit-Reviewer: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Reviewer: Meera Ravindranath <meera.ravindranath(a)intel.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Reviewer: Ronak Kanabar <ronak.kanabar(a)intel.com>
Gerrit-Reviewer: Sooraj Govindan <sooraj.govindan(a)intel.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: Usha P <usha.p(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Sun, 01 Mar 2020 15:21:37 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38978 )
Change subject: [WIP] mainboard: Add Acer ES1-572
......................................................................
Patch Set 4:
(1 comment)
https://review.coreboot.org/c/coreboot/+/38978/3//COMMIT_MSG
Commit Message:
https://review.coreboot.org/c/coreboot/+/38978/3//COMMIT_MSG@31
PS3, Line 31: UART 2
> The FSP has a UPD to enable these. Try using: […]
Yes, I had issues with that before. Now it works.
Also, I forgot to send this message yesterday. It's not really verbose, most of the fun stuff is done inside FSP anyway.
--
To view, visit https://review.coreboot.org/c/coreboot/+/38978
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Id98788a2c5e54f70fd1cacbd70d636f5e63b2619
Gerrit-Change-Number: 38978
Gerrit-PatchSet: 4
Gerrit-Owner: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Comment-Date: Sun, 01 Mar 2020 11:42:24 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-MessageType: comment
Subrata Banik has uploaded a new patch set (#10) to the change originally created by Maulik V Vaghela. ( https://review.coreboot.org/c/coreboot/+/39111 )
Change subject: soc/intel/tigerlake: Add Jasper lake GPIO support
......................................................................
soc/intel/tigerlake: Add Jasper lake GPIO support
Add gpio definition for Jasper Lake gpio controller.
Also created a separate file for JSL and TGL gpios along with separate
ASL files for JSL.
GPIO controller includes 4 gpio community and 10 groups. Patch adds
definition for all gpio within community and groups
Updated IRQ mapping for all gpios
TEST=Check if jslrvp code is compiling
Change-Id: Iae4e694ecb30658e43c5ed99e5436579fd7d2ed2
Signed-off-by: Ronak Kanabar <ronak.kanabar(a)intel.com>
Signed-off-by: Usha P <usha.p(a)intel.com>
Signed-off-by: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
---
M src/soc/intel/tigerlake/Makefile.inc
M src/soc/intel/tigerlake/acpi/gpio.asl
A src/soc/intel/tigerlake/acpi/gpio_jsl.asl
A src/soc/intel/tigerlake/acpi/gpio_op.asl
A src/soc/intel/tigerlake/acpi/gpio_tgl.asl
M src/soc/intel/tigerlake/chip.h
A src/soc/intel/tigerlake/gpio_jsl.c
R src/soc/intel/tigerlake/gpio_tgl.c
M src/soc/intel/tigerlake/include/soc/gpio.h
M src/soc/intel/tigerlake/include/soc/gpio_defs.h
A src/soc/intel/tigerlake/include/soc/gpio_defs_jsl.h
A src/soc/intel/tigerlake/include/soc/gpio_defs_tgl.h
A src/soc/intel/tigerlake/include/soc/gpio_soc_defs_jsl.h
M src/soc/intel/tigerlake/include/soc/pmc.h
14 files changed, 1,634 insertions(+), 520 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/11/39111/10
--
To view, visit https://review.coreboot.org/c/coreboot/+/39111
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iae4e694ecb30658e43c5ed99e5436579fd7d2ed2
Gerrit-Change-Number: 39111
Gerrit-PatchSet: 10
Gerrit-Owner: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Reviewer: Aamir Bohra <aamir.bohra(a)intel.com>
Gerrit-Reviewer: Balaji Manigandan <balaji.manigandan(a)intel.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Justin TerAvest <teravest(a)chromium.org>
Gerrit-Reviewer: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Maulik V Vaghela <maulik.v.vaghela(a)intel.com>
Gerrit-Reviewer: Meera Ravindranath <meera.ravindranath(a)intel.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi(a)intel.com>
Gerrit-Reviewer: Ronak Kanabar <ronak.kanabar(a)intel.com>
Gerrit-Reviewer: Sooraj Govindan <sooraj.govindan(a)intel.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: Usha P <usha.p(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: newpatchset
Benjamin Doron has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38978 )
Change subject: [WIP] mainboard: Add Acer ES1-572
......................................................................
Patch Set 4:
(1 comment)
https://review.coreboot.org/c/coreboot/+/38978/3//COMMIT_MSG
Commit Message:
https://review.coreboot.org/c/coreboot/+/38978/3//COMMIT_MSG@31
PS3, Line 31: UART 2
> The FSP has a UPD to enable these. Try using: […]
Done. I guess if it's too verbose, you can disable it if you want?
--
To view, visit https://review.coreboot.org/c/coreboot/+/38978
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Id98788a2c5e54f70fd1cacbd70d636f5e63b2619
Gerrit-Change-Number: 38978
Gerrit-PatchSet: 4
Gerrit-Owner: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Comment-Date: Sun, 01 Mar 2020 01:20:59 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-MessageType: comment
Hello build bot (Jenkins), Patrick Georgi, Martin Roth,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/38978
to look at the new patch set (#4).
Change subject: [WIP] mainboard: Add Acer ES1-572
......................................................................
[WIP] mainboard: Add Acer ES1-572
Both the Acer Aspire ES1-572 and the Acer Extensa 2540 use the same
mainboard. (Compal LA-E061P)
Tested and working:
- Intel Core i3-6006U with KabyLakeFspBinPkg
- Both SO-DIMM DDR4 slots
- Ethernet
- USB 3.0 port
- HDMI with libgfxinit
- SATA with an SSD
- Loading Linux with SeaBIOS and tianocore
- Power button soldered onto the board's debug pads.
- UART 2 to get yelled at
- S3 suspend and resume, for some reason
Not working or need testing:
- Wi-Fi card. Probably fixed with correct PCIe settings.
- Fan is pretending to be a leaf blower.
- All EC-related stuffs. The thing is an ENE KB9022, similar to KB9012.
- Audio is missing configuration.
- Internal eDP LCD. I think I have the cable somewhere.
- Lid. It was brutally damaged, even though the LCD managed to survive.
- Battery. I have it somewhere.
- Keyboard and touchpad.
- Card reader.
Change-Id: Id98788a2c5e54f70fd1cacbd70d636f5e63b2619
Signed-off-by: Angel Pons <th3fanbus(a)gmail.com>
---
A acer_a_thing.defconfig
A src/mainboard/acer/Kconfig
A src/mainboard/acer/Kconfig.name
A src/mainboard/acer/es1-572/Kconfig
A src/mainboard/acer/es1-572/Kconfig.name
A src/mainboard/acer/es1-572/Makefile.inc
A src/mainboard/acer/es1-572/acpi/ec.asl
A src/mainboard/acer/es1-572/acpi/mainboard.asl
A src/mainboard/acer/es1-572/acpi/superio.asl
A src/mainboard/acer/es1-572/board_info.txt
A src/mainboard/acer/es1-572/devicetree.cb
A src/mainboard/acer/es1-572/dsdt.asl
A src/mainboard/acer/es1-572/gma-mainboard.ads
A src/mainboard/acer/es1-572/gpio.h
A src/mainboard/acer/es1-572/hda_verb.c
A src/mainboard/acer/es1-572/ramstage.c
A src/mainboard/acer/es1-572/romstage.c
17 files changed, 1,514 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/78/38978/4
--
To view, visit https://review.coreboot.org/c/coreboot/+/38978
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Id98788a2c5e54f70fd1cacbd70d636f5e63b2619
Gerrit-Change-Number: 38978
Gerrit-PatchSet: 4
Gerrit-Owner: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-MessageType: newpatchset