> -----Original Message-----
> From: Qemu-devel <qemu-devel-bounces+wentong.wu=intel.com@nongnu.org> On Behalf Of Peter Maydell
> Sent: Monday, July 13, 2020 11:59 PM
> To: QEMU Developers <qemu-devel@nongnu.org>
> Subject: Re: [PULL 00/25] target-arm queue
> 
> On Mon, 13 Jul 2020 at 15:11, Peter Maydell <peter.maydell@linaro.org> wrote:
> >
> > Last lot of target-arm changes to squeeze in before rc1:
> >  * various minor Arm bug fixes
> >  * David Carlier's Haiku build portability fixes
> >  * Wentong Wu's fixes for icount handling in the nios2 target
> >
> > The following changes since commit 00ce6c36b35e0eb8cc5d68a28f288a6335848813:
> >
> >   Merge remote-tracking branch 'remotes/huth-gitlab/tags/pull-request-2020-07-13' into staging (2020-07-13 13:01:30 +0100)
> >
> > are available in the Git repository at:
> >
> >   https://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20200713
> >
> > for you to fetch changes up to 756f739b1682bf131994ec96dad7fbdf8b54493a:
> >
> >   hw/arm/aspeed: Do not create and attach empty SD cards by default (2020-07-13 14:36:12 +0100)
> >
> > ----------------------------------------------------------------
> > target-arm queue:
> >  * hw/arm/bcm2836: Remove unused 'cpu_type' field
> >  * target/arm: Fix mtedesc for do_mem_zpz
> >  * Add the ability to change the FEC PHY MDIO device number on i.MX25/i.MX6/i.MX7
> >  * target/arm: Don't do raw writes for PMINTENCLR
> >  * virtio-iommu: Fix coverity issue in virtio_iommu_handle_command()
> >  * build: Fix various issues with building on Haiku
> >  * target/nios2: fix wrctl behaviour when using icount
> >  * hw/arm/tosa: Encapsulate misc GPIO handling in a device
> >  * hw/arm/palm.c: Encapsulate misc GPIO handling in a device
> >  * hw/arm/aspeed: Do not create and attach empty SD cards by default
>
>
> Applied, thanks.
> 
> Please update the changelog at https://wiki.qemu.org/ChangeLog/5.1
> for any user-visible changes.

Who will be responsible updating the changelog? Patch author or the person who has the special access for that wiki page?

Thanks

> -- PMM

On Tue, 14 Jul 2020 at 15:52, Wu, Wentong <wentong.wu@intel.com> wrote:
>
> > On Mon, 13 Jul 2020 at 15:11, Peter Maydell <peter.maydell@linaro.org> wrote:
> > > ----------------------------------------------------------------
> > > target-arm queue:
> > >  * hw/arm/bcm2836: Remove unused 'cpu_type' field
> > >  * target/arm: Fix mtedesc for do_mem_zpz
> > >  * Add the ability to change the FEC PHY MDIO device number on i.MX25/i.MX6/i.MX7
> > >  * target/arm: Don't do raw writes for PMINTENCLR
> > >  * virtio-iommu: Fix coverity issue in virtio_iommu_handle_command()
> > >  * build: Fix various issues with building on Haiku
> > >  * target/nios2: fix wrctl behaviour when using icount
> > >  * hw/arm/tosa: Encapsulate misc GPIO handling in a device
> > >  * hw/arm/palm.c: Encapsulate misc GPIO handling in a device
> > >  * hw/arm/aspeed: Do not create and attach empty SD cards by default
> >
> >
> > Applied, thanks.
> >
> > Please update the changelog at https://wiki.qemu.org/ChangeLog/5.1
> > for any user-visible changes.
>
> Who will be responsible updating the changelog? Patch author or
> the person who has the special access for that wiki page?

Usually it's the person who sends the pullrequest (me in this case),
unless they specifically ask a patch author to write some changelog
text. In this case I didn't think anything in this set of patches
needed a changelog entry except for the empty-SD-card change.
If you think there's some text worth adding I can add it for you.

The changelog wiki page, incidentally, can be edited by anybody
with a wiki account. We don't have an automatic account-creation
process because it was heavily hit by spammers, but anybody with
an existing wiki account can create one for developers who want one.

thanks
-- PMM

> -----Original Message-----
> From: Peter Maydell <peter.maydell@linaro.org> 
> Sent: Tuesday, July 14, 2020 10:55 PM
> To: Wu, Wentong <wentong.wu@intel.com>
> Cc: QEMU Developers <qemu-devel@nongnu.org>
> Subject: Re: [PULL 00/25] target-arm queue
> 
> On Tue, 14 Jul 2020 at 15:52, Wu, Wentong <wentong.wu@intel.com> wrote:
> >
> > > On Mon, 13 Jul 2020 at 15:11, Peter Maydell <peter.maydell@linaro.org> wrote:
> > > > ----------------------------------------------------------------
> > > > target-arm queue:
> > > >  * hw/arm/bcm2836: Remove unused 'cpu_type' field
> > > >  * target/arm: Fix mtedesc for do_mem_zpz
> > > >  * Add the ability to change the FEC PHY MDIO device number on 
> > > > i.MX25/i.MX6/i.MX7
> > > >  * target/arm: Don't do raw writes for PMINTENCLR
> > > >  * virtio-iommu: Fix coverity issue in 
> > > > virtio_iommu_handle_command()
> > > >  * build: Fix various issues with building on Haiku
> > > >  * target/nios2: fix wrctl behaviour when using icount
> > > >  * hw/arm/tosa: Encapsulate misc GPIO handling in a device
> > > >  * hw/arm/palm.c: Encapsulate misc GPIO handling in a device
> > > >  * hw/arm/aspeed: Do not create and attach empty SD cards by 
> > > > default
> > >
> > >
> > > Applied, thanks.
> > >
> > > Please update the changelog at https://wiki.qemu.org/ChangeLog/5.1
> > > for any user-visible changes.
> >
> > Who will be responsible updating the changelog? Patch author or the 
> > person who has the special access for that wiki page?
> 
> Usually it's the person who sends the pullrequest (me in this case), unless they specifically ask a patch author to write some changelog text. In this case I didn't think anything in this set of patches needed a changelog entry except for the empty-SD-card change.

Thanks for the detail, I asked this because I don't want to break any working process in case patch author should do that. Now I understand it and no text worth adding, thanks again!

> If you think there's some text worth adding I can add it for you.
>
> The changelog wiki page, incidentally, can be edited by anybody with a wiki account. We don't have an automatic account-creation process because it was heavily hit by spammers, but anybody with an existing wiki account can create one for developers who want one.

> thanks
> -- PMM