Only skimmed over the patches, but in general looks good to me.

Feel free to add an Acked-by: Christian König <christian.koe...@amd.com> to the whole series.

But somebody with more ras knowledge than I have should probably take a look as well.

Christian.

Am 29.08.19 um 10:59 schrieb Guchun Chen:
These patches are to remove additional external lib-jsonc
dependence, and to put all test configurations into C code.

Guchun Chen (4):
   amdgpu: remove json package dependence
   amdgpu: delete test configuration file
   amdgpu: add ras inject unit test
   amdgpu: add ras feature capability check in inject test

  configure.ac             |  18 ---
  data/amdgpu_ras.json     | 267 ----------------------------------
  meson.build              |   1 -
  tests/amdgpu/Makefile.am |   5 +-
  tests/amdgpu/meson.build |  16 +-
  tests/amdgpu/ras_tests.c | 305 ++++++++++++++-------------------------
  6 files changed, 116 insertions(+), 496 deletions(-)
  delete mode 100644 data/amdgpu_ras.json


_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to