Is it just me or is the 5th patch lost? Am Dienstag, 11. Oktober 2016 schrieb Alexandre Courbot :
> Hi everyone, > > Apologies for the big patchset. This is a rework of the secure boot code > that > moves the building of the blob into its own set of source files (and own > hooks), > making the code more flexible and (hopefully) easier to understand as well. > > This rework is needed to support more signed firmware for existing and new > chips. Since the firmwares in question are not available yet I cannot send > the > code for them yet, but hopefully the gain in clarity will be enough to > merge > this series ahead of the rest. > > Patch 1 is just to make nv*_printk() more tolerent of my use of const > pointers. > Patch 2 lays the foundation of a small common falcon library that could > avoid > duplicated code in various parts of the driver. Right now only secure boot > makes > use of it (patch 3), but there are many potential users (GR, PMU, etc). I > will > test users and send further patches making use of it. > > Patch 4 renames a hook to something better describing its actual use. > > Patch 5 is where the big refactoring occurs. It looks scary, but is really > just > moving code (and introducing the nvkm_acr structures). > > Patch 6, 7 and 8 remove a few HS hooks that turn out to be unneeded, and > add > support for LS hooks. > > The end result can be observed by looking at acr_v1_gm20b.c: all the > specifics > of GM20B's firmware are handled in a single file, with no data structures > shared > with acr_v1.c. The gm20b_acr_v1_ls_func variable also describes clearly > what > LS firmwares are to be loaded and how. > > Alexandre Courbot (8): > core: constify nv*_printk macros > core: add falcon library > secboot: use falcon library's IMEM/DMEM loading functions > secboot: rename init() hook to oneinit() > secboot: move ACR building logic into own source files > secboot: remove fixup_hs_desc hook > secboot: add low-secure firmware hooks > secboot: generate HS BL descriptor in hook > > drm/nouveau/include/nvkm/core/client.h | 4 +- > drm/nouveau/include/nvkm/core/device.h | 2 +- > drm/nouveau/include/nvkm/core/falcon.h | 50 + > drm/nouveau/include/nvkm/core/subdev.h | 2 +- > drm/nouveau/include/nvkm/subdev/secboot.h | 33 +- > drm/nouveau/nvkm/core/Kbuild | 1 + > drm/nouveau/nvkm/core/falcon.c | 62 ++ > drm/nouveau/nvkm/engine/gr/gf100.c | 16 +- > drm/nouveau/nvkm/engine/gr/gm200.c | 6 +- > drm/nouveau/nvkm/subdev/secboot/Kbuild | 2 + > drm/nouveau/nvkm/subdev/secboot/acr.h | 67 ++ > drm/nouveau/nvkm/subdev/secboot/acr_v1.c | 1193 > ++++++++++++++++++++ > drm/nouveau/nvkm/subdev/secboot/acr_v1.h | 346 ++++++ > drm/nouveau/nvkm/subdev/secboot/acr_v1_gm20b.c | 135 +++ > drm/nouveau/nvkm/subdev/secboot/base.c | 113 +- > drm/nouveau/nvkm/subdev/secboot/gm200.c | 1386 > +----------------------- > drm/nouveau/nvkm/subdev/secboot/gm200.h | 43 + > drm/nouveau/nvkm/subdev/secboot/gm20b.c | 141 +-- > drm/nouveau/nvkm/subdev/secboot/priv.h | 208 +--- > 19 files changed, 2072 insertions(+), 1738 deletions(-) > create mode 100644 drm/nouveau/include/nvkm/core/falcon.h > create mode 100644 drm/nouveau/nvkm/core/falcon.c > create mode 100644 drm/nouveau/nvkm/subdev/secboot/acr.h > create mode 100644 drm/nouveau/nvkm/subdev/secboot/acr_v1.c > create mode 100644 drm/nouveau/nvkm/subdev/secboot/acr_v1.h > create mode 100644 drm/nouveau/nvkm/subdev/secboot/acr_v1_gm20b.c > create mode 100644 drm/nouveau/nvkm/subdev/secboot/gm200.h > > -- > 2.10.0 > > _______________________________________________ > Nouveau mailing list > Nouveau@lists.freedesktop.org <javascript:;> > https://lists.freedesktop.org/mailman/listinfo/nouveau >
_______________________________________________ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau