From: Yegor Yefremov <yegorslists@googlemail.com>
To: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Cc: Falco Hyfing <hyfinglists@gmail.com>,
barebox <barebox@lists.infradead.org>
Subject: Re: [PATCH] lib: xz: add support for bcj filters
Date: Wed, 22 Feb 2017 09:15:36 +0100 [thread overview]
Message-ID: <CAGm1_ku+mmxKRgdEGzQrTi37+8nmnM5MX+Cn2VQBGfxd-bjqog@mail.gmail.com> (raw)
In-Reply-To: <0CEC428C-E1EE-48A7-85CF-6FC90A22F47C@jcrosoft.com>
On Wed, Feb 22, 2017 at 5:29 AM, Jean-Christophe PLAGNIOL-VILLARD
<plagnioj@jcrosoft.com> wrote:
>
>> On Feb 21, 2017, at 10:47 PM, yegorslists@googlemail.com wrote:
>>
>> From: Yegor Yefremov <yegorslists@googlemail.com>
>>
>> Add missing configuration options for various bcj filters. Without
>> these options the lib/xz/xz_dec_bcj.c file will be compiled, but all
>> filters will be disabled.
>>
>> Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>
>> ---
>> lib/Kconfig | 28 ++++++++++++++++++++++++++++
>> 1 file changed, 28 insertions(+)
>>
>> diff --git a/lib/Kconfig b/lib/Kconfig
>> index f9f25bdef..83dd8e0a4 100644
>> --- a/lib/Kconfig
>> +++ b/lib/Kconfig
>> @@ -22,6 +22,34 @@ config XZ_DECOMPRESS
>> bool "include xz uncompression support"
>> select UNCOMPRESS
>>
>> +if XZ_DECOMPRESS
>> +
>> +config XZ_DEC_X86
>> + bool "x86 BCJ filter decoder"
>> + default y
> this need to be ARCH dependant
Why? In Linux kernel it is not arch dependent, because AFAIK it
describes only compression particularities. This way you can extract
sqaushfs images on your ARM machine, that were compressed on SPARK
etc.
>> +
>> +config XZ_DEC_POWERPC
>> + bool "PowerPC BCJ filter decoder"
>> + default y
>> +
> ditto
>> +config XZ_DEC_IA64
>> + bool "IA-64 BCJ filter decoder"
>> + default y
> ditto
>> +config XZ_DEC_ARM
>> + bool "ARM BCJ filter decoder"
>> + default y
>> +
> ditto
>> +config XZ_DEC_ARMTHUMB
>> + bool "ARM-Thumb BCJ filter decoder"
>> + default y
>> +
> ditto
>> +config XZ_DEC_SPARC
>> + bool "SPARC BCJ filter decoder"
>> + default y
>> +
>> +endif
>> +
>> config REED_SOLOMON
>> bool
>>
>> --
>> 2.11.0
>>
>>
>> _______________________________________________
>> barebox mailing list
>> barebox@lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/barebox
>
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2017-02-22 8:16 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-21 14:47 yegorslists
2017-02-22 4:29 ` Jean-Christophe PLAGNIOL-VILLARD
2017-02-22 8:15 ` Yegor Yefremov [this message]
2017-02-22 10:03 ` Sascha Hauer
2017-02-22 10:10 ` Yegor Yefremov
2017-02-22 10:21 ` Sascha Hauer
2017-02-22 10:35 ` Yegor Yefremov
2017-02-22 11:34 ` Sascha Hauer
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=CAGm1_ku+mmxKRgdEGzQrTi37+8nmnM5MX+Cn2VQBGfxd-bjqog@mail.gmail.com \
--to=yegorslists@googlemail.com \
--cc=barebox@lists.infradead.org \
--cc=hyfinglists@gmail.com \
--cc=plagnioj@jcrosoft.com \
/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