Re: [PATCH 09/14] x86, boot: add new runtime_address and runtime_size bzImage fields

From: Sam Ravnborg
Date: Fri May 08 2009 - 17:33:22 EST


On Fri, May 08, 2009 at 02:09:20PM -0700, H. Peter Anvin wrote:
> Sam Ravnborg wrote:
> >> @:
> >>@@ -50,6 +53,9 @@ suffix_$(CONFIG_KERNEL_GZIP) = gz
> >> suffix_$(CONFIG_KERNEL_BZIP2) = bz2
> >> suffix_$(CONFIG_KERNEL_LZMA) = lzma
> >
> >No need for a recursive assignment here. Use ":="
> >Ann we usually uses "-y" not "_y".
> >
>
> Wasn't part of the patch, but OK :)

I overlooked that fact - I often just review what I can see.
(In other words - I should have caught that detail when the original
patch was posted...)

Sam
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/