- flashrom-gerrit - mail.coreboot.org

Change in flashrom[master]: it8212: remove unused rget_io_perms()
by Thomas Heijligen (Code Review) 07 Jan '22

07 Jan '22

Attention is currently required from: Felix Singer, Nico Huber. Hello Felix Singer, build bot (Jenkins), Nico Huber, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/flashrom/+/60869 to look at the new patch set (#2). Change subject: it8212: remove unused rget_io_perms() ...................................................................... it8212: remove unused rget_io_perms() The it8212 programmer does not use x86 IO Ports. Remove the dependency to it. Change-Id: Iab859fb6be6cab094c9c6ce4b3e8eac0e5ff84ab Signed-off-by: Thomas Heijligen <thomas.heijligen(a)secunet.com> --- M it8212.c 1 file changed, 0 insertions(+), 4 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/69/60869/2 -- To view, visit https://review.coreboot.org/c/flashrom/+/60869 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: Iab859fb6be6cab094c9c6ce4b3e8eac0e5ff84ab Gerrit-Change-Number: 60869 Gerrit-PatchSet: 2 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-Reviewer: Felix Singer <felixsinger(a)posteo.net> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Felix Singer <felixsinger(a)posteo.net> Gerrit-Attention: Nico Huber <nico.h(a)gmx.de> Gerrit-MessageType: newpatchset

1 0

Change in flashrom[master]: drkaiser: remove unused rget_io_perms()
by Angel Pons (Code Review) 07 Jan '22

07 Jan '22

Attention is currently required from: Felix Singer, Nico Huber, Thomas Heijligen. Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/60842 ) Change subject: drkaiser: remove unused rget_io_perms() ...................................................................... Patch Set 1: (1 comment) Commit Message: https://review.coreboot.org/c/flashrom/+/60842/comment/428316bb_9738d549 PS1, Line 9: Prots > typo: Ports Just noticed this typo is on the other commits as well -- To view, visit https://review.coreboot.org/c/flashrom/+/60842 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I86bad947298a1166ff1e768f7d0b75a90e574696 Gerrit-Change-Number: 60842 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-Reviewer: Felix Singer <felixsinger(a)posteo.net> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Attention: Felix Singer <felixsinger(a)posteo.net> Gerrit-Attention: Nico Huber <nico.h(a)gmx.de> Gerrit-Attention: Thomas Heijligen <src(a)posteo.de> Gerrit-Comment-Date: Thu, 06 Jan 2022 14:25:28 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Angel Pons <th3fanbus(a)gmail.com> Gerrit-MessageType: comment

1 0

Change in flashrom[master]: drkaiser: remove unused rget_io_perms()
by Angel Pons (Code Review) 07 Jan '22

07 Jan '22

Attention is currently required from: Felix Singer, Nico Huber, Thomas Heijligen. Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/60842 ) Change subject: drkaiser: remove unused rget_io_perms() ...................................................................... Patch Set 1: (1 comment) Commit Message: https://review.coreboot.org/c/flashrom/+/60842/comment/5762f540_8a136710 PS1, Line 9: Prots typo: Ports -- To view, visit https://review.coreboot.org/c/flashrom/+/60842 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I86bad947298a1166ff1e768f7d0b75a90e574696 Gerrit-Change-Number: 60842 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-Reviewer: Felix Singer <felixsinger(a)posteo.net> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Attention: Felix Singer <felixsinger(a)posteo.net> Gerrit-Attention: Nico Huber <nico.h(a)gmx.de> Gerrit-Attention: Thomas Heijligen <src(a)posteo.de> Gerrit-Comment-Date: Thu, 06 Jan 2022 14:24:56 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in flashrom[master]: it8212: remove unused rget_io_perms()
by Thomas Heijligen (Code Review) 07 Jan '22

07 Jan '22

Thomas Heijligen has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/60869 ) Change subject: it8212: remove unused rget_io_perms() ...................................................................... it8212: remove unused rget_io_perms() The it8212 programmer does not use x86 IO Prots. Remove the dependency to it. Change-Id: Iab859fb6be6cab094c9c6ce4b3e8eac0e5ff84ab Signed-off-by: Thomas Heijligen <thomas.heijligen(a)secunet.com> --- M it8212.c 1 file changed, 0 insertions(+), 4 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/69/60869/1 diff --git a/it8212.c b/it8212.c index 6488b15..ca82d9c 100644 --- a/it8212.c +++ b/it8212.c @@ -18,7 +18,6 @@ #include "flash.h" #include "programmer.h" #include "hwaccess.h" -#include "hwaccess_x86_io.h" #include "hwaccess_physmap.h" #include "platform/pci.h" @@ -58,9 +57,6 @@ static int it8212_init(void) { - if (rget_io_perms()) - return 1; - struct pci_dev *dev = pcidev_init(devs_it8212, PCI_ROM_ADDRESS); if (!dev) return 1; -- To view, visit https://review.coreboot.org/c/flashrom/+/60869 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: Iab859fb6be6cab094c9c6ce4b3e8eac0e5ff84ab Gerrit-Change-Number: 60869 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-MessageType: newchange

1 0

Change in flashrom[master]: satasii: remove unused rget_io_perms()
by Thomas Heijligen (Code Review) 07 Jan '22

07 Jan '22

Thomas Heijligen has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/60848 ) Change subject: satasii: remove unused rget_io_perms() ...................................................................... satasii: remove unused rget_io_perms() The satasii programmer does not use x86 IO Prots. Remove the dependency to it. Change-Id: I54d65561ff024d3c181d11c6518a4612c2ab0399 Signed-off-by: Thomas Heijligen <thomas.heijligen(a)secunet.com> --- M Makefile M satasii.c 2 files changed, 0 insertions(+), 5 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/48/60848/1 diff --git a/Makefile b/Makefile index b351990..b8287a9 100644 --- a/Makefile +++ b/Makefile @@ -132,7 +132,6 @@ CONFIG_NICREALTEK \ CONFIG_RAYER_SPI \ CONFIG_SATAMV \ - CONFIG_SATASII \ DEPENDS_ON_LIBPCI := \ CONFIG_ATAHPT \ diff --git a/satasii.c b/satasii.c index f9cb5da..a27fc29 100644 --- a/satasii.c +++ b/satasii.c @@ -18,7 +18,6 @@ #include "programmer.h" #include "hwaccess.h" -#include "hwaccess_x86_io.h" #include "hwaccess_physmap.h" #include "platform/pci.h" @@ -102,9 +101,6 @@ uint32_t addr; uint16_t reg_offset; - if (rget_io_perms()) - return 1; - dev = pcidev_init(satas_sii, PCI_BASE_ADDRESS_0); if (!dev) return 1; -- To view, visit https://review.coreboot.org/c/flashrom/+/60848 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I54d65561ff024d3c181d11c6518a4612c2ab0399 Gerrit-Change-Number: 60848 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-MessageType: newchange

1 0

Change in flashrom[master]: nicintel_spi: remove unused rget_io_perms()
by Thomas Heijligen (Code Review) 07 Jan '22

07 Jan '22

Thomas Heijligen has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/60846 ) Change subject: nicintel_spi: remove unused rget_io_perms() ...................................................................... nicintel_spi: remove unused rget_io_perms() The nicintel_spi programmer does not use x86 IO Prots. Remove the dependency to it. Change-Id: I3e7451eceb1f01de21da934c9559dbf2f06e7e54 Signed-off-by: Thomas Heijligen <thomas.heijligen(a)secunet.com> --- M Makefile M nicintel_spi.c 2 files changed, 0 insertions(+), 5 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/46/60846/1 diff --git a/Makefile b/Makefile index 93fc657..34d517e 100644 --- a/Makefile +++ b/Makefile @@ -128,7 +128,6 @@ CONFIG_ATAPROMISE \ CONFIG_INTERNAL \ CONFIG_NIC3COM \ - CONFIG_NICINTEL_SPI \ CONFIG_NICNATSEMI \ CONFIG_NICREALTEK \ CONFIG_OGP_SPI \ diff --git a/nicintel_spi.c b/nicintel_spi.c index 852c042..004af66 100644 --- a/nicintel_spi.c +++ b/nicintel_spi.c @@ -35,7 +35,6 @@ #include "flash.h" #include "programmer.h" #include "hwaccess.h" -#include "hwaccess_x86_io.h" #include "hwaccess_physmap.h" #include "platform/pci.h" @@ -293,9 +292,6 @@ { struct pci_dev *dev = NULL; - if (rget_io_perms()) - return 1; - dev = pcidev_init(nics_intel_spi, PCI_BASE_ADDRESS_0); if (!dev) return 1; -- To view, visit https://review.coreboot.org/c/flashrom/+/60846 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I3e7451eceb1f01de21da934c9559dbf2f06e7e54 Gerrit-Change-Number: 60846 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-MessageType: newchange

1 0

Change in flashrom[master]: ogp_spi: remove unused rget_io_perms()
by Thomas Heijligen (Code Review) 07 Jan '22

07 Jan '22

Thomas Heijligen has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/60847 ) Change subject: ogp_spi: remove unused rget_io_perms() ...................................................................... ogp_spi: remove unused rget_io_perms() The ogp_spi programmer does not use x86 IO Prots. Remove the dependency to it. Change-Id: I1f3f34e33f77159fa0cdea150e1f408ce9d943f0 Signed-off-by: Thomas Heijligen <thomas.heijligen(a)secunet.com> --- M Makefile M ogp_spi.c 2 files changed, 0 insertions(+), 5 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/47/60847/1 diff --git a/Makefile b/Makefile index 34d517e..b351990 100644 --- a/Makefile +++ b/Makefile @@ -130,7 +130,6 @@ CONFIG_NIC3COM \ CONFIG_NICNATSEMI \ CONFIG_NICREALTEK \ - CONFIG_OGP_SPI \ CONFIG_RAYER_SPI \ CONFIG_SATAMV \ CONFIG_SATASII \ diff --git a/ogp_spi.c b/ogp_spi.c index d4218d1..830cfe7 100644 --- a/ogp_spi.c +++ b/ogp_spi.c @@ -19,7 +19,6 @@ #include "flash.h" #include "programmer.h" #include "hwaccess.h" -#include "hwaccess_x86_io.h" #include "hwaccess_physmap.h" #include "platform/pci.h" @@ -142,9 +141,6 @@ } free(type); - if (rget_io_perms()) - return 1; - dev = pcidev_init(ogp_spi, PCI_BASE_ADDRESS_0); if (!dev) return 1; -- To view, visit https://review.coreboot.org/c/flashrom/+/60847 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I1f3f34e33f77159fa0cdea150e1f408ce9d943f0 Gerrit-Change-Number: 60847 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-MessageType: newchange

1 0

Change in flashrom[master]: nicintel_eeprom: remove unused rget_io_perms()
by Thomas Heijligen (Code Review) 07 Jan '22

07 Jan '22

Thomas Heijligen has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/60845 ) Change subject: nicintel_eeprom: remove unused rget_io_perms() ...................................................................... nicintel_eeprom: remove unused rget_io_perms() The nicintel_eeprom programmer does not use x86 IO Prots. Remove the dependency to it. Change-Id: I5fd42572fd29f5d7fd749c2836eac3e68c947946 Signed-off-by: Thomas Heijligen <thomas.heijligen(a)secunet.com> --- M Makefile M nicintel_eeprom.c 2 files changed, 0 insertions(+), 5 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/45/60845/1 diff --git a/Makefile b/Makefile index da38d6b..93fc657 100644 --- a/Makefile +++ b/Makefile @@ -128,7 +128,6 @@ CONFIG_ATAPROMISE \ CONFIG_INTERNAL \ CONFIG_NIC3COM \ - CONFIG_NICINTEL_EEPROM \ CONFIG_NICINTEL_SPI \ CONFIG_NICNATSEMI \ CONFIG_NICREALTEK \ diff --git a/nicintel_eeprom.c b/nicintel_eeprom.c index 7b92958..ea0e768 100644 --- a/nicintel_eeprom.c +++ b/nicintel_eeprom.c @@ -35,7 +35,6 @@ #include "spi.h" #include "programmer.h" #include "hwaccess.h" -#include "hwaccess_x86_io.h" #include "hwaccess_physmap.h" #include "platform/pci.h" @@ -462,9 +461,6 @@ static int nicintel_ee_init(void) { - if (rget_io_perms()) - return 1; - struct pci_dev *dev = pcidev_init(nics_intel_ee, PCI_BASE_ADDRESS_0); if (!dev) return 1; -- To view, visit https://review.coreboot.org/c/flashrom/+/60845 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I5fd42572fd29f5d7fd749c2836eac3e68c947946 Gerrit-Change-Number: 60845 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-MessageType: newchange

1 0

Change in flashrom[master]: nicintel: remove unused rget_io_perms()
by Thomas Heijligen (Code Review) 07 Jan '22

07 Jan '22

Thomas Heijligen has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/60844 ) Change subject: nicintel: remove unused rget_io_perms() ...................................................................... nicintel: remove unused rget_io_perms() The nicintel programmer does not use x86 IO Prots. Remove the dependency to it. Change-Id: Ibe609ff8f8fdbdf2a7de8e1922325ca4ad56a9e7 Signed-off-by: Thomas Heijligen <thomas.heijligen(a)secunet.com> --- M Makefile M nicintel.c 2 files changed, 0 insertions(+), 8 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/44/60844/1 diff --git a/Makefile b/Makefile index 7859362..da38d6b 100644 --- a/Makefile +++ b/Makefile @@ -128,7 +128,6 @@ CONFIG_ATAPROMISE \ CONFIG_INTERNAL \ CONFIG_NIC3COM \ - CONFIG_NICINTEL \ CONFIG_NICINTEL_EEPROM \ CONFIG_NICINTEL_SPI \ CONFIG_NICNATSEMI \ diff --git a/nicintel.c b/nicintel.c index 21dfad2..c8a8789 100644 --- a/nicintel.c +++ b/nicintel.c @@ -19,7 +19,6 @@ #include "flash.h" #include "programmer.h" #include "hwaccess.h" -#include "hwaccess_x86_io.h" #include "hwaccess_physmap.h" #include "platform/pci.h" @@ -71,12 +70,6 @@ struct pci_dev *dev = NULL; uintptr_t addr; - /* Needed only for PCI accesses on some platforms. - * FIXME: Refactor that into get_mem_perms/rget_io_perms/get_pci_perms? - */ - if (rget_io_perms()) - return 1; - /* FIXME: BAR2 is not available if the device uses the CardBus function. */ dev = pcidev_init(nics_intel, PCI_BASE_ADDRESS_2); if (!dev) -- To view, visit https://review.coreboot.org/c/flashrom/+/60844 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: Ibe609ff8f8fdbdf2a7de8e1922325ca4ad56a9e7 Gerrit-Change-Number: 60844 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-MessageType: newchange

1 0

Change in flashrom[master]: gfxnvidia: remove unused rget_io_perms()
by Thomas Heijligen (Code Review) 07 Jan '22

07 Jan '22

Thomas Heijligen has uploaded this change for review. ( https://review.coreboot.org/c/flashrom/+/60843 ) Change subject: gfxnvidia: remove unused rget_io_perms() ...................................................................... gfxnvidia: remove unused rget_io_perms() The gfxnvidia programmer does not use x86 IO Prots. Remove the dependency to it. Change-Id: I1f0341390ccb698bc435760f4ead7de54e429a6e Signed-off-by: Thomas Heijligen <thomas.heijligen(a)secunet.com> --- M Makefile M gfxnvidia.c 2 files changed, 0 insertions(+), 5 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/43/60843/1 diff --git a/Makefile b/Makefile index 595c9cf..7859362 100644 --- a/Makefile +++ b/Makefile @@ -126,7 +126,6 @@ DEPENDS_ON_X86_PORT_IO := \ CONFIG_ATAHPT \ CONFIG_ATAPROMISE \ - CONFIG_GFXNVIDIA \ CONFIG_INTERNAL \ CONFIG_NIC3COM \ CONFIG_NICINTEL \ diff --git a/gfxnvidia.c b/gfxnvidia.c index 30c94b9..535467e 100644 --- a/gfxnvidia.c +++ b/gfxnvidia.c @@ -19,7 +19,6 @@ #include "flash.h" #include "programmer.h" #include "hwaccess.h" -#include "hwaccess_x86_io.h" #include "hwaccess_physmap.h" #include "platform/pci.h" @@ -89,9 +88,6 @@ struct pci_dev *dev = NULL; uint32_t reg32; - if (rget_io_perms()) - return 1; - dev = pcidev_init(gfx_nvidia, PCI_BASE_ADDRESS_0); if (!dev) return 1; -- To view, visit https://review.coreboot.org/c/flashrom/+/60843 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-Change-Id: I1f0341390ccb698bc435760f4ead7de54e429a6e Gerrit-Change-Number: 60843 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Heijligen <src(a)posteo.de> Gerrit-MessageType: newchange

1 0

2024

2023

2022

2021

2020

2019

2018

2017

flashrom-gerrit