On Thu, Apr 20, 2017 at 12:54:39AM +0100, Ben Hutchings wrote:
> On Wed, 2017-04-19 at 11:30 +0200, Lukas Wunner wrote:
> > Remove merged patches and rebase remaining patches.
> > 
> > A portion of the secureboot patches have been upstreamed, but were
> > changed substantially during review, primarily to avoid code
> > duplication among arches.  I've stripped the patches of the merged
> > bits and rebased the remainder.
> 
> This is great work!  I did find a few errors, though:
> 
> - You made some changes to context in
>   bugfix/all/firmware-remove-redundant-log-messages-from-drivers.patch
>   which didn't match 4.11-rc6.  Huh?

Ugh, my apologies, I forgot to mention that my kernel tree was based on
drm-tip as of last Saturday, rather than Linus' v4.11-rc6 tag, so I was
already taking into account all the drm and sound changes queued for 4.12.

> - You deleted part of
>   bugfix/all/tools-lib-traceevent-fix-use-of-uninitialized-variables.patch
>   but it looks like it can actually be dropped entirely.
> - In
>   
> features/all/securelevel/arm64-add-kernel-config-option-to-set-securelevel-wh.patch
>   the return value of efi_get_secureboot() is treated as a boolean,
>   but that's no longer true.

Sorry, missed that.  Thanks a lot for double-checking.  Good work pulling
in David Howells' new version of the lockdown patches, I knew of them but
had already purged them from my mailbox, was too lazy to scrape them together
from mailing list archives or patchwork, and had forgotten that he had
provided a branch that I just could have pulled from.

Kind regards,

Lukas

Reply via email to