diff mbox

[LEDE-DEV] ..ar71xx/image/Makefile: Remove tplink-safeloader def (moved to include/image.mk)

Message ID 1463124856-29927-1-git-send-email-ash.benz@bk.ru
State Superseded
Delegated to: Alexander Couzens
Headers show

Commit Message

Ash Benz May 13, 2016, 7:34 a.m. UTC
Atten: lynxis

Signed-off-by: Ash Benz <ash.benz@bk.ru>
---
 target/linux/ar71xx/image/Makefile | 12 ------------
 1 file changed, 12 deletions(-)

Comments

Rafał Miłecki May 13, 2016, 7:52 a.m. UTC | #1
Please use a proper prefix for your patch. You can find it e.g. by checking
git log target/linux/ar71xx/image/Makefile
I guess it'll be something like:
ar71xx: image: foo

On 13 May 2016 at 09:34, Ash Benz <ash.benz@bk.ru> wrote:
> Atten: lynxis
>
> Signed-off-by: Ash Benz <ash.benz@bk.ru>

We have some weird "Atten" in the description and there is no real
description at all. I suggest moving "(moved to include/image.mk)" out
of topic to the description and maybe make it a proper English
sentence then.
diff mbox

Patch

diff --git a/target/linux/ar71xx/image/Makefile b/target/linux/ar71xx/image/Makefile
index 3bf005f..a5b1a86 100644
--- a/target/linux/ar71xx/image/Makefile
+++ b/target/linux/ar71xx/image/Makefile
@@ -74,18 +74,6 @@  define Build/mktplinkfw-initramfs
 	@mv $@.new $@
 endef
 
-define Build/tplink-safeloader
-       -$(STAGING_DIR_HOST)/bin/tplink-safeloader \
-		-B $(TPLINK_BOARD_NAME) \
-		-V $(REVISION) \
-		-k $(word 1,$^) \
-		-r $@ \
-		-o $@.new \
-		-j \
-		$(wordlist 2,$(words $(1)),$(1)) \
-		$(if $(findstring sysupgrade,$(word 1,$(1))),-S) && mv $@.new $@ || rm -f $@
-endef
-
 define Build/loader-common
 	rm -rf $@.src
 	$(MAKE) -C lzma-loader \