mirror of https://github.com/hak5/openwrt.git
image: remove padding parameter from append-kernel/append-rootfs
Using pad-to instead of passing the optional padding to append-kernel or append-rootfs. It could be that the value of a variable is passed. In case the variable is empty no error is thrown. Furthermore the purpose of the extra parameter is hard to get without reading the code. Signed-off-by: Mathias Kresin <dev@kresin.me>lede-17.01
parent
14b40d61e1
commit
61c2a7339a
|
@ -103,11 +103,11 @@ define Build/patch-cmdline
|
||||||
endef
|
endef
|
||||||
|
|
||||||
define Build/append-kernel
|
define Build/append-kernel
|
||||||
dd if=$(IMAGE_KERNEL) $(if $(1),bs=$(1) conv=sync) >> $@
|
dd if=$(IMAGE_KERNEL) >> $@
|
||||||
endef
|
endef
|
||||||
|
|
||||||
define Build/append-rootfs
|
define Build/append-rootfs
|
||||||
dd if=$(IMAGE_ROOTFS) $(if $(1),bs=$(1) conv=sync) >> $@
|
dd if=$(IMAGE_ROOTFS) >> $@
|
||||||
endef
|
endef
|
||||||
|
|
||||||
define Build/append-ubi
|
define Build/append-ubi
|
||||||
|
|
|
@ -94,7 +94,7 @@ define Device/Default
|
||||||
KERNEL := kernel-bin | patch-cmdline | lzma | uImage lzma
|
KERNEL := kernel-bin | patch-cmdline | lzma | uImage lzma
|
||||||
COMPILE :=
|
COMPILE :=
|
||||||
IMAGES := sysupgrade.bin
|
IMAGES := sysupgrade.bin
|
||||||
IMAGE/sysupgrade.bin = append-kernel $$$$(BLOCKSIZE) | append-rootfs | pad-rootfs | check-size $$$$(IMAGE_SIZE)
|
IMAGE/sysupgrade.bin = append-kernel | pad-to $$$$(BLOCKSIZE) | append-rootfs | pad-rootfs | check-size $$$$(IMAGE_SIZE)
|
||||||
endef
|
endef
|
||||||
|
|
||||||
$(eval $(call BuildImage))
|
$(eval $(call BuildImage))
|
||||||
|
|
|
@ -106,7 +106,7 @@ define Device/wndr3700
|
||||||
MTDPARTS = spi0.0:320k(u-boot)ro,128k(u-boot-env)ro,7680k(firmware),64k(art)ro
|
MTDPARTS = spi0.0:320k(u-boot)ro,128k(u-boot-env)ro,7680k(firmware),64k(art)ro
|
||||||
IMAGES := sysupgrade.bin factory.img factory-NA.img
|
IMAGES := sysupgrade.bin factory.img factory-NA.img
|
||||||
KERNEL := kernel-bin | patch-cmdline | lzma -d20 | netgear-uImage lzma
|
KERNEL := kernel-bin | patch-cmdline | lzma -d20 | netgear-uImage lzma
|
||||||
IMAGE/default = append-kernel $$$$(BLOCKSIZE) | netgear-squashfs | append-rootfs | pad-rootfs
|
IMAGE/default = append-kernel | pad-to $$$$(BLOCKSIZE) | netgear-squashfs | append-rootfs | pad-rootfs
|
||||||
IMAGE/sysupgrade.bin = $$(IMAGE/default) | check-size $$$$(IMAGE_SIZE)
|
IMAGE/sysupgrade.bin = $$(IMAGE/default) | check-size $$$$(IMAGE_SIZE)
|
||||||
IMAGE/factory.img = $$(IMAGE/default) | netgear-dni | check-size $$$$(IMAGE_SIZE)
|
IMAGE/factory.img = $$(IMAGE/default) | netgear-dni | check-size $$$$(IMAGE_SIZE)
|
||||||
IMAGE/factory-NA.img = $$(IMAGE/default) | netgear-dni NA | check-size $$$$(IMAGE_SIZE)
|
IMAGE/factory-NA.img = $$(IMAGE/default) | netgear-dni NA | check-size $$$$(IMAGE_SIZE)
|
||||||
|
@ -410,7 +410,7 @@ define Device/NBG6616
|
||||||
CMDLINE += mem=128M
|
CMDLINE += mem=128M
|
||||||
IMAGES := sysupgrade.bin
|
IMAGES := sysupgrade.bin
|
||||||
KERNEL := kernel-bin | patch-cmdline | lzma | uImage lzma | jffs2 boot/vmlinux.lzma.uImage
|
KERNEL := kernel-bin | patch-cmdline | lzma | uImage lzma | jffs2 boot/vmlinux.lzma.uImage
|
||||||
IMAGE/sysupgrade.bin = append-kernel $$$$(KERNEL_SIZE) | append-rootfs | pad-rootfs | check-size $$$$(IMAGE_SIZE)
|
IMAGE/sysupgrade.bin = append-kernel | pad-to $$$$(KERNEL_SIZE) | append-rootfs | pad-rootfs | check-size $$$$(IMAGE_SIZE)
|
||||||
# We cannot currently build a factory image. It is the sysupgrade image
|
# We cannot currently build a factory image. It is the sysupgrade image
|
||||||
# prefixed with a header (which is actually written into the MTD device).
|
# prefixed with a header (which is actually written into the MTD device).
|
||||||
# The header is 2kiB and is filled with 0xff. The format seems to be:
|
# The header is 2kiB and is filled with 0xff. The format seems to be:
|
||||||
|
@ -443,7 +443,7 @@ define Device/c-55
|
||||||
KERNEL_SIZE = 2048k
|
KERNEL_SIZE = 2048k
|
||||||
IMAGE_SIZE = 15872k
|
IMAGE_SIZE = 15872k
|
||||||
MTDPARTS = spi0.0:256k(u-boot)ro,128k(u-boot-env)ro,2048k(kernel),13824k(rootfs),13824k(opt)ro,2624k(failsafe)ro,64k(art)ro,15872k@0x60000(firmware)
|
MTDPARTS = spi0.0:256k(u-boot)ro,128k(u-boot-env)ro,2048k(kernel),13824k(rootfs),13824k(opt)ro,2624k(failsafe)ro,64k(art)ro,15872k@0x60000(firmware)
|
||||||
IMAGE/sysupgrade.bin = append-kernel $$$$(KERNEL_SIZE) | append-rootfs | pad-rootfs | check-size $$$$(IMAGE_SIZE)
|
IMAGE/sysupgrade.bin = append-kernel | pad-to $$$$(KERNEL_SIZE) | append-rootfs | pad-rootfs | check-size $$$$(IMAGE_SIZE)
|
||||||
endef
|
endef
|
||||||
|
|
||||||
TARGET_DEVICES += c-55
|
TARGET_DEVICES += c-55
|
||||||
|
|
|
@ -48,7 +48,7 @@ define Device/ubnt-xm
|
||||||
UBNT_CHIP := ar7240
|
UBNT_CHIP := ar7240
|
||||||
IMAGES := sysupgrade.bin factory.bin
|
IMAGES := sysupgrade.bin factory.bin
|
||||||
IMAGE/factory.bin = $$(IMAGE/sysupgrade.bin) | mkubntimage-split
|
IMAGE/factory.bin = $$(IMAGE/sysupgrade.bin) | mkubntimage-split
|
||||||
IMAGE/sysupgrade.bin = append-kernel $$$$(BLOCKSIZE) | append-rootfs | pad-rootfs | check-size $$$$(IMAGE_SIZE)
|
IMAGE/sysupgrade.bin = append-kernel | pad-to $$$$(BLOCKSIZE) | append-rootfs | pad-rootfs | check-size $$$$(IMAGE_SIZE)
|
||||||
endef
|
endef
|
||||||
|
|
||||||
define Device/ubnt-xw
|
define Device/ubnt-xw
|
||||||
|
@ -61,7 +61,7 @@ define Device/ubnt-xw
|
||||||
UBNT_CHIP := ar934x
|
UBNT_CHIP := ar934x
|
||||||
IMAGES := sysupgrade.bin factory.bin
|
IMAGES := sysupgrade.bin factory.bin
|
||||||
IMAGE/factory.bin = $$(IMAGE/sysupgrade.bin) | mkubntimage-split
|
IMAGE/factory.bin = $$(IMAGE/sysupgrade.bin) | mkubntimage-split
|
||||||
IMAGE/sysupgrade.bin = append-kernel $$$$(BLOCKSIZE) | append-rootfs | pad-rootfs | check-size $$$$(IMAGE_SIZE)
|
IMAGE/sysupgrade.bin = append-kernel | pad-to $$$$(BLOCKSIZE) | append-rootfs | pad-rootfs | check-size $$$$(IMAGE_SIZE)
|
||||||
endef
|
endef
|
||||||
|
|
||||||
define Device/ubnt-bz
|
define Device/ubnt-bz
|
||||||
|
@ -74,7 +74,7 @@ define Device/ubnt-bz
|
||||||
UBNT_CHIP := ar934x
|
UBNT_CHIP := ar934x
|
||||||
IMAGES := sysupgrade.bin factory.bin
|
IMAGES := sysupgrade.bin factory.bin
|
||||||
IMAGE/factory.bin = $$(IMAGE/sysupgrade.bin) | mkubntimage-split
|
IMAGE/factory.bin = $$(IMAGE/sysupgrade.bin) | mkubntimage-split
|
||||||
IMAGE/sysupgrade.bin = append-kernel $$$$(BLOCKSIZE) | append-rootfs | pad-rootfs | check-size $$$$(IMAGE_SIZE)
|
IMAGE/sysupgrade.bin = append-kernel | pad-to $$$$(BLOCKSIZE) | append-rootfs | pad-rootfs | check-size $$$$(IMAGE_SIZE)
|
||||||
endef
|
endef
|
||||||
|
|
||||||
define Device/ubnt-unifiac
|
define Device/ubnt-unifiac
|
||||||
|
@ -83,7 +83,7 @@ define Device/ubnt-unifiac
|
||||||
IMAGE_SIZE := 7744k
|
IMAGE_SIZE := 7744k
|
||||||
MTDPARTS = spi0.0:384k(u-boot)ro,64k(u-boot-env)ro,7744k(firmware),7744k(ubnt-airos)ro,128k(bs)ro,256k(cfg)ro,64k(EEPROM)ro
|
MTDPARTS = spi0.0:384k(u-boot)ro,64k(u-boot-env)ro,7744k(firmware),7744k(ubnt-airos)ro,128k(bs)ro,256k(cfg)ro,64k(EEPROM)ro
|
||||||
IMAGES := sysupgrade.bin
|
IMAGES := sysupgrade.bin
|
||||||
IMAGE/sysupgrade.bin = append-kernel $$$$(BLOCKSIZE) | append-rootfs | pad-rootfs | check-size $$$$(IMAGE_SIZE)
|
IMAGE/sysupgrade.bin = append-kernel | pad-to $$$$(BLOCKSIZE) | append-rootfs | pad-rootfs | check-size $$$$(IMAGE_SIZE)
|
||||||
endef
|
endef
|
||||||
|
|
||||||
define Device/rw2458n
|
define Device/rw2458n
|
||||||
|
@ -261,7 +261,7 @@ define Device/ubnt-uap-pro
|
||||||
DEVICE_PROFILE := UBNT UAPPRO
|
DEVICE_PROFILE := UBNT UAPPRO
|
||||||
KERNEL := kernel-bin | patch-cmdline | lzma | uImage lzma | jffs2 kernel0
|
KERNEL := kernel-bin | patch-cmdline | lzma | uImage lzma | jffs2 kernel0
|
||||||
IMAGES := sysupgrade.bin factory.bin
|
IMAGES := sysupgrade.bin factory.bin
|
||||||
IMAGE/sysupgrade.bin = append-kernel $$$$(KERNEL_SIZE) | append-rootfs | pad-rootfs | check-size $$$$(IMAGE_SIZE)
|
IMAGE/sysupgrade.bin = append-kernel | pad-to $$$$(KERNEL_SIZE) | append-rootfs | pad-rootfs | check-size $$$$(IMAGE_SIZE)
|
||||||
IMAGE/factory.bin = $$(IMAGE/sysupgrade.bin) | mkubntimage2
|
IMAGE/factory.bin = $$(IMAGE/sysupgrade.bin) | mkubntimage2
|
||||||
endef
|
endef
|
||||||
|
|
||||||
|
|
|
@ -59,7 +59,7 @@ define Device/linksys-audi
|
||||||
FILESYSTEMS := squashfs
|
FILESYSTEMS := squashfs
|
||||||
PROFILES := Generic AUDI
|
PROFILES := Generic AUDI
|
||||||
IMAGES += factory.bin
|
IMAGES += factory.bin
|
||||||
IMAGE/factory.bin := append-kernel $$$$(KERNEL_SIZE) | append-ubi
|
IMAGE/factory.bin := append-kernel | pad-to $$$$(KERNEL_SIZE) | append-ubi
|
||||||
endef
|
endef
|
||||||
|
|
||||||
define Device/linksys-viper
|
define Device/linksys-viper
|
||||||
|
@ -68,7 +68,7 @@ define Device/linksys-viper
|
||||||
FILESYSTEMS := squashfs
|
FILESYSTEMS := squashfs
|
||||||
PROFILES := Generic VIPER
|
PROFILES := Generic VIPER
|
||||||
IMAGES += factory.bin
|
IMAGES += factory.bin
|
||||||
IMAGE/factory.bin := append-kernel $$$$(KERNEL_SIZE) | append-ubi
|
IMAGE/factory.bin := append-kernel | pad-to $$$$(KERNEL_SIZE) | append-ubi
|
||||||
endef
|
endef
|
||||||
|
|
||||||
define Image/BuildKernel/Template
|
define Image/BuildKernel/Template
|
||||||
|
|
|
@ -71,7 +71,7 @@ define Device/UBI-factory
|
||||||
$(Device/UBI)
|
$(Device/UBI)
|
||||||
UBINIZE_OPTS := -E 5
|
UBINIZE_OPTS := -E 5
|
||||||
IMAGES += factory.img
|
IMAGES += factory.img
|
||||||
IMAGE/factory.img := append-kernel $$$$(KERNEL_SIZE) | append-ubi | pad-to $$$$(PAGESIZE)
|
IMAGE/factory.img := append-kernel | pad-to $$$$(KERNEL_SIZE) | append-ubi | pad-to $$$$(PAGESIZE)
|
||||||
endef
|
endef
|
||||||
|
|
||||||
define Device/NAND-128K
|
define Device/NAND-128K
|
||||||
|
@ -167,7 +167,7 @@ TARGET_DEVICES += armada-370-db armada-370-rd armada-xp-db armada-xp-gp
|
||||||
define Device/armada-388-rd
|
define Device/armada-388-rd
|
||||||
DEVICE_TITLE := Marvell Armada 388 RD (RD-88F6820-AP)
|
DEVICE_TITLE := Marvell Armada 388 RD (RD-88F6820-AP)
|
||||||
IMAGES := firmware.bin
|
IMAGES := firmware.bin
|
||||||
IMAGE/firmware.bin := append-kernel 256k | append-rootfs | pad-rootfs
|
IMAGE/firmware.bin := append-kernel | pad-to 256k | append-rootfs | pad-rootfs
|
||||||
endef
|
endef
|
||||||
TARGET_DEVICES += armada-388-rd
|
TARGET_DEVICES += armada-388-rd
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue