On Thu, 4 Feb 2021, Jiaxun Yang wrote:
在 2021/2/2 上午4:04, BALATON Zoltan 写道:
On Sun, 10 Jan 2021, BALATON Zoltan wrote:
On Sun, 10 Jan 2021, Philippe Mathieu-Daudé wrote:
+PCI experts
On 1/10/21 1:43 AM, BALATON Zoltan wrote:
On Sun, 10 Jan 2021, Philippe Mathieu-Daudé wrote:
Hi Zoltan,
On 1/9/21 9:16 PM, BALATON Zoltan wrote:
Currently the ISA devices that are part of the VIA south bridge,
superio chip are wired up by board code. Move creation of these ISA
devices to the VIA ISA bridge model so that board code does not need
to access ISA bus. This also allows vt82c686b-superio to be made
internal to vt82c686 which allows implementing its configuration via
registers in subseqent commits.
Is this patch dependent of the VT82C686B_PM changes
or can it be applied before them?
I don't know but why would that be better? I thought it's clearer to
clean up pm related parts first before moving more stuff to this file so
that's why this patch comes after (and also because that's the order I
did it).
Not any better, but easier for me to get your patches integrated,
as I'm reviewing your patches slowly. Finding other reviewers
would certainly help.
No problem, I'll wait for your review. Merging parts of the series does
not help much because the whole series is needed for vt8231 which is
prerequisite for pegasos2 so eventually all of these are needed so it does
not matter if this one patch gets in earlier or later.
Not sure who could help with review. Maybe Jiaxun or Huacai as this is
used by fuloong2e so they might be interested and could have info on this
chip. Most of these patches just cleaning up the vt82c686b and adding some
missing features so these can be reused by the vt8231 model in last 3
patches (which is very similar to 686b only some reg addresses and ids
seem to be different for what we are concerned).
Ping? There are still a few patches needing review:
http://patchwork.ozlabs.org/project/qemu-devel/list/?series=223512
Jiaxun, Hiacai, or anybody else could you please help reviewing or testing
if this works with fuloong2e?
Tested the series against Fuloong2E PMON. Fuloong's kernel doesn't have much
to do with
VIA ISA.
Which patch is pending for test or review?
Thank you. In the above patchwork link there are numbers next to patches
showing the number of tags, those which don't have any need review, mostly
the last few. Unfortunately patchwork seems to be down at the moment from
here. Pathes that don't have review yet are: vt82c686: Fix up power
management io base and config and those starting with vt82c686: Fix
superio_cfg_{read,write}() functions and after that. The patchew link for
the series is here:
https://patchew.org/QEMU/cover.1610223396.git.bala...@eik.bme.hu/
Regards,
BALATON Zoltan
