This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH 4/5] ARM: read_pieced_value do big endian processing only in case of valid gdb_regnum
- From: Yao Qi <yao at codesourcery dot com>
- To: Victor Kamensky <victor dot kamensky at linaro dot org>
- Cc: "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Date: Thu, 23 Oct 2014 14:20:12 +0800
- Subject: Re: [PATCH 4/5] ARM: read_pieced_value do big endian processing only in case of valid gdb_regnum
- Authentication-results: sourceware.org; auth=none
- References: <1413853021-4393-1-git-send-email-victor dot kamensky at linaro dot org> <1413853021-4393-5-git-send-email-victor dot kamensky at linaro dot org> <877fzsihdr dot fsf at codesourcery dot com> <CAA3XUr354bJ-bz7TaxtgzpdvK4D9VEaQ1gsed0pW3rYxwKoViw at mail dot gmail dot com> <87tx2vh3rz dot fsf at codesourcery dot com> <CAA3XUr2U5jeRZOi3o9+gr-Fk+J6oRM=Moxf8EfgX4Rv7tDvHJQ at mail dot gmail dot com>
Victor Kamensky <victor.kamensky@linaro.org> writes:
> I've posted updated V3 version only for this patch. I've modified
> commit message to include more details as you suggested.
> And I moved reg_offset var to more specific blocks as you noted.
> Please take a look.
Thanks, I'll take a look.
> Would you like me to repost the whole series again (all 4
> patches) or it would be OK just like this?
Well, we have two options usually, supposing you post a patch series,
- if you only update one or two patches after review, you can
just reply to reviewer's mail and post the updated patches in the same
mail thread as your original post. It'll be convenient to search in
archive, because all of them are in the same thread.
- if you update all the patches, for example, change the design, better
to post the updated series, like what you did for V2.
In this case, you don't have to post them again.
>
> Also it came up on binutils@ patch discussion with Alan -
> I do not have git commit permission. Is it my correct
> expectation once folks are OK with the patches, you or
> some other gdb maintainer will commit those?
It is better for you to create your own account, and commit your patches
after approval in the future.
Please fill in the form here https://sourceware.org/cgi-bin/pdw/ps_form.cgi
I think I can approve your request, so please fill in my mail address in
box "email address of person who approved request".
Once your account is ready, you can commit approved patches first. I am
still thinking about patch 4/4, it may take some time.
--
Yao (éå)