hw/ide/pci.c | 47 ++++++++++++++++++++++++++++++++++++++++ hw/ide/piix.c | 50 ++----------------------------------------- hw/ide/via.c | 51 ++------------------------------------------ include/hw/ide/pci.h | 4 ++++ 4 files changed, 55 insertions(+), 97 deletions(-)
This is a preparation before I send v3 of ich6-ide controller emulation patch. I figured that it's more trivial to split the changes this way, by extracting the bmdma functions from via.c and piix.c and sharing them together. Then, I could easily put these into use when I send v3 of the ich6-ide patch by just using the already separated functions. This was suggested by BALATON Zoltan when he submitted a code review on my ich6-ide controller emulation patch. Liav Albani (1): hw/ide: share bmdma read and write functions between piix.c and via.c hw/ide/pci.c | 47 ++++++++++++++++++++++++++++++++++++++++ hw/ide/piix.c | 50 ++----------------------------------------- hw/ide/via.c | 51 ++------------------------------------------ include/hw/ide/pci.h | 4 ++++ 4 files changed, 55 insertions(+), 97 deletions(-) -- 2.35.1
Am 19. Februar 2022 08:08:17 UTC schrieb Liav Albani <liavalb@gmail.com>: >This is a preparation before I send v3 of ich6-ide controller emulation patch. >I figured that it's more trivial to split the changes this way, by extracting >the bmdma functions from via.c and piix.c and sharing them together. Then, >I could easily put these into use when I send v3 of the ich6-ide patch by just >using the already separated functions. This was suggested by BALATON Zoltan when >he submitted a code review on my ich6-ide controller emulation patch. Ping. Any news? >Liav Albani (1): > hw/ide: share bmdma read and write functions between piix.c and via.c > > hw/ide/pci.c | 47 ++++++++++++++++++++++++++++++++++++++++ > hw/ide/piix.c | 50 ++----------------------------------------- > hw/ide/via.c | 51 ++------------------------------------------ > include/hw/ide/pci.h | 4 ++++ > 4 files changed, 55 insertions(+), 97 deletions(-) >
On Tue, Sep 6, 2022 at 10:27 AM Bernhard Beschow <shentey@gmail.com> wrote: > > Am 19. Februar 2022 08:08:17 UTC schrieb Liav Albani <liavalb@gmail.com>: > >This is a preparation before I send v3 of ich6-ide controller emulation patch. > >I figured that it's more trivial to split the changes this way, by extracting > >the bmdma functions from via.c and piix.c and sharing them together. Then, > >I could easily put these into use when I send v3 of the ich6-ide patch by just > >using the already separated functions. This was suggested by BALATON Zoltan when > >he submitted a code review on my ich6-ide controller emulation patch. > > Ping. Any news? *cough*. Has this been folded into subsequent series, or does this still need attention? > > >Liav Albani (1): > > hw/ide: share bmdma read and write functions between piix.c and via.c > > > > hw/ide/pci.c | 47 ++++++++++++++++++++++++++++++++++++++++ > > hw/ide/piix.c | 50 ++----------------------------------------- > > hw/ide/via.c | 51 ++------------------------------------------ > > include/hw/ide/pci.h | 4 ++++ > > 4 files changed, 55 insertions(+), 97 deletions(-) > > >
Am 9. Januar 2023 19:24:16 UTC schrieb John Snow <jsnow@redhat.com>: >On Tue, Sep 6, 2022 at 10:27 AM Bernhard Beschow <shentey@gmail.com> wrote: >> >> Am 19. Februar 2022 08:08:17 UTC schrieb Liav Albani <liavalb@gmail.com>: >> >This is a preparation before I send v3 of ich6-ide controller emulation patch. >> >I figured that it's more trivial to split the changes this way, by extracting >> >the bmdma functions from via.c and piix.c and sharing them together. Then, >> >I could easily put these into use when I send v3 of the ich6-ide patch by just >> >using the already separated functions. This was suggested by BALATON Zoltan when >> >he submitted a code review on my ich6-ide controller emulation patch. >> >> Ping. Any news? > >*cough*. > >Has this been folded into subsequent series, or does this still need attention? Both piix and via still have their own bmdma implementations. This patch might be worth having. Best regards, Bernhard > >> >> >Liav Albani (1): >> > hw/ide: share bmdma read and write functions between piix.c and via.c >> > >> > hw/ide/pci.c | 47 ++++++++++++++++++++++++++++++++++++++++ >> > hw/ide/piix.c | 50 ++----------------------------------------- >> > hw/ide/via.c | 51 ++------------------------------------------ >> > include/hw/ide/pci.h | 4 ++++ >> > 4 files changed, 55 insertions(+), 97 deletions(-) >> > >> >
On Fri, Jan 13, 2023 at 9:10 AM Liav Albani <liavalb@gmail.com> wrote: > > > On 1/11/23 01:07, Bernhard Beschow wrote: > > Am 9. Januar 2023 19:24:16 UTC schrieb John Snow <jsnow@redhat.com>: > > On Tue, Sep 6, 2022 at 10:27 AM Bernhard Beschow <shentey@gmail.com> wrote: > > Am 19. Februar 2022 08:08:17 UTC schrieb Liav Albani <liavalb@gmail.com>: > > This is a preparation before I send v3 of ich6-ide controller emulation patch. > I figured that it's more trivial to split the changes this way, by extracting > the bmdma functions from via.c and piix.c and sharing them together. Then, > I could easily put these into use when I send v3 of the ich6-ide patch by just > using the already separated functions. This was suggested by BALATON Zoltan when > he submitted a code review on my ich6-ide controller emulation patch. > > Ping. Any news? > > *cough*. > > Has this been folded into subsequent series, or does this still need attention? > > Both piix and via still have their own bmdma implementations. This patch might be worth having. > > Best regards, > Bernhard > > I see. Since you are still interested, I will try to see what was the outcome of that patch as I really don't remember if it passed the CI tests, etc. If applicable, I will send this as v2, or if it's already approved, then I guess we could just let it be merged to the tree? > I was just going to run some smoke tests on it and as long as it didn't hurt anything, I'd wave it in. If you want it alongside other patches that I also should stage, you can bundle them if you'd like. Just let me know what you plan on doing. --js
© 2016 - 2024 Red Hat, Inc.