Re: [Qemu-devel] [PATCH] hw/arm: use defined type name instead of hard-coded string

Re: [Qemu-devel] [PATCH] hw/arm: use defined type name instead of hard-coded string

[Peter Maydell]

On 17 July 2017 at 14:30, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
>
> Hi Peter, this patch is waiting the IDE queue to enters since it depends of
> commit de09efcc7b67ada3593354711b12a03dea23a9d0 to fix AHCI includes.

There's no commit with that hash in master -- do you have
a commit subject or something that I can use to find out
whether it's landed or not?

thanks
-- PMM

Re: [Qemu-devel] [PATCH] hw/arm: use defined type name instead of hard-coded string

[Philippe Mathieu-Daudé]

On 07/27/2017 09:36 AM, Peter Maydell wrote:
> On 17 July 2017 at 14:30, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>> ---
>>
>> Hi Peter, this patch is waiting the IDE queue to enters since it depends of
>> commit de09efcc7b67ada3593354711b12a03dea23a9d0 to fix AHCI includes.
> 
> There's no commit with that hash in master -- do you have
> a commit subject or something that I can use to find out
> whether it's landed or not?

Oops I guess the IDE queue was rebased between this patch and the merge, 
or I messed my copy/paste :)

The dependent commit is 70e2337030f and has how entered /master, do you 
want me to send an v2 with fixed message? Anyway I delayed this patch 
for 2.11 since this is not a bugfix.

Regards,

Phil.

Re: [Qemu-devel] [PATCH] hw/arm: use defined type name instead of hard-coded string

[Peter Maydell]

On 27 July 2017 at 14:48, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
> On 07/27/2017 09:36 AM, Peter Maydell wrote:
>>
>> On 17 July 2017 at 14:30, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>>>
>>> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>>> ---
>>>
>>> Hi Peter, this patch is waiting the IDE queue to enters since it depends
>>> of
>>> commit de09efcc7b67ada3593354711b12a03dea23a9d0 to fix AHCI includes.
>>
>>
>> There's no commit with that hash in master -- do you have
>> a commit subject or something that I can use to find out
>> whether it's landed or not?
>
>
> Oops I guess the IDE queue was rebased between this patch and the merge, or
> I messed my copy/paste :)
>
> The dependent commit is 70e2337030f and has how entered /master, do you want
> me to send an v2 with fixed message? Anyway I delayed this patch for 2.11
> since this is not a bugfix.

You don't need to resend, I'll just keep it on my list of things
to deal with after 2.11. (Anything below the '---' isn't really
in the commit message anyway, it gets stripped out when the
patch is applied.)

thanks
-- PMM

Re: [Qemu-devel] [PATCH] hw/arm: use defined type name instead of hard-coded string

[Peter Maydell]

On 31 July 2017 at 13:09, Peter Maydell <peter.maydell@linaro.org> wrote:
> On 27 July 2017 at 14:48, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>> On 07/27/2017 09:36 AM, Peter Maydell wrote:
>>>
>>> On 17 July 2017 at 14:30, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>>>>
>>>> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>>>> ---
>>>>
>>>> Hi Peter, this patch is waiting the IDE queue to enters since it depends
>>>> of
>>>> commit de09efcc7b67ada3593354711b12a03dea23a9d0 to fix AHCI includes.
>>>
>>>
>>> There's no commit with that hash in master -- do you have
>>> a commit subject or something that I can use to find out
>>> whether it's landed or not?
>>
>>
>> Oops I guess the IDE queue was rebased between this patch and the merge, or
>> I messed my copy/paste :)
>>
>> The dependent commit is 70e2337030f and has how entered /master, do you want
>> me to send an v2 with fixed message? Anyway I delayed this patch for 2.11
>> since this is not a bugfix.
>
> You don't need to resend, I'll just keep it on my list of things
> to deal with after 2.11. (Anything below the '---' isn't really
> in the commit message anyway, it gets stripped out when the
> patch is applied.)

Patch now in target-arm.next ready for post-2.11 pullreq.

thanks
-- PMM