linux-kernel - Re: [PATCHv2 0/5] remoteproc: Custom firmware handling.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAK=WgbboTiyjn5W9zr4TE0xk9_YHDmcGzxnMVovs8P=c607r2w@mail.gmail.com>
Date:	Sun, 17 Jun 2012 11:53:27 +0300
From:	Ohad Ben-Cohen <ohad@...ery.com>
To:	sjur.brandeland@...ricsson.com
Cc:	Loic PALLARDY <loic.pallardy@...ricsson.com>,
	Ludovic BARRE <ludovic.barre@...ricsson.com>,
	linux-kernel@...r.kernel.org, Arnd Bergmann <arnd@...db.de>,
	Linus Walleij <linus.walleij@...aro.org>,
	Sjur Brændeland <sjurbren@...il.com>
Subject: Re: [PATCHv2 0/5] remoteproc: Custom firmware handling.

Hi Sjur,

On Mon, Jun 11, 2012 at 8:29 PM,  <sjur.brandeland@...ricsson.com> wrote:
> Here is a new patch series. I dropped the two bug-fix patches
> since last time, as you already included these. I've tried to
> address all of your review comments since last time, hope I
> didn't miss any.

Thanks, this looks good.

I have a few small mostly-style comments, but I could just do them
while I apply the patches.

I've tried applying the (first four) patches, but since they are still
based on the two bug-fixes patches, they don't apply.

So we can either wait until the bug fixes are merged by Linus (I'm
going to send them soon) or you can rebase the patches on a current
mainline tree (which I could then push to linux-next) - however you
prefer.

> Status:
> I have done a simple simulated test now with the STE firmware
> loader and main-flow seems ok, but I haven't done any negative
> tests.

Ok, thanks. The first four patches seem trivial enough for me to merge
even without any extensive testing.

> - The latest patch is a bit odd. The ste_modem_remoteproc module
>  contains only the firmware handler, and nothing else.

What are your plans regarding the rest of that module? we may want to
wait with the fifth patch until we have some basic functionality.

Thanks,
Ohad.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/