hw/mips: patch queue for 2020-05-26

[PATCH 00/14] hw/mips: patch queue for 2020-05-26

Posted by Philippe Mathieu-Daudé 3 years, 10 months ago

Hi,

This is the current state of my hw/mips/next tree.

- MAINTAINERS updated to welcome Huacai Chen and Jiaxun Yang,
  and update Aleksandar Rikalo's email address,

- Trivial improvements in the Bonito64 North Bridge and the
  Fuloong 2e machine,

- MIPS Machines names unified without 'mips_' prefix.

The following patches need review:

- #6  "hw/pci-host/bonito: Map the different PCI ranges more detailled"
- #10 "hw/mips/fuloong2e: Fix typo in Fuloong machine name"

Aleksandar Markovic (3):
  hw/mips: Rename malta/mipssim/r4k/jazz files
  hw/mips/malta: Add some logging for bad register offset cases
  MAINTAINERS: Change Aleksandar Rikalo's email address

Huacai Chen (1):
  MAINTAINERS: Add Huacai Chen as fuloong2e co-maintainer

Philippe Mathieu-Daudé (10):
  hw/pci-host: Use CONFIG_PCI_BONITO to select the Bonito North Bridge
  hw/pci-host/bonito: Fix DPRINTF() format strings
  hw/pci-host/bonito: Map peripheral using physical address
  hw/pci-host/bonito: Map all the Bonito64 I/O range
  hw/pci-host/bonito: Map the different PCI ranges more detailled
  hw/pci-host/bonito: Better describe the I/O CS regions
  hw/pci-host/bonito: Set the Config register reset value with
    FIELD_DP32
  hw/mips/fuloong2e: Move code and update a comment
  hw/mips/fuloong2e: Fix typo in Fuloong machine name
  hw/mips/mips_int: De-duplicate KVM interrupt delivery

 docs/system/deprecated.rst               |  5 ++
 docs/system/target-mips.rst              |  2 +-
 default-configs/mips64el-softmmu.mak     |  2 +-
 hw/isa/vt82c686.c                        |  2 +-
 hw/mips/{mips_fulong2e.c => fuloong2e.c} | 48 ++++++-------
 hw/mips/{mips_jazz.c => jazz.c}          |  0
 hw/mips/{mips_malta.c => malta.c}        | 14 ++--
 hw/mips/mips_int.c                       | 11 +--
 hw/mips/{mips_mipssim.c => mipssim.c}    |  0
 hw/mips/{mips_r4k.c => r4k.c}            |  0
 hw/pci-host/bonito.c                     | 87 +++++++++++++++++++-----
 tests/qtest/endianness-test.c            |  2 +-
 .mailmap                                 |  3 +-
 MAINTAINERS                              | 26 +++----
 hw/mips/Kconfig                          |  3 +-
 hw/mips/Makefile.objs                    | 10 +--
 hw/pci-host/Kconfig                      |  5 ++
 hw/pci-host/Makefile.objs                |  2 +-
 18 files changed, 142 insertions(+), 80 deletions(-)
 rename hw/mips/{mips_fulong2e.c => fuloong2e.c} (91%)
 rename hw/mips/{mips_jazz.c => jazz.c} (100%)
 rename hw/mips/{mips_malta.c => malta.c} (99%)
 rename hw/mips/{mips_mipssim.c => mipssim.c} (100%)
 rename hw/mips/{mips_r4k.c => r4k.c} (100%)

-- 
2.21.3

Re: [PATCH 00/14] hw/mips: patch queue for 2020-05-26

Posted by Aleksandar Markovic 3 years, 10 months ago

уто, 26. мај 2020. у 12:47 Philippe Mathieu-Daudé <f4bug@amsat.org> је
написао/ла:
>
> Hi,
>
> This is the current state of my hw/mips/next tree.
>
> - MAINTAINERS updated to welcome Huacai Chen and Jiaxun Yang,
>   and update Aleksandar Rikalo's email address,
>
> - Trivial improvements in the Bonito64 North Bridge and the
>   Fuloong 2e machine,
>
> - MIPS Machines names unified without 'mips_' prefix.
>
> The following patches need review:
>
> - #6  "hw/pci-host/bonito: Map the different PCI ranges more detailled"
> - #10 "hw/mips/fuloong2e: Fix typo in Fuloong machine name"
>

I agreed with these two patches, thanks for your help, patch 6 looks
much better now.

Hope you will be able to send pull request sooner rather than later.
Please work with Gerd to speed up the process, resolving the
dependencies efficiently.

Yours,
Aleksandar




> Aleksandar Markovic (3):
>   hw/mips: Rename malta/mipssim/r4k/jazz files
>   hw/mips/malta: Add some logging for bad register offset cases
>   MAINTAINERS: Change Aleksandar Rikalo's email address
>
> Huacai Chen (1):
>   MAINTAINERS: Add Huacai Chen as fuloong2e co-maintainer
>
> Philippe Mathieu-Daudé (10):
>   hw/pci-host: Use CONFIG_PCI_BONITO to select the Bonito North Bridge
>   hw/pci-host/bonito: Fix DPRINTF() format strings
>   hw/pci-host/bonito: Map peripheral using physical address
>   hw/pci-host/bonito: Map all the Bonito64 I/O range
>   hw/pci-host/bonito: Map the different PCI ranges more detailled
>   hw/pci-host/bonito: Better describe the I/O CS regions
>   hw/pci-host/bonito: Set the Config register reset value with
>     FIELD_DP32
>   hw/mips/fuloong2e: Move code and update a comment
>   hw/mips/fuloong2e: Fix typo in Fuloong machine name
>   hw/mips/mips_int: De-duplicate KVM interrupt delivery
>
>  docs/system/deprecated.rst               |  5 ++
>  docs/system/target-mips.rst              |  2 +-
>  default-configs/mips64el-softmmu.mak     |  2 +-
>  hw/isa/vt82c686.c                        |  2 +-
>  hw/mips/{mips_fulong2e.c => fuloong2e.c} | 48 ++++++-------
>  hw/mips/{mips_jazz.c => jazz.c}          |  0
>  hw/mips/{mips_malta.c => malta.c}        | 14 ++--
>  hw/mips/mips_int.c                       | 11 +--
>  hw/mips/{mips_mipssim.c => mipssim.c}    |  0
>  hw/mips/{mips_r4k.c => r4k.c}            |  0
>  hw/pci-host/bonito.c                     | 87 +++++++++++++++++++-----
>  tests/qtest/endianness-test.c            |  2 +-
>  .mailmap                                 |  3 +-
>  MAINTAINERS                              | 26 +++----
>  hw/mips/Kconfig                          |  3 +-
>  hw/mips/Makefile.objs                    | 10 +--
>  hw/pci-host/Kconfig                      |  5 ++
>  hw/pci-host/Makefile.objs                |  2 +-
>  18 files changed, 142 insertions(+), 80 deletions(-)
>  rename hw/mips/{mips_fulong2e.c => fuloong2e.c} (91%)
>  rename hw/mips/{mips_jazz.c => jazz.c} (100%)
>  rename hw/mips/{mips_malta.c => malta.c} (99%)
>  rename hw/mips/{mips_mipssim.c => mipssim.c} (100%)
>  rename hw/mips/{mips_r4k.c => r4k.c} (100%)
>
> --
> 2.21.3
>

Re: [PATCH 00/14] hw/mips: patch queue for 2020-05-26

Posted by Aleksandar Markovic 3 years, 10 months ago

уто, 26. мај 2020. у 13:06 Aleksandar Markovic
<aleksandar.qemu.devel@gmail.com> је написао/ла:
>
> уто, 26. мај 2020. у 12:47 Philippe Mathieu-Daudé <f4bug@amsat.org> је
> написао/ла:
> >
> > Hi,
> >
> > This is the current state of my hw/mips/next tree.
> >
> > - MAINTAINERS updated to welcome Huacai Chen and Jiaxun Yang,
> >   and update Aleksandar Rikalo's email address,
> >
> > - Trivial improvements in the Bonito64 North Bridge and the
> >   Fuloong 2e machine,
> >
> > - MIPS Machines names unified without 'mips_' prefix.
> >
> > The following patches need review:
> >
> > - #6  "hw/pci-host/bonito: Map the different PCI ranges more detailled"
> > - #10 "hw/mips/fuloong2e: Fix typo in Fuloong machine name"
> >
>
> I agreed with these two patches, thanks for your help, patch 6 looks
> much better now.
>
> Hope you will be able to send pull request sooner rather than later.
> Please work with Gerd to speed up the process, resolving the
> dependencies efficiently.
>

Hi, Gerd,

My understanding is that a rather trivial issue (removing an #include)
from a mips-specific file, makes your potential pull request, and this
Philippe's pull request dependent between themself. I think actually
Philippe is waiting for your pull request to be sent and applied.
Could you update us on the planned timing of your pull request?

Thanks,
Aleksandar

P.S Too bad such trivial issues from time to time cause stalls in our
workflows, but I guess it is sometimes unavoidable, and we have to
live with that.


> Yours,
> Aleksandar
>
>
>
>
> > Aleksandar Markovic (3):
> >   hw/mips: Rename malta/mipssim/r4k/jazz files
> >   hw/mips/malta: Add some logging for bad register offset cases
> >   MAINTAINERS: Change Aleksandar Rikalo's email address
> >
> > Huacai Chen (1):
> >   MAINTAINERS: Add Huacai Chen as fuloong2e co-maintainer
> >
> > Philippe Mathieu-Daudé (10):
> >   hw/pci-host: Use CONFIG_PCI_BONITO to select the Bonito North Bridge
> >   hw/pci-host/bonito: Fix DPRINTF() format strings
> >   hw/pci-host/bonito: Map peripheral using physical address
> >   hw/pci-host/bonito: Map all the Bonito64 I/O range
> >   hw/pci-host/bonito: Map the different PCI ranges more detailled
> >   hw/pci-host/bonito: Better describe the I/O CS regions
> >   hw/pci-host/bonito: Set the Config register reset value with
> >     FIELD_DP32
> >   hw/mips/fuloong2e: Move code and update a comment
> >   hw/mips/fuloong2e: Fix typo in Fuloong machine name
> >   hw/mips/mips_int: De-duplicate KVM interrupt delivery
> >
> >  docs/system/deprecated.rst               |  5 ++
> >  docs/system/target-mips.rst              |  2 +-
> >  default-configs/mips64el-softmmu.mak     |  2 +-
> >  hw/isa/vt82c686.c                        |  2 +-
> >  hw/mips/{mips_fulong2e.c => fuloong2e.c} | 48 ++++++-------
> >  hw/mips/{mips_jazz.c => jazz.c}          |  0
> >  hw/mips/{mips_malta.c => malta.c}        | 14 ++--
> >  hw/mips/mips_int.c                       | 11 +--
> >  hw/mips/{mips_mipssim.c => mipssim.c}    |  0
> >  hw/mips/{mips_r4k.c => r4k.c}            |  0
> >  hw/pci-host/bonito.c                     | 87 +++++++++++++++++++-----
> >  tests/qtest/endianness-test.c            |  2 +-
> >  .mailmap                                 |  3 +-
> >  MAINTAINERS                              | 26 +++----
> >  hw/mips/Kconfig                          |  3 +-
> >  hw/mips/Makefile.objs                    | 10 +--
> >  hw/pci-host/Kconfig                      |  5 ++
> >  hw/pci-host/Makefile.objs                |  2 +-
> >  18 files changed, 142 insertions(+), 80 deletions(-)
> >  rename hw/mips/{mips_fulong2e.c => fuloong2e.c} (91%)
> >  rename hw/mips/{mips_jazz.c => jazz.c} (100%)
> >  rename hw/mips/{mips_malta.c => malta.c} (99%)
> >  rename hw/mips/{mips_mipssim.c => mipssim.c} (100%)
> >  rename hw/mips/{mips_r4k.c => r4k.c} (100%)
> >
> > --
> > 2.21.3
> >

Re: [PATCH 00/14] hw/mips: patch queue for 2020-05-26

Posted by Philippe Mathieu-Daudé 3 years, 10 months ago

On 5/26/20 3:14 PM, Aleksandar Markovic wrote:
> уто, 26. мај 2020. у 13:06 Aleksandar Markovic
> <aleksandar.qemu.devel@gmail.com> је написао/ла:
>>
>> уто, 26. мај 2020. у 12:47 Philippe Mathieu-Daudé <f4bug@amsat.org> је
>> написао/ла:
>>>
>>> Hi,
>>>
>>> This is the current state of my hw/mips/next tree.
>>>
>>> - MAINTAINERS updated to welcome Huacai Chen and Jiaxun Yang,
>>>   and update Aleksandar Rikalo's email address,
>>>
>>> - Trivial improvements in the Bonito64 North Bridge and the
>>>   Fuloong 2e machine,
>>>
>>> - MIPS Machines names unified without 'mips_' prefix.
>>>
>>> The following patches need review:
>>>
>>> - #6  "hw/pci-host/bonito: Map the different PCI ranges more detailled"
>>> - #10 "hw/mips/fuloong2e: Fix typo in Fuloong machine name"
>>>
>>
>> I agreed with these two patches, thanks for your help, patch 6 looks
>> much better now.
>>
>> Hope you will be able to send pull request sooner rather than later.
>> Please work with Gerd to speed up the process, resolving the
>> dependencies efficiently.
>>
> 
> Hi, Gerd,
> 
> My understanding is that a rather trivial issue (removing an #include)
> from a mips-specific file, makes your potential pull request, and this
> Philippe's pull request dependent between themself. I think actually
> Philippe is waiting for your pull request to be sent and applied.
> Could you update us on the planned timing of your pull request?

Don't worry, Gerd sent his pullreq and Peter is testing it. I'm waiting
Peter's testing done to rebase/send. No hurry anyway ;)

> 
> Thanks,
> Aleksandar
> 
> P.S Too bad such trivial issues from time to time cause stalls in our
> workflows, but I guess it is sometimes unavoidable, and we have to
> live with that.
> 
> 
>> Yours,
>> Aleksandar
>>
>>
>>
>>
>>> Aleksandar Markovic (3):
>>>   hw/mips: Rename malta/mipssim/r4k/jazz files
>>>   hw/mips/malta: Add some logging for bad register offset cases
>>>   MAINTAINERS: Change Aleksandar Rikalo's email address
>>>
>>> Huacai Chen (1):
>>>   MAINTAINERS: Add Huacai Chen as fuloong2e co-maintainer
>>>
>>> Philippe Mathieu-Daudé (10):
>>>   hw/pci-host: Use CONFIG_PCI_BONITO to select the Bonito North Bridge
>>>   hw/pci-host/bonito: Fix DPRINTF() format strings
>>>   hw/pci-host/bonito: Map peripheral using physical address
>>>   hw/pci-host/bonito: Map all the Bonito64 I/O range
>>>   hw/pci-host/bonito: Map the different PCI ranges more detailled
>>>   hw/pci-host/bonito: Better describe the I/O CS regions
>>>   hw/pci-host/bonito: Set the Config register reset value with
>>>     FIELD_DP32
>>>   hw/mips/fuloong2e: Move code and update a comment
>>>   hw/mips/fuloong2e: Fix typo in Fuloong machine name
>>>   hw/mips/mips_int: De-duplicate KVM interrupt delivery
>>>
>>>  docs/system/deprecated.rst               |  5 ++
>>>  docs/system/target-mips.rst              |  2 +-
>>>  default-configs/mips64el-softmmu.mak     |  2 +-
>>>  hw/isa/vt82c686.c                        |  2 +-
>>>  hw/mips/{mips_fulong2e.c => fuloong2e.c} | 48 ++++++-------
>>>  hw/mips/{mips_jazz.c => jazz.c}          |  0
>>>  hw/mips/{mips_malta.c => malta.c}        | 14 ++--
>>>  hw/mips/mips_int.c                       | 11 +--
>>>  hw/mips/{mips_mipssim.c => mipssim.c}    |  0
>>>  hw/mips/{mips_r4k.c => r4k.c}            |  0
>>>  hw/pci-host/bonito.c                     | 87 +++++++++++++++++++-----
>>>  tests/qtest/endianness-test.c            |  2 +-
>>>  .mailmap                                 |  3 +-
>>>  MAINTAINERS                              | 26 +++----
>>>  hw/mips/Kconfig                          |  3 +-
>>>  hw/mips/Makefile.objs                    | 10 +--
>>>  hw/pci-host/Kconfig                      |  5 ++
>>>  hw/pci-host/Makefile.objs                |  2 +-
>>>  18 files changed, 142 insertions(+), 80 deletions(-)
>>>  rename hw/mips/{mips_fulong2e.c => fuloong2e.c} (91%)
>>>  rename hw/mips/{mips_jazz.c => jazz.c} (100%)
>>>  rename hw/mips/{mips_malta.c => malta.c} (99%)
>>>  rename hw/mips/{mips_mipssim.c => mipssim.c} (100%)
>>>  rename hw/mips/{mips_r4k.c => r4k.c} (100%)
>>>
>>> --
>>> 2.21.3
>>>
>

Re: [PATCH 00/14] hw/mips: patch queue for 2020-05-26

Posted by Aleksandar Markovic 3 years, 10 months ago

уто, 26. мај 2020. у 15:20 Philippe Mathieu-Daudé <f4bug@amsat.org> је
написао/ла:
>
> On 5/26/20 3:14 PM, Aleksandar Markovic wrote:
> > уто, 26. мај 2020. у 13:06 Aleksandar Markovic
> > <aleksandar.qemu.devel@gmail.com> је написао/ла:
> >>
> >> уто, 26. мај 2020. у 12:47 Philippe Mathieu-Daudé <f4bug@amsat.org> је
> >> написао/ла:
> >>>
> >>> Hi,
> >>>
> >>> This is the current state of my hw/mips/next tree.
> >>>
> >>> - MAINTAINERS updated to welcome Huacai Chen and Jiaxun Yang,
> >>>   and update Aleksandar Rikalo's email address,
> >>>
> >>> - Trivial improvements in the Bonito64 North Bridge and the
> >>>   Fuloong 2e machine,
> >>>
> >>> - MIPS Machines names unified without 'mips_' prefix.
> >>>
> >>> The following patches need review:
> >>>
> >>> - #6  "hw/pci-host/bonito: Map the different PCI ranges more detailled"
> >>> - #10 "hw/mips/fuloong2e: Fix typo in Fuloong machine name"
> >>>
> >>
> >> I agreed with these two patches, thanks for your help, patch 6 looks
> >> much better now.
> >>
> >> Hope you will be able to send pull request sooner rather than later.
> >> Please work with Gerd to speed up the process, resolving the
> >> dependencies efficiently.
> >>
> >
> > Hi, Gerd,
> >
> > My understanding is that a rather trivial issue (removing an #include)
> > from a mips-specific file, makes your potential pull request, and this
> > Philippe's pull request dependent between themself. I think actually
> > Philippe is waiting for your pull request to be sent and applied.
> > Could you update us on the planned timing of your pull request?
>
> Don't worry, Gerd sent his pullreq and Peter is testing it. I'm waiting
> Peter's testing done to rebase/send. No hurry anyway ;)
>

Peter accepted Gerd's pull request 17 mins ago.

There is a hurry, Philippe, because, some other pending series needs
to be rebased because of file name changes in this series, and I
really would not like to prolong and complicate your efforts in case
we clash with some other pull request over such trivial stuff. i
advice you to send a pull request as soon as you do your regular
preparation and tests - I would really appreciate today, if possible.
Thanks for for your involvement, and meticulousness regarding all
patches in this series and future pull request!

Aleksandar


> >
> > Thanks,
> > Aleksandar
> >
> > P.S Too bad such trivial issues from time to time cause stalls in our
> > workflows, but I guess it is sometimes unavoidable, and we have to
> > live with that.
> >
> >
> >> Yours,
> >> Aleksandar
> >>
> >>
> >>
> >>
> >>> Aleksandar Markovic (3):
> >>>   hw/mips: Rename malta/mipssim/r4k/jazz files
> >>>   hw/mips/malta: Add some logging for bad register offset cases
> >>>   MAINTAINERS: Change Aleksandar Rikalo's email address
> >>>
> >>> Huacai Chen (1):
> >>>   MAINTAINERS: Add Huacai Chen as fuloong2e co-maintainer
> >>>
> >>> Philippe Mathieu-Daudé (10):
> >>>   hw/pci-host: Use CONFIG_PCI_BONITO to select the Bonito North Bridge
> >>>   hw/pci-host/bonito: Fix DPRINTF() format strings
> >>>   hw/pci-host/bonito: Map peripheral using physical address
> >>>   hw/pci-host/bonito: Map all the Bonito64 I/O range
> >>>   hw/pci-host/bonito: Map the different PCI ranges more detailled
> >>>   hw/pci-host/bonito: Better describe the I/O CS regions
> >>>   hw/pci-host/bonito: Set the Config register reset value with
> >>>     FIELD_DP32
> >>>   hw/mips/fuloong2e: Move code and update a comment
> >>>   hw/mips/fuloong2e: Fix typo in Fuloong machine name
> >>>   hw/mips/mips_int: De-duplicate KVM interrupt delivery
> >>>
> >>>  docs/system/deprecated.rst               |  5 ++
> >>>  docs/system/target-mips.rst              |  2 +-
> >>>  default-configs/mips64el-softmmu.mak     |  2 +-
> >>>  hw/isa/vt82c686.c                        |  2 +-
> >>>  hw/mips/{mips_fulong2e.c => fuloong2e.c} | 48 ++++++-------
> >>>  hw/mips/{mips_jazz.c => jazz.c}          |  0
> >>>  hw/mips/{mips_malta.c => malta.c}        | 14 ++--
> >>>  hw/mips/mips_int.c                       | 11 +--
> >>>  hw/mips/{mips_mipssim.c => mipssim.c}    |  0
> >>>  hw/mips/{mips_r4k.c => r4k.c}            |  0
> >>>  hw/pci-host/bonito.c                     | 87 +++++++++++++++++++-----
> >>>  tests/qtest/endianness-test.c            |  2 +-
> >>>  .mailmap                                 |  3 +-
> >>>  MAINTAINERS                              | 26 +++----
> >>>  hw/mips/Kconfig                          |  3 +-
> >>>  hw/mips/Makefile.objs                    | 10 +--
> >>>  hw/pci-host/Kconfig                      |  5 ++
> >>>  hw/pci-host/Makefile.objs                |  2 +-
> >>>  18 files changed, 142 insertions(+), 80 deletions(-)
> >>>  rename hw/mips/{mips_fulong2e.c => fuloong2e.c} (91%)
> >>>  rename hw/mips/{mips_jazz.c => jazz.c} (100%)
> >>>  rename hw/mips/{mips_malta.c => malta.c} (99%)
> >>>  rename hw/mips/{mips_mipssim.c => mipssim.c} (100%)
> >>>  rename hw/mips/{mips_r4k.c => r4k.c} (100%)
> >>>
> >>> --
> >>> 2.21.3
> >>>
> >

Re: [PATCH 00/14] hw/mips: patch queue for 2020-05-26

Posted by Philippe Mathieu-Daudé 3 years, 10 months ago

On 5/26/20 3:29 PM, Aleksandar Markovic wrote:
> уто, 26. мај 2020. у 15:20 Philippe Mathieu-Daudé <f4bug@amsat.org> је
> написао/ла:
>>
>> On 5/26/20 3:14 PM, Aleksandar Markovic wrote:
>>> уто, 26. мај 2020. у 13:06 Aleksandar Markovic
>>> <aleksandar.qemu.devel@gmail.com> је написао/ла:
>>>>
>>>> уто, 26. мај 2020. у 12:47 Philippe Mathieu-Daudé <f4bug@amsat.org> је
>>>> написао/ла:
>>>>>
>>>>> Hi,
>>>>>
>>>>> This is the current state of my hw/mips/next tree.
>>>>>
>>>>> - MAINTAINERS updated to welcome Huacai Chen and Jiaxun Yang,
>>>>>   and update Aleksandar Rikalo's email address,
>>>>>
>>>>> - Trivial improvements in the Bonito64 North Bridge and the
>>>>>   Fuloong 2e machine,
>>>>>
>>>>> - MIPS Machines names unified without 'mips_' prefix.
>>>>>
>>>>> The following patches need review:
>>>>>
>>>>> - #6  "hw/pci-host/bonito: Map the different PCI ranges more detailled"
>>>>> - #10 "hw/mips/fuloong2e: Fix typo in Fuloong machine name"
>>>>>
>>>>
>>>> I agreed with these two patches, thanks for your help, patch 6 looks
>>>> much better now.
>>>>
>>>> Hope you will be able to send pull request sooner rather than later.
>>>> Please work with Gerd to speed up the process, resolving the
>>>> dependencies efficiently.
>>>>
>>>
>>> Hi, Gerd,
>>>
>>> My understanding is that a rather trivial issue (removing an #include)
>>> from a mips-specific file, makes your potential pull request, and this
>>> Philippe's pull request dependent between themself. I think actually
>>> Philippe is waiting for your pull request to be sent and applied.
>>> Could you update us on the planned timing of your pull request?
>>
>> Don't worry, Gerd sent his pullreq and Peter is testing it. I'm waiting
>> Peter's testing done to rebase/send. No hurry anyway ;)
>>
> 
> Peter accepted Gerd's pull request 17 mins ago.
> 
> There is a hurry, Philippe, because, some other pending series needs
> to be rebased because of file name changes in this series, and I
> really would not like to prolong and complicate your efforts in case
> we clash with some other pull request over such trivial stuff. i
> advice you to send a pull request as soon as you do your regular
> preparation and tests - I would really appreciate today, if possible.

Sorry I took a 30min break to eat something for lunch.

The pull request is now sent, thanks for your patience.

Regards,

Phil.

> Thanks for for your involvement, and meticulousness regarding all
> patches in this series and future pull request!
> 
> Aleksandar
> 
> 
>>>
>>> Thanks,
>>> Aleksandar
>>>
>>> P.S Too bad such trivial issues from time to time cause stalls in our
>>> workflows, but I guess it is sometimes unavoidable, and we have to
>>> live with that.
>>>
>>>
>>>> Yours,
>>>> Aleksandar
>>>>
>>>>
>>>>
>>>>
>>>>> Aleksandar Markovic (3):
>>>>>   hw/mips: Rename malta/mipssim/r4k/jazz files
>>>>>   hw/mips/malta: Add some logging for bad register offset cases
>>>>>   MAINTAINERS: Change Aleksandar Rikalo's email address
>>>>>
>>>>> Huacai Chen (1):
>>>>>   MAINTAINERS: Add Huacai Chen as fuloong2e co-maintainer
>>>>>
>>>>> Philippe Mathieu-Daudé (10):
>>>>>   hw/pci-host: Use CONFIG_PCI_BONITO to select the Bonito North Bridge
>>>>>   hw/pci-host/bonito: Fix DPRINTF() format strings
>>>>>   hw/pci-host/bonito: Map peripheral using physical address
>>>>>   hw/pci-host/bonito: Map all the Bonito64 I/O range
>>>>>   hw/pci-host/bonito: Map the different PCI ranges more detailled
>>>>>   hw/pci-host/bonito: Better describe the I/O CS regions
>>>>>   hw/pci-host/bonito: Set the Config register reset value with
>>>>>     FIELD_DP32
>>>>>   hw/mips/fuloong2e: Move code and update a comment
>>>>>   hw/mips/fuloong2e: Fix typo in Fuloong machine name
>>>>>   hw/mips/mips_int: De-duplicate KVM interrupt delivery
>>>>>
>>>>>  docs/system/deprecated.rst               |  5 ++
>>>>>  docs/system/target-mips.rst              |  2 +-
>>>>>  default-configs/mips64el-softmmu.mak     |  2 +-
>>>>>  hw/isa/vt82c686.c                        |  2 +-
>>>>>  hw/mips/{mips_fulong2e.c => fuloong2e.c} | 48 ++++++-------
>>>>>  hw/mips/{mips_jazz.c => jazz.c}          |  0
>>>>>  hw/mips/{mips_malta.c => malta.c}        | 14 ++--
>>>>>  hw/mips/mips_int.c                       | 11 +--
>>>>>  hw/mips/{mips_mipssim.c => mipssim.c}    |  0
>>>>>  hw/mips/{mips_r4k.c => r4k.c}            |  0
>>>>>  hw/pci-host/bonito.c                     | 87 +++++++++++++++++++-----
>>>>>  tests/qtest/endianness-test.c            |  2 +-
>>>>>  .mailmap                                 |  3 +-
>>>>>  MAINTAINERS                              | 26 +++----
>>>>>  hw/mips/Kconfig                          |  3 +-
>>>>>  hw/mips/Makefile.objs                    | 10 +--
>>>>>  hw/pci-host/Kconfig                      |  5 ++
>>>>>  hw/pci-host/Makefile.objs                |  2 +-
>>>>>  18 files changed, 142 insertions(+), 80 deletions(-)
>>>>>  rename hw/mips/{mips_fulong2e.c => fuloong2e.c} (91%)
>>>>>  rename hw/mips/{mips_jazz.c => jazz.c} (100%)
>>>>>  rename hw/mips/{mips_malta.c => malta.c} (99%)
>>>>>  rename hw/mips/{mips_mipssim.c => mipssim.c} (100%)
>>>>>  rename hw/mips/{mips_r4k.c => r4k.c} (100%)
>>>>>
>>>>> --
>>>>> 2.21.3
>>>>>
>>>
>

Re: [PATCH 00/14] hw/mips: patch queue for 2020-05-26

Posted by Aleksandar Markovic 3 years, 10 months ago

уто, 26. мај 2020. у 15:14 Aleksandar Markovic
<aleksandar.qemu.devel@gmail.com> је написао/ла:
>
> уто, 26. мај 2020. у 13:06 Aleksandar Markovic
> <aleksandar.qemu.devel@gmail.com> је написао/ла:
> >
> > уто, 26. мај 2020. у 12:47 Philippe Mathieu-Daudé <f4bug@amsat.org> је
> > написао/ла:
> > >
> > > Hi,
> > >
> > > This is the current state of my hw/mips/next tree.
> > >
> > > - MAINTAINERS updated to welcome Huacai Chen and Jiaxun Yang,
> > >   and update Aleksandar Rikalo's email address,
> > >
> > > - Trivial improvements in the Bonito64 North Bridge and the
> > >   Fuloong 2e machine,
> > >
> > > - MIPS Machines names unified without 'mips_' prefix.
> > >
> > > The following patches need review:
> > >
> > > - #6  "hw/pci-host/bonito: Map the different PCI ranges more detailled"
> > > - #10 "hw/mips/fuloong2e: Fix typo in Fuloong machine name"
> > >
> >
> > I agreed with these two patches, thanks for your help, patch 6 looks
> > much better now.
> >
> > Hope you will be able to send pull request sooner rather than later.
> > Please work with Gerd to speed up the process, resolving the
> > dependencies efficiently.
> >
>
> Hi, Gerd,
>
> My understanding is that a rather trivial issue (removing an #include)
> from a mips-specific file, makes your potential pull request, and this
> Philippe's pull request dependent between themself. I think actually
> Philippe is waiting for your pull request to be sent and applied.
> Could you update us on the planned timing of your pull request?
>
> Thanks,
> Aleksandar
>
> P.S Too bad such trivial issues from time to time cause stalls in our
> workflows, but I guess it is sometimes unavoidable, and we have to
> live with that.
>

Ok,Gerd,  it looks your pull request is accepted by Peter.

Philippe, ready to go with pull request? :)

Thanks,
Aleksandar


>
> > Yours,
> > Aleksandar
> >
> >
> >
> >
> > > Aleksandar Markovic (3):
> > >   hw/mips: Rename malta/mipssim/r4k/jazz files
> > >   hw/mips/malta: Add some logging for bad register offset cases
> > >   MAINTAINERS: Change Aleksandar Rikalo's email address
> > >
> > > Huacai Chen (1):
> > >   MAINTAINERS: Add Huacai Chen as fuloong2e co-maintainer
> > >
> > > Philippe Mathieu-Daudé (10):
> > >   hw/pci-host: Use CONFIG_PCI_BONITO to select the Bonito North Bridge
> > >   hw/pci-host/bonito: Fix DPRINTF() format strings
> > >   hw/pci-host/bonito: Map peripheral using physical address
> > >   hw/pci-host/bonito: Map all the Bonito64 I/O range
> > >   hw/pci-host/bonito: Map the different PCI ranges more detailled
> > >   hw/pci-host/bonito: Better describe the I/O CS regions
> > >   hw/pci-host/bonito: Set the Config register reset value with
> > >     FIELD_DP32
> > >   hw/mips/fuloong2e: Move code and update a comment
> > >   hw/mips/fuloong2e: Fix typo in Fuloong machine name
> > >   hw/mips/mips_int: De-duplicate KVM interrupt delivery
> > >
> > >  docs/system/deprecated.rst               |  5 ++
> > >  docs/system/target-mips.rst              |  2 +-
> > >  default-configs/mips64el-softmmu.mak     |  2 +-
> > >  hw/isa/vt82c686.c                        |  2 +-
> > >  hw/mips/{mips_fulong2e.c => fuloong2e.c} | 48 ++++++-------
> > >  hw/mips/{mips_jazz.c => jazz.c}          |  0
> > >  hw/mips/{mips_malta.c => malta.c}        | 14 ++--
> > >  hw/mips/mips_int.c                       | 11 +--
> > >  hw/mips/{mips_mipssim.c => mipssim.c}    |  0
> > >  hw/mips/{mips_r4k.c => r4k.c}            |  0
> > >  hw/pci-host/bonito.c                     | 87 +++++++++++++++++++-----
> > >  tests/qtest/endianness-test.c            |  2 +-
> > >  .mailmap                                 |  3 +-
> > >  MAINTAINERS                              | 26 +++----
> > >  hw/mips/Kconfig                          |  3 +-
> > >  hw/mips/Makefile.objs                    | 10 +--
> > >  hw/pci-host/Kconfig                      |  5 ++
> > >  hw/pci-host/Makefile.objs                |  2 +-
> > >  18 files changed, 142 insertions(+), 80 deletions(-)
> > >  rename hw/mips/{mips_fulong2e.c => fuloong2e.c} (91%)
> > >  rename hw/mips/{mips_jazz.c => jazz.c} (100%)
> > >  rename hw/mips/{mips_malta.c => malta.c} (99%)
> > >  rename hw/mips/{mips_mipssim.c => mipssim.c} (100%)
> > >  rename hw/mips/{mips_r4k.c => r4k.c} (100%)
> > >
> > > --
> > > 2.21.3
> > >