28 Jan
2014
28 Jan
'14
9:17 a.m.
Hello,
Here's the patch that enables lzmadec command in sandbox. Should I send it as a new thread or is it fine like that ?
Regards
Patrice
--- include/configs/sandbox.h | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/include/configs/sandbox.h b/include/configs/sandbox.h index a6d5582..c995882 100644 --- a/include/configs/sandbox.h +++ b/include/configs/sandbox.h @@ -131,4 +131,6 @@
#define CONFIG_TPM_TIS_SANDBOX
+#define CONFIG_CMD_LZMADEC + #endif
--
2014-01-26 Simon Glass sjg@chromium.org:
> Hi Patrice,
>
>
> On 24 January 2014 14:39, Tom Rini trini@ti.com wrote:
>>
>> On Thu, Dec 19, 2013 at 11:10:24AM +0100, Patrice B wrote:
>>
>> > Hi,
>> >
>> > I needed to be able to uncompress lzma files. I did this command
>> > based on unzip command and propose it if it could help. Hopping the
>> > patch is correctly done...
>> >
>> > Best regards.
>> >
>> > Patrice
>> >
>> >
>> > Signed-off-by: Patrice Bouchand <pbfwdlist_gmail_com>
>
>
>>
>>
>> The code looks fine, but we don't add code that isn't enabled anywhere.
>> So unless Simon wants to pick this up for sandbox, I'm marking this as
>> deferred in patchwork, thanks!
>>
>
> It's fine to enable this in sandbox if you want to - just send a separate
> patch to add the CONFIG to include/configs/sandbox.h, test it in sandbox
> (make sandbox_config; make; ./u-boot) and Cc me. I will pick it up.
>
> Regards,
> Simon
>
>>
>> --
>> Tom
>
>