coreboot-gerrit June 2017

coreboot-gerrit@coreboot.org

1 participants
2269 discussions

Change in coreboot[master]: device/pnp: remove struct io_info
by build bot (Jenkins) (Code Review) 07 Jun '17

07 Jun '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20078 ) Change subject: device/pnp: remove struct io_info ...................................................................... Patch Set 3: Verified-1 Build Unstable https://qa.coreboot.org/job/coreboot-gerrit/55044/ : UNSTABLE https://qa.coreboot.org/job/coreboot-checkpatch/10728/ : SUCCESS -- To view, visit https://review.coreboot.org/20078 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1e7853844605cd2a6d568caf05488e1218fb53f9 Gerrit-Change-Number: 20078 Gerrit-PatchSet: 3 Gerrit-Owner: Samuel Holland <samuel(a)sholland.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 07 Jun 2017 05:11:22 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: device/pnp: remove struct io_info
by Samuel Holland (Code Review) 07 Jun '17

07 Jun '17

Hello build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/20078 to look at the new patch set (#3). Change subject: device/pnp: remove struct io_info ...................................................................... device/pnp: remove struct io_info The 'set' field was not used anywhere. Replace the struct with a simple integer. ec.c and superio.c updates performed with: sed -i -r 's/\{ ?0(x([[:digit:]abcdefABCDEF]{3,4}))?, (0x)?[04]? ?\}/0\1/g' src/ec/*/*/ec.c sed -i -r 's/\{ ?0(x([[:digit:]abcdefABCDEF]{3,4}))?, (0x)?[04] ?\}/0\1/g' src/ec/*/*/ec_lpc.c sed -i -r 's/\{ ?0(x([[:digit:]abcdefABCDEF]{3,4}))?, (0x)?[04] ?\}/0\1/g' src/superio/*/*/superio.c sed -i -r 's/\{ ?0(x([[:digit:]abcdefABCDEF]{3,4}))?, (0x)?[04] ?\}/0\1/g' src/superio/smsc/sio10n268/sio10n268.c sed -i -r 's/\{ ?0(x([[:digit:]abcdefABCDEF]{3,4}))?, (0x)?[04] ?\}/0\1/g' src/superio/via/vt1211/vt1211.c Change-Id: I1e7853844605cd2a6d568caf05488e1218fb53f9 Signed-off-by: Samuel Holland <samuel(a)sholland.org> --- M src/device/pnp_device.c M src/ec/compal/ene932/ec.c M src/ec/google/chromeec/ec_lpc.c M src/ec/kontron/it8516e/ec.c M src/ec/quanta/ene_kb3940q/ec.c M src/ec/quanta/it8518/ec.c M src/ec/roda/it8518/ec.c M src/include/device/pnp.h M src/superio/fintek/f71805f/superio.c M src/superio/fintek/f71808a/superio.c M src/superio/fintek/f71859/superio.c M src/superio/fintek/f71863fg/superio.c M src/superio/fintek/f71869ad/superio.c M src/superio/fintek/f71872/superio.c M src/superio/fintek/f81216h/superio.c M src/superio/fintek/f81865f/superio.c M src/superio/fintek/f81866d/superio.c M src/superio/intel/i3100/superio.c M src/superio/intel/i8900/superio.c M src/superio/ite/it8671f/superio.c M src/superio/ite/it8712f/superio.c M src/superio/ite/it8716f/superio.c M src/superio/ite/it8718f/superio.c M src/superio/ite/it8721f/superio.c M src/superio/ite/it8728f/superio.c M src/superio/ite/it8772f/superio.c M src/superio/ite/it8783ef/superio.c M src/superio/nsc/pc87309/superio.c M src/superio/nsc/pc87360/superio.c M src/superio/nsc/pc87366/superio.c M src/superio/nsc/pc87382/superio.c M src/superio/nsc/pc87384/superio.c M src/superio/nsc/pc87392/superio.c M src/superio/nsc/pc87417/superio.c M src/superio/nsc/pc97317/superio.c M src/superio/nuvoton/nct5104d/superio.c M src/superio/nuvoton/nct5572d/superio.c M src/superio/nuvoton/nct6776/superio.c M src/superio/nuvoton/nct6779d/superio.c M src/superio/nuvoton/nct6791d/superio.c M src/superio/nuvoton/wpcm450/superio.c M src/superio/renesas/m3885x/superio.c M src/superio/smsc/dme1737/superio.c M src/superio/smsc/kbc1100/superio.c M src/superio/smsc/lpc47b272/superio.c M src/superio/smsc/lpc47b397/superio.c M src/superio/smsc/lpc47m10x/superio.c M src/superio/smsc/lpc47m15x/superio.c M src/superio/smsc/lpc47n217/superio.c M src/superio/smsc/lpc47n227/superio.c M src/superio/smsc/mec1308/superio.c M src/superio/smsc/sch4037/superio.c M src/superio/smsc/sio1036/superio.c M src/superio/smsc/sio10n268/sio10n268.c M src/superio/smsc/smscsuperio/superio.c M src/superio/via/vt1211/vt1211.c M src/superio/winbond/w83627dhg/superio.c M src/superio/winbond/w83627ehg/superio.c M src/superio/winbond/w83627hf/superio.c M src/superio/winbond/w83627thg/superio.c M src/superio/winbond/w83627uhg/superio.c M src/superio/winbond/w83667hg-a/superio.c M src/superio/winbond/w83697hf/superio.c M src/superio/winbond/w83977tf/superio.c M src/superio/winbond/wpcd376i/superio.c 65 files changed, 366 insertions(+), 370 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/78/20078/3 -- To view, visit https://review.coreboot.org/20078 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I1e7853844605cd2a6d568caf05488e1218fb53f9 Gerrit-Change-Number: 20078 Gerrit-PatchSet: 3 Gerrit-Owner: Samuel Holland <samuel(a)sholland.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in coreboot[master]: soc/baytrail: fix ACPI table by recollecting TOLM
by build bot (Jenkins) (Code Review) 07 Jun '17

07 Jun '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20081 ) Change subject: soc/baytrail: fix ACPI table by recollecting TOLM ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/10727/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/55043/ : SUCCESS -- To view, visit https://review.coreboot.org/20081 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib26d4fe229b3f7d8ee664f5d89774d1f4a997f51 Gerrit-Change-Number: 20081 Gerrit-PatchSet: 1 Gerrit-Owner: Matt DeVillier <matt.devillier(a)gmail.com> Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 07 Jun 2017 05:02:42 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: soc/baytrail: fix ACPI table by recollecting TOLM
by Matt DeVillier (Code Review) 07 Jun '17

07 Jun '17

Matt DeVillier has uploaded this change for review. ( https://review.coreboot.org/20081 Change subject: soc/baytrail: fix ACPI table by recollecting TOLM ...................................................................... soc/baytrail: fix ACPI table by recollecting TOLM Adapted from Chromium commit 8fbe1e7 for soc/braswell (also review.coreboot.org/#/c/20060/); same issue affects baytrail as well. This patch recollects TOLM accessing; as Aaron recalled some core_msr_script turns off access to TOLM register, he suggests to store tolm to avoid getting back a zero while setting acpi nvs space. Change-Id: Ib26d4fe229b3f7d8ee664f5d89774d1f4a997f51 Signed-off-by: Matt DeVillier <matt.devillier(a)gmail.com> --- M src/soc/intel/baytrail/northcluster.c 1 file changed, 9 insertions(+), 2 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/81/20081/1 diff --git a/src/soc/intel/baytrail/northcluster.c b/src/soc/intel/baytrail/northcluster.c index 55ace59..c8ba9c1 100644 --- a/src/soc/intel/baytrail/northcluster.c +++ b/src/soc/intel/baytrail/northcluster.c @@ -20,7 +20,7 @@ #include <device/pci_ids.h> #include <vendorcode/google/chromeos/chromeos.h> #include <arch/acpi.h> - +#include <stddef.h> #include <soc/iomap.h> #include <soc/iosf.h> #include <soc/pci_devs.h> @@ -65,7 +65,14 @@ uint32_t nc_read_top_of_low_memory(void) { - return iosf_bunit_read(BUNIT_BMBOUND) & ~((1 << 27) - 1); + MAYBE_STATIC uint32_t tolm = 0; + + if (tolm) + return tolm; + + tolm = iosf_bunit_read(BUNIT_BMBOUND) & ~((1 << 27) - 1); + + return tolm; } static void nc_read_resources(device_t dev) -- To view, visit https://review.coreboot.org/20081 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ib26d4fe229b3f7d8ee664f5d89774d1f4a997f51 Gerrit-Change-Number: 20081 Gerrit-PatchSet: 1 Gerrit-Owner: Matt DeVillier <matt.devillier(a)gmail.com>

1 0

Change in coreboot[master]: soc/baytrail: assign unique DMA request lines to I2C control...
by build bot (Jenkins) (Code Review) 07 Jun '17

07 Jun '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20080 ) Change subject: soc/baytrail: assign unique DMA request lines to I2C controllers ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/10726/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/55042/ : SUCCESS -- To view, visit https://review.coreboot.org/20080 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Icc9b27aaa14583d11d325e43d9165ddda72ca865 Gerrit-Change-Number: 20080 Gerrit-PatchSet: 1 Gerrit-Owner: Matt DeVillier <matt.devillier(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 07 Jun 2017 04:56:53 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: soc/baytrail: assign unique DMA request lines to I2C control...
by Matt DeVillier (Code Review) 07 Jun '17

07 Jun '17

Matt DeVillier has uploaded this change for review. ( https://review.coreboot.org/20080 Change subject: soc/baytrail: assign unique DMA request lines to I2C controllers ...................................................................... soc/baytrail: assign unique DMA request lines to I2C controllers Each I2C controller should have a unique pair of DMA request lines, and DMA channels should be assigned incrementally, rolling over as necessary. Source: Intel Baytrail/ValleyView UEFI reference code Change-Id: Icc9b27aaa14583d11d325e43d9165ddda72ca865 Signed-off-by: Matt DeVillier <matt.devillier(a)gmail.com> --- M src/soc/intel/baytrail/acpi/lpss.asl 1 file changed, 12 insertions(+), 12 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/80/20080/1 diff --git a/src/soc/intel/baytrail/acpi/lpss.asl b/src/soc/intel/baytrail/acpi/lpss.asl index 6cac06a..6473a78 100644 --- a/src/soc/intel/baytrail/acpi/lpss.asl +++ b/src/soc/intel/baytrail/acpi/lpss.asl @@ -156,8 +156,8 @@ { LPSS_I2C2_IRQ } - FixedDMA (0x10, 0x0, Width32Bit, ) - FixedDMA (0x11, 0x1, Width32Bit, ) + FixedDMA (0x12, 0x2, Width32Bit, ) + FixedDMA (0x13, 0x3, Width32Bit, ) }) Method (_CRS) @@ -215,8 +215,8 @@ { LPSS_I2C3_IRQ } - FixedDMA (0x10, 0x0, Width32Bit, ) - FixedDMA (0x11, 0x1, Width32Bit, ) + FixedDMA (0x14, 0x4, Width32Bit, ) + FixedDMA (0x15, 0x5, Width32Bit, ) }) Method (_CRS) @@ -274,8 +274,8 @@ { LPSS_I2C4_IRQ } - FixedDMA (0x10, 0x0, Width32Bit, ) - FixedDMA (0x11, 0x1, Width32Bit, ) + FixedDMA (0x16, 0x6, Width32Bit, ) + FixedDMA (0x17, 0x7, Width32Bit, ) }) Method (_CRS) @@ -333,8 +333,8 @@ { LPSS_I2C5_IRQ } - FixedDMA (0x10, 0x0, Width32Bit, ) - FixedDMA (0x11, 0x1, Width32Bit, ) + FixedDMA (0x18, 0x0, Width32Bit, ) + FixedDMA (0x19, 0x1, Width32Bit, ) }) Method (_CRS) @@ -392,8 +392,8 @@ { LPSS_I2C6_IRQ } - FixedDMA (0x10, 0x0, Width32Bit, ) - FixedDMA (0x11, 0x1, Width32Bit, ) + FixedDMA (0x1A, 0x2, Width32Bit, ) + FixedDMA (0x1B, 0x3, Width32Bit, ) }) Method (_CRS) @@ -451,8 +451,8 @@ { LPSS_I2C7_IRQ } - FixedDMA (0x10, 0x0, Width32Bit, ) - FixedDMA (0x11, 0x1, Width32Bit, ) + FixedDMA (0x1C, 0x4, Width32Bit, ) + FixedDMA (0x1D, 0x5, Width32Bit, ) }) Method (_CRS) -- To view, visit https://review.coreboot.org/20080 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Icc9b27aaa14583d11d325e43d9165ddda72ca865 Gerrit-Change-Number: 20080 Gerrit-PatchSet: 1 Gerrit-Owner: Matt DeVillier <matt.devillier(a)gmail.com>

1 0

Change in coreboot[master]: device/pnp: remove struct io_info
by build bot (Jenkins) (Code Review) 07 Jun '17

07 Jun '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20078 ) Change subject: device/pnp: remove struct io_info ...................................................................... Patch Set 2: Verified-1 Build Unstable https://qa.coreboot.org/job/coreboot-gerrit/55041/ : UNSTABLE https://qa.coreboot.org/job/coreboot-checkpatch/10725/ : SUCCESS -- To view, visit https://review.coreboot.org/20078 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1e7853844605cd2a6d568caf05488e1218fb53f9 Gerrit-Change-Number: 20078 Gerrit-PatchSet: 2 Gerrit-Owner: Samuel Holland <samuel(a)sholland.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 07 Jun 2017 04:49:44 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: soc/braswell: assign unique DMA request lines to I2C control...
by build bot (Jenkins) (Code Review) 07 Jun '17

07 Jun '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/20079 ) Change subject: soc/braswell: assign unique DMA request lines to I2C controllers ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/coreboot-checkpatch/10724/ : SUCCESS https://qa.coreboot.org/job/coreboot-gerrit/55040/ : SUCCESS -- To view, visit https://review.coreboot.org/20079 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1d97b5a07bf732c27caf57904c138b120b93ca81 Gerrit-Change-Number: 20079 Gerrit-PatchSet: 1 Gerrit-Owner: Matt DeVillier <matt.devillier(a)gmail.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Wed, 07 Jun 2017 04:46:16 +0000 Gerrit-HasComments: No

1 0

Change in coreboot[master]: device/pnp: remove struct io_info
by Samuel Holland (Code Review) 07 Jun '17

07 Jun '17

Hello build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/20078 to look at the new patch set (#2). Change subject: device/pnp: remove struct io_info ...................................................................... device/pnp: remove struct io_info The 'set' field was not used anywhere. Replace the struct with a simple integer. superio.c updates performed with: sed -i -r 's/\{ ?0(x([[:digit:]abcdefABCDEF]{3,4}))?, (0x)?[04] ?\}/0\1/g' src/ec/*/*/ec.c sed -i -r 's/\{ ?0(x([[:digit:]abcdefABCDEF]{3,4}))?, (0x)?[04] ?\}/0\1/g' src/ec/*/*/ec_lpc.c sed -i -r 's/\{ ?0(x([[:digit:]abcdefABCDEF]{3,4}))?, (0x)?[04] ?\}/0\1/g' src/superio/*/*/superio.c sed -i -r 's/\{ ?0(x([[:digit:]abcdefABCDEF]{3,4}))?, (0x)?[04] ?\}/0\1/g' src/superio/via/vt1211/vt1211.c sed -i 's/{},/0,/g' src/superio/*/*/superio.c Change-Id: I1e7853844605cd2a6d568caf05488e1218fb53f9 Signed-off-by: Samuel Holland <samuel(a)sholland.org> --- M src/device/pnp_device.c M src/ec/compal/ene932/ec.c M src/ec/google/chromeec/ec_lpc.c M src/ec/quanta/ene_kb3940q/ec.c M src/ec/quanta/it8518/ec.c M src/ec/roda/it8518/ec.c M src/include/device/pnp.h M src/superio/fintek/f71805f/superio.c M src/superio/fintek/f71808a/superio.c M src/superio/fintek/f71859/superio.c M src/superio/fintek/f71863fg/superio.c M src/superio/fintek/f71869ad/superio.c M src/superio/fintek/f71872/superio.c M src/superio/fintek/f81216h/superio.c M src/superio/fintek/f81865f/superio.c M src/superio/fintek/f81866d/superio.c M src/superio/intel/i3100/superio.c M src/superio/intel/i8900/superio.c M src/superio/ite/it8671f/superio.c M src/superio/ite/it8712f/superio.c M src/superio/ite/it8716f/superio.c M src/superio/ite/it8718f/superio.c M src/superio/ite/it8721f/superio.c M src/superio/ite/it8728f/superio.c M src/superio/ite/it8772f/superio.c M src/superio/ite/it8783ef/superio.c M src/superio/nsc/pc87309/superio.c M src/superio/nsc/pc87360/superio.c M src/superio/nsc/pc87366/superio.c M src/superio/nsc/pc87382/superio.c M src/superio/nsc/pc87384/superio.c M src/superio/nsc/pc87392/superio.c M src/superio/nsc/pc87417/superio.c M src/superio/nsc/pc97317/superio.c M src/superio/nuvoton/nct5104d/superio.c M src/superio/nuvoton/nct5572d/superio.c M src/superio/nuvoton/nct6776/superio.c M src/superio/nuvoton/nct6779d/superio.c M src/superio/nuvoton/nct6791d/superio.c M src/superio/nuvoton/wpcm450/superio.c M src/superio/renesas/m3885x/superio.c M src/superio/smsc/dme1737/superio.c M src/superio/smsc/kbc1100/superio.c M src/superio/smsc/lpc47b272/superio.c M src/superio/smsc/lpc47b397/superio.c M src/superio/smsc/lpc47m10x/superio.c M src/superio/smsc/lpc47m15x/superio.c M src/superio/smsc/lpc47n217/superio.c M src/superio/smsc/lpc47n227/superio.c M src/superio/smsc/mec1308/superio.c M src/superio/smsc/sch4037/superio.c M src/superio/smsc/sio1036/superio.c M src/superio/smsc/smscsuperio/superio.c M src/superio/via/vt1211/vt1211.c M src/superio/winbond/w83627dhg/superio.c M src/superio/winbond/w83627ehg/superio.c M src/superio/winbond/w83627hf/superio.c M src/superio/winbond/w83627thg/superio.c M src/superio/winbond/w83627uhg/superio.c M src/superio/winbond/w83667hg-a/superio.c M src/superio/winbond/w83697hf/superio.c M src/superio/winbond/w83977tf/superio.c M src/superio/winbond/wpcd376i/superio.c 63 files changed, 357 insertions(+), 361 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/78/20078/2 -- To view, visit https://review.coreboot.org/20078 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I1e7853844605cd2a6d568caf05488e1218fb53f9 Gerrit-Change-Number: 20078 Gerrit-PatchSet: 2 Gerrit-Owner: Samuel Holland <samuel(a)sholland.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in coreboot[master]: soc/braswell: assign unique DMA request lines to I2C control...
by Matt DeVillier (Code Review) 07 Jun '17

07 Jun '17

Matt DeVillier has uploaded this change for review. ( https://review.coreboot.org/20079 Change subject: soc/braswell: assign unique DMA request lines to I2C controllers ...................................................................... soc/braswell: assign unique DMA request lines to I2C controllers Each I2C controller should have a unique pair of DMA request lines, and DMA channels should be assigned incrementally, rolling over as necessary. Source: Intel Braswell UEFI reference code Change-Id: I1d97b5a07bf732c27caf57904c138b120b93ca81 Signed-off-by: Matt DeVillier <matt.devillier(a)gmail.com> --- M src/soc/intel/braswell/acpi/lpss.asl 1 file changed, 12 insertions(+), 12 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/79/20079/1 diff --git a/src/soc/intel/braswell/acpi/lpss.asl b/src/soc/intel/braswell/acpi/lpss.asl index 34d1e4d..4cc93cc 100644 --- a/src/soc/intel/braswell/acpi/lpss.asl +++ b/src/soc/intel/braswell/acpi/lpss.asl @@ -187,8 +187,8 @@ { LPSS_I2C2_IRQ } - FixedDMA (0x10, 0x0, Width32Bit, ) - FixedDMA (0x11, 0x1, Width32Bit, ) + FixedDMA (0x12, 0x2, Width32Bit, ) + FixedDMA (0x13, 0x3, Width32Bit, ) }) Method (_CRS) @@ -246,8 +246,8 @@ { LPSS_I2C3_IRQ } - FixedDMA (0x10, 0x0, Width32Bit, ) - FixedDMA (0x11, 0x1, Width32Bit, ) + FixedDMA (0x14, 0x4, Width32Bit, ) + FixedDMA (0x15, 0x5, Width32Bit, ) }) Method (_CRS) @@ -305,8 +305,8 @@ { LPSS_I2C4_IRQ } - FixedDMA (0x10, 0x0, Width32Bit, ) - FixedDMA (0x11, 0x1, Width32Bit, ) + FixedDMA (0x16, 0x6, Width32Bit, ) + FixedDMA (0x17, 0x7, Width32Bit, ) }) Method (_CRS) @@ -364,8 +364,8 @@ { LPSS_I2C5_IRQ } - FixedDMA (0x10, 0x0, Width32Bit, ) - FixedDMA (0x11, 0x1, Width32Bit, ) + FixedDMA (0x18, 0x0, Width32Bit, ) + FixedDMA (0x19, 0x1, Width32Bit, ) }) Method (_CRS) @@ -423,8 +423,8 @@ { LPSS_I2C6_IRQ } - FixedDMA (0x10, 0x0, Width32Bit, ) - FixedDMA (0x11, 0x1, Width32Bit, ) + FixedDMA (0x1A, 0x2, Width32Bit, ) + FixedDMA (0x1B, 0x3, Width32Bit, ) }) Method (_CRS) @@ -482,8 +482,8 @@ { LPSS_I2C7_IRQ } - FixedDMA (0x10, 0x0, Width32Bit, ) - FixedDMA (0x11, 0x1, Width32Bit, ) + FixedDMA (0x1C, 0x4, Width32Bit, ) + FixedDMA (0x1D, 0x5, Width32Bit, ) }) Method (_CRS) -- To view, visit https://review.coreboot.org/20079 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I1d97b5a07bf732c27caf57904c138b120b93ca81 Gerrit-Change-Number: 20079 Gerrit-PatchSet: 1 Gerrit-Owner: Matt DeVillier <matt.devillier(a)gmail.com>

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit June 2017