* [PATCH v2 0/3] Add support for Fujitsu A64FX processor
@ 2021-07-30 1:40 Shuuichirou Ishii
0 siblings, 0 replies; 5+ messages in thread
From: Shuuichirou Ishii @ 2021-07-30 1:40 UTC (permalink / raw)
To: peter.maydell, qemu-arm, qemu-devel; +Cc: ishii.shuuichir
This is the v2 patch series.
v2:
No features have been added or removed from the v1 patch series. Removal
of unused definitions that were added in excess, and consolidation of
patches for the purpose of functional consistency.
For patch 1, ARM_FEATURE_A64FX is not used in the v1 patch series, so it
was deleted this time, and will be added again when it is used.
For patch 2, since the a64fx_cp_reginfo structure is not used in the v1
patch series, I deleted the empty definition and added the TODO in the
aarch64_a64fx_initfn function. Also fixed the appearance, and cleaned up
and removed some things for patch consistency.
For patch 3, a64fx was added to docs/system/arm/virt.rst and
hw/arm/virt.c respectively, as a modification to the patch consistency
cleanup done in patch 2.
Shuuichirou Ishii (3):
target-arm: delete ARM_FEATURE_A64FX
target-arm: cpu64: Add support for Fujitsu A64FX
target-arm: Add A64FX processor support to virt machine
hw/arm/virt.c | 2 +-
target/arm/cpu.h | 1 -
target/arm/cpu64.c | 10 +++-------
3 files changed, 4 insertions(+), 9 deletions(-)
--
2.27.0
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v2 0/3] Add support for Fujitsu A64FX processor
@ 2021-07-30 3:08 Shuuichirou Ishii
2021-07-30 10:38 ` Peter Maydell
0 siblings, 1 reply; 5+ messages in thread
From: Shuuichirou Ishii @ 2021-07-30 3:08 UTC (permalink / raw)
To: peter.maydell, qemu-arm, qemu-devel; +Cc: ishii.shuuichir
This is the v2 patch series.
v2:
No features have been added or removed from the v1 patch series. Removal
of unused definitions that were added in excess, and consolidation of
patches for the purpose of functional consistency.
For patch 1, ARM_FEATURE_A64FX is not used in the v1 patch series, so it
was deleted this time, and will be added again when it is used.
For patch 2, since the a64fx_cp_reginfo structure is not used in the v1
patch series, I deleted the empty definition and added the TODO in the
aarch64_a64fx_initfn function. Also fixed the appearance, and cleaned up
and removed some things for patch consistency.
For patch 3, a64fx was added to docs/system/arm/virt.rst and
hw/arm/virt.c respectively, as a modification to the patch consistency
cleanup done in patch 2.
Shuuichirou Ishii (3):
target-arm: delete ARM_FEATURE_A64FX
target-arm: cpu64: Add support for Fujitsu A64FX
target-arm: Add A64FX processor support to virt machine
hw/arm/virt.c | 2 +-
target/arm/cpu.h | 1 -
target/arm/cpu64.c | 10 +++-------
3 files changed, 4 insertions(+), 9 deletions(-)
--
2.27.0
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 0/3] Add support for Fujitsu A64FX processor
2021-07-30 3:08 Shuuichirou Ishii
@ 2021-07-30 10:38 ` Peter Maydell
2021-08-03 0:36 ` ishii.shuuichir
0 siblings, 1 reply; 5+ messages in thread
From: Peter Maydell @ 2021-07-30 10:38 UTC (permalink / raw)
To: Shuuichirou Ishii; +Cc: qemu-arm, QEMU Developers
On Fri, 30 Jul 2021 at 04:08, Shuuichirou Ishii
<ishii.shuuichir@fujitsu.com> wrote:
>
> This is the v2 patch series.
>
> v2:
> No features have been added or removed from the v1 patch series. Removal
> of unused definitions that were added in excess, and consolidation of
> patches for the purpose of functional consistency.
>
> For patch 1, ARM_FEATURE_A64FX is not used in the v1 patch series, so it
> was deleted this time, and will be added again when it is used.
>
> For patch 2, since the a64fx_cp_reginfo structure is not used in the v1
> patch series, I deleted the empty definition and added the TODO in the
> aarch64_a64fx_initfn function. Also fixed the appearance, and cleaned up
> and removed some things for patch consistency.
>
> For patch 3, a64fx was added to docs/system/arm/virt.rst and
> hw/arm/virt.c respectively, as a modification to the patch consistency
> cleanup done in patch 2.
I'm afraid this isn't the way a v2 patchseries should be structured.
The idea is that a v2 series should be complete in itself, not based
on whatever v1 was. So when you make the changes requested in review
of v1, you update the commits in your local git branch, and then you
send out the patches as the v2. v2 should apply cleanly on to master,
and all the patches in it should be logically separated out changes
(with no "patch 1 makes a change and then patch 2 changes the code
that was added in patch 1" effects).
thanks
-- PMM
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH v2 0/3] Add support for Fujitsu A64FX processor
2021-07-30 10:38 ` Peter Maydell
@ 2021-08-03 0:36 ` ishii.shuuichir
2021-08-03 9:03 ` Peter Maydell
0 siblings, 1 reply; 5+ messages in thread
From: ishii.shuuichir @ 2021-08-03 0:36 UTC (permalink / raw)
To: 'Peter Maydell'; +Cc: qemu-arm, QEMU Developers, ishii.shuuichir
> I'm afraid this isn't the way a v2 patchseries should be structured.
> The idea is that a v2 series should be complete in itself, not based on whatever v1
> was. So when you make the changes requested in review of v1, you update the
> commits in your local git branch, and then you send out the patches as the v2. v2
> should apply cleanly on to master, and all the patches in it should be logically
> separated out changes (with no "patch 1 makes a change and then patch 2
> changes the code that was added in patch 1" effects).
Thank you for comments.
We apologize for the inconvenience caused by our lack of understanding.
I understood your point.
Just to confirm,
should I update to v3 and resubmit it as a patch series based on the points you mentioned?
Best regards.
> -----Original Message-----
> From: Peter Maydell <peter.maydell@linaro.org>
> Sent: Friday, July 30, 2021 7:39 PM
> To: Ishii, Shuuichirou/石井 周一郎 <ishii.shuuichir@fujitsu.com>
> Cc: qemu-arm <qemu-arm@nongnu.org>; QEMU Developers
> <qemu-devel@nongnu.org>
> Subject: Re: [PATCH v2 0/3] Add support for Fujitsu A64FX processor
>
> On Fri, 30 Jul 2021 at 04:08, Shuuichirou Ishii <ishii.shuuichir@fujitsu.com>
> wrote:
> >
> > This is the v2 patch series.
> >
> > v2:
> > No features have been added or removed from the v1 patch series.
> > Removal of unused definitions that were added in excess, and
> > consolidation of patches for the purpose of functional consistency.
> >
> > For patch 1, ARM_FEATURE_A64FX is not used in the v1 patch series, so
> > it was deleted this time, and will be added again when it is used.
> >
> > For patch 2, since the a64fx_cp_reginfo structure is not used in the
> > v1 patch series, I deleted the empty definition and added the TODO in
> > the aarch64_a64fx_initfn function. Also fixed the appearance, and
> > cleaned up and removed some things for patch consistency.
> >
> > For patch 3, a64fx was added to docs/system/arm/virt.rst and
> > hw/arm/virt.c respectively, as a modification to the patch consistency
> > cleanup done in patch 2.
>
> I'm afraid this isn't the way a v2 patchseries should be structured.
> The idea is that a v2 series should be complete in itself, not based on whatever v1
> was. So when you make the changes requested in review of v1, you update the
> commits in your local git branch, and then you send out the patches as the v2. v2
> should apply cleanly on to master, and all the patches in it should be logically
> separated out changes (with no "patch 1 makes a change and then patch 2
> changes the code that was added in patch 1" effects).
>
> thanks
> -- PMM
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 0/3] Add support for Fujitsu A64FX processor
2021-08-03 0:36 ` ishii.shuuichir
@ 2021-08-03 9:03 ` Peter Maydell
0 siblings, 0 replies; 5+ messages in thread
From: Peter Maydell @ 2021-08-03 9:03 UTC (permalink / raw)
To: ishii.shuuichir; +Cc: qemu-arm, QEMU Developers
On Tue, 3 Aug 2021 at 01:37, ishii.shuuichir@fujitsu.com
<ishii.shuuichir@fujitsu.com> wrote:
>
> > I'm afraid this isn't the way a v2 patchseries should be structured.
> > The idea is that a v2 series should be complete in itself, not based on whatever v1
> > was. So when you make the changes requested in review of v1, you update the
> > commits in your local git branch, and then you send out the patches as the v2. v2
> > should apply cleanly on to master, and all the patches in it should be logically
> > separated out changes (with no "patch 1 makes a change and then patch 2
> > changes the code that was added in patch 1" effects).
>
> Thank you for comments.
> We apologize for the inconvenience caused by our lack of understanding.
> I understood your point.
>
> Just to confirm,
> should I update to v3 and resubmit it as a patch series based on the points you mentioned?
Yes, please.
thanks
-- PMM
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-08-03 9:05 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-30 1:40 [PATCH v2 0/3] Add support for Fujitsu A64FX processor Shuuichirou Ishii
2021-07-30 3:08 Shuuichirou Ishii
2021-07-30 10:38 ` Peter Maydell
2021-08-03 0:36 ` ishii.shuuichir
2021-08-03 9:03 ` Peter Maydell
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).