The following changes since commit d3768d885c6ccbf8a137276843177d76c49033a7:
x86, efi: retry ExitBootServices() on failure (2013-06-11 07:51:54 +0100)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/mfleming/efi.git tags/efi-next
for you to fetch changes up to 5d32cf1e590bcdc879f8f862e836c542f6ba7ecb:
[IA64] sim: Add casts to avoid assignment warnings (2013-06-21 11:22:21 +0100)
----------------------------------------------------------------
* Add casts to avoid compiler warnings about assignments making
pointers from integers of different types. The warnings were
introduced with Borislav's patch to move the EFI runtime service
pointers to void * - Tony Luck
----------------------------------------------------------------
Tony Luck (1):
[IA64] sim: Add casts to avoid assignment warnings
arch/ia64/hp/sim/boot/fw-emu.c | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
--
Matt Fleming, Intel Open Source Technology Center
On Wed, 26 Jun, at 01:23:31PM, Matt Fleming wrote:
> The following changes since commit d3768d885c6ccbf8a137276843177d76c49033a7:
>
> x86, efi: retry ExitBootServices() on failure (2013-06-11 07:51:54 +0100)
>
> are available in the git repository at:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/mfleming/efi.git tags/efi-next
>
> for you to fetch changes up to 5d32cf1e590bcdc879f8f862e836c542f6ba7ecb:
>
> [IA64] sim: Add casts to avoid assignment warnings (2013-06-21 11:22:21 +0100)
>
> ----------------------------------------------------------------
> * Add casts to avoid compiler warnings about assignments making
> pointers from integers of different types. The warnings were
> introduced with Borislav's patch to move the EFI runtime service
> pointers to void * - Tony Luck
>
> ----------------------------------------------------------------
> Tony Luck (1):
> [IA64] sim: Add casts to avoid assignment warnings
>
> arch/ia64/hp/sim/boot/fw-emu.c | 20 ++++++++++----------
> 1 file changed, 10 insertions(+), 10 deletions(-)
I don't see this commit in Linus' tree so presumably Tony is still
seeing these warnings.
--
Matt Fleming, Intel Open Source Technology Center
>> ----------------------------------------------------------------
>> Tony Luck (1):
>> [IA64] sim: Add casts to avoid assignment warnings
>>
>> arch/ia64/hp/sim/boot/fw-emu.c | 20 ++++++++++----------
>> 1 file changed, 10 insertions(+), 10 deletions(-)
>
> I don't see this commit in Linus' tree so presumably Tony is still
> seeing these warnings.
Correct - I see 10 warning about "assignment makes pointer from integer"
when building Linus' tree (HEAD = d2b4a646).
My patch doesn't appear to be in linux-next either (next-20130708).
I had hoped to have this patch follow in the same path that the
one that changed the types and introduced the warnings took ...
but since that didn't work perhaps I should just ask Linus to pull
it from my ia64 tree.
-Tony
On 07/08/2013 10:29 AM, Luck, Tony wrote:
>>> ----------------------------------------------------------------
>>> Tony Luck (1):
>>> [IA64] sim: Add casts to avoid assignment warnings
>>>
>>> arch/ia64/hp/sim/boot/fw-emu.c | 20 ++++++++++----------
>>> 1 file changed, 10 insertions(+), 10 deletions(-)
>>
>> I don't see this commit in Linus' tree so presumably Tony is still
>> seeing these warnings.
>
> Correct - I see 10 warning about "assignment makes pointer from integer"
> when building Linus' tree (HEAD = d2b4a646).
>
> My patch doesn't appear to be in linux-next either (next-20130708).
>
> I had hoped to have this patch follow in the same path that the
> one that changed the types and introduced the warnings took ...
> but since that didn't work perhaps I should just ask Linus to pull
> it from my ia64 tree.
>
I can push it, although it seems a bit odd to me to push an ia64-only
patch through the x86 tree.
Let me know what you prefer.
-hpa
On Mon, Jul 8, 2013 at 11:36 AM, H. Peter Anvin <[email protected]> wrote:
>> I had hoped to have this patch follow in the same path that the
>> one that changed the types and introduced the warnings took ...
>> but since that didn't work perhaps I should just ask Linus to pull
>> it from my ia64 tree.
>>
>
> I can push it, although it seems a bit odd to me to push an ia64-only
> patch through the x86 tree.
>
> Let me know what you prefer.
I've sent Linus a "please pull" for this from my ia64 tree.
-Tony