mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Michael Olbrich <m.olbrich@pengutronix.de>
To: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH] Update mtd-utils to 1.4.9
Date: Tue, 13 Mar 2012 14:34:42 +0100	[thread overview]
Message-ID: <20120313133442.GT29221@pengutronix.de> (raw)
In-Reply-To: <20120313133114.GS29221@pengutronix.de>

On Tue, Mar 13, 2012 at 02:31:14PM +0100, Michael Olbrich wrote:
> On Tue, Mar 13, 2012 at 01:00:11PM +0100, bernhard@bwalle.de wrote:
> > From: Bernhard Walle <bernhard@bwalle.de>
> > 
> > I had to rework some patches. Parallel build has been fixed upstream due
> > in commits 5b1cfb9d09a8a40577e79262c6d79dfcde36b1b7 and
> > 05f56bf49ee0de8979c29c11f61a01e30e964444, so we should be able to enable
> > it here.
> > 
> > flash-info has been dropped and mtdinfo should be used instead.
> 
> This patch does not apply. The last one did. Maybe you have some other
> local changes?

Arg, probably the other patch you sent. I have some more comment for that,
then sent them together.

> Michael
> 
> > Signed-off-by: Bernhard Walle <bernhard@bwalle.de>
> > ---
> >  .../0001-make-liblzo-optional.patch                |  107 --------------------
> >  .../0002-make-mkubifs-optional.patch               |   26 -----
> >  patches/mtd-utils-1.4.6/series                     |    2 -
> >  .../mtd-utils-1.4.9/0001-make-ubifs-optional.patch |   24 +++++
> >  ...0002-Make-liblzo-optional-for-ubifs-tools.patch |   54 ++++++++++
> >  patches/mtd-utils-1.4.9/series                     |    5 +
> >  rules/host-mtd-utils.make                          |    2 -
> >  rules/mtd-utils.in                                 |   10 +-
> >  rules/mtd-utils.make                               |    9 +-
> >  9 files changed, 87 insertions(+), 152 deletions(-)
> >  delete mode 100644 patches/mtd-utils-1.4.6/0001-make-liblzo-optional.patch
> >  delete mode 100644 patches/mtd-utils-1.4.6/0002-make-mkubifs-optional.patch
> >  delete mode 100644 patches/mtd-utils-1.4.6/series
> >  create mode 100644 patches/mtd-utils-1.4.9/0001-make-ubifs-optional.patch
> >  create mode 100644 patches/mtd-utils-1.4.9/0002-Make-liblzo-optional-for-ubifs-tools.patch
> >  create mode 100644 patches/mtd-utils-1.4.9/series
> > 
> > diff --git a/patches/mtd-utils-1.4.6/0001-make-liblzo-optional.patch b/patches/mtd-utils-1.4.6/0001-make-liblzo-optional.patch
> > deleted file mode 100644
> > index 00845a1..0000000
> > --- a/patches/mtd-utils-1.4.6/0001-make-liblzo-optional.patch
> > +++ /dev/null
> > @@ -1,107 +0,0 @@
> > -From: Marc Kleine-Budde <mkl@pengutronix.de>
> > -Date: Mon, 20 Jul 2009 19:37:09 +0200
> > -Subject: [PATCH] make liblzo optional
> > -
> > -Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
> > ----
> > - Makefile            |    3 ++-
> > - compr.h             |    2 ++
> > - mkfs.ubifs/Makefile |    6 +++++-
> > - mkfs.ubifs/compr.c  |   17 +++++++++++++++++
> > - 4 files changed, 26 insertions(+), 2 deletions(-)
> > -
> > -Index: mtd-utils-1.4.6/Makefile
> > -===================================================================
> > ---- mtd-utils-1.4.6.orig/Makefile
> > -+++ mtd-utils-1.4.6/Makefile
> > -@@ -10,6 +10,7 @@ ifeq ($(WITHOUT_LZO), 1)
> > -   CPPFLAGS += -DWITHOUT_LZO
> > - else
> > -   LZOLDLIBS = -llzo2
> > -+  JFFS2_COMPR_LZO_O := compr_lzo.o
> > - endif
> > - 
> > - SUBDIRS = lib ubi-utils mkfs.ubifs
> > -@@ -50,7 +51,7 @@ $(SYMLINKS):
> > - 	ln -sf ../fs/jffs2/$@ $@
> > - 
> > - $(BUILDDIR)/mkfs.jffs2: $(addprefix $(BUILDDIR)/,\
> > --	compr_rtime.o mkfs.jffs2.o compr_zlib.o compr_lzo.o \
> > -+	compr_rtime.o mkfs.jffs2.o compr_zlib.o $(JFFS2_COMPR_LZO_O) \
> > - 	compr.o rbtree.o)
> > - LDFLAGS_mkfs.jffs2 = $(ZLIBLDFLAGS) $(LZOLDFLAGS)
> > - LDLIBS_mkfs.jffs2  = -lz $(LZOLDLIBS)
> > -Index: mtd-utils-1.4.6/compr.h
> > -===================================================================
> > ---- mtd-utils-1.4.6.orig/compr.h
> > -+++ mtd-utils-1.4.6/compr.h
> > -@@ -18,7 +18,9 @@
> > - 
> > - #define CONFIG_JFFS2_ZLIB
> > - #define CONFIG_JFFS2_RTIME
> > -+#ifndef WITHOUT_LZO
> > - #define CONFIG_JFFS2_LZO
> > -+#endif
> > - 
> > - #define JFFS2_RUBINMIPS_PRIORITY 10
> > - #define JFFS2_DYNRUBIN_PRIORITY  20
> > -Index: mtd-utils-1.4.6/mkfs.ubifs/Makefile
> > -===================================================================
> > ---- mtd-utils-1.4.6.orig/mkfs.ubifs/Makefile
> > -+++ mtd-utils-1.4.6/mkfs.ubifs/Makefile
> > -@@ -6,7 +6,11 @@ ALL_SOURCES=*.[ch] hashtable/*.[ch]
> > - 
> > - TARGETS = mkfs.ubifs
> > - 
> > --LDLIBS_mkfs.ubifs = -lz -llzo2 -lm -luuid -L$(BUILDDIR)/../ubi-utils/ -lubi
> > -+ifneq ($(WITHOUT_LZO), 1)
> > -+mkfs.ubifs_LIBLZO = -llzo2
> > -+endif
> > -+
> > -+LDLIBS_mkfs.ubifs = -lz $(mkfs.ubifs_LIBLZO) -lm -luuid -L$(BUILDDIR)/../ubi-utils/ -lubi
> > - LDLIBS_mkfs.ubifs += -L$(BUILDDIR)/../lib -lmtd
> > - LDLIBS_mkfs.ubifs += $(ZLIBLDFLAGS) $(LZOLDFLAGS)
> > - 
> > -Index: mtd-utils-1.4.6/mkfs.ubifs/compr.c
> > -===================================================================
> > ---- mtd-utils-1.4.6.orig/mkfs.ubifs/compr.c
> > -+++ mtd-utils-1.4.6/mkfs.ubifs/compr.c
> > -@@ -24,7 +24,11 @@
> > - #include <stdio.h>
> > - #include <stdint.h>
> > - #include <string.h>
> > -+#ifndef WITHOUT_LZO
> > - #include <lzo/lzo1x.h>
> > -+#else
> > -+#define LZO1X_999_MEM_COMPRESS	0
> > -+#endif
> > - #include <linux/types.h>
> > - 
> > - #define crc32 __zlib_crc32
> > -@@ -86,6 +90,8 @@ static int zlib_deflate(void *in_buf, si
> > - 	return 0;
> > - }
> > - 
> > -+
> > -+#ifndef WITHOUT_LZO
> > - static int lzo_compress(void *in_buf, size_t in_len, void *out_buf,
> > - 			size_t *out_len)
> > - {
> > -@@ -103,6 +109,17 @@ static int lzo_compress(void *in_buf, si
> > - 
> > - 	return 0;
> > - }
> > -+#else
> > -+static inline int lzo_compress(void *in_buf, size_t in_len, void *out_buf,
> > -+			size_t *out_len)
> > -+{
> > -+	(void)in_buf;
> > -+	(void)in_len;
> > -+	(void)out_buf;
> > -+	(void)out_len;
> > -+	return -1;
> > -+}
> > -+#endif
> > - 
> > - static int no_compress(void *in_buf, size_t in_len, void *out_buf,
> > - 		       size_t *out_len)
> > diff --git a/patches/mtd-utils-1.4.6/0002-make-mkubifs-optional.patch b/patches/mtd-utils-1.4.6/0002-make-mkubifs-optional.patch
> > deleted file mode 100644
> > index 23d0111..0000000
> > --- a/patches/mtd-utils-1.4.6/0002-make-mkubifs-optional.patch
> > +++ /dev/null
> > @@ -1,26 +0,0 @@
> > -From: Ladislav Michl <ladis@linux-mips.org>
> > -Date: Tue, 3 Aug 2010 13:00:00 +0200
> > -Subject: [PATCH] make mkubifs optional
> > -
> > -Signed-off-by: Ladislav Michl <ladis@linux-mips.org>
> > ----
> > - Makefile |    6 +++++-
> > - 1 file changed, 5 insertions(+), 1 deletion(-)
> > -
> > -Index: mtd-utils-1.4.6/Makefile
> > -===================================================================
> > ---- mtd-utils-1.4.6.orig/Makefile
> > -+++ mtd-utils-1.4.6/Makefile
> > -@@ -13,7 +13,11 @@ else
> > -   JFFS2_COMPR_LZO_O := compr_lzo.o
> > - endif
> > - 
> > --SUBDIRS = lib ubi-utils mkfs.ubifs
> > -+SUBDIRS = lib ubi-utils
> > -+ifneq ($(WITHOUT_MKUBIFS), 1)
> > -+  SUBDIRS += mkfs.ubifs
> > -+endif
> > -+
> > - TESTS = tests
> > - 
> > - TARGETS = ftl_format flash_erase nanddump doc_loadbios \
> > diff --git a/patches/mtd-utils-1.4.6/series b/patches/mtd-utils-1.4.6/series
> > deleted file mode 100644
> > index 4888771..0000000
> > --- a/patches/mtd-utils-1.4.6/series
> > +++ /dev/null
> > @@ -1,2 +0,0 @@
> > -0001-make-liblzo-optional.patch
> > -0002-make-mkubifs-optional.patch
> > diff --git a/patches/mtd-utils-1.4.9/0001-make-ubifs-optional.patch b/patches/mtd-utils-1.4.9/0001-make-ubifs-optional.patch
> > new file mode 100644
> > index 0000000..01c4139
> > --- /dev/null
> > +++ b/patches/mtd-utils-1.4.9/0001-make-ubifs-optional.patch
> > @@ -0,0 +1,24 @@
> > +From: Bernhard Walle <bernhard@bwalle.de>
> > +Date: Sun, 11 Mar 2012 20:03:50 +0100
> > +Subject: [PATCH] make ubifs optional
> > +
> > +Signed-off-by: Bernhard Walle <bernhard@bwalle.de>
> > +---
> > + Makefile |    2 ++
> > + 1 file changed, 2 insertions(+)
> > +
> > +diff --git a/Makefile b/Makefile
> > +index 17a1216..000dc86 100644
> > +--- a/Makefile
> > ++++ b/Makefile
> > +@@ -30,8 +30,10 @@ UBI_BINS = \
> > + 	ubidetach ubinize ubiformat ubirename mtdinfo ubirsvol
> > + 
> > + BINS = $(MTD_BINS)
> > ++ifneq ($(WITHOUT_MKUBIFS), 1)
> > + BINS += mkfs.ubifs/mkfs.ubifs
> > + BINS += $(addprefix ubi-utils/,$(UBI_BINS))
> > ++endif
> > + SCRIPTS = flash_eraseall
> > + 
> > + TARGETS = $(BINS)
> > diff --git a/patches/mtd-utils-1.4.9/0002-Make-liblzo-optional-for-ubifs-tools.patch b/patches/mtd-utils-1.4.9/0002-Make-liblzo-optional-for-ubifs-tools.patch
> > new file mode 100644
> > index 0000000..7e008de
> > --- /dev/null
> > +++ b/patches/mtd-utils-1.4.9/0002-Make-liblzo-optional-for-ubifs-tools.patch
> > @@ -0,0 +1,54 @@
> > +From: Bernhard Walle <bernhard@bwalle.de>
> > +Date: Sun, 11 Mar 2012 20:11:14 +0100
> > +Subject: [PATCH] Make liblzo optional for ubifs tools
> > +
> > +Based on patch from Marc Kleine-Budde <mkl@pengutronix.de>.
> > +
> > +Signed-off-by: Bernhard Walle <bernhard@bwalle.de>
> > +---
> > + mkfs.ubifs/compr.c |   17 +++++++++++++++++
> > + 1 file changed, 17 insertions(+)
> > +
> > +diff --git a/mkfs.ubifs/compr.c b/mkfs.ubifs/compr.c
> > +index 4152b6a..244d450 100644
> > +--- a/mkfs.ubifs/compr.c
> > ++++ b/mkfs.ubifs/compr.c
> > +@@ -24,7 +24,11 @@
> > + #include <stdio.h>
> > + #include <stdint.h>
> > + #include <string.h>
> > ++#ifndef WITHOUT_LZO
> > + #include <lzo/lzo1x.h>
> > ++#else
> > ++#define LZO1X_999_MEM_COMPRESS	0
> > ++#endif
> > + #include <linux/types.h>
> > + 
> > + #define crc32 __zlib_crc32
> > +@@ -86,6 +90,8 @@ static int zlib_deflate(void *in_buf, size_t in_len, void *out_buf,
> > + 	return 0;
> > + }
> > + 
> > ++
> > ++#ifndef WITHOUT_LZO
> > + static int lzo_compress(void *in_buf, size_t in_len, void *out_buf,
> > + 			size_t *out_len)
> > + {
> > +@@ -103,6 +109,17 @@ static int lzo_compress(void *in_buf, size_t in_len, void *out_buf,
> > + 
> > + 	return 0;
> > + }
> > ++#else
> > ++static inline int lzo_compress(void *in_buf, size_t in_len, void *out_buf,
> > ++			size_t *out_len)
> > ++{
> > ++	(void)in_buf;
> > ++	(void)in_len;
> > ++	(void)out_buf;
> > ++	(void)out_len;
> > ++	return -1;
> > ++}
> > ++#endif
> > + 
> > + static int no_compress(void *in_buf, size_t in_len, void *out_buf,
> > + 		       size_t *out_len)
> > diff --git a/patches/mtd-utils-1.4.9/series b/patches/mtd-utils-1.4.9/series
> > new file mode 100644
> > index 0000000..38ac3d1
> > --- /dev/null
> > +++ b/patches/mtd-utils-1.4.9/series
> > @@ -0,0 +1,5 @@
> > +# generated by git-ptx-patches
> > +#tag:base --start-number 1
> > +0001-make-ubifs-optional.patch
> > +0002-Make-liblzo-optional-for-ubifs-tools.patch
> > +# c7b83aac3f84d721cecd9e53605999f9  - git-ptx-patches magic
> > diff --git a/rules/host-mtd-utils.make b/rules/host-mtd-utils.make
> > index ddba9b8..64a0468 100644
> > --- a/rules/host-mtd-utils.make
> > +++ b/rules/host-mtd-utils.make
> > @@ -33,6 +33,4 @@ HOST_MTD_UTILS_MAKEVARS	= \
> >  	BUILDDIR=$(HOST_MTD_UTILS_DIR) \
> >  	DESTDIR=$(HOST_MTD_UTILS_PKGDIR)
> >  
> > -HOST_MTD_UTILS_MAKE_PAR := NO
> > -
> >  # vim: syntax=make
> > diff --git a/rules/mtd-utils.in b/rules/mtd-utils.in
> > index 800202b..50a489e 100644
> > --- a/rules/mtd-utils.in
> > +++ b/rules/mtd-utils.in
> > @@ -50,12 +50,6 @@ config MTD_UTILS_FLASH_ERASEALL
> >  comment "flash_eraseall is disabled by busybox flash_eraseall"
> >  	depends on BUSYBOX_FLASH_ERASEALL
> >  
> > -config MTD_UTILS_FLASH_INFO
> > -	bool
> > -	prompt "flash_info"
> > -	help
> > -	  print info about a MTD device
> > -
> >  config MTD_UTILS_FLASH_LOCK
> >  	bool
> >  	prompt "flash_lock"
> > @@ -308,8 +302,6 @@ config MTD_UTILS_MKFS_UBIFS
> >  	help
> >  	  Create ubifs filesystems
> >  
> > -endif
> > -
> >  config MTD_UTILS_MTDINFO
> >  	bool
> >  	prompt "mtdinfo"
> > @@ -317,3 +309,5 @@ config MTD_UTILS_MTDINFO
> >  	  Print UBI specific information about mtd devices
> >  
> >  endif
> > +
> > +endif
> > diff --git a/rules/mtd-utils.make b/rules/mtd-utils.make
> > index b52a170..14df787 100644
> > --- a/rules/mtd-utils.make
> > +++ b/rules/mtd-utils.make
> > @@ -17,8 +17,8 @@ PACKAGES-$(PTXCONF_MTD_UTILS) += mtd-utils
> >  #
> >  # Paths and names
> >  #
> > -MTD_UTILS_VERSION	:= 1.4.6
> > -MTD_UTILS_MD5		:= a0418afcf6026824a5fa5e9051793f3c
> > +MTD_UTILS_VERSION	:= 1.4.9
> > +MTD_UTILS_MD5		:= 82b8e714b90674896570968f70ca778b
> >  MTD_UTILS		:= mtd-utils-$(MTD_UTILS_VERSION)
> >  MTD_UTILS_SUFFIX	:= tar.bz2
> >  MTD_UTILS_URL		:= ftp://ftp.infradead.org/pub/mtd-utils/$(MTD_UTILS).$(MTD_UTILS_SUFFIX)
> > @@ -53,7 +53,6 @@ MTD_UTILS_COMPILE_ENV += WITHOUT_MKUBIFS=1
> >  endif
> >  
> >  MTD_UTILS_MAKEVARS := BUILDDIR=$(MTD_UTILS_DIR)
> > -MTD_UTILS_MAKE_PAR := NO
> >  
> >  $(STATEDIR)/mtd-utils.prepare:
> >  	@$(call targetinfo)
> > @@ -88,10 +87,6 @@ ifdef PTXCONF_MTD_UTILS_FLASH_ERASEALL
> >  	@$(call install_copy, mtd-utils, 0, 0, 0755, -, \
> >  		/usr/sbin/flash_eraseall)
> >  endif
> > -ifdef PTXCONF_MTD_UTILS_FLASH_INFO
> > -	@$(call install_copy, mtd-utils, 0, 0, 0755, -, \
> > -		/usr/sbin/flash_info)
> > -endif
> >  ifdef PTXCONF_MTD_UTILS_FLASH_LOCK
> >  	@$(call install_copy, mtd-utils, 0, 0, 0755, -, \
> >  		/usr/sbin/flash_lock)
> > -- 
> > 1.7.9.4
> > 
> > 
> > -- 
> > ptxdist mailing list
> > ptxdist@pengutronix.de
> > 
> 
> -- 
> Pengutronix e.K.                           |                             |
> Industrial Linux Solutions                 | http://www.pengutronix.de/  |
> Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
> 
> -- 
> ptxdist mailing list
> ptxdist@pengutronix.de
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
ptxdist mailing list
ptxdist@pengutronix.de

  reply	other threads:[~2012-03-13 13:34 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-12 14:09 [ptxdist] [PATCH 1/3] " bernhard
2012-03-12 14:09 ` [ptxdist] [PATCH 2/3] mtd-utils.in: Add busybox dependencies bernhard
2012-03-13  9:12   ` Michael Olbrich
2012-03-13  9:18     ` Bernhard Walle
2012-03-13  9:41       ` Michael Olbrich
2012-03-12 14:09 ` [ptxdist] [PATCH 3/3] procps.in: Unify wording of busybox comments bernhard
2012-03-13  9:14   ` Michael Olbrich
2012-03-13  9:09 ` [ptxdist] [PATCH 1/3] Update mtd-utils to 1.4.9 Michael Olbrich
2012-03-13 12:02   ` Bernhard Walle
2012-03-13 12:00     ` [ptxdist] [PATCH] " bernhard
2012-03-13 13:31       ` Michael Olbrich
2012-03-13 13:34         ` Michael Olbrich [this message]
2012-07-23 20:24 Bernhard Walle
2012-07-23 20:28 ` Thomas Petazzoni
2012-07-23 20:30   ` Bernhard Walle

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120313133442.GT29221@pengutronix.de \
    --to=m.olbrich@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox