[LinuxBIOS] [BUILDROM] Missing Serengeti-Cheetah file

[Jordan Crouse]

I missed a patch file when I added the S-C stuff to buildrom last week.
THis just patches LB to use ../payload.elf as the target (an unfortunate
v2 side effect).

[Peter Stuge]

On Mon, Oct 29, 2007 at 04:08:33PM -0600, Jordan Crouse wrote:
> [BUILDROM] Missing Serengeti-Cheetah file
>
> This file should have gone with the rest of the patch - it patches LinuxBIOS
> to look for payload in ../
>
> Signed-off-by: Jordan Crouse

Acked-by: Peter Stuge

[LinuxBIOS] Missing Serengeti-Cheetah file

[Jordan Crouse]

On 29/10/07 23:42 +0100, Peter Stuge wrote:
> On Mon, Oct 29, 2007 at 04:08:33PM -0600, Jordan Crouse wrote:
> > [BUILDROM] Missing Serengeti-Cheetah file
> >
> > This file should have gone with the rest of the patch - it patches LinuxBIOS
> > to look for payload in ../
> >
> > Signed-off-by: Jordan Crouse
>
> Acked-by: Peter Stuge

r56.   Sorry for the delay.
>

[Uwe Hermann]

On Mon, Oct 29, 2007 at 04:08:33PM -0600, Jordan Crouse wrote:
> I missed a patch file when I added the S-C stuff to buildrom last week.
> THis just patches LB to use ../payload.elf as the target (an unfortunate
> v2 side effect).
> [BUILDROM] Missing Serengeti-Cheetah file
>
> This file should have gone with the rest of the patch - it patches LinuxBIOS
> to look for payload in ../
>
> Signed-off-by: Jordan Crouse

Peter already ACKed, but here's one more:

Acked-by: Uwe Hermann
> +@@ -29,7 +29,7 @@ romimage "normal"
> #	payload ../../../payloads/tg3.zelf
> #	payload ../../../../payloads/tg3_vga.zelf
> #	payload ../../../../payloads/tg3--filo_hda2_vga.zelf
>
> +-	payload ../../../../payloads/tg3--filo_hda2_vga_5.4.1.zelf
> ++#	payload ../../../../payloads/tg3--filo_hda2_vga_5.4.1.zelf
> #	payload ../../../../payloads/e1000_vga.zelf
> #	payload ../../../../payloads/tg3--filo_hda2_vga_5_4.zelf
> #	payload ../../../payloads/tg3_com2.zelf
>
> +@@ -38,6 +38,7 @@ romimage "normal"
> #        payload ../../../payloads/tg3--eepro100--e1000--filo_hda2.zelf
> #	payload ../../../payloads/tg3--eepro100--e1000--filo_hda2_5.3.zelf
> #	payload ../../../payloads/tg3--eepro100--e1000--filo_hda2_com2.zelf
>
> ++	payload ../payload.elf
> end
> romimage "fallback"
>
> +@@ -62,7 +63,7 @@ romimage "fallback"
> #	payload ../../../../payloads/filo_hda.zelf
> #	payload ../../../../payloads/tg3--filo_hda2_vga.zelf
> #	payload ../../../../payloads/tg3--filo_hda2_vga_5_4.zelf
>
> +-	payload ../../../../payloads/tg3--filo_hda2_vga_5.4.1.zelf
> ++#	payload ../../../../payloads/tg3--filo_hda2_vga_5.4.1.zelf
> #	payload ../../../../payloads/filo_hda2_novga.zelf
> #	payload ../../../payloads/tg3_com2.zelf
> #       payload ../../../payloads/e1000--filo.zelf
>
> +@@ -70,6 +71,7 @@ romimage "fallback"
> #        payload ../../../payloads/tg3--eepro100--e1000--filo_hda2.zelf
> #	payload ../../../payloads/tg3--eepro100--e1000--filo_hda2_5.3.zelf
> #	payload ../../../payloads/tg3--eepro100--e1000--filo_hda2_com2.zelf
>
> ++	payload ../payload.elf

But why all of the commented stuff? It's absolutely not needed in
buildrom, so why not just drop it? The single line with payload.elf is
fully sufficient.

Uwe.