diff mbox

ARM: zImage/virt: remove hyp-stub.S during distclean

Message ID 20130610060325.2335.65815.sendpatchset@w520 (mailing list archive)
State New, archived
Headers show

Commit Message

Magnus Damm June 10, 2013, 6:03 a.m. UTC
From: Magnus Damm <damm@opensource.se>

Make sure hyp-stub.S gets removed during make distclean,
this left over file was introduced in commit:

424e599 ARM: zImage/virt: hyp mode entry support for the zImage loader

Signed-off-by: Magnus Damm <damm@opensource.se>
---

 arch/arm/boot/compressed/Makefile |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

tip-bot for Dave Martin June 10, 2013, 11:26 a.m. UTC | #1
On Mon, Jun 10, 2013 at 03:03:25PM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm@opensource.se>
> 
> Make sure hyp-stub.S gets removed during make distclean,
> this left over file was introduced in commit:
> 
> 424e599 ARM: zImage/virt: hyp mode entry support for the zImage loader
> 
> Signed-off-by: Magnus Damm <damm@opensource.se>

Thanks for spotting this.  Your fix looks sensible to me.

Acked-by: Dave Martin <dave.martin@linaro.org>

> ---
> 
>  arch/arm/boot/compressed/Makefile |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> --- 0001/arch/arm/boot/compressed/Makefile
> +++ work/arch/arm/boot/compressed/Makefile	2013-06-05 17:21:20.000000000 +0900
> @@ -116,7 +116,8 @@ targets       := vmlinux vmlinux.lds \
>  
>  # Make sure files are removed during clean
>  extra-y       += piggy.gzip piggy.lzo piggy.lzma piggy.xzkern \
> -		 lib1funcs.S ashldi3.S $(libfdt) $(libfdt_hdrs)
> +		 lib1funcs.S ashldi3.S $(libfdt) $(libfdt_hdrs) \
> +		 hyp-stub.S
>  
>  ifeq ($(CONFIG_FUNCTION_TRACER),y)
>  ORIG_CFLAGS := $(KBUILD_CFLAGS)
Simon Horman June 11, 2013, 7:48 a.m. UTC | #2
On Mon, Jun 10, 2013 at 03:03:25PM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm@opensource.se>
> 
> Make sure hyp-stub.S gets removed during make distclean,
> this left over file was introduced in commit:
> 
> 424e599 ARM: zImage/virt: hyp mode entry support for the zImage loader
> 
> Signed-off-by: Magnus Damm <damm@opensource.se>

Reviewed-by: Simon Horman <horms+renesas@verge.net.au>

> ---
> 
>  arch/arm/boot/compressed/Makefile |    3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> --- 0001/arch/arm/boot/compressed/Makefile
> +++ work/arch/arm/boot/compressed/Makefile	2013-06-05 17:21:20.000000000 +0900
> @@ -116,7 +116,8 @@ targets       := vmlinux vmlinux.lds \
>  
>  # Make sure files are removed during clean
>  extra-y       += piggy.gzip piggy.lzo piggy.lzma piggy.xzkern \
> -		 lib1funcs.S ashldi3.S $(libfdt) $(libfdt_hdrs)
> +		 lib1funcs.S ashldi3.S $(libfdt) $(libfdt_hdrs) \
> +		 hyp-stub.S
>  
>  ifeq ($(CONFIG_FUNCTION_TRACER),y)
>  ORIG_CFLAGS := $(KBUILD_CFLAGS)
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
Magnus Damm June 11, 2013, 9:31 a.m. UTC | #3
Hi Simon, Dave, everyone,

On Tue, Jun 11, 2013 at 4:48 PM, Simon Horman <horms@verge.net.au> wrote:
> On Mon, Jun 10, 2013 at 03:03:25PM +0900, Magnus Damm wrote:
>> From: Magnus Damm <damm@opensource.se>
>>
>> Make sure hyp-stub.S gets removed during make distclean,
>> this left over file was introduced in commit:
>>
>> 424e599 ARM: zImage/virt: hyp mode entry support for the zImage loader
>>
>> Signed-off-by: Magnus Damm <damm@opensource.se>

Acked-by: Dave Martin <dave.martin@linaro.org>

> Reviewed-by: Simon Horman <horms+renesas@verge.net.au>

Thanks guys, who can pick this up?

Cheers,

/ magnus
tip-bot for Dave Martin June 11, 2013, 10:06 a.m. UTC | #4
On Tue, Jun 11, 2013 at 06:31:37PM +0900, Magnus Damm wrote:
> Hi Simon, Dave, everyone,
> 
> On Tue, Jun 11, 2013 at 4:48 PM, Simon Horman <horms@verge.net.au> wrote:
> > On Mon, Jun 10, 2013 at 03:03:25PM +0900, Magnus Damm wrote:
> >> From: Magnus Damm <damm@opensource.se>
> >>
> >> Make sure hyp-stub.S gets removed during make distclean,
> >> this left over file was introduced in commit:
> >>
> >> 424e599 ARM: zImage/virt: hyp mode entry support for the zImage loader
> >>
> >> Signed-off-by: Magnus Damm <damm@opensource.se>
> 
> Acked-by: Dave Martin <dave.martin@linaro.org>
> 
> > Reviewed-by: Simon Horman <horms+renesas@verge.net.au>
> 
> Thanks guys, who can pick this up?

I suggest you send this to Russell's patch system.

Cheers
---Dave
Magnus Damm June 12, 2013, 9:52 a.m. UTC | #5
On Tue, Jun 11, 2013 at 7:06 PM, Dave Martin <dave.martin@linaro.org> wrote:
> On Tue, Jun 11, 2013 at 06:31:37PM +0900, Magnus Damm wrote:
>> Hi Simon, Dave, everyone,
>>
>> On Tue, Jun 11, 2013 at 4:48 PM, Simon Horman <horms@verge.net.au> wrote:
>> > On Mon, Jun 10, 2013 at 03:03:25PM +0900, Magnus Damm wrote:
>> >> From: Magnus Damm <damm@opensource.se>
>> >>
>> >> Make sure hyp-stub.S gets removed during make distclean,
>> >> this left over file was introduced in commit:
>> >>
>> >> 424e599 ARM: zImage/virt: hyp mode entry support for the zImage loader
>> >>
>> >> Signed-off-by: Magnus Damm <damm@opensource.se>
>>
>> Acked-by: Dave Martin <dave.martin@linaro.org>
>>
>> > Reviewed-by: Simon Horman <horms+renesas@verge.net.au>
>>
>> Thanks guys, who can pick this up?
>
> I suggest you send this to Russell's patch system.

Done!

http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=7756/1

Thanks for your help!

/ magnus
diff mbox

Patch

--- 0001/arch/arm/boot/compressed/Makefile
+++ work/arch/arm/boot/compressed/Makefile	2013-06-05 17:21:20.000000000 +0900
@@ -116,7 +116,8 @@  targets       := vmlinux vmlinux.lds \
 
 # Make sure files are removed during clean
 extra-y       += piggy.gzip piggy.lzo piggy.lzma piggy.xzkern \
-		 lib1funcs.S ashldi3.S $(libfdt) $(libfdt_hdrs)
+		 lib1funcs.S ashldi3.S $(libfdt) $(libfdt_hdrs) \
+		 hyp-stub.S
 
 ifeq ($(CONFIG_FUNCTION_TRACER),y)
 ORIG_CFLAGS := $(KBUILD_CFLAGS)