On 1/3/21 3:27 PM, BALATON Zoltan via wrote: > On Sun, 3 Jan 2021, Philippe Mathieu-Daudé wrote: >> On 1/2/21 11:43 AM, BALATON Zoltan via wrote: >>> Hello, >>> >>> This is an all in one series containing all the patches from my >>> previous part I and part II vt82c686b clean up series plus the end of >>> it to finally add vt8231 emulation that will be used by subsequent >>> ppc/pegasos2 emulation. I consider this finished for now and good >>> enough to get in also cleaning up and improving fuloong2e emulation a >>> bit but previous disclaimer is still valid: It does not aim to fix all >>> existing bugs or make the model perfectly emulate the real chip just >>> reach the level where we can have working emulation to boot guests >>> which can then be improved later. (E.g. I think state saving was >>> broken before and it remains broken after because I don't know all the >>> details how to add vmstate for all kinds of data structures and this >>> could be addressed separately when fixing the already broken state >>> saving if someone wants to test and fix it.) With this it boots at >>> least MorphOS on pegasos2 and works with the pmon_2e.bin for fuloong2e >>> which needs more fixes for Linux that are currently under review. More >>> testing is welcome. >>> >>> It still needs the Bonito BONITO_PCICONF_REG_MASK fix for fuloong2e >>> because it no longer maps SMBus but due to the Bonito bug guest cannot >>> write register 0xd2 to map it. With that fix pmon_2e.bin from here: >>> http://www.anheng.com.cn/loongson/pmon/ works for me with this >>> command: qemu-system-mips64el -M fuloong2e -net none -bios pmon_2e.bin >>> >>> After rolling this for two years now I hope it can finally be merged >>> and eventually also get pegasos2 emulation that will need this. >>> >>> Regards, >>> BALATON Zoltan >>> >>> BALATON Zoltan (24): >>> vt82c686: Rename AC97/MC97 parts from VT82C686B to VIA >>> vt82c686: Remove unnecessary _DEVICE suffix from type macros >>> vt82c686b: Rename VT82C686B to VT82C686B_ISA >>> vt82c686: Remove vt82c686b_[am]c97_init() functions >>> vt82c686: Split off via-[am]c97 into separate file in hw/audio >>> audio/via-ac97: Simplify code and set user_creatable to false >>> vt82c686: Remove legacy vt82c686b_isa_init() function >>> vt82c686: Remove legacy vt82c686b_pm_init() function >>> vt82c686: Convert debug printf to trace points >>> vt82c686: Remove unneeded includes and defines >>> vt82c686: Use shorter name for local variable holding object state >>> vt82c686: Rename superio config related parts >> >> As the first half of this series is reviewed, I'm queuing it >> (patches 1-12) via mips-next. > > Thanks, You may want to fix the single vt82c686b: in the above commit > title that was a typo, just for consistency.
Fixed (along with your git author email, instead of the list). > Will this pull include the > Bonito BONITO_PCICONF_REG_MASK fix or some replacement for that? That > would be needed to get the rest of this series starting with 15/24 > working. Up to that it's just clean up which should be OK. I checked the Bonito(32) ASIC and Bonito64 (FPGA) manuals, and for (vendor_id = 0xdf53, device_id = 0x00d5) our implementation is correct. I am waiting for further news from Jiaxun who asked someone at Loongson for a manual of their Bonito variant. I expect a different PCI device_id, so we can add it with your change. > I can repost > the remaining patches rebased once it's merged in master or if you tell > me which branch should I use. IMHO no need (at least not worthwhile until mips/next is merged). Regards, Phil.
