SerialICE July 2013

serialice@serialice.com

2 participants
23 discussions

Patch set updated for serialice: 30b671a VX900: Remove filters for PCI device 0:00.0
by Alexandru Gagniuc 23 Jul '13

23 Jul '13

Alexandru Gagniuc (mr.nuke.me(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3800 -gerrit commit 30b671aebf140e64f0610fc3115543f23b85f54c Author: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> Date: Mon Jul 22 16:06:57 2013 -0500 VX900: Remove filters for PCI device 0:00.0 This BDF does not have any writable MMIO or IO BARs, so don't bother filtering it. Change-Id: I2f8f82bb51f5ffa175e7dd37b0fc20ad9c26f57d Signed-off-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> --- SerialICE/simba/chipset/via_bars.lua | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/SerialICE/simba/chipset/via_bars.lua b/SerialICE/simba/chipset/via_bars.lua index 838aba2..458005e 100644 --- a/SerialICE/simba/chipset/via_bars.lua +++ b/SerialICE/simba/chipset/via_bars.lua @@ -15,21 +15,8 @@ dev_sb = { mmio = { f = nil }, } -function nb_pcie_bar(dev, reg, base) - local size = 64*1024 - - pcie_mm_cfg_bar(base, size) -end - -dev_nb = { - pci_dev = pci_bdf(0,0,0,0), - name = "nb", - bar = {}, -} - function northbridge_vx900() pci_cfg32_hook(dev_sb, 0xbc, "SB_MMIO", sb_mmio_bar) - pci_cfg32_hook(dev_nb, 0x0, "NB_PCI", nb_pcie_bar) end dev_nb_traf_ctl = {

1 0

New patch to review for serialice: c754960 Move VX900 filters from epia_m_850.lua to via_bars.lua
by Alexandru Gagniuc 22 Jul '13

22 Jul '13

Alexandru Gagniuc (mr.nuke.me(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3803 -gerrit commit c754960f6e32ff195d98e5a2e6f03841f93d1634 Author: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> Date: Mon Jul 22 16:32:09 2013 -0500 Move VX900 filters from epia_m_850.lua to via_bars.lua As the comment said, "MOVE THIS TO CHIPSET FILE". Now we can remove that comment and be happy that VX900 filtering is done properly. Change-Id: I47294211be073aad205c6727b2649ec0b38ba2c4 Signed-off-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> --- SerialICE/simba/chipset/via_bars.lua | 44 ++++++++++++++++++++++++++++ SerialICE/simba/mainboard/via_epia_m_850.lua | 39 +----------------------- 2 files changed, 45 insertions(+), 38 deletions(-) diff --git a/SerialICE/simba/chipset/via_bars.lua b/SerialICE/simba/chipset/via_bars.lua index 8bbff35..5610230 100644 --- a/SerialICE/simba/chipset/via_bars.lua +++ b/SerialICE/simba/chipset/via_bars.lua @@ -1,3 +1,32 @@ +-- SerialICE +-- +-- Copyright (c) 2012 Kyösti Mälkki <kyosti.malkki(a)gmail.com> +-- Copyright (c) 2013 Alexandru Gagniuc <mr.nuke.me(a)gmail.com> +-- +-- Permission is hereby granted, free of charge, to any person obtaining a copy +-- of this software and associated documentation files (the "Software"), to deal +-- in the Software without restriction, including without limitation the rights +-- to use, copy, modify, merge, publish, distribute, sublicense, and/or sell +-- copies of the Software, and to permit persons to whom the Software is +-- furnished to do so, subject to the following conditions: +-- +-- The above copyright notice and this permission notice shall be included in +-- all copies or substantial portions of the Software. +-- +-- THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR +-- IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, +-- FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL +-- THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER +-- LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, +-- OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN +-- THE SOFTWARE. +-- + + +-- ********************************************************** +-- + +load_filter("intel_smbus") dev_nb_traf_ctl = { pci_dev = pci_bdf(0,0,5,0), @@ -22,6 +51,7 @@ dev_sb = { name = "sb", bar = {}, mmio = { f = nil }, + acpi = { f = nil }, } function sb_mmio_bar(f, action) @@ -33,7 +63,21 @@ function sb_mmio_bar(f, action) generic_mmio_bar(f.dev.mmio) end +function pm_io_bar(f, action) + f.dev.acpi.name = "ACPI" + f.dev.acpi.val = bit32.band(action.data, 0xff80) + f.dev.acpi.size = 0x80 + generic_io_bar(f.dev.acpi) +end + +function smbus_bar_hook(f, action) + local base = bit32.band(action.data, 0xfff0) + intel_smbus_setup(base, 0x10) +end + function enable_hooks_vx900() pci_cfg8_hook(dev_nb_traf_ctl, 0x61, "PCI", vx900_pcie_bar) pci_cfg32_hook(dev_sb, 0xbc, "SB_MMIO", sb_mmio_bar) + pci_cfg16_hook(dev_sb, 0x88, "PM", pm_io_bar) + pci_cfg16_hook(dev_sb, 0xd0, "SMBus", smbus_bar_hook) end diff --git a/SerialICE/simba/mainboard/via_epia_m_850.lua b/SerialICE/simba/mainboard/via_epia_m_850.lua index fb908b2..4f5dc14 100644 --- a/SerialICE/simba/mainboard/via_epia_m_850.lua +++ b/SerialICE/simba/mainboard/via_epia_m_850.lua @@ -1,6 +1,7 @@ -- SerialICE -- -- Copyright (c) 2012 Kyösti Mälkki <kyosti.malkki(a)gmail.com> +-- Copyright (c) 2013 Alexandru Gagniuc <mr.nuke.me(a)gmail.com> -- -- Permission is hereby granted, free of charge, to any person obtaining a copy -- of this software and associated documentation files (the "Software"), to deal @@ -104,43 +105,8 @@ filter_mainboard = { size = 0x10000 } - - --- MOVE THIS TO CHIPSET FILE - -load_filter("intel_smbus") load_filter("via_bars") -function smbus_bar_hook(f, action) - local base = bit32.band(action.data, 0xfff0) - intel_smbus_setup(base, 0x10) -end - -dev_sb_lpc = { - pci_dev = pci_bdf(0x0,0x1f,0x3,0x0), - name = "Smbus", - bar = {}, -} - -dev_power = { - pci_dev = pci_bdf(0x0,0x11,0x0,0x0), - name = "SYS", - bar = {}, - acpi = { f = nil }, - tco = { f = nil }, -} - -function pm_io_bar(f, action) - f.dev.acpi.name = "ACPI" - f.dev.acpi.val = bit32.band(action.data, 0xff80) - f.dev.acpi.size = 0x80 - generic_io_bar(f.dev.acpi) -end - - - --- **************** - function do_mainboard_setup() enable_hook(io_hooks, filter_pci_io_cfg) enable_hook(mem_hooks, filter_lapic) @@ -155,10 +121,7 @@ function do_mainboard_setup() enable_hook_pc80() enable_hook_superio(0x4e, 0x07) - enable_hooks_vx900() - pci_cfg16_hook(dev_power, 0x88, "PM", pm_io_bar) - pci_cfg16_hook(dev_power, 0xd0, "SMBus", smbus_bar_hook) -- Apply mainboard hooks last, so they are the first ones to check enable_hook(io_hooks, filter_mainboard)

1 0

New patch to review for serialice: f1cb328 VX900: Rename northbridge_vx900 to enable_hooks_vx900
by Alexandru Gagniuc 22 Jul '13

22 Jul '13

Alexandru Gagniuc (mr.nuke.me(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3802 -gerrit commit f1cb32804d76a8e16ac0c5d52a60e0c804f1eb04 Author: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> Date: Mon Jul 22 16:16:06 2013 -0500 VX900: Rename northbridge_vx900 to enable_hooks_vx900 The VX900 is not just a northbridge, and the function enables hooks for both the north and south modules of the chipset, so rename it for better readability. Change-Id: If851652830f178668dd3fa9ea85b8d72790b40ed Signed-off-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> --- SerialICE/simba/chipset/via_bars.lua | 2 +- SerialICE/simba/mainboard/via_epia_m_850.lua | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/SerialICE/simba/chipset/via_bars.lua b/SerialICE/simba/chipset/via_bars.lua index bafc53b..8bbff35 100644 --- a/SerialICE/simba/chipset/via_bars.lua +++ b/SerialICE/simba/chipset/via_bars.lua @@ -33,7 +33,7 @@ function sb_mmio_bar(f, action) generic_mmio_bar(f.dev.mmio) end -function northbridge_vx900() +function enable_hooks_vx900() pci_cfg8_hook(dev_nb_traf_ctl, 0x61, "PCI", vx900_pcie_bar) pci_cfg32_hook(dev_sb, 0xbc, "SB_MMIO", sb_mmio_bar) end diff --git a/SerialICE/simba/mainboard/via_epia_m_850.lua b/SerialICE/simba/mainboard/via_epia_m_850.lua index 6737399..fb908b2 100644 --- a/SerialICE/simba/mainboard/via_epia_m_850.lua +++ b/SerialICE/simba/mainboard/via_epia_m_850.lua @@ -156,7 +156,7 @@ function do_mainboard_setup() enable_hook_pc80() enable_hook_superio(0x4e, 0x07) - northbridge_vx900() + enable_hooks_vx900() pci_cfg16_hook(dev_power, 0x88, "PM", pm_io_bar) pci_cfg16_hook(dev_power, 0xd0, "SMBus", smbus_bar_hook)

1 0

New patch to review for serialice: d3c68a4 VX900: Unify northbridge_vx900 and northbridge_vx900_traf filters
by Alexandru Gagniuc 22 Jul '13

22 Jul '13

Alexandru Gagniuc (mr.nuke.me(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3801 -gerrit commit d3c68a43894abb29f41d8d94915afe2c1fa9e5bc Author: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> Date: Mon Jul 22 16:13:25 2013 -0500 VX900: Unify northbridge_vx900 and northbridge_vx900_traf filters The VX900 is a monolithic chipset, so it does not make sense to treat the south and north modules separately. Change-Id: I4f7eb686bdc7d5b08cf41317fd10546c156e48cc Signed-off-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> --- SerialICE/simba/chipset/via_bars.lua | 39 +++++++++++++--------------- SerialICE/simba/mainboard/via_epia_m_850.lua | 1 - 2 files changed, 18 insertions(+), 22 deletions(-) diff --git a/SerialICE/simba/chipset/via_bars.lua b/SerialICE/simba/chipset/via_bars.lua index 458005e..bafc53b 100644 --- a/SerialICE/simba/chipset/via_bars.lua +++ b/SerialICE/simba/chipset/via_bars.lua @@ -1,24 +1,4 @@ -function sb_mmio_bar(f, action) - -- This MMIO space is used for SPI and CEC control - f.dev.mmio.name = "SB_MMIO" - f.dev.mmio.val = bit32.lshift(bit32.band(action.data, 0xfff0), 8) - f.dev.mmio.size = 0x10000 - - generic_mmio_bar(f.dev.mmio) -end - -dev_sb = { - pci_dev = pci_bdf(0,0x11,0,0), - name = "sb", - bar = {}, - mmio = { f = nil }, -} - -function northbridge_vx900() - pci_cfg32_hook(dev_sb, 0xbc, "SB_MMIO", sb_mmio_bar) -end - dev_nb_traf_ctl = { pci_dev = pci_bdf(0,0,5,0), name = "trf", @@ -37,6 +17,23 @@ function vx900_pcie_bar(f, action) end end -function northbridge_vx900_traf() +dev_sb = { + pci_dev = pci_bdf(0,0x11,0,0), + name = "sb", + bar = {}, + mmio = { f = nil }, +} + +function sb_mmio_bar(f, action) + -- This MMIO space is used for SPI and CEC control + f.dev.mmio.name = "SB_MMIO" + f.dev.mmio.val = bit32.lshift(bit32.band(action.data, 0xfff0), 8) + f.dev.mmio.size = 0x10000 + + generic_mmio_bar(f.dev.mmio) +end + +function northbridge_vx900() pci_cfg8_hook(dev_nb_traf_ctl, 0x61, "PCI", vx900_pcie_bar) + pci_cfg32_hook(dev_sb, 0xbc, "SB_MMIO", sb_mmio_bar) end diff --git a/SerialICE/simba/mainboard/via_epia_m_850.lua b/SerialICE/simba/mainboard/via_epia_m_850.lua index aa5ebb3..6737399 100644 --- a/SerialICE/simba/mainboard/via_epia_m_850.lua +++ b/SerialICE/simba/mainboard/via_epia_m_850.lua @@ -157,7 +157,6 @@ function do_mainboard_setup() enable_hook_superio(0x4e, 0x07) northbridge_vx900() - northbridge_vx900_traf() pci_cfg16_hook(dev_power, 0x88, "PM", pm_io_bar) pci_cfg16_hook(dev_power, 0xd0, "SMBus", smbus_bar_hook)

1 0

New patch to review for serialice: 449d68f VX900: Remove filters for PCI devide 0:00.0
by Alexandru Gagniuc 22 Jul '13

22 Jul '13

Alexandru Gagniuc (mr.nuke.me(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3800 -gerrit commit 449d68f4eecc32b47c433d6f5f2ce3fec39e596a Author: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> Date: Mon Jul 22 16:06:57 2013 -0500 VX900: Remove filters for PCI devide 0:00.0 This BDF does not have any writeable MMIO or IO BARs, so don't bother filtering it. Change-Id: I2f8f82bb51f5ffa175e7dd37b0fc20ad9c26f57d Signed-off-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> --- SerialICE/simba/chipset/via_bars.lua | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/SerialICE/simba/chipset/via_bars.lua b/SerialICE/simba/chipset/via_bars.lua index 838aba2..458005e 100644 --- a/SerialICE/simba/chipset/via_bars.lua +++ b/SerialICE/simba/chipset/via_bars.lua @@ -15,21 +15,8 @@ dev_sb = { mmio = { f = nil }, } -function nb_pcie_bar(dev, reg, base) - local size = 64*1024 - - pcie_mm_cfg_bar(base, size) -end - -dev_nb = { - pci_dev = pci_bdf(0,0,0,0), - name = "nb", - bar = {}, -} - function northbridge_vx900() pci_cfg32_hook(dev_sb, 0xbc, "SB_MMIO", sb_mmio_bar) - pci_cfg32_hook(dev_nb, 0x0, "NB_PCI", nb_pcie_bar) end dev_nb_traf_ctl = {

1 0

New patch to review for serialice: e9102c9 VX900: Fix decoding of southbridge MMIO BAR
by Alexandru Gagniuc 22 Jul '13

22 Jul '13

Alexandru Gagniuc (mr.nuke.me(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3799 -gerrit commit e9102c9a51691b3727d88a8b47b0566523718cb7 Author: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> Date: Mon Jul 22 16:00:34 2013 -0500 VX900: Fix decoding of southbridge MMIO BAR Change-Id: I94ef658fb90e0c36a038e4f87a0b25e444af7e40 Signed-off-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> --- SerialICE/simba/chipset/via_bars.lua | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/SerialICE/simba/chipset/via_bars.lua b/SerialICE/simba/chipset/via_bars.lua index 1dca148..838aba2 100644 --- a/SerialICE/simba/chipset/via_bars.lua +++ b/SerialICE/simba/chipset/via_bars.lua @@ -1,15 +1,18 @@ -function sb_pcie_bar(dev, reg, base) - local baseaddr = bit32.lshift(base, 16) - local size = 64*1024 +function sb_mmio_bar(f, action) + -- This MMIO space is used for SPI and CEC control + f.dev.mmio.name = "SB_MMIO" + f.dev.mmio.val = bit32.lshift(bit32.band(action.data, 0xfff0), 8) + f.dev.mmio.size = 0x10000 - pcie_mm_cfg_bar(baseaddr, size) + generic_mmio_bar(f.dev.mmio) end dev_sb = { pci_dev = pci_bdf(0,0x11,0,0), name = "sb", bar = {}, + mmio = { f = nil }, } function nb_pcie_bar(dev, reg, base) @@ -25,7 +28,7 @@ dev_nb = { } function northbridge_vx900() - pci_cfg16_hook(dev_sb, 0xbd, "SB_PCI", sb_pcie_bar) + pci_cfg32_hook(dev_sb, 0xbc, "SB_MMIO", sb_mmio_bar) pci_cfg32_hook(dev_nb, 0x0, "NB_PCI", nb_pcie_bar) end

1 0

New patch to review for serialice: 47c3f20 EPIA-M850: Fix decoding of SMBus IO BAR to prevent 'nil' error
by Alexandru Gagniuc 22 Jul '13

22 Jul '13

Alexandru Gagniuc (mr.nuke.me(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3798 -gerrit commit 47c3f201fc4c23c4f5720bdae8cdb1a47fc0a6ed Author: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> Date: Mon Jul 22 14:41:57 2013 -0500 EPIA-M850: Fix decoding of SMBus IO BAR to prevent 'nil' error The filter seems to have been using an obsolete form, thus causing a 'nil' error when SerialICE tried to run it. Fix this error. Change-Id: Ib650e9bd1661574a1f7987803cfc131c12035a5d Signed-off-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> --- SerialICE/simba/mainboard/via_epia_m_850.lua | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/SerialICE/simba/mainboard/via_epia_m_850.lua b/SerialICE/simba/mainboard/via_epia_m_850.lua index 8f10ba3..aa5ebb3 100644 --- a/SerialICE/simba/mainboard/via_epia_m_850.lua +++ b/SerialICE/simba/mainboard/via_epia_m_850.lua @@ -111,8 +111,9 @@ filter_mainboard = { load_filter("intel_smbus") load_filter("via_bars") -function smbus_bar_hook(dev, reg, base) - intel_smbus_setup(base, 0x20) +function smbus_bar_hook(f, action) + local base = bit32.band(action.data, 0xfff0) + intel_smbus_setup(base, 0x10) end dev_sb_lpc = {

1 0

Patch set updated for serialice: 9baee69 EPIA-M850: Fix decoding of ACPI IO BAR to prevent 'nil' error
by Alexandru Gagniuc 22 Jul '13

22 Jul '13

Alexandru Gagniuc (mr.nuke.me(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3797 -gerrit commit 9baee690a83a5e6220cea9fb855bb632bcf3c9a6 Author: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> Date: Mon Jul 22 14:06:12 2013 -0500 EPIA-M850: Fix decoding of ACPI IO BAR to prevent 'nil' error When SerialICE got to the ACPI IO filter, it complained about a 'nil' value and died. Make this filter work again. Change-Id: Id8ff78e2f94d3f4d55d00824a8eb2e5ccecd23f4 Signed-off-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> --- SerialICE/simba/mainboard/via_epia_m_850.lua | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/SerialICE/simba/mainboard/via_epia_m_850.lua b/SerialICE/simba/mainboard/via_epia_m_850.lua index f791e54..8f10ba3 100644 --- a/SerialICE/simba/mainboard/via_epia_m_850.lua +++ b/SerialICE/simba/mainboard/via_epia_m_850.lua @@ -129,11 +129,11 @@ dev_power = { tco = { f = nil }, } -function pm_io_bar(dev, reg, base) - dev.acpi.name = "ACPI" - dev.acpi.base = base - dev.acpi.size = 0x60 - generic_io_bar(dev.acpi) +function pm_io_bar(f, action) + f.dev.acpi.name = "ACPI" + f.dev.acpi.val = bit32.band(action.data, 0xff80) + f.dev.acpi.size = 0x80 + generic_io_bar(f.dev.acpi) end

1 0

New patch to review for serialice: ab3052f NOTFORMERGE: EPIA-M850: Fix decoding of ACPI IO BAR
by Alexandru Gagniuc 22 Jul '13

22 Jul '13

Alexandru Gagniuc (mr.nuke.me(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3797 -gerrit commit ab3052fa00e89b359b0c80fee49a0ba71f70b297 Author: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> Date: Mon Jul 22 14:06:12 2013 -0500 NOTFORMERGE: EPIA-M850: Fix decoding of ACPI IO BAR When SerialICE got to the ACPI IO filter, it complained about a 'nil' value and died. Make this filter work again. Change-Id: Id8ff78e2f94d3f4d55d00824a8eb2e5ccecd23f4 Signed-off-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> --- SerialICE/simba/mainboard/via_epia_m_850.lua | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/SerialICE/simba/mainboard/via_epia_m_850.lua b/SerialICE/simba/mainboard/via_epia_m_850.lua index f791e54..7d1743c 100644 --- a/SerialICE/simba/mainboard/via_epia_m_850.lua +++ b/SerialICE/simba/mainboard/via_epia_m_850.lua @@ -129,11 +129,11 @@ dev_power = { tco = { f = nil }, } -function pm_io_bar(dev, reg, base) - dev.acpi.name = "ACPI" - dev.acpi.base = base - dev.acpi.size = 0x60 - generic_io_bar(dev.acpi) +function pm_io_bar(f, action) + f.dev.acpi.name = "ACPI" + f.dev.acpi.base = bit32.band(action.data, 0xff80) + f.dev.acpi.size = 0x60 + generic_io_bar(f.dev.acpi) end

1 0

Patch set updated for serialice: 723c37c EPIA-M850: Add hook for PCI MMIO accesses
by Alexandru Gagniuc 22 Jul '13

22 Jul '13

Alexandru Gagniuc (mr.nuke.me(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3795 -gerrit commit 723c37cc31ab0f81c12f93f44183ae2d2ae66f17 Author: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> Date: Mon Jul 22 02:27:53 2013 -0500 EPIA-M850: Add hook for PCI MMIO accesses Add a hook that enables decoding of PCI MMIO base on the VIA VX900. Change-Id: I5a66366fc4dbaf4e2f60f58febad90f7bb7cd2ff Signed-off-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com> --- SerialICE/simba/chipset/via_bars.lua | 22 ++++++++++++++++++++++ SerialICE/simba/mainboard/via_epia_m_850.lua | 1 + 2 files changed, 23 insertions(+) diff --git a/SerialICE/simba/chipset/via_bars.lua b/SerialICE/simba/chipset/via_bars.lua index 7868535..1dca148 100644 --- a/SerialICE/simba/chipset/via_bars.lua +++ b/SerialICE/simba/chipset/via_bars.lua @@ -28,3 +28,25 @@ function northbridge_vx900() pci_cfg16_hook(dev_sb, 0xbd, "SB_PCI", sb_pcie_bar) pci_cfg32_hook(dev_nb, 0x0, "NB_PCI", nb_pcie_bar) end + +dev_nb_traf_ctl = { + pci_dev = pci_bdf(0,0,5,0), + name = "trf", + bar = {}, +} + +function vx900_pcie_bar(f, action) + local baseaddr = bit32.lshift(action.data, 28) + local size = 256*1024*1024 + + -- enable is 0:00.5 [060] .10 + if baseaddr then + pcie_mm_enable(f.dev, f.reg, baseaddr, size) + else + pcie_mm_disable(f.dev, f.reg, baseaddr, size) + end +end + +function northbridge_vx900_traf() + pci_cfg8_hook(dev_nb_traf_ctl, 0x61, "PCI", vx900_pcie_bar) +end diff --git a/SerialICE/simba/mainboard/via_epia_m_850.lua b/SerialICE/simba/mainboard/via_epia_m_850.lua index 44f0fde..f791e54 100644 --- a/SerialICE/simba/mainboard/via_epia_m_850.lua +++ b/SerialICE/simba/mainboard/via_epia_m_850.lua @@ -156,6 +156,7 @@ function do_mainboard_setup() enable_hook_superio(0x4e, 0x07) northbridge_vx900() + northbridge_vx900_traf() pci_cfg16_hook(dev_power, 0x88, "PM", pm_io_bar) pci_cfg16_hook(dev_power, 0xd0, "SMBus", smbus_bar_hook)

1 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

SerialICE July 2013