2013.01.22. 12:18 keltezéssel, Yuval Adam írta:
> 
> Signed-off-by: Yuval Adam <[email protected]>
> ---
>  target/linux/ramips/image/Makefile |   56 
> +++++++++++++++++++++++++++++++++---
>  1 file changed, 52 insertions(+), 4 deletions(-)
> 
> diff --git a/target/linux/ramips/image/Makefile 
> b/target/linux/ramips/image/Makefile
> index daceb15..1f9f363 100644
> --- a/target/linux/ramips/image/Makefile
> +++ b/target/linux/ramips/image/Makefile
> @@ -72,6 +72,18 @@ define MkCombineduImage
>       $(call MkImage,lzma,$(KDIR)/vmlinux-$(2).bin.lzma.combined,$(call 
> sysupname,$(1),$(2)),$(6))
>  endef
>  
> +define PrepareEdimaxImage
> +     $(call PatchKernelLzma,$(2),$(3))
> +     if [ `stat -c%s "$(KDIR)/vmlinux-$(2).bin.lzma"` -gt `expr $(4) - 84` 
> ]; then \
> +             echo "Warning: $(KDIR)/vmlinux-$(2).bin.lzma is too big"; \
> +     else if [ `stat -c%s "$(KDIR)/root.$(1)"` -gt $(5) ]; then \
> +             echo "Warning: $(KDIR)/root.$(1) is too big"; \
> +     else \
> +             ( dd if=$(KDIR)/vmlinux-$(2).bin.lzma bs=`expr $(4) - 84` 
> conv=sync ; dd if=$(KDIR)/root.$(1) ) > 
> $(KDIR)/vmlinux-$(2).bin.lzma.combined ; \
> +     fi ; fi
> +     $(call 
> MkImage,lzma,$(KDIR)/vmlinux-$(2).bin.lzma.combined,$(KDIR)/vmlinux-$(2).bin.lzma.combined.uImage,$(6))
> +endef

Why do you need a different kind of image for this board?

> +
>  define CatFiles
>       if [ `stat -c%s "$(1)"` -gt $(2) ]; then \
>               echo "Warning: $(1) is too big"; \
> @@ -119,14 +131,32 @@ define BuildFirmware/Generic/initramfs
>  endef
>  
>  
> mtdlayout_4M=192k(u-boot)ro,64k(u-boot-env)ro,64k(factory)ro,896k(kernel),2880k(rootfs),3776k@0x50000(firmware)
> +kernel_size_4M=917504
> +rootfs_size_4M=2949120
>  define BuildFirmware/GENERIC_4M
> -     $(call BuildFirmware/Generic,$(1),$(2),$(call mkcmdline,$(3),$(4),$(5)) 
> $(call mkmtd/$(6),$(mtdlayout_4M)),917504,2949120,$(7),$(8))
> +     $(call BuildFirmware/Generic,$(1),$(2),$(call mkcmdline,$(3),$(4),$(5)) 
> $(call mkmtd/$(6),$(mtdlayout_4M)),$(kernel_size_4M),$(rootfs_size_4M))
>  endef

This should be a separate change.

>  
>  define BuildFirmware/GENERIC_4M/initramfs
>       $(call BuildFirmware/Generic/initramfs,$(1),$(2),$(call 
> mkcmdline,$(3),$(4),$(5)) $(call mkmtd/$(6),$(mtdlayout_4M)))
>  endef
>  
> +define BuildFirmware/UIMAGE_4M
> +     $(call MkCombineduImage,$(1),$(2),$(call mkcmdline,$(3),$(4),$(5)) 
> $(call mkmtd/$(6),$(mtdlayout_4M)),$(kernel_size_4M),$(rootfs_size_4M),$(7))
> +endef
> +
> +define BuildFirmware/UIMAGE_4M/initramfs
> +     $(call BuildFirmware/GENERIC_4M/initramfs,$(1),$(2),$(3),$(4),$(5),$(6))
> +endef

After the patch, the UIMAGE_4M/8M stuff is not used by any board. So it should
be not here at all.

> +
> +define BuildFirmware/EDIMAX_4M
> +     $(call PrepareEdimaxImage,$(1),$(2),$(call mkcmdline,$(3),$(4),$(5)) 
> $(call mkmtd/$(6),$(mtdlayout_4M)),$(kernel_size_4M),$(rootfs_size_4M),$(7))
> +endef
> +
> +define BuildFirmware/EDIMAX_4M/initramfs
> +     $(call BuildFirmware/GENERIC_4M/initramfs,$(1),$(2),$(3),$(4),$(5),$(6))
> +endef
> +
>  
> mtdlayout_8M=192k(u-boot)ro,64k(u-boot-env)ro,64k(factory)ro,896k(kernel),6976k(rootfs),7872k@0x50000(firmware)
>  kernel_size_8M=917504
>  rootfs_size_8M=7143424
> @@ -142,6 +172,18 @@ define BuildFirmware/UIMAGE_8M
>       $(call MkCombineduImage,$(1),$(2),$(call mkcmdline,$(3),$(4),$(5)) 
> $(call mkmtd/$(6),$(mtdlayout_8M)),$(kernel_size_8M),$(rootfs_size_8M),$(7))
>  endef
>  
> +define BuildFirmware/UIMAGE_8M/initramfs
> +     $(call BuildFirmware/GENERIC_8M/initramfs,$(1),$(2),$(3),$(4),$(5),$(6))
> +endef
> +
> +define BuildFirmware/EDIMAX_8M
> +     $(call PrepareEdimaxImage,$(1),$(2),$(call mkcmdline,$(3),$(4),$(5)) 
> $(call mkmtd/$(6),$(mtdlayout_8M)),$(kernel_size_8M),$(rootfs_size_8M),$(7))
> +endef
> +
> +define BuildFirmware/EDIMAX_8M/initramfs
> +     $(call BuildFirmware/GENERIC_8M/initramfs,$(1),$(2),$(3),$(4),$(5),$(6))
> +endef
> +
>  
> mtdlayout_edimax_3g6200n=192k(u-boot)ro,64k(u-boot-env)ro,64k(factory)ro,896k(kernel),2752k(rootfs),128k@0x3e0000(cimage)ro,3648k@0x50000(firmware)
>  define BuildFirmware/3G6200N
>       $(call BuildFirmware/Generic,$(1),$(2),$(call mkcmdline,$(3),$(4),$(5)) 
> $(call mkmtd/$(6),$(mtdlayout_edimax_3g6200n)),917504,2818048)
> @@ -250,8 +292,8 @@ define BuildFirmware/DIR620A1/initramfs
>  endef
>  
>  define BuildFirmware/Edimax
> -     if [ -e "$(call sysupname,$(1),$(2))" ]; then \
> -             mkedimaximg -i $(call sysupname,$(1),$(2)) \
> +     if [ -e "$(KDIR)/vmlinux-$(2).bin.lzma.combined" ]; then \
> +             mkedimaximg -i $(KDIR)/vmlinux-$(2).bin.lzma.combined.uImage \
>                       -o $(call imgname,$(1),$(2))-factory.bin \
>                       -s $(3) -m $(4) -f $(5) -S $(6); \
>       fi
> @@ -270,7 +312,7 @@ define BuildFirmware/F5D8235
>  endef
>  
>  define BuildFirmware/FONERA20N
> -     $(call 
> BuildFirmware/GENERIC_8M,$(1),fonera20n,FONERA20N,ttyS1,57600,phys)
> +     $(call 
> BuildFirmware/EDIMAX_8M,$(1),fonera20n,FONERA20N,ttyS1,57600,phys)

Again, what is the problem with the plain uImage on the Fonera 2.0N?

-Gabor
_______________________________________________
openwrt-devel mailing list
[email protected]
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to